summaryrefslogtreecommitdiff
path: root/src/script
diff options
context:
space:
mode:
authoralberth <alberth@openttd.org>2012-09-18 19:29:29 +0000
committeralberth <alberth@openttd.org>2012-09-18 19:29:29 +0000
commit461fc1e268cb5db0394221f1b61383b453e6fc0a (patch)
tree09e47a7d2107524293c09567eafdf5a88a26aa92 /src/script
parent72639deb84b7b4ca61a10f87ac2dc4841f4c8cdb (diff)
downloadopenttd-461fc1e268cb5db0394221f1b61383b453e6fc0a.tar.xz
(svn r24531) -Fix: Max script chance was too big.
Diffstat (limited to 'src/script')
-rw-r--r--src/script/api/script_base.cpp4
-rw-r--r--src/script/api/script_base.hpp1
2 files changed, 4 insertions, 1 deletions
diff --git a/src/script/api/script_base.cpp b/src/script/api/script_base.cpp
index 5be6dd9a5..2472fe1c7 100644
--- a/src/script/api/script_base.cpp
+++ b/src/script/api/script_base.cpp
@@ -11,6 +11,7 @@
#include "../../stdafx.h"
#include "script_base.hpp"
+#include "script_error.hpp"
#include "../../network/network.h"
#include "../../core/random_func.hpp"
@@ -42,7 +43,8 @@
/* static */ bool ScriptBase::Chance(uint out, uint max)
{
- return (uint16)Rand() <= (uint16)((65536 * out) / max);
+ EnforcePrecondition(false, out <= max);
+ return (uint16)Rand() <= (uint16)((65535 * out) / max);
}
/* static */ bool ScriptBase::ChanceItem(int unused_param, uint out, uint max)
diff --git a/src/script/api/script_base.hpp b/src/script/api/script_base.hpp
index 9384cbf80..a01325820 100644
--- a/src/script/api/script_base.hpp
+++ b/src/script/api/script_base.hpp
@@ -59,6 +59,7 @@ public:
* After all, it is a random function.
* @param out How many times it should return true.
* @param max Out of this many times.
+ * @pre \a out is at most equal to \a max.
* @return True if the chance worked out.
*/
static bool Chance(uint out, uint max);