adf88 opened the ticket and wrote:

1. start scenario editor
2. select "lower/raise land" tool
3. increase tool size
4. move cursor toward SE or SW map border
5. notice that selection "dots" are being drawn outside the map leaving artifacts behind

Bug introduced by "(svn r26899) -Codechange: reduce the amount of tiles that needs to be drawn by taking the height of tiles into account instead of drawing way too many (ic111)"

There is also a small issue that doesn't cause problems currently, but is dangerous and may cause bugs. ViewportAddLandscape fills in a TileInfo struct with the information of the tile that is being drawn currently. Before r26899, the TileInfo::tile for MP_VOID tiles was being set correctly (index of the MP_VOID tile). After r26899 TileInfo::tile is set to INVALID_TILE for booth MP_VOID tiles and non-existent tiles. It may cause problems e.g. the DrawPointSelection procedure when drawing MP_VOID tiles at SE and SW edges gets INVALID_TILE in the given TileInfo::tile. Fortunately, it doesn't refer to the TileInfo::tile field...

For sanity, the TileInfo::tile should be set correctly for every existing tile.

I'm attaching a fix (-p0), it requires #6206.

The fix changes the code in a way that tile selection is not being drawn for non-existent tiles. Also TileInfo::tile is being set correctly for every existent tile.

Attachments

• fix-don-t-draw-tile-selection-for-non-existent-tiles-r27117.diff (1.84 KiB)

Reported version: trunk
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/6208