diff options
author | fonsinchen <fonsinchen@openttd.org> | 2013-10-20 14:48:08 +0000 |
---|---|---|
committer | fonsinchen <fonsinchen@openttd.org> | 2013-10-20 14:48:08 +0000 |
commit | fe550c2db3ede0cb5235d5918f1634358b56e5db (patch) | |
tree | 0a4dcf478154030ad1b1e4d2b3e71195ba92bd99 /src/core/random_func.cpp | |
parent | 9337c561b0cc37a9156d4132178d54322cf6d06e (diff) | |
download | openttd-fe550c2db3ede0cb5235d5918f1634358b56e5db.tar.xz |
(svn r25893) -Fix: Document RandomRange and change misleading parameter name 'max' in random functions.
Diffstat (limited to 'src/core/random_func.cpp')
-rw-r--r-- | src/core/random_func.cpp | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/src/core/random_func.cpp b/src/core/random_func.cpp index 7800591e6..8cf21b54f 100644 --- a/src/core/random_func.cpp +++ b/src/core/random_func.cpp @@ -29,13 +29,14 @@ uint32 Randomizer::Next() } /** - * Generate the next pseudo random number scaled to max - * @param max the maximum value of the returned random number - * @return the random number + * Generate the next pseudo random number scaled to \a limit, excluding \a limit + * itself. + * @param limit Limit of the range to be generated from. + * @return Random number in [0,\a limit) */ -uint32 Randomizer::Next(uint32 max) +uint32 Randomizer::Next(uint32 limit) { - return ((uint64)this->Next() * (uint64)max) >> 32; + return ((uint64)this->Next() * (uint64)limit) >> 32; } /** @@ -75,8 +76,8 @@ uint32 DoRandom(int line, const char *file) return _random.Next(); } -uint32 DoRandomRange(uint32 max, int line, const char *file) +uint32 DoRandomRange(uint32 limit, int line, const char *file) { - return ((uint64)DoRandom(line, file) * (uint64)max) >> 32; + return ((uint64)DoRandom(line, file) * (uint64)limit) >> 32; } #endif /* RANDOM_DEBUG */ |