diff --git a/src/players.cpp b/src/players.cpp index 5a06a90aee..0773bb0ab1 100644 --- a/src/players.cpp +++ b/src/players.cpp @@ -676,11 +676,11 @@ CommandCost CmdSetAutoReplace(TileIndex tile, uint32 flags, uint32 p1, uint32 p2 p = GetPlayer(_current_player); switch (GB(p1, 0, 3)) { case 0: - if (p->engine_renew == (bool)GB(p2, 0, 1)) + if (p->engine_renew == HASBIT(p2, 0)) return CMD_ERROR; if (flags & DC_EXEC) { - p->engine_renew = (bool)GB(p2, 0, 1); + p->engine_renew = HASBIT(p2, 0); if (IsLocalPlayer()) { _patches.autorenew = p->engine_renew; InvalidateWindow(WC_GAME_OPTIONS, 0); @@ -749,7 +749,7 @@ CommandCost CmdSetAutoReplace(TileIndex tile, uint32 flags, uint32 p1, uint32 p2 case 4: if (flags & DC_EXEC) { - p->engine_renew = (bool)GB(p1, 15, 1); + p->engine_renew = HASBIT(p1, 15); p->engine_renew_months = (int16)GB(p1, 16, 16); p->engine_renew_money = (uint32)p2; @@ -762,11 +762,11 @@ CommandCost CmdSetAutoReplace(TileIndex tile, uint32 flags, uint32 p1, uint32 p2 } break; case 5: - if (p->renew_keep_length == (bool)GB(p2, 0, 1)) + if (p->renew_keep_length == HASBIT(p2, 0)) return CMD_ERROR; if (flags & DC_EXEC) { - p->renew_keep_length = (bool)GB(p2, 0, 1); + p->renew_keep_length = HASBIT(p2, 0); if (IsLocalPlayer()) { InvalidateWindow(WC_REPLACE_VEHICLE, VEH_TRAIN); } diff --git a/src/station_gui.cpp b/src/station_gui.cpp index bff5fa97c2..d8722adbbd 100644 --- a/src/station_gui.cpp +++ b/src/station_gui.cpp @@ -443,7 +443,7 @@ static void PlayerStationsWndProc(Window *w, WindowEvent *e) case STATIONLIST_WIDGET_SORTBY: /*flip sorting method asc/desc*/ sl->flags ^= SL_ORDER; //DESC-flag - station_sort.order = GB(sl->flags, 0, 1); + station_sort.order = HASBIT(sl->flags, 0); sl->flags |= SL_RESORT; w->flags4 |= 5 << WF_TIMEOUT_SHL; LowerWindowWidget(w, STATIONLIST_WIDGET_SORTBY);