# HG changeset patch
# User frosch <frosch@openttd.org>
# Date 2015-02-22 15:05:48
# Node ID c9b1aedf684a6235d20c202e2a1847d8ed399051
# Parent  8c411f3a210b41ee675128973e5f25b0dc66a0ae

(svn r27162) -Fix [FS#6208]: Tile selection was drawn outside of map in some cases. (adf88)

diff --git a/src/viewport.cpp b/src/viewport.cpp
--- a/src/viewport.cpp
+++ b/src/viewport.cpp
@@ -1142,16 +1142,21 @@ static void ViewportAddLandscape()
 			tile_info.x = tilecoord.x * TILE_SIZE; // FIXME tile_info should use signed integers
 			tile_info.y = tilecoord.y * TILE_SIZE;
 
-			if (IsInsideBS(tilecoord.x, 0, MapMaxX()) && IsInsideBS(tilecoord.y, 0, MapMaxY())) {
-				/* We are inside the map => paint landscape. */
+			if (IsInsideBS(tilecoord.x, 0, MapSizeX()) && IsInsideBS(tilecoord.y, 0, MapSizeY())) {
+				/* This includes the south border at MapMaxX / MapMaxY. When terraforming we still draw tile selections there. */
 				tile_info.tile = TileXY(tilecoord.x, tilecoord.y);
-				tile_info.tileh = GetTilePixelSlope(tile_info.tile, &tile_info.z);
 				tile_type = GetTileType(tile_info.tile);
 			} else {
+				tile_info.tile = INVALID_TILE;
+				tile_type = MP_VOID;
+			}
+
+			if (tile_type != MP_VOID) {
+				/* We are inside the map => paint landscape. */
+				tile_info.tileh = GetTilePixelSlope(tile_info.tile, &tile_info.z);
+			} else {
 				/* We are outside the map => paint black. */
-				tile_info.tile = INVALID_TILE;
 				tile_info.tileh = GetTilePixelSlopeOutsideMap(tilecoord.x, tilecoord.y, &tile_info.z);
-				tile_type = MP_VOID;
 			}
 
 			int viewport_y = GetViewportY(tilecoord);
@@ -1195,7 +1200,7 @@ static void ViewportAddLandscape()
 				_vd.last_foundation_child[1] = NULL;
 
 				_tile_type_procs[tile_type]->draw_tile_proc(&tile_info);
-				DrawTileSelection(&tile_info);
+				if (tile_info.tile != INVALID_TILE) DrawTileSelection(&tile_info);
 			}
 		}
 	}