diff options
author | ache <ache@FreeBSD.org> | 2008-08-08 01:42:17 +0000 |
---|---|---|
committer | ache <ache@FreeBSD.org> | 2008-08-08 01:42:17 +0000 |
commit | 823689b09c9db6b0a03a78837e4c02b453e37d8d (patch) | |
tree | 8f6ee5f63b889dc3ddd277538eb981661d977ec6 /games/random/randomize_fd.c | |
parent | c14ceb129d096f937348a276508fee6f9493f9ae (diff) | |
download | FreeBSD-src-823689b09c9db6b0a03a78837e4c02b453e37d8d.zip FreeBSD-src-823689b09c9db6b0a03a78837e4c02b453e37d8d.tar.gz |
I was confused a bit by the wrong construction with RAND_MAX
used in randomize_fd.c.
Although the max value is the same currently, RAND_MAX is for rand(),
not for random().
So move RANDOM_MAX const to the common file now, make it UL and
use in randomize_fd.c too.
(in any case its old value was 1 less then must be, as noted in
the prev. commit)
Diffstat (limited to 'games/random/randomize_fd.c')
-rw-r--r-- | games/random/randomize_fd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/games/random/randomize_fd.c b/games/random/randomize_fd.c index b60c53c..e1011fb 100644 --- a/games/random/randomize_fd.c +++ b/games/random/randomize_fd.c @@ -202,7 +202,7 @@ randomize_fd(int fd, int type, int unique, double denom) } for (i = numnode; i > 0; i--) { - selected = ((int)denom * random())/(((double)RAND_MAX + 1) / numnode); + selected = ((int)denom * random())/(RANDOM_MAX / numnode); for (j = 0, prev = n = rand_root; n != NULL; j++, prev = n, n = n->next) { if (j == selected) { |