archives/open-adventure
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Once again, take srand()/random() out of the initialization chain.

Browse source 
They have exactly the wrong kind of randomness for this job - not
returning consistent sequences across different platforms or C library
versions, and because pseodorandom not really better than sampling
the clock.
This commit is contained in:
Eric S. Raymond 2017-05-24 23:36:25 -04:00
parent 6146406990
commit 6ba9a7de7f
3 changed files with 6 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

9
misc.c
View file

@ -724,7 +724,8 @@ L2: ATDWRF=I;
/* Utility routines (SETBIT, TSTBIT, set_seed_from_time, get_next_lcg_value, randrange, RNDVOC, BUG) */
/* Utility routines (SETBIT, TSTBIT, set_seed, get_next_lcg_value,
* randrange, RNDVOC, BUG) */
#undef SETBIT
long fSETBIT(long BIT) {
@ -759,11 +760,9 @@ long TSTBIT;
#define TSTBIT(MASK,BIT) fTSTBIT(MASK,BIT)
void set_seed_from_time(void)
void set_seed(long seedval)
{
/* Use the current system time to get seed the ISO rand() function, from which we get a seed for the LCG. */
srand(time(NULL));
lcgstate.x = (unsigned long) rand() % lcgstate.m;
lcgstate.x = (unsigned long) seedval % lcgstate.m;
}
unsigned long get_next_lcg_value(void)