(svn r16996) -Fix (r16995): the tiles under the bridge were not marked dirty when a bridge was replaced with another type

1 files changed, 9 insertions, 3 deletions

diff --git a/src/tunnelbridge_cmd.cpp b/src/tunnelbridge_cmd.cpp
index 06d65d077..b57095768 100644
--- a/src/tunnelbridge_cmd.cpp
+++ b/src/tunnelbridge_cmd.cpp
@@ -373,8 +373,10 @@ CommandCost CmdBuildBridge(TileIndex end_tile, DoCommandFlag flags, uint32 p1, u
 			}
 
 			if (flags & DC_EXEC) {
+				/* We do this here because when replacing a bridge with another
+				 * type calling SetBridgeMiddle isn't needed. After all, the
+				 * tile alread has the has_bridge_above bits set. */
 				SetBridgeMiddle(tile, direction);
-				MarkTileDirtyByTile(tile);
 			}
 		}
 
@@ -404,8 +406,12 @@ CommandCost CmdBuildBridge(TileIndex end_tile, DoCommandFlag flags, uint32 p1, u
 			default:
 				NOT_REACHED();
 		}
-		MarkTileDirtyByTile(tile_start);
-		MarkTileDirtyByTile(tile_end);
+
+		/* Mark all tiles dirty */
+		TileIndexDiff delta = (direction == AXIS_X ? TileDiffXY(1, 0) : TileDiffXY(0, 1));
+		for (TileIndex tile = tile_start; tile <= tile_end; tile += delta) {
+			MarkTileDirtyByTile(tile);
+		}
 	}
 
 	if ((flags & DC_EXEC) && transport_type == TRANSPORT_RAIL) {