diff options
45 files changed, 161 insertions, 166 deletions
diff --git a/src/ai/default/default.cpp b/src/ai/default/default.cpp index 419dd39bb..61f816c1f 100644 --- a/src/ai/default/default.cpp +++ b/src/ai/default/default.cpp @@ -3847,7 +3847,7 @@ static void AiHandleTakeover(Player *p) return; } - SETBIT(p->bankrupt_asked, best_pl->index); + SetBit(p->bankrupt_asked, best_pl->index); if (best_pl->index == _local_player) { p->bankrupt_timeout = 4440; diff --git a/src/aircraft_cmd.cpp b/src/aircraft_cmd.cpp index 5afad4f3f..221fdaca1 100644 --- a/src/aircraft_cmd.cpp +++ b/src/aircraft_cmd.cpp @@ -418,7 +418,7 @@ CommandCost CmdBuildAircraft(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) u->random_bits = VehicleRandomBits(); v->vehicle_flags = 0; - if (e->flags & ENGINE_EXCLUSIVE_PREVIEW) SETBIT(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE); + if (e->flags & ENGINE_EXCLUSIVE_PREVIEW) SetBit(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE); UpdateAircraftCache(v); @@ -606,7 +606,7 @@ CommandCost CmdSendAircraftToHangar(TileIndex tile, uint32 flags, uint32 p1, uin v->current_order.type = OT_GOTO_DEPOT; v->current_order.flags = OF_NON_STOP; - if (!(p2 & DEPOT_SERVICE)) SETBIT(v->current_order.flags, OFB_HALT_IN_DEPOT); + if (!(p2 & DEPOT_SERVICE)) SetBit(v->current_order.flags, OFB_HALT_IN_DEPOT); v->current_order.refit_cargo = CT_INVALID; v->current_order.dest = next_airport_index; InvalidateWindowWidget(WC_VEHICLE_VIEW, v->index, STATUS_BAR); @@ -1960,7 +1960,7 @@ static bool FreeTerminal(Vehicle *v, byte i, byte last_terminal) if (!HasBit(st->airport_flags, _airport_terminal_flag[i])) { /* TERMINAL# HELIPAD# */ v->u.air.state = _airport_terminal_state[i]; // start moving to that terminal/helipad - SETBIT(st->airport_flags, _airport_terminal_flag[i]); // occupy terminal/helipad + SetBit(st->airport_flags, _airport_terminal_flag[i]); // occupy terminal/helipad return true; } } diff --git a/src/airport.cpp b/src/airport.cpp index 73bfdc430..937adca2f 100644 --- a/src/airport.cpp +++ b/src/airport.cpp @@ -488,14 +488,14 @@ uint32 GetValidAirports() { uint32 mask = 0; - if (_cur_year < 1960 || _patches.always_small_airport) SETBIT(mask, 0); // small airport - if (_cur_year >= 1955) SETBIT(mask, 1); // city airport - if (_cur_year >= 1963) SETBIT(mask, 2); // heliport - if (_cur_year >= 1980) SETBIT(mask, 3); // metropolitan airport - if (_cur_year >= 1990) SETBIT(mask, 4); // international airport - if (_cur_year >= 1983) SETBIT(mask, 5); // commuter airport - if (_cur_year >= 1976) SETBIT(mask, 6); // helidepot - if (_cur_year >= 2002) SETBIT(mask, 7); // intercontinental airport - if (_cur_year >= 1980) SETBIT(mask, 8); // helistation + if (_cur_year < 1960 || _patches.always_small_airport) SetBit(mask, 0); // small airport + if (_cur_year >= 1955) SetBit(mask, 1); // city airport + if (_cur_year >= 1963) SetBit(mask, 2); // heliport + if (_cur_year >= 1980) SetBit(mask, 3); // metropolitan airport + if (_cur_year >= 1990) SetBit(mask, 4); // international airport + if (_cur_year >= 1983) SetBit(mask, 5); // commuter airport + if (_cur_year >= 1976) SetBit(mask, 6); // helidepot + if (_cur_year >= 2002) SetBit(mask, 7); // intercontinental airport + if (_cur_year >= 1980) SetBit(mask, 8); // helistation return mask; } diff --git a/src/autoreplace_cmd.cpp b/src/autoreplace_cmd.cpp index e86acca76..60f7b826e 100644 --- a/src/autoreplace_cmd.cpp +++ b/src/autoreplace_cmd.cpp @@ -191,7 +191,7 @@ static CommandCost ReplaceVehicle(Vehicle **w, byte flags, Money total_cost) if (new_v->type == VEH_TRAIN && HasBit(old_v->u.rail.flags, VRF_REVERSE_DIRECTION) && !IsMultiheaded(new_v) && !(new_v->Next() != NULL && IsArticulatedPart(new_v->Next()))) { // we are autorenewing to a single engine, so we will turn it as the old one was turned as well - SETBIT(new_v->u.rail.flags, VRF_REVERSE_DIRECTION); + SetBit(new_v->u.rail.flags, VRF_REVERSE_DIRECTION); } if (old_v->type == VEH_TRAIN && !IsFrontEngine(old_v)) { diff --git a/src/bridge_map.h b/src/bridge_map.h index d381762fb..b7f24ff83 100644 --- a/src/bridge_map.h +++ b/src/bridge_map.h @@ -208,7 +208,7 @@ static inline void ClearBridgeMiddle(TileIndex t) static inline void SetBridgeMiddle(TileIndex t, Axis a) { assert(MayHaveBridgeAbove(t)); - SETBIT(_m[t].m6, 6 + a); + SetBit(_m[t].m6, 6 + a); } /** diff --git a/src/cargotype.cpp b/src/cargotype.cpp index 1b10b680d..7c8cb40de 100644 --- a/src/cargotype.cpp +++ b/src/cargotype.cpp @@ -37,7 +37,7 @@ void SetupCargoForClimate(LandscapeID l) if (cl < lengthof(_default_cargo)) { /* Copy the indexed cargo */ _cargo[i] = _default_cargo[cl]; - SETBIT(_cargo_mask, i); + SetBit(_cargo_mask, i); continue; } @@ -48,7 +48,7 @@ void SetupCargoForClimate(LandscapeID l) _cargo[i] = _default_cargo[j]; /* Populate the available cargo mask */ - SETBIT(_cargo_mask, i); + SetBit(_cargo_mask, i); break; } } diff --git a/src/console.cpp b/src/console.cpp index 5f0a523ea..d4208e256 100644 --- a/src/console.cpp +++ b/src/console.cpp @@ -305,7 +305,7 @@ void IConsoleSwitch() w->height = _screen.height / 3; w->width = _screen.width; _iconsole_mode = ICONSOLE_OPENED; - SETBIT(_no_scroll, SCROLL_CON); // override cursor arrows; the gamefield will not scroll + SetBit(_no_scroll, SCROLL_CON); // override cursor arrows; the gamefield will not scroll } break; case ICONSOLE_OPENED: case ICONSOLE_FULL: DeleteWindowById(WC_CONSOLE, 0); diff --git a/src/economy.cpp b/src/economy.cpp index 63cc2d0db..022ba5e49 100644 --- a/src/economy.cpp +++ b/src/economy.cpp @@ -333,7 +333,7 @@ void ChangeOwnershipOfPlayerItems(PlayerID old_player, PlayerID new_player) // use max of the two ratings. t->ratings[new_player] = max(t->ratings[new_player], t->ratings[old_player]); } else { - SETBIT(t->have_ratings, new_player); + SetBit(t->have_ratings, new_player); t->ratings[new_player] = t->ratings[old_player]; } } @@ -1371,7 +1371,7 @@ static Money DeliverGoods(int num_pieces, CargoID cargo_type, StationID source, { Player *p = GetPlayer(_current_player); p->cur_economy.delivered_cargo += num_pieces; - SETBIT(p->cargo_types, cargo_type); + SetBit(p->cargo_types, cargo_type); } /* Get station pointers. */ @@ -1436,7 +1436,7 @@ void VehiclePayment(Vehicle *front_v) /* All cargo has already been paid for, no need to pay again */ if (!v->cargo.UnpaidCargo()) { - SETBIT(v->vehicle_flags, VF_CARGO_UNLOADING); + SetBit(v->vehicle_flags, VF_CARGO_UNLOADING); continue; } @@ -1460,7 +1460,7 @@ void VehiclePayment(Vehicle *front_v) result |= 1; - SETBIT(v->vehicle_flags, VF_CARGO_UNLOADING); + SetBit(v->vehicle_flags, VF_CARGO_UNLOADING); } else if (front_v->current_order.flags & (OF_UNLOAD | OF_TRANSFER)) { if (!cp->paid_for && (front_v->current_order.flags & OF_TRANSFER) != 0) { Money profit = GetTransportedGoodsIncome( @@ -1477,7 +1477,7 @@ void VehiclePayment(Vehicle *front_v) } result |= 2; - SETBIT(v->vehicle_flags, VF_CARGO_UNLOADING); + SetBit(v->vehicle_flags, VF_CARGO_UNLOADING); } } v->cargo.InvalidateCache(); @@ -1527,7 +1527,7 @@ static void LoadUnloadVehicle(Vehicle *v, int *cargo_left) if (v->type == VEH_TRAIN && !IsTileType(v->tile, MP_STATION)) { /* The train reversed in the station. Take the "easy" way * out and let the train just leave as it always did. */ - SETBIT(v->vehicle_flags, VF_LOADING_FINISHED); + SetBit(v->vehicle_flags, VF_LOADING_FINISHED); return; } @@ -1570,7 +1570,7 @@ static void LoadUnloadVehicle(Vehicle *v, int *cargo_left) result |= 1; } else if (u->current_order.flags & (OF_UNLOAD | OF_TRANSFER)) { remaining = v->cargo.MoveTo(&ge->cargo, amount_unloaded); - SETBIT(ge->acceptance_pickup, GoodsEntry::PICKUP); + SetBit(ge->acceptance_pickup, GoodsEntry::PICKUP); result |= 2; } else { @@ -1621,7 +1621,7 @@ static void LoadUnloadVehicle(Vehicle *v, int *cargo_left) /* Skip loading this vehicle if another train/vehicle is already handling * the same cargo type at this station */ if (_patches.improved_load && cargo_left[v->cargo_type] <= 0) { - SETBIT(cargo_not_full, v->cargo_type); + SetBit(cargo_not_full, v->cargo_type); continue; } @@ -1656,9 +1656,9 @@ static void LoadUnloadVehicle(Vehicle *v, int *cargo_left) } if (v->cargo.Count() == v->cargo_cap) { - SETBIT(cargo_full, v->cargo_type); + SetBit(cargo_full, v->cargo_type); } else { - SETBIT(cargo_not_full, v->cargo_type); + SetBit(cargo_not_full, v->cargo_type); } } diff --git a/src/elrail.cpp b/src/elrail.cpp index b62c23456..ffd567f89 100644 --- a/src/elrail.cpp +++ b/src/elrail.cpp @@ -274,7 +274,7 @@ static void DrawCatenaryRailway(const TileInfo *ti) /* track found, if track is in the neighbour tile, adjust the number * of the PCP for preferred/allowed determination*/ DiagDirection PCPpos = (TrackSourceTile[i][k] == TS_HOME) ? i : ReverseDiagDir(i); - SETBIT(PCPstatus, i); // This PCP is in use + SetBit(PCPstatus, i); // This PCP is in use PPPpreferred[i] &= PreferredPPPofTrackAtPCP[TracksAtPCP[i][k]][PCPpos]; PPPallowed[i] &= ~DisallowedPPPofTrackAtPCP[TracksAtPCP[i][k]][PCPpos]; @@ -322,7 +322,7 @@ static void DrawCatenaryRailway(const TileInfo *ti) if ((height <= GetTileMaxZ(ti->tile) + TILE_HEIGHT) && (i == PCPpositions[bridgetrack][0] || i == PCPpositions[bridgetrack][1])) { - SETBIT(OverridePCP, i); + SetBit(OverridePCP, i); } } @@ -504,7 +504,7 @@ int32 SettingsDisableElrail(int32 p1) * so add there also normal rail compatibility */ v->u.rail.compatible_railtypes |= (1 << RAILTYPE_RAIL); v->u.rail.railtype = RAILTYPE_RAIL; - SETBIT(v->u.rail.flags, VRF_EL_ENGINE_ALLOWED_NORMAL_RAIL); + SetBit(v->u.rail.flags, VRF_EL_ENGINE_ALLOWED_NORMAL_RAIL); } } } diff --git a/src/engine.cpp b/src/engine.cpp index ba9fba42e..9c54efd48 100644 --- a/src/engine.cpp +++ b/src/engine.cpp @@ -183,14 +183,14 @@ static void AcceptEnginePreview(EngineID eid, PlayerID player) Engine *e = GetEngine(eid); Player *p = GetPlayer(player); - SETBIT(e->player_avail, player); + SetBit(e->player_avail, player); if (e->type == VEH_TRAIN) { const RailVehicleInfo *rvi = RailVehInfo(eid); assert(rvi->railtype < RAILTYPE_END); - SETBIT(p->avail_railtypes, rvi->railtype); + SetBit(p->avail_railtypes, rvi->railtype); } else if (e->type == VEH_ROAD) { - SETBIT(p->avail_roadtypes, HasBit(EngInfo(eid)->misc_flags, EF_ROAD_TRAM) ? ROADTYPE_TRAM : ROADTYPE_ROAD); + SetBit(p->avail_roadtypes, HasBit(EngInfo(eid)->misc_flags, EF_ROAD_TRAM) ? ROADTYPE_TRAM : ROADTYPE_ROAD); } e->preview_player = INVALID_PLAYER; @@ -219,7 +219,7 @@ static PlayerID GetBestPlayer(PlayerID pp) if (best_player == PLAYER_SPECTATOR) return PLAYER_SPECTATOR; - SETBIT(mask, best_player); + SetBit(mask, best_player); } while (pp--, pp != 0); return best_player; @@ -332,13 +332,13 @@ static void NewVehicleAvailable(Engine *e) RailType railtype = RailVehInfo(index)->railtype; assert(railtype < RAILTYPE_END); FOR_ALL_PLAYERS(p) { - if (p->is_active) SETBIT(p->avail_railtypes, railtype); + if (p->is_active) SetBit(p->avail_railtypes, railtype); } } if ((index - NUM_TRAIN_ENGINES) < NUM_ROAD_ENGINES) { /* maybe make another road type available */ FOR_ALL_PLAYERS(p) { - if (p->is_active) SETBIT(p->avail_roadtypes, HasBit(EngInfo(index)->misc_flags, EF_ROAD_TRAM) ? ROADTYPE_TRAM : ROADTYPE_ROAD); + if (p->is_active) SetBit(p->avail_roadtypes, HasBit(EngInfo(index)->misc_flags, EF_ROAD_TRAM) ? ROADTYPE_TRAM : ROADTYPE_ROAD); } } AddNewsItem(index, NEWS_FLAGS(NM_CALLBACK, 0, NT_NEW_VEHICLES, DNC_VEHICLEAVAIL), 0, 0); diff --git a/src/graph_gui.cpp b/src/graph_gui.cpp index 4caf34b2d..fb1105e62 100644 --- a/src/graph_gui.cpp +++ b/src/graph_gui.cpp @@ -260,7 +260,7 @@ static void GraphLegendWndProc(Window *w, WindowEvent *e) FOR_ALL_PLAYERS(p) { if (p->is_active) continue; - SETBIT(_legend_excluded_players, p->index); + SetBit(_legend_excluded_players, p->index); RaiseWindowWidget(w, p->index + 3); } @@ -334,7 +334,7 @@ static void SetupGraphDrawerForPlayers(GraphDrawer *gd) /* Exclude the players which aren't valid */ FOR_ALL_PLAYERS(p) { - if (!p->is_active) SETBIT(excluded_players, p->index); + if (!p->is_active) SetBit(excluded_players, p->index); } gd->excluded_data = excluded_players; gd->num_vert_lines = 24; diff --git a/src/helpers.hpp b/src/helpers.hpp index d7834c47d..e873d38ee 100644 --- a/src/helpers.hpp +++ b/src/helpers.hpp @@ -149,11 +149,6 @@ template <typename Tenum_t> struct TinyEnumT } }; -template <typename T> void SetBitT(T &t, int bit_index) -{ - t = (T)(t | ((T)1 << bit_index)); -} - template <typename T> void ToggleBitT(T &t, int bit_index) { t = (T)(t ^ ((T)1 << bit_index)); diff --git a/src/macros.h b/src/macros.h index f2d6d89dd..4ad64c032 100644 --- a/src/macros.h +++ b/src/macros.h @@ -247,9 +247,9 @@ template<typename T> static inline bool HasBit(const T x, const uint8 y) * @param y The bit position to set * @return The new value of the old value with the bit set */ -template<typename T> static inline T SETBIT(T& x, const uint8 y) +template<typename T> static inline T SetBit(T& x, const uint8 y) { - return x |= (T)1U << y; + return x = (T)(x | (T)(1U << y)); } /** diff --git a/src/main_gui.cpp b/src/main_gui.cpp index 5f7c8ea5d..9d55ff35b 100644 --- a/src/main_gui.cpp +++ b/src/main_gui.cpp @@ -971,14 +971,14 @@ static void ToolbarOptionsClick(Window *w) w = PopupMainToolbMenu(w, 2, STR_02C3_GAME_OPTIONS, 14, 0); - if (HasBit(_display_opt, DO_SHOW_TOWN_NAMES)) SETBIT(x, 6); - if (HasBit(_display_opt, DO_SHOW_STATION_NAMES)) SETBIT(x, 7); - if (HasBit(_display_opt, DO_SHOW_SIGNS)) SETBIT(x, 8); - if (HasBit(_display_opt, DO_WAYPOINTS)) SETBIT(x, 9); - if (HasBit(_display_opt, DO_FULL_ANIMATION)) SETBIT(x, 10); - if (HasBit(_display_opt, DO_FULL_DETAIL)) SETBIT(x, 11); - if (IsTransparencySet(TO_HOUSES) && IsTransparencySet(TO_TREES)) SETBIT(x, 12); - if (IsTransparencySet(TO_SIGNS)) SETBIT(x, 13); + if (HasBit(_display_opt, DO_SHOW_TOWN_NAMES)) SetBit(x, 6); + if (HasBit(_display_opt, DO_SHOW_STATION_NAMES)) SetBit(x, 7); + if (HasBit(_display_opt, DO_SHOW_SIGNS)) SetBit(x, 8); + if (HasBit(_display_opt, DO_WAYPOINTS)) SetBit(x, 9); + if (HasBit(_display_opt, DO_FULL_ANIMATION)) SetBit(x, 10); + if (HasBit(_display_opt, DO_FULL_DETAIL)) SetBit(x, 11); + if (IsTransparencySet(TO_HOUSES) && IsTransparencySet(TO_TREES)) SetBit(x, 12); + if (IsTransparencySet(TO_SIGNS)) SetBit(x, 13); WP(w,menu_d).checked_items = x; } diff --git a/src/misc_gui.cpp b/src/misc_gui.cpp index bc8b9384c..26b47bad3 100644 --- a/src/misc_gui.cpp +++ b/src/misc_gui.cpp @@ -1106,7 +1106,7 @@ static void QueryStringWndProc(Window *w, WindowEvent *e) switch (e->event) { case WE_CREATE: - SETBIT(_no_scroll, SCROLL_EDIT); + SetBit(_no_scroll, SCROLL_EDIT); break; case WE_PAINT: @@ -1680,7 +1680,7 @@ void ShowSaveLoadDialog(int mode) DeleteWindowById(WC_SAVELOAD, 0); _saveload_mode = mode; - SETBIT(_no_scroll, SCROLL_SAVE); + SetBit(_no_scroll, SCROLL_SAVE); switch (mode) { case SLD_SAVE_GAME: GenerateFileName(); break; diff --git a/src/network/network_gui.cpp b/src/network/network_gui.cpp index 09f904ebd..5b76331c5 100644 --- a/src/network/network_gui.cpp +++ b/src/network/network_gui.cpp @@ -1686,7 +1686,7 @@ static void ChatWindowWndProc(Window *w, WindowEvent *e) switch (e->event) { case WE_CREATE: SendWindowMessage(WC_NEWS_WINDOW, 0, WE_CREATE, w->height, 0); - SETBIT(_no_scroll, SCROLL_CHAT); // do not scroll the game with the arrow-keys + SetBit(_no_scroll, SCROLL_CHAT); // do not scroll the game with the arrow-keys break; case WE_PAINT: { diff --git a/src/network/network_udp.cpp b/src/network/network_udp.cpp index 5a1980a9d..c12ac21c6 100644 --- a/src/network/network_udp.cpp +++ b/src/network/network_udp.cpp @@ -398,7 +398,7 @@ void ClientNetworkUDPSocketHandler::HandleIncomingNetworkGameInfoGRFConfig(GRFCo config->name = f->name; config->info = f->info; } - SETBIT(config->flags, GCF_COPY); + SetBit(config->flags, GCF_COPY); } // Close UDP connection diff --git a/src/newgrf.cpp b/src/newgrf.cpp index eb462b706..9411649d3 100644 --- a/src/newgrf.cpp +++ b/src/newgrf.cpp @@ -950,7 +950,7 @@ static bool StationChangeInfo(uint stid, int numinfo, int prop, byte **bufp, int if (dts->ground_sprite == 0) continue; if (HasBit(dts->ground_pal, 15)) { ClrBit(dts->ground_pal, 15); - SETBIT(dts->ground_sprite, SPRITE_MODIFIER_USE_OFFSET); + SetBit(dts->ground_sprite, SPRITE_MODIFIER_USE_OFFSET); } while (buf < *bufp + len) { @@ -973,16 +973,16 @@ static bool StationChangeInfo(uint stid, int numinfo, int prop, byte **bufp, int /* Remap flags as ours collide */ if (HasBit(dtss->pal, 15)) { ClrBit(dtss->pal, 15); - SETBIT(dtss->image, SPRITE_MODIFIER_USE_OFFSET); + SetBit(dtss->image, SPRITE_MODIFIER_USE_OFFSET); } if (HasBit(dtss->image, 15)) { ClrBit(dtss->image, 15); - SETBIT(dtss->image, PALETTE_MODIFIER_COLOR); + SetBit(dtss->image, PALETTE_MODIFIER_COLOR); } if (HasBit(dtss->image, 14)) { ClrBit(dtss->image, 14); - SETBIT(dtss->image, PALETTE_MODIFIER_TRANSPARENT); + SetBit(dtss->image, PALETTE_MODIFIER_TRANSPARENT); } } } @@ -1176,7 +1176,7 @@ static bool BridgeChangeInfo(uint brid, int numinfo, int prop, byte **bufp, int SpriteID pal = grf_load_word(&buf); if (HasBit(pal, 15)) { - SETBIT(image, PALETTE_MODIFIER_TRANSPARENT); + SetBit(image, PALETTE_MODIFIER_TRANSPARENT); } /* Clear old color modifer bit */ @@ -1572,7 +1572,7 @@ static bool CargoChangeInfo(uint cid, int numinfo, int prop, byte **bufp, int le cs->bitnum = grf_load_byte(&buf); if (cs->IsValid()) { cs->grfid = _cur_grffile->grfid; - SETBIT(_cargo_mask, cid + i); + SetBit(_cargo_mask, cid + i); } else { ClrBit(_cargo_mask, cid + i); } @@ -1965,7 +1965,7 @@ static bool IndustriesChangeInfo(uint indid, int numinfo, int prop, byte **bufp, } /* Install final layout construction in the industry spec */ indsp->table = tile_table; - SETBIT(indsp->cleanup_flag, 1); + SetBit(indsp->cleanup_flag, 1); free(itt); } break; @@ -2017,7 +2017,7 @@ static bool IndustriesChangeInfo(uint indid, int numinfo, int prop, byte **bufp, for (uint8 j = 0; j < indsp->number_of_sounds; j++) sounds[j] = grf_load_byte(&buf); indsp->random_sounds = sounds; - SETBIT(indsp->cleanup_flag, 0); + SetBit(indsp->cleanup_flag, 0); } break; case 0x16: // Conflicting industry types @@ -2212,7 +2212,7 @@ static void SafeChangeInfo(byte *buf, int len) if (prop == 0x0D) return; } - SETBIT(_cur_grfconfig->flags, GCF_UNSAFE); + SetBit(_cur_grfconfig->flags, GCF_UNSAFE); /* Skip remainder of GRF */ _skip_sprites = -1; @@ -2584,15 +2584,15 @@ static void NewSpriteGroup(byte *buf, int len) /* Remap transparent/colour modifier bits */ if (HasBit(group->g.layout.dts->ground_sprite, 14)) { ClrBit(group->g.layout.dts->ground_sprite, 14); - SETBIT(group->g.layout.dts->ground_sprite, PALETTE_MODIFIER_TRANSPARENT); + SetBit(group->g.layout.dts->ground_sprite, PALETTE_MODIFIER_TRANSPARENT); } if (HasBit(group->g.layout.dts->ground_sprite, 15)) { ClrBit(group->g.layout.dts->ground_sprite, 15); - SETBIT(group->g.layout.dts->ground_sprite, PALETTE_MODIFIER_COLOR); + SetBit(group->g.layout.dts->ground_sprite, PALETTE_MODIFIER_COLOR); } if (HasBit(group->g.layout.dts->ground_pal, 14)) { ClrBit(group->g.layout.dts->ground_pal, 14); - SETBIT(group->g.layout.dts->ground_sprite, SPRITE_MODIFIER_OPAQUE); + SetBit(group->g.layout.dts->ground_sprite, SPRITE_MODIFIER_OPAQUE); } if (HasBit(group->g.layout.dts->ground_pal, 15)) { /* Bit 31 set means this is a custom sprite, so rewrite it to the @@ -2614,15 +2614,15 @@ static void NewSpriteGroup(byte *buf, int len) if (HasBit(seq->image, 14)) { ClrBit(seq->image, 14); - SETBIT(seq->image, PALETTE_MODIFIER_TRANSPARENT); + SetBit(seq->image, PALETTE_MODIFIER_TRANSPARENT); } if (HasBit(seq->image, 15)) { ClrBit(seq->image, 15); - SETBIT(seq->image, PALETTE_MODIFIER_COLOR); + SetBit(seq->image, PALETTE_MODIFIER_COLOR); } if (HasBit(seq->pal, 14)) { ClrBit(seq->pal, 14); - SETBIT(seq->image, SPRITE_MODIFIER_OPAQUE); + SetBit(seq->image, SPRITE_MODIFIER_OPAQUE); } if (HasBit(seq->pal, 15)) { /* Bit 31 set means this is a custom sprite, so rewrite it to the @@ -3417,9 +3417,9 @@ static uint32 GetParamVal(byte param, uint32 *cond_val) case 0x84: { // GRF loading stage uint32 res = 0; - if (_cur_stage > GLS_INIT) SETBIT(res, 0); - if (_cur_stage == GLS_RESERVE) SETBIT(res, 8); - if (_cur_stage == GLS_ACTIVATION) SETBIT(res, 9); + if (_cur_stage > GLS_INIT) SetBit(res, 0); + if (_cur_stage == GLS_RESERVE) SetBit(res, 8); + if (_cur_stage == GLS_ACTIVATION) SetBit(res, 9); return res; } @@ -3734,7 +3734,7 @@ static void ScanInfo(byte *buf, int len) _cur_grfconfig->grfid = grfid; /* GRF IDs starting with 0xFF are reserved for internal TTDPatch use */ - if (GB(grfid, 24, 8) == 0xFF) SETBIT(_cur_grfconfig->flags, GCF_SYSTEM); + if (GB(grfid, 24, 8) == 0xFF) SetBit(_cur_grfconfig->flags, GCF_SYSTEM); len -= 6; const char *name = grf_load_string(&buf, len); @@ -3958,7 +3958,7 @@ static void SafeParamSet(byte *buf, int len) * reserved, it would be marked unsafe anyway. GRM for (e.g. bridge) * sprites is considered safe. */ - SETBIT(_cur_grfconfig->flags, GCF_UNSAFE); + SetBit(_cur_grfconfig->flags, GCF_UNSAFE); /* Skip remainder of GRF */ _skip_sprites = -1; @@ -4313,7 +4313,7 @@ static void SafeGRFInhibit(byte *buf, int len) /* GRF is unsafe it if tries to deactivate other GRFs */ if (grfid != _cur_grfconfig->grfid) { - SETBIT(_cur_grfconfig->flags, GCF_UNSAFE); + SetBit(_cur_grfconfig->flags, GCF_UNSAFE); /* Skip remainder of GRF */ _skip_sprites = -1; @@ -4736,7 +4736,7 @@ static void GRFDataBlock(byte *buf, int len) /* Used during safety scan on unsafe actions */ static void GRFUnsafe(byte *buf, int len) { - SETBIT(_cur_grfconfig->flags, GCF_UNSAFE); + SetBit(_cur_grfconfig->flags, GCF_UNSAFE); /* Skip remainder of GRF */ _skip_sprites = -1; @@ -5181,7 +5181,7 @@ static void CalculateRefitMasks() CargoID c = GetCargoIDByLabel(file->cargo_list[i]); if (c == CT_INVALID) continue; - SETBIT(xor_mask, c); + SetBit(xor_mask, c); } } else { /* No cargo table, so use the cargo bitnum values */ @@ -5189,7 +5189,7 @@ static void CalculateRefitMasks() const CargoSpec *cs = GetCargo(c); if (!cs->IsValid()) continue; - if (HasBit(_engine_info[engine].refit_mask, cs->bitnum)) SETBIT(xor_mask, c); + if (HasBit(_engine_info[engine].refit_mask, cs->bitnum)) SetBit(xor_mask, c); } } } @@ -5198,8 +5198,8 @@ static void CalculateRefitMasks() /* Build up the list of cargo types from the set cargo classes. */ for (CargoID i = 0; i < NUM_CARGO; i++) { const CargoSpec *cs = GetCargo(i); - if (cargo_allowed[engine] & cs->classes) SETBIT(mask, i); - if (cargo_disallowed[engine] & cs->classes) SETBIT(not_mask, i); + if (cargo_allowed[engine] & cs->classes) SetBit(mask, i); + if (cargo_disallowed[engine] & cs->classes) SetBit(not_mask, i); } } else { /* Don't apply default refit mask to wagons or engines with no capacity */ @@ -5216,7 +5216,7 @@ static void CalculateRefitMasks() CargoID cargo = GetCargoIDByLabel(cl[i]); if (cargo == CT_INVALID) continue; - SETBIT(xor_mask, cargo); + SetBit(xor_mask, cargo); } } } @@ -5605,7 +5605,7 @@ void LoadNewGRF(uint load_index, uint file_index) if (stage == GLS_LABELSCAN) InitNewGRFFile(c, _cur_spriteid); LoadNewGRFFile(c, slot++, stage); if (stage == GLS_RESERVE) { - SETBIT(c->flags, GCF_RESERVED); + SetBit(c->flags, GCF_RESERVED); } else if (stage == GLS_ACTIVATION) { ClrBit(c->flags, GCF_RESERVED); ClearTemporaryNewGRFData(); diff --git a/src/newgrf_config.cpp b/src/newgrf_config.cpp index 4d9de7489..8a26ad40f 100644 --- a/src/newgrf_config.cpp +++ b/src/newgrf_config.cpp @@ -137,7 +137,7 @@ GRFConfig **CopyGRFConfigList(GRFConfig **dst, const GRFConfig *src, bool init_o } ClrBit(c->flags, GCF_INIT_ONLY); - if (init_only) SETBIT(c->flags, GCF_INIT_ONLY); + if (init_only) SetBit(c->flags, GCF_INIT_ONLY); *dst = c; dst = &c->next; @@ -234,7 +234,7 @@ GRFListCompatibility IsGoodGRFConfigList() if (f != NULL) { md5sumToString(buf, lastof(buf), c->md5sum); DEBUG(grf, 1, "NewGRF %08X (%s) not found; checksum %s. Compatibility mode on", BSWAP32(c->grfid), c->filename, buf); - SETBIT(c->flags, GCF_COMPATIBLE); + SetBit(c->flags, GCF_COMPATIBLE); /* Non-found has precedence over compatibility load */ if (res != GLC_NOT_FOUND) res = GLC_COMPATIBLE; diff --git a/src/newgrf_engine.cpp b/src/newgrf_engine.cpp index 70ec67d14..3f6f90dfb 100644 --- a/src/newgrf_engine.cpp +++ b/src/newgrf_engine.cpp @@ -660,7 +660,7 @@ static uint32 VehicleGetVariable(const ResolverObject *object, byte variable, by * bit 8: (Maybe?) Toggled whenever the train reverses. */ - if (HasBit(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE)) SETBIT(modflags, 10); + if (HasBit(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE)) SetBit(modflags, 10); return variable == 0xFE ? modflags : GB(modflags, 8, 8); } diff --git a/src/newgrf_industries.cpp b/src/newgrf_industries.cpp index 99d5ed5fd..7bd447a21 100644 --- a/src/newgrf_industries.cpp +++ b/src/newgrf_industries.cpp @@ -158,7 +158,7 @@ static uint32 GetCountAndDistanceOfClosestInstance(byte param_setID, byte layout /* Fall through */ default: //use the grfid specified in register 100h - SETBIT(param_setID, 7); // bit 7 means it is not an old type + SetBit(param_setID, 7); // bit 7 means it is not an old type ind_index = MapNewGRFIndustryType(param_setID, GrfID); break; } diff --git a/src/newgrf_station.cpp b/src/newgrf_station.cpp index 9b8663677..07502a139 100644 --- a/src/newgrf_station.cpp +++ b/src/newgrf_station.cpp @@ -311,10 +311,10 @@ static uint32 GetRailContinuationInfo(TileIndex tile) uint32 ts = GetTileTrackStatus(tile + TileOffsByDir(*dir), TRANSPORT_RAIL, 0); if (ts != 0) { /* If there is any track on the tile, set the bit in the second byte */ - SETBIT(res, i + 8); + SetBit(res, i + 8); /* If any track reaches our exit direction, set the bit in the lower byte */ - if (ts & DiagdirReachesTracks(*diagdir)) SETBIT(res, i); + if (ts & DiagdirReachesTracks(*diagdir)) SetBit(res, i); } } @@ -402,26 +402,26 @@ static uint32 StationGetVariable(const ResolverObject *object, byte variable, by switch (variable) { /* Calculated station variables */ case 0x40: - if (!HasBit(_svc.valid, 0)) { _svc.v40 = GetPlatformInfoHelper(tile, false, false, false); SETBIT(_svc.valid, 0); } + if (!HasBit(_svc.valid, 0)) { _svc.v40 = GetPlatformInfoHelper(tile, false, false, false); SetBit(_svc.valid, 0); } return _svc.v40; case 0x41: - if (!HasBit(_svc.valid, 1)) { _svc.v41 = GetPlatformInfoHelper(tile, true, false, false); SETBIT(_svc.valid, 1); } + if (!HasBit(_svc.valid, 1)) { _svc.v41 = GetPlatformInfoHelper(tile, true, false, false); SetBit(_svc.valid, 1); } return _svc.v41; case 0x42: return GetTerrainType(tile) | (GetRailType(tile) << 8); case 0x43: return st->owner; // Station owner case 0x44: return 2; // PBS status case 0x45: - if (!HasBit(_svc.valid, 2)) { _svc.v45 = GetRailContinuationInfo(tile); SETBIT(_svc.valid, 2); } + if (!HasBit(_svc.valid, 2)) { _svc.v45 = GetRailContinuationInfo(tile); SetBit(_svc.valid, 2); } return _svc.v45; case 0x46: - if (!HasBit(_svc.valid, 3)) { _svc.v46 = GetPlatformInfoHelper(tile, false, false, true); SETBIT(_svc.valid, 3); } + if (!HasBit(_svc.valid, 3)) { _svc.v46 = GetPlatformInfoHelper(tile, false, false, true); SetBit(_svc.valid, 3); } return _svc.v46; case 0x47: - if (!HasBit(_svc.valid, 4)) { _svc.v47 = GetPlatformInfoHelper(tile, true, false, true); SETBIT(_svc.valid, 4); } + if (!HasBit(_svc.valid, 4)) { _svc.v47 = GetPlatformInfoHelper(tile, true, false, true); SetBit(_svc.valid, 4); } return _svc.v47; case 0x48: { // Accepted cargo types @@ -429,12 +429,12 @@ static uint32 StationGetVariable(const ResolverObject *object, byte variable, by uint32 value = 0; for (cargo_type = 0; cargo_type < NUM_CARGO; cargo_type++) { - if (HasBit(st->goods[cargo_type].acceptance_pickup, GoodsEntry::PICKUP)) SETBIT(value, cargo_type); + if (HasBit(st->goods[cargo_type].acceptance_pickup, GoodsEntry::PICKUP)) SetBit(value, cargo_type); } return value; } case 0x49: - if (!HasBit(_svc.valid, 5)) { _svc.v49 = GetPlatformInfoHelper(tile, false, true, false); SETBIT(_svc.valid, 5); } + if (!HasBit(_svc.valid, 5)) { _svc.v49 = GetPlatformInfoHelper(tile, false, true, false); SetBit(_svc.valid, 5); } return _svc.v49; /* Variables which use the parameter */ @@ -121,7 +121,7 @@ static inline bool NPFGetFlag(const AyStarNode* node, NPFNodeFlag flag) static inline void NPFSetFlag(AyStarNode* node, NPFNodeFlag flag, bool value) { if (value) - SETBIT(node->user_data[NPF_NODE_FLAGS], flag); + SetBit(node->user_data[NPF_NODE_FLAGS], flag); else ClrBit(node->user_data[NPF_NODE_FLAGS], flag); } diff --git a/src/openttd.cpp b/src/openttd.cpp index 7dfc91b6e..27f83900b 100644 --- a/src/openttd.cpp +++ b/src/openttd.cpp @@ -1375,7 +1375,7 @@ bool AfterLoadGame() break; case MP_STATION: { - if (HasBit(_m[t].m6, 3)) SETBIT(_m[t].m6, 2); + if (HasBit(_m[t].m6, 3)) SetBit(_m[t].m6, 2); StationGfx gfx = GetStationGfx(t); StationType st; if ( IS_INT_INSIDE(gfx, 0, 8)) { // Railway station @@ -2146,7 +2146,7 @@ bool AfterLoadGame() Vehicle *v; FOR_ALL_VEHICLES(v) { if (v->type == VEH_ROAD && (v->u.road.state == 250 || v->u.road.state == 251)) { - SETBIT(v->u.road.state, RVS_IS_STOPPING); + SetBit(v->u.road.state, RVS_IS_STOPPING); } } } @@ -2188,7 +2188,7 @@ bool AfterLoadGame() FOR_ALL_STATIONS(st) { for (CargoID c = 0; c < NUM_CARGO; c++) { st->goods[c].last_speed = 0; - if (st->goods[c].cargo.Count() != 0) SETBIT(st->goods[c].acceptance_pickup, GoodsEntry::PICKUP); + if (st->goods[c].cargo.Count() != 0) SetBit(st->goods[c].acceptance_pickup, GoodsEntry::PICKUP); } } } diff --git a/src/player_gui.cpp b/src/player_gui.cpp index 1218f305d..87a3266ac 100644 --- a/src/player_gui.cpp +++ b/src/player_gui.cpp @@ -275,7 +275,7 @@ static void ShowColourDropDownMenu(Window *w, uint32 widget) if (HasBit(WP(w, livery_d).sel, LS_DEFAULT) && widget == 10) { const Player *p; FOR_ALL_PLAYERS(p) { - if (p->is_active && p->index != _local_player) SETBIT(used_colours, p->player_color); + if (p->is_active && p->index != _local_player) SetBit(used_colours, p->player_color); } } diff --git a/src/players.cpp b/src/players.cpp index 4d2f21b26..13aff85bc 100644 --- a/src/players.cpp +++ b/src/players.cpp @@ -90,8 +90,8 @@ PlayerFace ConvertFromOldPlayerFace(uint32 face) PlayerFace pf = 0; GenderEthnicity ge = GE_WM; - if (HasBit(face, 31)) SetBitT(ge, GENDER_FEMALE); - if (HasBit(face, 27) && (HasBit(face, 26) == HasBit(face, 19))) SetBitT(ge, ETHNICITY_BLACK); + if (HasBit(face, 31)) SetBit(ge, GENDER_FEMALE); + if (HasBit(face, 27) && (HasBit(face, 26) == HasBit(face, 19))) SetBit(ge, ETHNICITY_BLACK); SetPlayerFaceBits(pf, PFV_GEN_ETHN, ge, ge); SetPlayerFaceBits(pf, PFV_HAS_GLASSES, ge, GB(face, 28, 3) <= 1); @@ -603,7 +603,7 @@ byte GetPlayerRailtypes(PlayerID p) if (rvi->railveh_type != RAILVEH_WAGON) { assert(rvi->railtype < RAILTYPE_END); - SETBIT(rt, rvi->railtype); + SetBit(rt, rvi->railtype); } } } @@ -622,7 +622,7 @@ byte GetPlayerRoadtypes(PlayerID p) if (e->type == VEH_ROAD && HasBit(ei->climates, _opt.landscape) && (HasBit(e->player_avail, p) || _date >= e->intro_date + 365)) { - SETBIT(rt, HasBit(ei->misc_flags, EF_ROAD_TRAM) ? ROADTYPE_TRAM : ROADTYPE_ROAD); + SetBit(rt, HasBit(ei->misc_flags, EF_ROAD_TRAM) ? ROADTYPE_TRAM : ROADTYPE_ROAD); } } diff --git a/src/rail_cmd.cpp b/src/rail_cmd.cpp index e9422b21d..a25fe4853 100644 --- a/src/rail_cmd.cpp +++ b/src/rail_cmd.cpp @@ -602,7 +602,7 @@ static CommandCost ValidateAutoDrag(Trackdir *trackdir, TileIndex start, TileInd (trdy >= 0 && dy < 0) ) { if (!HasBit(*trackdir, 3)) { // first direction is invalid, try the other - SetBitT(*trackdir, 3); // reverse the direction + SetBit(*trackdir, 3); // reverse the direction trdx = -trdx; trdy = -trdy; } else { // other direction is invalid too, invalid drag @@ -701,7 +701,7 @@ CommandCost CmdBuildRailroadTrack(TileIndex tile, uint32 flags, uint32 p1, uint3 */ CommandCost CmdRemoveRailroadTrack(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) { - return CmdRailTrackHelper(tile, flags, p1, SETBIT(p2, 7)); + return CmdRailTrackHelper(tile, flags, p1, SetBit(p2, 7)); } /** Build a train depot @@ -985,8 +985,8 @@ static CommandCost CmdSignalTrackHelper(TileIndex tile, uint32 flags, uint32 p1, } byte signal_dir = 0; - if (signals & SignalAlongTrackdir(trackdir)) SETBIT(signal_dir, 0); - if (signals & SignalAgainstTrackdir(trackdir)) SETBIT(signal_dir, 1); + if (signals & SignalAlongTrackdir(trackdir)) SetBit(signal_dir, 0); + if (signals & SignalAgainstTrackdir(trackdir)) SetBit(signal_dir, 1); /* signal_ctr - amount of tiles already processed * signals_density - patch setting to put signal on every Nth tile (double space on |, -- tracks) @@ -1121,7 +1121,7 @@ CommandCost CmdRemoveSingleSignal(TileIndex tile, uint32 flags, uint32 p1, uint3 */ CommandCost CmdRemoveSignalTrack(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) { - return CmdSignalTrackHelper(tile, flags, p1, SETBIT(p2, 5)); // bit 5 is remove bit + return CmdSignalTrackHelper(tile, flags, p1, SetBit(p2, 5)); // bit 5 is remove bit } typedef CommandCost DoConvertRailProc(TileIndex tile, RailType totype, bool exec); diff --git a/src/road_gui.cpp b/src/road_gui.cpp index 30dac2391..3c3b821d6 100644 --- a/src/road_gui.cpp +++ b/src/road_gui.cpp @@ -204,7 +204,7 @@ static void PlaceRoadStop(TileIndex tile, uint32 p2, uint32 cmd) uint32 p1 = _road_station_picker_orientation; if (p1 >= DIAGDIR_END) { - SETBIT(p2, 1); // It's a drive-through stop + SetBit(p2, 1); // It's a drive-through stop p1 -= DIAGDIR_END; // Adjust picker result to actual direction } DoCommandP(tile, p1, p2, CcRoadDepot, cmd); diff --git a/src/road_map.h b/src/road_map.h index 99aef132e..b6860a146 100644 --- a/src/road_map.h +++ b/src/road_map.h @@ -200,7 +200,7 @@ static inline void UnbarCrossing(TileIndex t) static inline void BarCrossing(TileIndex t) { assert(GetRoadTileType(t) == ROAD_TILE_CROSSING); - SETBIT(_m[t].m4, 5); + SetBit(_m[t].m4, 5); } static inline bool IsCrossingBarred(TileIndex t) diff --git a/src/roadveh_cmd.cpp b/src/roadveh_cmd.cpp index 93ab47985..dd4b3f547 100644 --- a/src/roadveh_cmd.cpp +++ b/src/roadveh_cmd.cpp @@ -257,7 +257,7 @@ CommandCost CmdBuildRoadVeh(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) v->u.road.cached_veh_length = GetRoadVehLength(v); v->vehicle_flags = 0; - if (e->flags & ENGINE_EXCLUSIVE_PREVIEW) SETBIT(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE); + if (e->flags & ENGINE_EXCLUSIVE_PREVIEW) SetBit(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE); v->cargo_cap = GetVehicleProperty(v, 0x0F, rvi->capacity); @@ -511,7 +511,7 @@ CommandCost CmdSendRoadVehToDepot(TileIndex tile, uint32 flags, uint32 p1, uint3 ClearSlot(v); v->current_order.type = OT_GOTO_DEPOT; v->current_order.flags = OF_NON_STOP; - if (!(p2 & DEPOT_SERVICE)) SETBIT(v->current_order.flags, OFB_HALT_IN_DEPOT); + if (!(p2 & DEPOT_SERVICE)) SetBit(v->current_order.flags, OFB_HALT_IN_DEPOT); v->current_order.refit_cargo = CT_INVALID; v->current_order.dest = dep->index; v->dest_tile = dep->xy; diff --git a/src/settings.cpp b/src/settings.cpp index aab88256e..853719a71 100644 --- a/src/settings.cpp +++ b/src/settings.cpp @@ -435,7 +435,7 @@ static uint32 lookup_manyofmany(const char *many, const char *str) r = lookup_oneofmany(many, str, s - str); if (r == -1) return (uint32)-1; - SETBIT(res, r); // value found, set it + SetBit(res, r); // value found, set it if (*s == 0) break; str = s + 1; } @@ -1684,7 +1684,7 @@ static GRFConfig *GRFLoadConfig(IniFile *ini, const char *grpname, bool is_stati } /* Mark file as static to avoid saving in savegame. */ - if (is_static) SETBIT(c->flags, GCF_STATIC); + if (is_static) SetBit(c->flags, GCF_STATIC); /* Add item to list */ *curr = c; diff --git a/src/settings_gui.cpp b/src/settings_gui.cpp index abe7cc189..40ec04426 100644 --- a/src/settings_gui.cpp +++ b/src/settings_gui.cpp @@ -537,12 +537,12 @@ static void GameDifficultyWndProc(Window *w, WindowEvent *e) if (x >= 10) { // Increase button clicked val = min(val + info->step, info->max); - SETBIT(_difficulty_click_b, btn); + SetBit(_difficulty_click_b, btn); } else { // Decrease button clicked val -= info->step; val = max(val, info->min); - SETBIT(_difficulty_click_a, btn); + SetBit(_difficulty_click_a, btn); } // save value in temporary variable diff --git a/src/ship_cmd.cpp b/src/ship_cmd.cpp index 5cae48f59..6983bef2f 100644 --- a/src/ship_cmd.cpp +++ b/src/ship_cmd.cpp @@ -875,7 +875,7 @@ CommandCost CmdBuildShip(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) v->random_bits = VehicleRandomBits(); v->vehicle_flags = 0; - if (e->flags & ENGINE_EXCLUSIVE_PREVIEW) SETBIT(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE); + if (e->flags & ENGINE_EXCLUSIVE_PREVIEW) SetBit(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE); v->cargo_cap = GetVehicleProperty(v, 0x0D, svi->capacity); @@ -1039,7 +1039,7 @@ CommandCost CmdSendShipToDepot(TileIndex tile, uint32 flags, uint32 p1, uint32 p v->dest_tile = dep->xy; v->current_order.type = OT_GOTO_DEPOT; v->current_order.flags = OF_NON_STOP; - if (!(p2 & DEPOT_SERVICE)) SETBIT(v->current_order.flags, OFB_HALT_IN_DEPOT); + if (!(p2 & DEPOT_SERVICE)) SetBit(v->current_order.flags, OFB_HALT_IN_DEPOT); v->current_order.refit_cargo = CT_INVALID; v->current_order.dest = dep->index; InvalidateWindowWidget(WC_VEHICLE_VIEW, v->index, STATUS_BAR); diff --git a/src/signs_gui.cpp b/src/signs_gui.cpp index 79eca1899..b792ad488 100644 --- a/src/signs_gui.cpp +++ b/src/signs_gui.cpp @@ -201,7 +201,7 @@ static void QuerySignEditWndProc(Window *w, WindowEvent *e) switch (e->event) { case WE_CREATE: - SETBIT(_no_scroll, SCROLL_EDIT); + SetBit(_no_scroll, SCROLL_EDIT); break; case WE_PAINT: diff --git a/src/station.cpp b/src/station.cpp index 4eb56a776..e25d61252 100644 --- a/src/station.cpp +++ b/src/station.cpp @@ -476,7 +476,7 @@ void RoadStop::AllocateDriveThroughBay(uint nr) void RoadStop::FreeBay(uint nr) { assert(nr < MAX_BAY_COUNT); - SETBIT(status, nr); + SetBit(status, nr); } diff --git a/src/station_cmd.cpp b/src/station_cmd.cpp index 8e63b1a89..6f3eb7e11 100644 --- a/src/station_cmd.cpp +++ b/src/station_cmd.cpp @@ -990,7 +990,7 @@ CommandCost CmdBuildRailroadStation(TileIndex tile_org, uint32 flags, uint32 p1, if (!GenerateStationName(st, tile_org, STATIONNAMING_RAIL)) return CMD_ERROR; if (IsValidPlayer(_current_player) && (flags & DC_EXEC) != 0) { - SETBIT(st->town->have_ratings, _current_player); + SetBit(st->town->have_ratings, _current_player); } } @@ -1417,7 +1417,7 @@ CommandCost CmdBuildRoadStop(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) if (!GenerateStationName(st, tile, STATIONNAMING_ROAD)) return CMD_ERROR; if (IsValidPlayer(_current_player) && (flags & DC_EXEC) != 0) { - SETBIT(t->have_ratings, _current_player); + SetBit(t->have_ratings, _current_player); } st->sign.width_1 = 0; @@ -1715,7 +1715,7 @@ CommandCost CmdBuildAirport(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) st->town = t; if (IsValidPlayer(_current_player) && (flags & DC_EXEC) != 0) { - SETBIT(t->have_ratings, _current_player); + SetBit(t->have_ratings, _current_player); } st->sign.width_1 = 0; @@ -2002,7 +2002,7 @@ CommandCost CmdBuildDock(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) Town *t = st->town = ClosestTownFromTile(tile, (uint)-1); if (IsValidPlayer(_current_player) && (flags & DC_EXEC) != 0) { - SETBIT(t->have_ratings, _current_player); + SetBit(t->have_ratings, _current_player); } st->sign.width_1 = 0; @@ -2386,14 +2386,14 @@ static uint32 VehicleEnter_Station(Vehicle *v, TileIndex tile, int x, int y) /* Check if the vehicle is stopping at this road stop */ if (GetRoadStopType(tile) == (IsCargoInClass(v->cargo_type, CC_PASSENGERS) ? RoadStop::BUS : RoadStop::TRUCK) && v->current_order.dest == GetStationIndex(tile)) { - SETBIT(v->u.road.state, RVS_IS_STOPPING); + SetBit(v->u.road.state, RVS_IS_STOPPING); rs->AllocateDriveThroughBay(side); } /* Indicate if vehicle is using second bay. */ - if (side == 1) SETBIT(v->u.road.state, RVS_USING_SECOND_BAY); + if (side == 1) SetBit(v->u.road.state, RVS_USING_SECOND_BAY); /* Indicate a drive-through stop */ - SETBIT(v->u.road.state, RVS_IN_DT_ROAD_STOP); + SetBit(v->u.road.state, RVS_IN_DT_ROAD_STOP); return VETSB_CONTINUE; } @@ -2401,7 +2401,7 @@ static uint32 VehicleEnter_Station(Vehicle *v, TileIndex tile, int x, int y) /* Check if station is busy or if there are no free bays or whether it is a articulated vehicle. */ if (rs->IsEntranceBusy() || !rs->HasFreeBay() || RoadVehHasArticPart(v)) return VETSB_CANNOT_ENTER; - SETBIT(v->u.road.state, RVS_IN_ROAD_STOP); + SetBit(v->u.road.state, RVS_IN_ROAD_STOP); /* Allocate a bay and update the road state */ uint bay_nr = rs->AllocateBay(); @@ -2587,7 +2587,7 @@ void ModifyStationRatingAround(TileIndex tile, PlayerID owner, int amount, uint static void UpdateStationWaiting(Station *st, CargoID type, uint amount) { st->goods[type].cargo.Append(new CargoPacket(st->index, amount)); - SETBIT(st->goods[type].acceptance_pickup, GoodsEntry::PICKUP); + SetBit(st->goods[type].acceptance_pickup, GoodsEntry::PICKUP); InvalidateWindow(WC_STATION_VIEW, st->index); st->MarkTilesDirty(true); diff --git a/src/station_gui.cpp b/src/station_gui.cpp index 023cc400a..e92529375 100644 --- a/src/station_gui.cpp +++ b/src/station_gui.cpp @@ -406,7 +406,7 @@ static void PlayerStationsWndProc(Window *w, WindowEvent *e) for (uint i = 0; facilities != 0; i++, facilities >>= 1) { if (HasBit(facilities, 0)) RaiseWindowWidget(w, i + STATIONLIST_WIDGET_TRAIN); } - SETBIT(facilities, e->we.click.widget - STATIONLIST_WIDGET_TRAIN); + SetBit(facilities, e->we.click.widget - STATIONLIST_WIDGET_TRAIN); LowerWindowWidget(w, e->we.click.widget); } SetWindowWidgetLoweredState(w, STATIONLIST_WIDGET_FACILALL, facilities == (FACIL_TRAIN | FACIL_TRUCK_STOP | FACIL_BUS_STOP | FACIL_AIRPORT | FACIL_DOCK)); @@ -498,7 +498,7 @@ static void PlayerStationsWndProc(Window *w, WindowEvent *e) _cargo_filter = 0; include_empty = false; - SETBIT(_cargo_filter, c); + SetBit(_cargo_filter, c); LowerWindowWidget(w, e->we.click.widget); } sl->flags |= SL_REBUILD; diff --git a/src/timetable_cmd.cpp b/src/timetable_cmd.cpp index ceb82bd26..2678186f5 100644 --- a/src/timetable_cmd.cpp +++ b/src/timetable_cmd.cpp @@ -121,7 +121,7 @@ CommandCost CmdAutofillTimetable(TileIndex tile, uint32 flags, uint32 p1, uint32 if (p2 == 1) { /* Start autofilling the timetable, which clears all the current * timings and clears the "timetable has started" bit. */ - SETBIT(v->vehicle_flags, VF_AUTOFILL_TIMETABLE); + SetBit(v->vehicle_flags, VF_AUTOFILL_TIMETABLE); ClrBit(v->vehicle_flags, VF_TIMETABLE_STARTED); for (Order *order = GetVehicleOrder(v, 0); order != NULL; order = order->next) { @@ -155,7 +155,7 @@ void UpdateVehicleTimetable(Vehicle *v, bool travelling) /* Make sure the timetable only starts when the vehicle reaches the first * order, not when travelling from the depot to the first station. */ if (v->cur_order_index == 0 && !HasBit(v->vehicle_flags, VF_TIMETABLE_STARTED)) { - SETBIT(v->vehicle_flags, VF_TIMETABLE_STARTED); + SetBit(v->vehicle_flags, VF_TIMETABLE_STARTED); return; } diff --git a/src/town_cmd.cpp b/src/town_cmd.cpp index 76677f9e2..6c4f6a94a 100644 --- a/src/town_cmd.cpp +++ b/src/town_cmd.cpp @@ -1725,9 +1725,9 @@ static void DoBuildTownHouse(Town *t, TileIndex tile) /* Special houses that there can be only one of. */ if (hs->building_flags & BUILDING_IS_CHURCH) { - SETBIT(oneof, TOWN_HAS_CHURCH); + SetBit(oneof, TOWN_HAS_CHURCH); } else if (hs->building_flags & BUILDING_IS_STADIUM) { - SETBIT(oneof, TOWN_HAS_STADIUM); + SetBit(oneof, TOWN_HAS_STADIUM); } else { oneof = 0; } @@ -2012,7 +2012,7 @@ static void TownActionBuildStatue(Town* t) TileIndex tile = t->xy; if (CircularTileSearch(tile, 9, SearchTileForStatue, t->index)) - SETBIT(t->statues, _current_player); // Once found and built, "inform" the Town + SetBit(t->statues, _current_player); // Once found and built, "inform" the Town } static void TownActionFundBuildings(Town* t) @@ -2020,7 +2020,7 @@ static void TownActionFundBuildings(Town* t) /* Build next tick */ t->grow_counter = 1; /* If we were not already growing */ - SETBIT(t->flags12, TOWN_IS_FUNDED); + SetBit(t->flags12, TOWN_IS_FUNDED); /* And grow for 3 months */ t->fund_buildings_months = 3; } @@ -2177,7 +2177,7 @@ static void UpdateTownGrowRate(Town *t) if (m <= t->grow_counter) t->grow_counter = m; - SETBIT(t->flags12, TOWN_IS_FUNDED); + SetBit(t->flags12, TOWN_IS_FUNDED); } static void UpdateTownAmounts(Town *t) @@ -2267,7 +2267,7 @@ void ChangeTownRating(Town *t, int add, int max) return; } - SETBIT(t->have_ratings, _current_player); + SetBit(t->have_ratings, _current_player); rating = t->ratings[_current_player]; diff --git a/src/town_map.h b/src/town_map.h index 71c6dbaa6..20d0f9950 100644 --- a/src/town_map.h +++ b/src/town_map.h @@ -88,7 +88,7 @@ static inline bool LiftHasDestination(TileIndex t) */ static inline void SetLiftDestination(TileIndex t, byte dest) { - SETBIT(_me[t].m7, 0); + SetBit(_me[t].m7, 0); SB(_me[t].m7, 1, 3, dest); } diff --git a/src/train.h b/src/train.h index 43fc7b9ae..ad31a4475 100644 --- a/src/train.h +++ b/src/train.h @@ -41,7 +41,7 @@ static inline bool IsFrontEngine(const Vehicle *v) static inline void SetFrontEngine(Vehicle *v) { assert(v->type == VEH_TRAIN); - SETBIT(v->subtype, Train_Front); + SetBit(v->subtype, Train_Front); } /** Remove the front engine state @@ -69,7 +69,7 @@ static inline bool IsArticulatedPart(const Vehicle *v) static inline void SetArticulatedPart(Vehicle *v) { assert(v->type == VEH_TRAIN); - SETBIT(v->subtype, Train_Articulated_Part); + SetBit(v->subtype, Train_Articulated_Part); } /** Clear a vehicle from being an articulated part @@ -97,7 +97,7 @@ static inline bool IsTrainWagon(const Vehicle *v) static inline void SetTrainWagon(Vehicle *v) { assert(v->type == VEH_TRAIN); - SETBIT(v->subtype, Train_Wagon); + SetBit(v->subtype, Train_Wagon); } /** Clear wagon property @@ -125,7 +125,7 @@ static inline bool IsTrainEngine(const Vehicle *v) static inline void SetTrainEngine(Vehicle *v) { assert(v->type == VEH_TRAIN); - SETBIT(v->subtype, Train_Engine); + SetBit(v->subtype, Train_Engine); } /** Clear engine status @@ -153,7 +153,7 @@ static inline bool IsFreeWagon(const Vehicle *v) static inline void SetFreeWagon(Vehicle *v) { assert(v->type == VEH_TRAIN); - SETBIT(v->subtype, Train_Free_Wagon); + SetBit(v->subtype, Train_Free_Wagon); } /** Clear a vehicle from being a free wagon @@ -181,7 +181,7 @@ static inline bool IsMultiheaded(const Vehicle *v) static inline void SetMultiheaded(Vehicle *v) { assert(v->type == VEH_TRAIN); - SETBIT(v->subtype, Train_Multiheaded); + SetBit(v->subtype, Train_Multiheaded); } /** Clear multiheaded engine property diff --git a/src/train_cmd.cpp b/src/train_cmd.cpp index 99db615fe..18636387e 100644 --- a/src/train_cmd.cpp +++ b/src/train_cmd.cpp @@ -206,7 +206,7 @@ void TrainConsistChanged(Vehicle* v) if (rvi_v->pow_wag_power != 0 && rvi_u->railveh_type == RAILVEH_WAGON && UsesWagonOverride(u) && !HasBit(u->u.rail.cached_vis_effect, 7)) { /* wagon is powered */ - SETBIT(u->u.rail.flags, VRF_POWEREDWAGON); // cache 'powered' status + SetBit(u->u.rail.flags, VRF_POWEREDWAGON); // cache 'powered' status } else { ClrBit(u->u.rail.flags, VRF_POWEREDWAGON); } @@ -747,7 +747,7 @@ CommandCost CmdBuildRailVehicle(TileIndex tile, uint32 flags, uint32 p1, uint32 v->random_bits = VehicleRandomBits(); v->vehicle_flags = 0; - if (e->flags & ENGINE_EXCLUSIVE_PREVIEW) SETBIT(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE); + if (e->flags & ENGINE_EXCLUSIVE_PREVIEW) SetBit(v->vehicle_flags, VF_BUILT_AS_PROTOTYPE); v->group_id = DEFAULT_GROUP; @@ -1465,14 +1465,14 @@ static void SwapTrainFlags(byte *swap_flag1, byte *swap_flag2) /* Reverse the rail-flags (if needed) */ if (HasBit(flag1, VRF_GOINGUP)) { - SETBIT(*swap_flag2, VRF_GOINGDOWN); + SetBit(*swap_flag2, VRF_GOINGDOWN); } else if (HasBit(flag1, VRF_GOINGDOWN)) { - SETBIT(*swap_flag2, VRF_GOINGUP); + SetBit(*swap_flag2, VRF_GOINGUP); } if (HasBit(flag2, VRF_GOINGUP)) { - SETBIT(*swap_flag1, VRF_GOINGDOWN); + SetBit(*swap_flag1, VRF_GOINGDOWN); } else if (HasBit(flag2, VRF_GOINGDOWN)) { - SETBIT(*swap_flag1, VRF_GOINGUP); + SetBit(*swap_flag1, VRF_GOINGUP); } } @@ -1940,7 +1940,7 @@ CommandCost CmdSendTrainToDepot(TileIndex tile, uint32 flags, uint32 p1, uint32 v->dest_tile = tfdd.tile; v->current_order.type = OT_GOTO_DEPOT; v->current_order.flags = OF_NON_STOP; - if (!(p2 & DEPOT_SERVICE)) SETBIT(v->current_order.flags, OFB_HALT_IN_DEPOT); + if (!(p2 & DEPOT_SERVICE)) SetBit(v->current_order.flags, OFB_HALT_IN_DEPOT); v->current_order.dest = GetDepotByTile(tfdd.tile)->index; v->current_order.refit_cargo = CT_INVALID; InvalidateWindowWidget(WC_VEHICLE_VIEW, v->index, STATUS_BAR); @@ -2262,7 +2262,7 @@ static Track ChooseTrainTrack(Vehicle* v, TileIndex tile, DiagDirection enterdir /* PF didn't find the route */ if (!HasBit(v->u.rail.flags, VRF_NO_PATH_TO_DESTINATION)) { /* it is first time the problem occurred, set the "path not found" flag */ - SETBIT(v->u.rail.flags, VRF_NO_PATH_TO_DESTINATION); + SetBit(v->u.rail.flags, VRF_NO_PATH_TO_DESTINATION); /* and notify user about the event */ if (_patches.lost_train_warn && v->owner == _local_player) { SetDParam(0, v->unitnumber); @@ -2570,7 +2570,7 @@ static byte AfterSetTrainPos(Vehicle *v, bool new_tile) /* For some reason tunnel tiles are always given as sloped :( * But they are not sloped... */ if (middle_z != v->z_pos && !IsTunnelTile(TileVirtXY(v->x_pos, v->y_pos))) { - SETBIT(v->u.rail.flags, (middle_z > old_z) ? VRF_GOINGUP : VRF_GOINGDOWN); + SetBit(v->u.rail.flags, (middle_z > old_z) ? VRF_GOINGUP : VRF_GOINGDOWN); } } } @@ -3526,7 +3526,7 @@ void ConvertOldMultiheadToNew() Vehicle *v; FOR_ALL_VEHICLES(v) { if (v->type == VEH_TRAIN) { - SETBIT(v->subtype, 7); // indicates that it's the old format and needs to be converted in the next loop + SetBit(v->subtype, 7); // indicates that it's the old format and needs to be converted in the next loop } } diff --git a/src/tunnelbridge_cmd.cpp b/src/tunnelbridge_cmd.cpp index 3d2b1af2c..e0c1270f3 100644 --- a/src/tunnelbridge_cmd.cpp +++ b/src/tunnelbridge_cmd.cpp @@ -1198,7 +1198,7 @@ void DrawBridgeMiddle(const TileInfo* ti) SpriteID image = psid->sprite; SpriteID pal = psid->pal; if (IsTransparencySet(TO_BRIDGES)) { - SETBIT(image, PALETTE_MODIFIER_TRANSPARENT); + SetBit(image, PALETTE_MODIFIER_TRANSPARENT); pal = PALETTE_TO_TRANSPARENT; } diff --git a/src/vehicle.cpp b/src/vehicle.cpp index 957093552..960298870 100644 --- a/src/vehicle.cpp +++ b/src/vehicle.cpp @@ -1755,7 +1755,7 @@ CommandCost CmdCloneVehicle(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) w = GetVehicle(_new_vehicle_id); if (v->type == VEH_TRAIN && HasBit(v->u.rail.flags, VRF_REVERSE_DIRECTION)) { - SETBIT(w->u.rail.flags, VRF_REVERSE_DIRECTION); + SetBit(w->u.rail.flags, VRF_REVERSE_DIRECTION); } if (v->type == VEH_TRAIN && !IsFrontEngine(v)) { diff --git a/src/viewport.cpp b/src/viewport.cpp index 57f09c926..21ce36560 100644 --- a/src/viewport.cpp +++ b/src/viewport.cpp @@ -179,7 +179,7 @@ void AssignWindowViewport(Window *w, int x, int y, assert(vp != endof(_viewports)); if (vp->width == 0) break; } - SETBIT(_active_viewports, bit); + SetBit(_active_viewports, bit); vp->left = x + w->left; vp->top = y + w->top; @@ -627,7 +627,7 @@ void AddSortableSpriteToDraw(SpriteID image, SpriteID pal, int x, int y, int w, /* make the sprites transparent with the right palette */ if (transparent) { - SETBIT(image, PALETTE_MODIFIER_TRANSPARENT); + SetBit(image, PALETTE_MODIFIER_TRANSPARENT); pal = PALETTE_TO_TRANSPARENT; } @@ -739,7 +739,7 @@ void AddChildSpriteScreen(SpriteID image, SpriteID pal, int x, int y, bool trans /* make the sprites transparent with the right palette */ if (transparent) { - SETBIT(image, PALETTE_MODIFIER_TRANSPARENT); + SetBit(image, PALETTE_MODIFIER_TRANSPARENT); pal = PALETTE_TO_TRANSPARENT; } |