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/random.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/random.c')
-rw-r--r-- | games/random/random.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/games/random/random.c b/games/random/random.c index 601036f..122ce45 100644 --- a/games/random/random.c +++ b/games/random/random.c @@ -162,7 +162,7 @@ main(int argc, char *argv[]) /* Compute a random exit status between 0 and denom - 1. */ if (random_exit) - return (int)(denom * random() / ((double)RAND_MAX + 1)); + return (int)(denom * random() / RANDOM_MAX); /* * Select whether to print the first line. (Prime the pump.) @@ -170,7 +170,7 @@ main(int argc, char *argv[]) * 0 (which has a 1 / denom chance of being true), we select the * line. */ - selected = (int)(denom * random() / ((double)RAND_MAX + 1)) == 0; + selected = (int)(denom * random() / RANDOM_MAX) == 0; while ((ch = getchar()) != EOF) { if (selected) (void)putchar(ch); @@ -180,7 +180,7 @@ main(int argc, char *argv[]) err(2, "stdout"); /* Now see if the next line is to be printed. */ - selected = (int)(denom * random() / ((double)RAND_MAX + 1)) == 0; + selected = (int)(denom * random() / RANDOM_MAX) == 0; } } if (ferror(stdin)) |