From dda3793eb932eb86ff93632910b7c804caf4144c Mon Sep 17 00:00:00 2001 From: belugas Date: Thu, 8 Nov 2007 16:09:04 +0000 Subject: (svn r11392) -Codechange: Remove some MSVC warnings by removing boolean typecast that can be avoided by using HASBIT instead. Although the code is old (r2817), it is the recent changes made in macros.h that brough it up. --- src/players.cpp | 10 +++++----- src/station_gui.cpp | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/players.cpp b/src/players.cpp index 5a06a90ae..0773bb0ab 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 bff5fa97c..d8722adbb 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); -- cgit v1.2.3-70-g09d2