# HG changeset patch
# User frosch <frosch@openttd.org>
# Date 2010-04-16 21:21:54
# Node ID e5877a75e8437f0f0482736b4918f95150e7c012
# Parent  fb9c32d7987cb4b4da8feefbccdd763e9013e53d

(svn r19643) -Fix (r19120): Industry generation failed for large maps and lots of industry types.

diff --git a/src/core/random_func.cpp b/src/core/random_func.cpp
--- a/src/core/random_func.cpp
+++ b/src/core/random_func.cpp
@@ -57,6 +57,7 @@ uint32 DoRandom(int line, const char *fi
 
 uint DoRandomRange(uint max, int line, const char *file)
 {
+	assert(max <= UINT16_MAX);
 	return GB(DoRandom(line, file), 0, 16) * max >> 16;
 }
 #endif /* RANDOM_DEBUG */
diff --git a/src/core/random_func.hpp b/src/core/random_func.hpp
--- a/src/core/random_func.hpp
+++ b/src/core/random_func.hpp
@@ -99,8 +99,9 @@ void SetRandomSeed(uint32 seed);
 		return _random.Next();
 	}
 
-	static FORCEINLINE uint32 RandomRange(uint16 max)
+	static FORCEINLINE uint32 RandomRange(uint max)
 	{
+		assert(max <= UINT16_MAX);
 		return _random.Next(max);
 	}
 #endif
diff --git a/src/industry_cmd.cpp b/src/industry_cmd.cpp
--- a/src/industry_cmd.cpp
+++ b/src/industry_cmd.cpp
@@ -1923,7 +1923,8 @@ void GenerateIndustries()
 
 	/* Add the remaining industries according to their probabilities */
 	for (uint i = 0; i < total_amount; i++) {
-		uint32 r = RandomRange(total_prob);
+		/* RandomRange() can only deal with 16 bit, which is not enough here. */
+		uint32 r = ((uint64)Random() * (uint64)total_prob) >> 32;
 		IndustryType it = 0;
 		while (it < NUM_INDUSTRYTYPES && r >= industry_probs[it]) {
 			r -= industry_probs[it];