From 5bdf6833b304618aef3a45b96a8458205bf85237 Mon Sep 17 00:00:00 2001 From: rubidium Date: Sat, 26 May 2007 22:19:04 +0000 Subject: (svn r9947) -Fix [FS#807]: don't check whether roadbits may be built when they are already built for another type. --- src/road_cmd.cpp | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/src/road_cmd.cpp b/src/road_cmd.cpp index 64f50c832..cd9ba5deb 100644 --- a/src/road_cmd.cpp +++ b/src/road_cmd.cpp @@ -337,6 +337,7 @@ int32 CmdBuildRoad(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) int32 cost = 0; int32 ret; RoadBits existing = ROAD_NONE; + RoadBits all_bits = ROAD_NONE; Slope tileh; SET_EXPENSES_TYPE(EXPENSES_CONSTRUCTION); @@ -359,6 +360,7 @@ int32 CmdBuildRoad(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) if (HasRoadWorks(tile)) return_cmd_error(STR_ROAD_WORKS_IN_PROGRESS); if (!HASBIT(GetRoadTypes(tile), rt)) break; + all_bits = GetAllRoadBits(tile); existing = GetRoadBits(tile, rt); if ((existing & pieces) == pieces) { return_cmd_error(STR_1007_ALREADY_BUILT); @@ -368,7 +370,8 @@ int32 CmdBuildRoad(TileIndex tile, uint32 flags, uint32 p1, uint32 p2) case ROAD_TILE_CROSSING: if (HASBIT(GetRoadTypes(tile), rt)) return_cmd_error(STR_1007_ALREADY_BUILT); - if (pieces & ComplementRoadBits(GetCrossingRoadBits(tile))) goto do_clear; + all_bits = GetCrossingRoadBits(tile); + if (pieces & ComplementRoadBits(all_bits)) goto do_clear; break; default: @@ -435,13 +438,16 @@ do_clear:; cost += ret; } - ret = CheckRoadSlope(tileh, &pieces, (IsTileType(tile, MP_STREET) ? GetAllRoadBits(tile) : ROAD_NONE) | existing); - /* Return an error if we need to build a foundation (ret != 0) but the - * current patch-setting is turned off (or stupid AI@work) */ - if (CmdFailed(ret) || (ret != 0 && (!_patches.build_on_slopes || _is_old_ai_player))) - return_cmd_error(STR_1000_LAND_SLOPED_IN_WRONG_DIRECTION); - - cost += ret; + if (all_bits != existing) { + /* Check the foundation/slopes when adding road/tram bits */ + ret = CheckRoadSlope(tileh, &pieces, all_bits | existing); + /* Return an error if we need to build a foundation (ret != 0) but the + * current patch-setting is turned off (or stupid AI@work) */ + if (CmdFailed(ret) || (ret != 0 && (!_patches.build_on_slopes || _is_old_ai_player))) { + return_cmd_error(STR_1000_LAND_SLOPED_IN_WRONG_DIRECTION); + } + cost += ret; + } if (IsTileType(tile, MP_STREET)) { /* Don't put the pieces that already exist */ -- cgit v1.2.3-70-g09d2