SamuXarick

Got a savegame of yet another situation I just found. [crash20220609155448.zip](https://github.com/OpenTTD/OpenTTD/files/8872171/crash20220609155448.zip) Vehicle is blocked, but still displays a speed over 0 km/h: 

@frosch123 - UpdateInclination calls UpdateViewport, which was the main intention. When the diagonal direction is modified, it is not yet reflected in the viewport, only x_pos and y_pos were. There...

> I think most of the > `bool river = HasTileCanalOnRiver(tile_cur);` > variables are unnecessary becuase you only use them once. They are getting the state of the tile before...

> If I turn off the `Allow removal of rivers` setting, and try to demolish a river tile, I can hear the sound of the demolition however nothing happens. I...

> > > I think most of the > > > `bool river = HasTileCanalOnRiver(tile_cur);` > > > variables are unnecessary becuase you only use them once. > > >...

> > If I turn off the `Allow removal of rivers` setting, and try to demolish a river tile, I can hear the sound of the demolition however nothing happens....

> Please explain why this change should be made My idea here was to change the logic on what to do about the missing cargo that can originate from "a...

Worst case station scenario file for testing: [single house.zip](https://github.com/OpenTTD/OpenTTD/files/5742293/single.house.zip)  1 house covered by 124 stations

You can use this commit https://github.com/SamuXarick/OpenTTD/commit/39390976e1d408fa94900a682ffdfb761718440a for debugging purposes, to check the existance of flooded tiles after a world is generated.

AIs available on bananas that might be affected (by searching if they used both `AIRoadTypeList `and `AIRoad.HasRoadType` functions, not really tested): - AAAHogEx - RailwAI