diff --git a/src/ai/default/default.cpp b/src/ai/default/default.cpp --- a/src/ai/default/default.cpp +++ b/src/ai/default/default.cpp @@ -698,11 +698,11 @@ static void AiWantLongIndustryRoute(Play for (;;) { // look for one from the subsidy list AiFindSubsidyIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 60, 90 + 1)) break; + if (IsInsideMM(fr.distance, 60, 90 + 1)) break; // try a random one AiFindRandomIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 60, 90 + 1)) break; + if (IsInsideMM(fr.distance, 60, 90 + 1)) break; // only test 60 times if (--i == 0) return; @@ -801,11 +801,11 @@ static void AiWantMediumIndustryRoute(Pl for (;;) { // look for one from the subsidy list AiFindSubsidyIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 40, 60 + 1)) break; + if (IsInsideMM(fr.distance, 40, 60 + 1)) break; // try a random one AiFindRandomIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 40, 60 + 1)) break; + if (IsInsideMM(fr.distance, 40, 60 + 1)) break; // only test 60 times if (--i == 0) return; @@ -869,11 +869,11 @@ static void AiWantShortIndustryRoute(Pla for (;;) { // look for one from the subsidy list AiFindSubsidyIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 15, 40 + 1)) break; + if (IsInsideMM(fr.distance, 15, 40 + 1)) break; // try a random one AiFindRandomIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 15, 40 + 1)) break; + if (IsInsideMM(fr.distance, 15, 40 + 1)) break; // only test 60 times if (--i == 0) return; @@ -937,11 +937,11 @@ static void AiWantMailRoute(Player *p) for (;;) { // look for one from the subsidy list AiFindSubsidyPassengerRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 60, 110 + 1)) break; + if (IsInsideMM(fr.distance, 60, 110 + 1)) break; // try a random one AiFindRandomPassengerRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 60, 110 + 1)) break; + if (IsInsideMM(fr.distance, 60, 110 + 1)) break; // only test 60 times if (--i == 0) return; @@ -1038,11 +1038,11 @@ static void AiWantPassengerRoute(Player for (;;) { // look for one from the subsidy list AiFindSubsidyPassengerRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 0, 55 + 1)) break; + if (IsInsideMM(fr.distance, 0, 55 + 1)) break; // try a random one AiFindRandomPassengerRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 0, 55 + 1)) break; + if (IsInsideMM(fr.distance, 0, 55 + 1)) break; // only test 60 times if (--i == 0) return; @@ -1126,11 +1126,11 @@ static void AiWantLongRoadIndustryRoute( for (;;) { // look for one from the subsidy list AiFindSubsidyIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 35, 55 + 1)) break; + if (IsInsideMM(fr.distance, 35, 55 + 1)) break; // try a random one AiFindRandomIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 35, 55 + 1)) break; + if (IsInsideMM(fr.distance, 35, 55 + 1)) break; // only test 60 times if (--i == 0) return; @@ -1182,11 +1182,11 @@ static void AiWantMediumRoadIndustryRout for (;;) { // look for one from the subsidy list AiFindSubsidyIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 15, 40 + 1)) break; + if (IsInsideMM(fr.distance, 15, 40 + 1)) break; // try a random one AiFindRandomIndustryRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 15, 40 + 1)) break; + if (IsInsideMM(fr.distance, 15, 40 + 1)) break; // only test 60 times if (--i == 0) return; @@ -1238,11 +1238,11 @@ static void AiWantLongRoadPassengerRoute for (;;) { // look for one from the subsidy list AiFindSubsidyPassengerRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 55, 180 + 1)) break; + if (IsInsideMM(fr.distance, 55, 180 + 1)) break; // try a random one AiFindRandomPassengerRoute(&fr); - if (IS_INT_INSIDE(fr.distance, 55, 180 + 1)) break; + if (IsInsideMM(fr.distance, 55, 180 + 1)) break; // only test 60 times if (--i == 0) return; @@ -1407,11 +1407,11 @@ static void AiWantPassengerAircraftRoute // look for one from the subsidy list AiFindSubsidyPassengerRoute(&fr); - if (IS_INT_INSIDE(fr.distance, min_squares, max_squares + 1)) break; + if (IsInsideMM(fr.distance, min_squares, max_squares + 1)) break; // try a random one AiFindRandomPassengerRoute(&fr); - if (IS_INT_INSIDE(fr.distance, min_squares, max_squares + 1)) break; + if (IsInsideMM(fr.distance, min_squares, max_squares + 1)) break; // only test 60 times if (--i == 0) return; diff --git a/src/console.cpp b/src/console.cpp --- a/src/console.cpp +++ b/src/console.cpp @@ -356,7 +356,7 @@ static void IConsoleHistoryNavigate(int _iconsole_historypos = i; IConsoleClearCommand(); /* copy history to 'command prompt / bash' */ - assert(_iconsole_history[i] != NULL && IS_INT_INSIDE(i, 0, ICON_HISTORY_SIZE)); + assert(_iconsole_history[i] != NULL && IsInsideMM(i, 0, ICON_HISTORY_SIZE)); ttd_strlcpy(_iconsole_cmdline.buf, _iconsole_history[i], _iconsole_cmdline.maxlength); UpdateTextBufferSize(&_iconsole_cmdline); } diff --git a/src/console_cmds.cpp b/src/console_cmds.cpp --- a/src/console_cmds.cpp +++ b/src/console_cmds.cpp @@ -233,7 +233,7 @@ static const FiosItem* GetFiosItem(const if (file == endptr || *endptr != '\0') i = -1; } - return IS_INT_INSIDE(i, 0, _fios_num) ? &_fios_list[i] : NULL; + return IsInsideMM(i, 0, _fios_num) ? &_fios_list[i] : NULL; } diff --git a/src/core/math_func.hpp b/src/core/math_func.hpp --- a/src/core/math_func.hpp +++ b/src/core/math_func.hpp @@ -82,7 +82,7 @@ static inline uint minu(const uint a, co * @param a The value we want to unsign * @return The unsigned value */ -template static inline T abs(T a) +template static inline T abs(const T a) { return (a < (T)0) ? -a : a; } @@ -191,37 +191,22 @@ template static inline T de * @param size The size of the interval * @return True if the value is in the interval, false else. */ -template static inline bool IS_INSIDE_1D(const T x, const int base, const uint size) +template static inline bool IsInsideBS(const T x, const uint base, const uint size) { return (uint)(x - base) < size; } /** - * Checks if a byte is in an interval. + * Checks if a value is in an interval. * - * Returns true if a byte value is in the interval of [min, max). + * Returns true if a value is in the interval of [min, max). * - * @param a The byte value to check + * @param a The value to check * @param min The minimum of the interval * @param max The maximum of the interval - * @see IS_INSIDE_1D + * @see IsInsideBS() */ -template static inline bool IS_BYTE_INSIDE(const T x, const byte min, const byte max) -{ - return (byte)(x - min) < (max - min); -} - -/** - * Checks if an int is in an interval. - * - * Returns true if a integer value is in the interval of [min, max). - * - * @param a The integer value to check - * @param min The minimum of the interval - * @param max The maximum of the interval - * @see IS_INSIDE_1D - */ -template static inline bool IS_INT_INSIDE(const T x, const int min, const uint max) +template static inline bool IsInsideMM(const T x, const uint min, const uint max) { return (uint)(x - min) < (max - min); } diff --git a/src/economy.cpp b/src/economy.cpp --- a/src/economy.cpp +++ b/src/economy.cpp @@ -365,7 +365,7 @@ void ChangeOwnershipOfPlayerItems(Player } FOR_ALL_VEHICLES(v) { - if (v->owner == old_player && IS_BYTE_INSIDE(v->type, VEH_TRAIN, VEH_AIRCRAFT + 1)) { + if (v->owner == old_player && IsInsideMM(v->type, VEH_TRAIN, VEH_AIRCRAFT + 1)) { if (new_player == PLAYER_SPECTATOR) { DeleteWindowById(WC_VEHICLE_VIEW, v->index); DeleteWindowById(WC_VEHICLE_DETAILS, v->index); diff --git a/src/graph_gui.cpp b/src/graph_gui.cpp --- a/src/graph_gui.cpp +++ b/src/graph_gui.cpp @@ -279,7 +279,7 @@ static void GraphLegendWndProc(Window *w } case WE_CLICK: - if (!IS_INT_INSIDE(e->we.click.widget, 3, 11)) return; + if (!IsInsideMM(e->we.click.widget, 3, 11)) return; ToggleBit(_legend_excluded_players, e->we.click.widget - 3); ToggleWidgetLoweredState(w, e->we.click.widget); @@ -1050,7 +1050,7 @@ static void PerformanceRatingDetailWndPr case WE_CLICK: /* Check which button is clicked */ - if (IS_INT_INSIDE(e->we.click.widget, 13, 21)) { + if (IsInsideMM(e->we.click.widget, 13, 21)) { /* Is it no on disable? */ if (!IsWindowWidgetDisabled(w, e->we.click.widget)) { RaiseWindowWidget(w, _performance_rating_detail_player + 13); diff --git a/src/industry_gui.cpp b/src/industry_gui.cpp --- a/src/industry_gui.cpp +++ b/src/industry_gui.cpp @@ -539,8 +539,8 @@ static void IndustryViewWndProc(Window * if (!IsProductionAlterable(i)) return; x = e->we.click.pt.x; line = (e->we.click.pt.y - WP(w, indview_d).production_offset_y) / 10; - if (e->we.click.pt.y >= WP(w, indview_d).production_offset_y && IS_INT_INSIDE(line, 0, 2) && i->produced_cargo[line] != CT_INVALID) { - if (IS_INT_INSIDE(x, 5, 25) ) { + if (e->we.click.pt.y >= WP(w, indview_d).production_offset_y && IsInsideMM(line, 0, 2) && i->produced_cargo[line] != CT_INVALID) { + if (IsInsideMM(x, 5, 25) ) { /* Clicked buttons, decrease or increase production */ if (x < 15) { if (isProductionMinimum(i, line)) return; @@ -557,7 +557,7 @@ static void IndustryViewWndProc(Window * w->flags4 |= 5 << WF_TIMEOUT_SHL; WP(w, indview_d).clicked_line = line + 1; WP(w, indview_d).clicked_button = (x < 15 ? 1 : 2); - } else if (IS_INT_INSIDE(x, 34, 160)) { + } else if (IsInsideMM(x, 34, 160)) { /* clicked the text */ WP(w, indview_d).editbox_line = line; SetDParam(0, i->production_rate[line] * 8); @@ -844,7 +844,7 @@ static void IndustryDirectoryWndProc(Win int y = (e->we.click.pt.y - 28) / 10; uint16 p; - if (!IS_INT_INSIDE(y, 0, w->vscroll.cap)) return; + if (!IsInsideMM(y, 0, w->vscroll.cap)) return; p = y + w->vscroll.pos; if (p < _num_industry_sort) { ScrollMainWindowToTile(_industry_sort[p]->xy); diff --git a/src/main_gui.cpp b/src/main_gui.cpp --- a/src/main_gui.cpp +++ b/src/main_gui.cpp @@ -1336,7 +1336,7 @@ static void ScenEditLandGenWndProc(Windo HandleButtonClick(w, e->we.click.widget); size += _terraform_size; - if (!IS_INT_INSIDE(size, 1, 8 + 1)) return; + if (!IsInsideMM(size, 1, 8 + 1)) return; _terraform_size = size; SndPlayFx(SND_15_BEEP); diff --git a/src/misc_gui.cpp b/src/misc_gui.cpp --- a/src/misc_gui.cpp +++ b/src/misc_gui.cpp @@ -1917,7 +1917,7 @@ static void CheatsWndProc(Window *w, Win uint x = e->we.click.pt.x; /* not clicking a button? */ - if (!IS_INT_INSIDE(x, 20, 40) || btn >= lengthof(_cheats_ui)) break; + if (!IsInsideMM(x, 20, 40) || btn >= lengthof(_cheats_ui)) break; ce = &_cheats_ui[btn]; oldvalue = value = (int32)ReadValue(ce->variable, ce->type); diff --git a/src/music_gui.cpp b/src/music_gui.cpp --- a/src/music_gui.cpp +++ b/src/music_gui.cpp @@ -247,7 +247,7 @@ static void MusicTrackSelectionWndProc(W byte *p; if (msf.playlist < 4) return; - if (!IS_INT_INSIDE(y, 0, NUM_SONGS_AVAILABLE)) return; + if (!IsInsideMM(y, 0, NUM_SONGS_AVAILABLE)) return; p = _playlists[msf.playlist]; for (i = 0; i != NUM_SONGS_PLAYLIST - 1; i++) { @@ -267,7 +267,7 @@ static void MusicTrackSelectionWndProc(W byte *p; if (msf.playlist < 4) return; - if (!IS_INT_INSIDE(y, 0, NUM_SONGS_AVAILABLE)) return; + if (!IsInsideMM(y, 0, NUM_SONGS_AVAILABLE)) return; p = _playlists[msf.playlist]; for (i = y; i != NUM_SONGS_PLAYLIST - 1; i++) { diff --git a/src/newgrf.cpp b/src/newgrf.cpp --- a/src/newgrf.cpp +++ b/src/newgrf.cpp @@ -2465,7 +2465,7 @@ static void NewSpriteGroup(byte *buf, in /* Link subroutine group */ adjust->subroutine = GetGroupFromGroupID(setid, type, grf_load_byte(&buf)); } else { - adjust->parameter = IS_BYTE_INSIDE(adjust->variable, 0x60, 0x80) ? grf_load_byte(&buf) : 0; + adjust->parameter = IsInsideMM(adjust->variable, 0x60, 0x80) ? grf_load_byte(&buf) : 0; } varadjust = grf_load_byte(&buf); diff --git a/src/oldloader.cpp b/src/oldloader.cpp --- a/src/oldloader.cpp +++ b/src/oldloader.cpp @@ -302,7 +302,7 @@ static void FixOldTowns() /* Convert town-names if needed */ FOR_ALL_TOWNS(town) { - if (IS_INT_INSIDE(town->townnametype, 0x20C1, 0x20C3)) { + if (IsInsideMM(town->townnametype, 0x20C1, 0x20C3)) { town->townnametype = SPECSTR_TOWNNAME_ENGLISH + _opt.town_name; town->townnameparts = GetOldTownName(town->townnameparts, _opt.town_name); } diff --git a/src/openttd.cpp b/src/openttd.cpp --- a/src/openttd.cpp +++ b/src/openttd.cpp @@ -1357,34 +1357,34 @@ bool AfterLoadGame() 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 + if ( IsInsideMM(gfx, 0, 8)) { // Railway station st = STATION_RAIL; SetStationGfx(t, gfx - 0); - } else if (IS_INT_INSIDE(gfx, 8, 67)) { // Airport + } else if (IsInsideMM(gfx, 8, 67)) { // Airport st = STATION_AIRPORT; SetStationGfx(t, gfx - 8); - } else if (IS_INT_INSIDE(gfx, 67, 71)) { // Truck + } else if (IsInsideMM(gfx, 67, 71)) { // Truck st = STATION_TRUCK; SetStationGfx(t, gfx - 67); - } else if (IS_INT_INSIDE(gfx, 71, 75)) { // Bus + } else if (IsInsideMM(gfx, 71, 75)) { // Bus st = STATION_BUS; SetStationGfx(t, gfx - 71); } else if (gfx == 75) { // Oil rig st = STATION_OILRIG; SetStationGfx(t, gfx - 75); - } else if (IS_INT_INSIDE(gfx, 76, 82)) { // Dock + } else if (IsInsideMM(gfx, 76, 82)) { // Dock st = STATION_DOCK; SetStationGfx(t, gfx - 76); } else if (gfx == 82) { // Buoy st = STATION_BUOY; SetStationGfx(t, gfx - 82); - } else if (IS_INT_INSIDE(gfx, 83, 168)) { // Extended airport + } else if (IsInsideMM(gfx, 83, 168)) { // Extended airport st = STATION_AIRPORT; SetStationGfx(t, gfx - 83 + 67 - 8); - } else if (IS_INT_INSIDE(gfx, 168, 170)) { // Drive through truck + } else if (IsInsideMM(gfx, 168, 170)) { // Drive through truck st = STATION_TRUCK; SetStationGfx(t, gfx - 168 + GFX_TRUCK_BUS_DRIVETHROUGH_OFFSET); - } else if (IS_INT_INSIDE(gfx, 170, 172)) { // Drive through bus + } else if (IsInsideMM(gfx, 170, 172)) { // Drive through bus st = STATION_BUS; SetStationGfx(t, gfx - 170 + GFX_TRUCK_BUS_DRIVETHROUGH_OFFSET); } else { diff --git a/src/player.h b/src/player.h --- a/src/player.h +++ b/src/player.h @@ -239,7 +239,7 @@ static inline byte ActivePlayerCount() static inline Player *GetPlayer(PlayerID i) { - assert(IS_INSIDE_1D(i, PLAYER_FIRST, lengthof(_players))); + assert(IsInsideBS(i, PLAYER_FIRST, lengthof(_players))); return &_players[i]; } @@ -250,7 +250,7 @@ static inline bool IsLocalPlayer() static inline bool IsValidPlayer(PlayerID pi) { - return IS_INSIDE_1D(pi, PLAYER_FIRST, MAX_PLAYERS); + return IsInsideBS(pi, PLAYER_FIRST, MAX_PLAYERS); } byte GetPlayerRailtypes(PlayerID p); diff --git a/src/players.cpp b/src/players.cpp --- a/src/players.cpp +++ b/src/players.cpp @@ -299,7 +299,7 @@ static void GenerateCompanyName(Player * t = ClosestTownFromTile(tile, (uint)-1); - if (IS_INT_INSIDE(t->townnametype, SPECSTR_TOWNNAME_START, SPECSTR_TOWNNAME_LAST+1)) { + if (IsInsideMM(t->townnametype, SPECSTR_TOWNNAME_START, SPECSTR_TOWNNAME_LAST+1)) { str = t->townnametype - SPECSTR_TOWNNAME_START + SPECSTR_PLAYERNAME_START; strp = t->townnameparts; diff --git a/src/road_cmd.cpp b/src/road_cmd.cpp --- a/src/road_cmd.cpp +++ b/src/road_cmd.cpp @@ -647,7 +647,7 @@ CommandCost CmdBuildLongRoad(TileIndex e TileIndex t = start_tile; start_tile = end_tile; end_tile = t; - p2 ^= IS_INT_INSIDE(p2 & 3, 1, 3) ? 3 : 0; + p2 ^= IsInsideMM(p2 & 3, 1, 3) ? 3 : 0; drd = DRD_SOUTHBOUND; } @@ -730,7 +730,7 @@ CommandCost CmdRemoveLongRoad(TileIndex TileIndex t = start_tile; start_tile = end_tile; end_tile = t; - p2 ^= IS_INT_INSIDE(p2 & 3, 1, 3) ? 3 : 0; + p2 ^= IsInsideMM(p2 & 3, 1, 3) ? 3 : 0; } money.AddCost(GetAvailableMoneyForCommand()); diff --git a/src/roadveh_cmd.cpp b/src/roadveh_cmd.cpp --- a/src/roadveh_cmd.cpp +++ b/src/roadveh_cmd.cpp @@ -1643,8 +1643,8 @@ again: goto again; } - if (IS_BYTE_INSIDE(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_DT_ROAD_STOP_END) && IsTileType(v->tile, MP_STATION)) { - if (IsReversingRoadTrackdir(dir) && IS_BYTE_INSIDE(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_ROAD_STOP_END)) { + if (IsInsideMM(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_DT_ROAD_STOP_END) && IsTileType(v->tile, MP_STATION)) { + if (IsReversingRoadTrackdir(dir) && IsInsideMM(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_ROAD_STOP_END)) { /* New direction is trying to turn vehicle around. * We can't turn at the exit of a road stop so wait.*/ v->cur_speed = 0; @@ -1766,7 +1766,7 @@ again: new_dir = RoadVehGetSlidingDirection(v, x, y); - if (IsRoadVehFront(v) && !IS_BYTE_INSIDE(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_ROAD_STOP_END)) { + if (IsRoadVehFront(v) && !IsInsideMM(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_ROAD_STOP_END)) { /* Vehicle is not in a road stop. * Check for another vehicle to overtake */ Vehicle* u = RoadVehFindCloseTo(v, x, y, new_dir); @@ -1801,9 +1801,9 @@ again: * and it's the correct type of stop (bus or truck) and the frame equals the stop frame... * (the station test and stop type test ensure that other vehicles, using the road stop as * a through route, do not stop) */ - if (IsRoadVehFront(v) && ((IS_BYTE_INSIDE(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_ROAD_STOP_END) && + if (IsRoadVehFront(v) && ((IsInsideMM(v->u.road.state, RVSB_IN_ROAD_STOP, RVSB_IN_ROAD_STOP_END) && _road_veh_data_1[v->u.road.state - RVSB_IN_ROAD_STOP + (_opt.road_side << RVS_DRIVE_SIDE)] == v->u.road.frame) || - (IS_BYTE_INSIDE(v->u.road.state, RVSB_IN_DT_ROAD_STOP, RVSB_IN_DT_ROAD_STOP_END) && + (IsInsideMM(v->u.road.state, RVSB_IN_DT_ROAD_STOP, RVSB_IN_DT_ROAD_STOP_END) && v->current_order.dest == GetStationIndex(v->tile) && GetRoadStopType(v->tile) == (IsCargoInClass(v->cargo_type, CC_PASSENGERS) ? RoadStop::BUS : RoadStop::TRUCK) && v->u.road.frame == RVC_DRIVE_THROUGH_STOP_FRAME))) { diff --git a/src/settings.cpp b/src/settings.cpp --- a/src/settings.cpp +++ b/src/settings.cpp @@ -1155,15 +1155,15 @@ static int32 CheckInterval(int32 p1) const Patches *ptc = (_game_mode == GM_MENU) ? &_patches_newgame : &_patches; if (p1) { - warning = ( (IS_INT_INSIDE(ptc->servint_trains, 5, 90 + 1) || ptc->servint_trains == 0) && - (IS_INT_INSIDE(ptc->servint_roadveh, 5, 90 + 1) || ptc->servint_roadveh == 0) && - (IS_INT_INSIDE(ptc->servint_aircraft, 5, 90 + 1) || ptc->servint_aircraft == 0) && - (IS_INT_INSIDE(ptc->servint_ships, 5, 90 + 1) || ptc->servint_ships == 0) ); + warning = ( (IsInsideMM(ptc->servint_trains, 5, 90 + 1) || ptc->servint_trains == 0) && + (IsInsideMM(ptc->servint_roadveh, 5, 90 + 1) || ptc->servint_roadveh == 0) && + (IsInsideMM(ptc->servint_aircraft, 5, 90 + 1) || ptc->servint_aircraft == 0) && + (IsInsideMM(ptc->servint_ships, 5, 90 + 1) || ptc->servint_ships == 0) ); } else { - warning = ( (IS_INT_INSIDE(ptc->servint_trains, 30, 800 + 1) || ptc->servint_trains == 0) && - (IS_INT_INSIDE(ptc->servint_roadveh, 30, 800 + 1) || ptc->servint_roadveh == 0) && - (IS_INT_INSIDE(ptc->servint_aircraft, 30, 800 + 1) || ptc->servint_aircraft == 0) && - (IS_INT_INSIDE(ptc->servint_ships, 30, 800 + 1) || ptc->servint_ships == 0) ); + warning = ( (IsInsideMM(ptc->servint_trains, 30, 800 + 1) || ptc->servint_trains == 0) && + (IsInsideMM(ptc->servint_roadveh, 30, 800 + 1) || ptc->servint_roadveh == 0) && + (IsInsideMM(ptc->servint_aircraft, 30, 800 + 1) || ptc->servint_aircraft == 0) && + (IsInsideMM(ptc->servint_ships, 30, 800 + 1) || ptc->servint_ships == 0) ); } if (!warning) diff --git a/src/settings_gui.cpp b/src/settings_gui.cpp --- a/src/settings_gui.cpp +++ b/src/settings_gui.cpp @@ -511,7 +511,7 @@ static void GameDifficultyWndProc(Window return; x = e->we.click.pt.x - 5; - if (!IS_INT_INSIDE(x, 0, 21)) // Button area + if (!IsInsideMM(x, 0, 21)) // Button area return; y = e->we.click.pt.y - GAMEDIFF_WND_TOP_OFFSET; @@ -1101,7 +1101,7 @@ static void CustCurrencyWndProc(Window * switch (line) { case 0: // rate - if (IS_INT_INSIDE(x, 10, 30)) { // clicked buttons + if (IsInsideMM(x, 10, 30)) { // clicked buttons if (x < 20) { if (_custom_currency.rate > 1) _custom_currency.rate--; WP(w,def_d).data_1 = 1 << (line * 2 + 0); @@ -1118,7 +1118,7 @@ static void CustCurrencyWndProc(Window * break; case 1: // separator - if (IS_INT_INSIDE(x, 10, 30)) { // clicked button + if (IsInsideMM(x, 10, 30)) { // clicked button WP(w,def_d).data_1 = 1 << (line * 2 + 1); } str = BindCString(_str_separator); @@ -1126,7 +1126,7 @@ static void CustCurrencyWndProc(Window * break; case 2: // prefix - if (IS_INT_INSIDE(x, 10, 30)) { // clicked button + if (IsInsideMM(x, 10, 30)) { // clicked button WP(w,def_d).data_1 = 1 << (line * 2 + 1); } str = BindCString(_custom_currency.prefix); @@ -1134,7 +1134,7 @@ static void CustCurrencyWndProc(Window * break; case 3: // suffix - if (IS_INT_INSIDE(x, 10, 30)) { // clicked button + if (IsInsideMM(x, 10, 30)) { // clicked button WP(w,def_d).data_1 = 1 << (line * 2 + 1); } str = BindCString(_custom_currency.suffix); @@ -1142,7 +1142,7 @@ static void CustCurrencyWndProc(Window * break; case 4: // to euro - if (IS_INT_INSIDE(x, 10, 30)) { // clicked buttons + if (IsInsideMM(x, 10, 30)) { // clicked buttons if (x < 20) { _custom_currency.to_euro = (_custom_currency.to_euro <= 2000) ? CF_NOEURO : _custom_currency.to_euro - 1; diff --git a/src/slope.h b/src/slope.h --- a/src/slope.h +++ b/src/slope.h @@ -75,7 +75,7 @@ enum Slope { */ static inline bool IsValidCorner(Corner corner) { - return IS_INT_INSIDE(corner, 0, CORNER_END); + return IsInsideMM(corner, 0, CORNER_END); } /** @@ -294,7 +294,7 @@ static inline bool IsInclinedFoundation( */ static inline bool IsNonContinuousFoundation(Foundation f) { - return IS_INT_INSIDE(f, FOUNDATION_STEEP_BOTH, FOUNDATION_HALFTILE_N + 1); + return IsInsideMM(f, FOUNDATION_STEEP_BOTH, FOUNDATION_HALFTILE_N + 1); } /** @@ -307,7 +307,7 @@ static inline bool IsNonContinuousFounda */ static inline Corner GetHalftileFoundationCorner(Foundation f) { - assert(IS_INT_INSIDE(f, FOUNDATION_HALFTILE_W, FOUNDATION_HALFTILE_N + 1)); + assert(IsInsideMM(f, FOUNDATION_HALFTILE_W, FOUNDATION_HALFTILE_N + 1)); return (Corner)(f - FOUNDATION_HALFTILE_W); } @@ -319,7 +319,7 @@ static inline Corner GetHalftileFoundati */ static inline bool IsSpecialRailFoundation(Foundation f) { - return IS_INT_INSIDE(f, FOUNDATION_RAIL_W, FOUNDATION_RAIL_N + 1); + return IsInsideMM(f, FOUNDATION_RAIL_W, FOUNDATION_RAIL_N + 1); } /** diff --git a/src/smallmap_gui.cpp b/src/smallmap_gui.cpp --- a/src/smallmap_gui.cpp +++ b/src/smallmap_gui.cpp @@ -697,7 +697,7 @@ skip_column: /* Check if y is out of bounds? */ y -= dpi->top; - if (!IS_INT_INSIDE(y, 0, dpi->height)) continue; + if (!IsInsideMM(y, 0, dpi->height)) continue; /* Default is to draw both pixels. */ skip = false; diff --git a/src/sound.cpp b/src/sound.cpp --- a/src/sound.cpp +++ b/src/sound.cpp @@ -209,8 +209,8 @@ static void SndPlayScreenCoordFx(SoundFx const ViewPort *vp = (*wz)->viewport; if (vp != NULL && - IS_INSIDE_1D(x, vp->virtual_left, vp->virtual_width) && - IS_INSIDE_1D(y, vp->virtual_top, vp->virtual_height)) { + IsInsideBS(x, vp->virtual_left, vp->virtual_width) && + IsInsideBS(y, vp->virtual_top, vp->virtual_height)) { int left = (x - vp->virtual_left); StartSound( diff --git a/src/station_cmd.cpp b/src/station_cmd.cpp --- a/src/station_cmd.cpp +++ b/src/station_cmd.cpp @@ -465,7 +465,7 @@ void GetProductionAroundTiles(AcceptedCa for (int yc = y1; yc != y2; yc++) { for (int xc = x1; xc != x2; xc++) { - if (!(IS_INSIDE_1D(xc, x, w) && IS_INSIDE_1D(yc, y, h))) { + if (!(IsInsideBS(xc, x, w) && IsInsideBS(yc, y, h))) { TileIndex tile = TileXY(xc, yc); GetProducedCargoProc *gpc = _tile_type_procs[GetTileType(tile)]->get_produced_cargo_proc; diff --git a/src/strings.cpp b/src/strings.cpp --- a/src/strings.cpp +++ b/src/strings.cpp @@ -1120,20 +1120,20 @@ static char *GetSpecialPlayerNameString( } /* town name? */ - if (IS_INT_INSIDE(ind - 6, 0, SPECSTR_TOWNNAME_LAST-SPECSTR_TOWNNAME_START + 1)) { + if (IsInsideMM(ind - 6, 0, SPECSTR_TOWNNAME_LAST-SPECSTR_TOWNNAME_START + 1)) { buff = GetSpecialTownNameString(buff, ind - 6, GetInt32(&argv), last); return strecpy(buff, " Transport", last); } /* language name? */ - if (IS_INT_INSIDE(ind, (SPECSTR_LANGUAGE_START - 0x70E4), (SPECSTR_LANGUAGE_END - 0x70E4) + 1)) { + if (IsInsideMM(ind, (SPECSTR_LANGUAGE_START - 0x70E4), (SPECSTR_LANGUAGE_END - 0x70E4) + 1)) { int i = ind - (SPECSTR_LANGUAGE_START - 0x70E4); return strecpy(buff, i == _dynlang.curr ? _langpack->own_name : _dynlang.ent[i].name, last); } /* resolution size? */ - if (IS_INT_INSIDE(ind, (SPECSTR_RESOLUTION_START - 0x70E4), (SPECSTR_RESOLUTION_END - 0x70E4) + 1)) { + if (IsInsideMM(ind, (SPECSTR_RESOLUTION_START - 0x70E4), (SPECSTR_RESOLUTION_END - 0x70E4) + 1)) { int i = ind - (SPECSTR_RESOLUTION_START - 0x70E4); buff += snprintf( buff, last - buff + 1, "%dx%d", _resolutions[i][0], _resolutions[i][1] @@ -1142,7 +1142,7 @@ static char *GetSpecialPlayerNameString( } /* screenshot format name? */ - if (IS_INT_INSIDE(ind, (SPECSTR_SCREENSHOT_START - 0x70E4), (SPECSTR_SCREENSHOT_END - 0x70E4) + 1)) { + if (IsInsideMM(ind, (SPECSTR_SCREENSHOT_START - 0x70E4), (SPECSTR_SCREENSHOT_END - 0x70E4) + 1)) { int i = ind - (SPECSTR_SCREENSHOT_START - 0x70E4); return strecpy(buff, GetScreenshotFormatDesc(i), last); } @@ -1168,7 +1168,7 @@ StringID RemapOldStringID(StringID s) case 0xA02F: return STR_SV_AIRCRAFT_NAME; default: - if (IS_INT_INSIDE(s, 0x300F, 0x3030)) { + if (IsInsideMM(s, 0x300F, 0x3030)) { return s - 0x300F + STR_SV_STNAME; } else { return s; diff --git a/src/town_gui.cpp b/src/town_gui.cpp --- a/src/town_gui.cpp +++ b/src/town_gui.cpp @@ -216,7 +216,7 @@ static void TownAuthorityWndProc(Window const Town *t = GetTown(w->window_number); int y = (e->we.click.pt.y - 0x6B) / 10; - if (!IS_INT_INSIDE(y, 0, 5)) return; + if (!IsInsideMM(y, 0, 5)) return; y = GetNthSetBit(GetMaskOfTownActions(NULL, _local_player, t), y + w->vscroll.pos - 1); if (y >= 0) { diff --git a/src/tree_cmd.cpp b/src/tree_cmd.cpp --- a/src/tree_cmd.cpp +++ b/src/tree_cmd.cpp @@ -354,7 +354,7 @@ CommandCost CmdPlantTree(TileIndex tile, } MarkTileDirtyByTile(tile); - if (_game_mode == GM_EDITOR && IS_INT_INSIDE(treetype, TREE_RAINFOREST, TREE_CACTUS)) + if (_game_mode == GM_EDITOR && IsInsideMM(treetype, TREE_RAINFOREST, TREE_CACTUS)) SetTropicZone(tile, TROPICZONE_RAINFOREST); } cost.AddCost(_price.build_trees); @@ -418,7 +418,7 @@ static void DrawTile_Trees(TileInfo *ti) /* different tree styles above one of the grounds */ if (GetTreeGround(ti->tile) == TREE_GROUND_SNOW_DESERT && GetTreeDensity(ti->tile) >= 2 && - IS_INT_INSIDE(index, TREE_SUB_ARCTIC << 2, TREE_RAINFOREST << 2)) { + IsInsideMM(index, TREE_SUB_ARCTIC << 2, TREE_RAINFOREST << 2)) { index += 164 - (TREE_SUB_ARCTIC << 2); } @@ -494,7 +494,7 @@ static CommandCost ClearTile_Trees(TileI } num = GetTreeCount(tile) + 1; - if (IS_INT_INSIDE(GetTreeType(tile), TREE_RAINFOREST, TREE_CACTUS)) num *= 4; + if (IsInsideMM(GetTreeType(tile), TREE_RAINFOREST, TREE_CACTUS)) num *= 4; if (flags & DC_EXEC) DoClearSquare(tile); @@ -510,7 +510,7 @@ static void GetTileDesc_Trees(TileIndex { TreeType tt = GetTreeType(tile); - if (IS_INT_INSIDE(tt, TREE_RAINFOREST, TREE_CACTUS)) { + if (IsInsideMM(tt, TREE_RAINFOREST, TREE_CACTUS)) { td->str = STR_280F_RAINFOREST; } else { td->str = tt == TREE_CACTUS ? STR_2810_CACTUS_PLANTS : STR_280E_TREES; diff --git a/src/unmovable_map.h b/src/unmovable_map.h --- a/src/unmovable_map.h +++ b/src/unmovable_map.h @@ -21,7 +21,7 @@ enum UnmovableType { UNMOVABLE_HQ_EAST = 0x82, ///< Offset for the eastern HQ tile UNMOVABLE_HQ_SOUTH = 0x83, ///< Offset for the southern HQ tile - /** End of the HQ (rather end + 1 for IS_INT_INSIDE) */ + /** End of the HQ (rather end + 1 for IsInside) */ UNMOVABLE_HQ_END = UNMOVABLE_HQ_NORTH + HQ_NUM_SIZE * HQ_NUM_TILE }; @@ -80,7 +80,7 @@ static inline bool IsOwnedLandTile(TileI static inline bool IsCompanyHQ(TileIndex t) { assert(IsTileType(t, MP_UNMOVABLE)); - return IS_INT_INSIDE(GetUnmovableType(t), UNMOVABLE_HQ_NORTH, UNMOVABLE_HQ_END); + return IsInsideMM(GetUnmovableType(t), UNMOVABLE_HQ_NORTH, UNMOVABLE_HQ_END); } /** diff --git a/src/video/dedicated_v.cpp b/src/video/dedicated_v.cpp --- a/src/video/dedicated_v.cpp +++ b/src/video/dedicated_v.cpp @@ -235,7 +235,7 @@ static void DedicatedHandleKeyInput() if (input_line[i] == '\0') break; - if (!IS_INT_INSIDE(input_line[i], ' ', 256)) + if (!IsInsideMM(input_line[i], ' ', 256)) input_line[i] = ' '; } } diff --git a/src/video/sdl_v.cpp b/src/video/sdl_v.cpp --- a/src/video/sdl_v.cpp +++ b/src/video/sdl_v.cpp @@ -128,8 +128,8 @@ static void GetVideoModes() for (i = 0; modes[i]; i++) { int w = modes[i]->w; int h = modes[i]->h; - if (IS_INT_INSIDE(w, 640, MAX_SCREEN_WIDTH + 1) && - IS_INT_INSIDE(h, 480, MAX_SCREEN_HEIGHT + 1)) { + if (IsInsideMM(w, 640, MAX_SCREEN_WIDTH + 1) && + IsInsideMM(h, 480, MAX_SCREEN_HEIGHT + 1)) { int j; for (j = 0; j < n; j++) { if (_resolutions[j][0] == w && _resolutions[j][1] == h) break; diff --git a/src/video/win32_v.cpp b/src/video/win32_v.cpp --- a/src/video/win32_v.cpp +++ b/src/video/win32_v.cpp @@ -737,8 +737,8 @@ static void FindResolutions() * Doesn't really matter since we don't pass a string anyways, but still * a letdown */ for (i = 0; EnumDisplaySettingsA(NULL, i, &dm) != 0; i++) { - if (dm.dmBitsPerPel == BlitterFactoryBase::GetCurrentBlitter()->GetScreenDepth() && IS_INT_INSIDE(dm.dmPelsWidth, 640, MAX_SCREEN_WIDTH + 1) && - IS_INT_INSIDE(dm.dmPelsHeight, 480, MAX_SCREEN_HEIGHT + 1)) { + if (dm.dmBitsPerPel == BlitterFactoryBase::GetCurrentBlitter()->GetScreenDepth() && IsInsideMM(dm.dmPelsWidth, 640, MAX_SCREEN_WIDTH + 1) && + IsInsideMM(dm.dmPelsHeight, 480, MAX_SCREEN_HEIGHT + 1)) { uint j; for (j = 0; j < n; j++) { diff --git a/src/viewport.cpp b/src/viewport.cpp --- a/src/viewport.cpp +++ b/src/viewport.cpp @@ -344,8 +344,8 @@ ViewPort *IsPtInWindowViewport(const Win ViewPort *vp = w->viewport; if (vp != NULL && - IS_INT_INSIDE(x, vp->left, vp->left + vp->width) && - IS_INT_INSIDE(y, vp->top, vp->top + vp->height)) + IsInsideMM(x, vp->left, vp->left + vp->width) && + IsInsideMM(y, vp->top, vp->top + vp->height)) return vp; return NULL; @@ -501,7 +501,7 @@ void DrawGroundSpriteAt(SpriteID image, static void AddChildSpriteToFoundation(SpriteID image, SpriteID pal, const SubSprite *sub, FoundationPart foundation_part, int extra_offs_x, int extra_offs_y) { ViewportDrawer *vd = _cur_vd; - assert(IS_INT_INSIDE(foundation_part, 0, FOUNDATION_PART_END)); + assert(IsInsideMM(foundation_part, 0, FOUNDATION_PART_END)); assert(vd->foundation[foundation_part] != NULL); Point offs = vd->foundation_offset[foundation_part]; @@ -932,8 +932,8 @@ static void DrawTileSelection(const Tile if (_thd.drawstyle == 0) return; /* Inside the inner area? */ - if (IS_INSIDE_1D(ti->x, _thd.pos.x, _thd.size.x) && - IS_INSIDE_1D(ti->y, _thd.pos.y, _thd.size.y)) { + if (IsInsideBS(ti->x, _thd.pos.x, _thd.size.x) && + IsInsideBS(ti->y, _thd.pos.y, _thd.size.y)) { if (_thd.drawstyle & HT_RECT) { DrawTileSelectionRect(ti, _thd.make_square_red ? PALETTE_SEL_TILE_RED : PAL_NONE); } else if (_thd.drawstyle & HT_POINT) { @@ -978,8 +978,8 @@ static void DrawTileSelection(const Tile /* Check if it's inside the outer area? */ if (_thd.outersize.x && _thd.size.x < _thd.size.x + _thd.outersize.x && - IS_INSIDE_1D(ti->x, _thd.pos.x + _thd.offs.x, _thd.size.x + _thd.outersize.x) && - IS_INSIDE_1D(ti->y, _thd.pos.y + _thd.offs.y, _thd.size.y + _thd.outersize.y)) { + IsInsideBS(ti->x, _thd.pos.x + _thd.offs.x, _thd.size.x + _thd.outersize.x) && + IsInsideBS(ti->y, _thd.pos.y + _thd.offs.y, _thd.size.y + _thd.outersize.y)) { /* Draw a blue rect. */ DrawTileSelectionRect(ti, PALETTE_SEL_TILE_BLUE); return; @@ -1754,8 +1754,8 @@ void MarkTileDirty(int x, int y) uint z = 0; Point pt; - if (IS_INT_INSIDE(x, 0, MapSizeX() * TILE_SIZE) && - IS_INT_INSIDE(y, 0, MapSizeY() * TILE_SIZE)) + if (IsInsideMM(x, 0, MapSizeX() * TILE_SIZE) && + IsInsideMM(y, 0, MapSizeY() * TILE_SIZE)) z = GetTileZ(TileVirtXY(x, y)); pt = RemapCoords(x, y, z); diff --git a/src/water_cmd.cpp b/src/water_cmd.cpp --- a/src/water_cmd.cpp +++ b/src/water_cmd.cpp @@ -334,8 +334,8 @@ static CommandCost ClearTile_Water(TileI if (flags & DC_NO_WATER) return_cmd_error(STR_3807_CAN_T_BUILD_ON_WATER); /* Make sure it's not an edge tile. */ - if (!IS_INT_INSIDE(TileX(tile), 1, MapMaxX() - 1) || - !IS_INT_INSIDE(TileY(tile), 1, MapMaxY() - 1)) { + if (!IsInsideMM(TileX(tile), 1, MapMaxX() - 1) || + !IsInsideMM(TileY(tile), 1, MapMaxY() - 1)) { return_cmd_error(STR_0002_TOO_CLOSE_TO_EDGE_OF_MAP); } @@ -771,8 +771,8 @@ void TileLoop_Water(TileIndex tile) if (IsTileType(tile, MP_WATER) && !(IsSea(tile) || IsCoast(tile))) return; /* floods in all four diagonal directions with the exception of the edges */ - if (IS_INT_INSIDE(TileX(tile), 1, MapSizeX() - 3 + 1) && - IS_INT_INSIDE(TileY(tile), 1, MapSizeY() - 3 + 1)) { + if (IsInsideMM(TileX(tile), 1, MapSizeX() - 3 + 1) && + IsInsideMM(TileY(tile), 1, MapSizeY() - 3 + 1)) { uint i; for (i = 0; i != lengthof(_tile_loop_offs_array); i++) { @@ -784,19 +784,19 @@ void TileLoop_Water(TileIndex tile) _current_player = OWNER_NONE; /* edges */ - if (TileX(tile) == 0 && IS_INT_INSIDE(TileY(tile), 1, MapSizeY() - 3 + 1)) { //NE + if (TileX(tile) == 0 && IsInsideMM(TileY(tile), 1, MapSizeY() - 3 + 1)) { //NE TileLoopWaterHelper(tile, _tile_loop_offs_array[2]); } - if (TileX(tile) == MapSizeX() - 2 && IS_INT_INSIDE(TileY(tile), 1, MapSizeY() - 3 + 1)) { //SW + if (TileX(tile) == MapSizeX() - 2 && IsInsideMM(TileY(tile), 1, MapSizeY() - 3 + 1)) { //SW TileLoopWaterHelper(tile, _tile_loop_offs_array[0]); } - if (TileY(tile) == 0 && IS_INT_INSIDE(TileX(tile), 1, MapSizeX() - 3 + 1)) { //NW + if (TileY(tile) == 0 && IsInsideMM(TileX(tile), 1, MapSizeX() - 3 + 1)) { //NW TileLoopWaterHelper(tile, _tile_loop_offs_array[1]); } - if (TileY(tile) == MapSizeY() - 2 && IS_INT_INSIDE(TileX(tile), 1, MapSizeX() - 3 + 1)) { //SE + if (TileY(tile) == MapSizeY() - 2 && IsInsideMM(TileX(tile), 1, MapSizeX() - 3 + 1)) { //SE TileLoopWaterHelper(tile, _tile_loop_offs_array[3]); } } diff --git a/src/water_map.h b/src/water_map.h --- a/src/water_map.h +++ b/src/water_map.h @@ -31,9 +31,9 @@ static inline WaterTileType GetWaterTile if (_m[t].m5 == 0) return WATER_TILE_CLEAR; if (_m[t].m5 == 1) return WATER_TILE_COAST; - if (IS_INT_INSIDE(_m[t].m5, LOCK_MIDDLE, LOCK_END)) return WATER_TILE_LOCK; + if (IsInsideMM(_m[t].m5, LOCK_MIDDLE, LOCK_END)) return WATER_TILE_LOCK; - assert(IS_INT_INSIDE(_m[t].m5, DEPOT_NORTH, DEPOT_END)); + assert(IsInsideMM(_m[t].m5, DEPOT_NORTH, DEPOT_END)); return WATER_TILE_DEPOT; } @@ -72,7 +72,7 @@ static inline TileIndex GetOtherShipDepo static inline TileIndex IsShipDepot(TileIndex t) { - return IS_INT_INSIDE(_m[t].m5, DEPOT_NORTH, DEPOT_END); + return IsInsideMM(_m[t].m5, DEPOT_NORTH, DEPOT_END); } static inline Axis GetShipDepotAxis(TileIndex t) diff --git a/src/window.cpp b/src/window.cpp --- a/src/window.cpp +++ b/src/window.cpp @@ -953,7 +953,7 @@ Window *FindWindowFromPt(int x, int y) for (wz = _last_z_window; wz != _z_windows;) { Window *w = *--wz; - if (IS_INSIDE_1D(x, w->left, w->width) && IS_INSIDE_1D(y, w->top, w->height)) { + if (IsInsideBS(x, w->left, w->width) && IsInsideBS(y, w->top, w->height)) { return w; } }