miguel.horta/openttd-cmclient
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Codechange: Prefer EnumBitSet's All() method over bitwise comparisons. (#14611)

Browse Source 
This makes the intent of the operation clearer.
This commit is contained in:
Peter Nelson
2025-09-12 19:37:17 +01:00
committed by dP
parent 74840d1b0a
commit 404e9bb506
3 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/townname.cpp
View File

@@ -667,7 +667,7 @@ static void MakeCzechTownName(StringBuilder &builder, uint32_t seed)
for (ending = ending_start; ending <= ending_stop; ending++) {
const CzechNameSubst *e = &_name_czech_subst_ending[ending];
if ((e->choose & choose) == choose && e->allow.Any(allow)) {
if (e->choose.All(choose) && e->allow.Any(allow)) {
map[i++] = ending;
}
}
@@ -681,7 +681,7 @@ static void MakeCzechTownName(StringBuilder &builder, uint32_t seed)
assert(gender != CZG_FREE && gender != CZG_NFREE);
}
if (do_prefix && (_name_czech_adj[prefix].choose & choose) != choose) {
if (do_prefix && !_name_czech_adj[prefix].choose.All(choose)) {
/* Throw away non-matching prefix. */
do_prefix = false;
}