aly/servatrice
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Commented a pitfall in case of RNG replacement.

Browse source
This commit is contained in:
Mark Morschhäuser 2014-03-11 08:18:54 +01:00
parent 800f7e6373
commit 9a20e5e2e2
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
common/rng_sfmt.cpp
View file

@ -47,6 +47,13 @@ RNG_SFMT::RNG_SFMT(QObject *parent)
* This can be compared to an ideal six sided die that is rolled until only sides * This can be compared to an ideal six sided die that is rolled until only sides
* 1-5 show up, while 6 represents something that you don't want. So you basically roll * 1-5 show up, while 6 represents something that you don't want. So you basically roll
* a five sided die. * a five sided die.
*
* Note: If you replace the SFMT RNG with some other rand() function in the future,
* then you _need_ to change the UINT64_MAX constant to the largest possible random
* number which can be created by the new rand() function. This value is often defined
* in a RAND_MAX constant.
* Otherwise you will probably skew the outcome of the getNumber() method or worsen the
* performance of the application.
*/ */
unsigned int RNG_SFMT::getNumber(unsigned int min, unsigned int max) unsigned int RNG_SFMT::getNumber(unsigned int min, unsigned int max)
{ {