mirror/OpenTTD
1
0
Fork 1
mirror of https://github.com/OpenTTD/OpenTTD.git synced 2025-08-31 02:19:09 +00:00
Code Issues Activity

Codechange: Warn if randomaction2 group count is not a power of 2.

Browse Source 
Previously noted by a comment, this does not need to be guarded against as non-powers of 2 will not cause issues beyond the choice of results being reduced.
This commit is contained in:
Peter Nelson
2021-05-02 00:34:17 +01:00
committed by PeterN
parent 6b0b1bb3de
commit 18fb1c3866
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/newgrf.cpp
View File

@@ -5130,6 +5130,10 @@ static void NewSpriteGroup(ByteReader *buf)
group->lowest_randbit = buf->ReadByte(); group->lowest_randbit = buf->ReadByte();
byte num_groups = buf->ReadByte(); byte num_groups = buf->ReadByte();
if (!HasExactlyOneBit(num_groups)) {
grfmsg(1, "NewSpriteGroup: Random Action 2 nrand should be power of 2");
}
for (uint i = 0; i < num_groups; i++) { for (uint i = 0; i < num_groups; i++) {
group->groups.push_back(GetGroupFromGroupID(setid, type, buf->ReadWord())); group->groups.push_back(GetGroupFromGroupID(setid, type, buf->ReadWord()));
} }