summaryrefslogtreecommitdiff
path: root/src/core/random_func.cpp
diff options
context:
space:
mode:
authorfonsinchen <fonsinchen@openttd.org>2013-10-20 14:48:08 +0000
committerfonsinchen <fonsinchen@openttd.org>2013-10-20 14:48:08 +0000
commitfe550c2db3ede0cb5235d5918f1634358b56e5db (patch)
tree0a4dcf478154030ad1b1e4d2b3e71195ba92bd99 /src/core/random_func.cpp
parent9337c561b0cc37a9156d4132178d54322cf6d06e (diff)
downloadopenttd-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.cpp15
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 */