summaryrefslogtreecommitdiffstats
path: root/games/random/randomize_fd.c
diff options
context:
space:
mode:
authorache <ache@FreeBSD.org>2008-08-08 01:42:17 +0000
committerache <ache@FreeBSD.org>2008-08-08 01:42:17 +0000
commit823689b09c9db6b0a03a78837e4c02b453e37d8d (patch)
tree8f6ee5f63b889dc3ddd277538eb981661d977ec6 /games/random/randomize_fd.c
parentc14ceb129d096f937348a276508fee6f9493f9ae (diff)
downloadFreeBSD-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.c2
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) {
OpenPOWER on IntegriCloud