diff options
author | matthijs <matthijs@openttd.org> | 2006-12-05 13:58:20 +0000 |
---|---|---|
committer | matthijs <matthijs@openttd.org> | 2006-12-05 13:58:20 +0000 |
commit | 9218fc16e6fff350e6209b6b90b8597fc431f00f (patch) | |
tree | e23ebc401e3e8f7037ad38701e00a6bf1fb61136 /industry.h | |
parent | 681f994d16a71813144352b3a6ef05a22cb8b3ff (diff) | |
download | openttd-9218fc16e6fff350e6209b6b90b8597fc431f00f.tar.xz |
(svn r7372) - CodeChange: Rename all GetXXXArraySize() functions to GetNumXXX() and add GetMaxXXXIndex() functions. This prepares for the new pool interface.
Diffstat (limited to 'industry.h')
-rw-r--r-- | industry.h | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/industry.h b/industry.h index db1b915f3..65da9bd0e 100644 --- a/industry.h +++ b/industry.h @@ -92,13 +92,18 @@ static inline bool IsValidIndustry(const Industry *industry) VARDEF int _total_industries; -static inline IndustryID GetIndustryArraySize(void) +static inline IndustryID GetMaxIndustryIndex(void) { /* TODO - This isn't the real content of the function, but * with the new pool-system this will be replaced with one that - * _really_ returns the highest index + 1. Now it just returns + * _really_ returns the highest index. Now it just returns * the next safe value we are sure about everything is below. */ + return _total_industries - 1; +} + +static inline uint GetNumIndustries(void) +{ return _total_industries; } @@ -107,10 +112,10 @@ static inline IndustryID GetIndustryArraySize(void) */ static inline Industry *GetRandomIndustry(void) { - uint num = RandomRange(GetIndustryArraySize()); + uint num = RandomRange(GetNumIndustries()); uint index = 0; - if (GetIndustryArraySize() == 0) return NULL; + if (GetNumIndustries() == 0) return NULL; while (num > 0) { num--; @@ -119,7 +124,7 @@ static inline Industry *GetRandomIndustry(void) /* Make sure we have a valid industry */ while (GetIndustry(index) == NULL) { index++; - if (index == GetIndustryArraySize()) index = 0; + if (index > GetMaxIndustryIndex()) index = 0; } } |