Game crashes when ship construction site cannot be cleared for a new ship

[Astuur]

When the shipwright builds a new ship the last one is moved out of the way to clear the construction site.
When this is not possible due to space restrictions, the game crashes.

[Hans Joachim Desserud (hjd)]

Thanks for splitting this out from bug 963799. This should be dealt with, but I doubt it will get into build17, thus targetting to build18.

[Hans Joachim Desserud (hjd)]

Could either of you please attach a savegame where this issue is reproducible and/or terminal output including an error message (and preferably backtrace) when this happens?

[Shevonar (shevonar)]

The duplicate bug #1009504 has a patch for this problem.

[Andreas Breitschopp (ab-tools)]

Ah, sorry for posting a duplicate bug - I searched before, but obviously with the wrong keywords...

I'll create a branch and propose it for merging as suggested. Thanks for your fast reply, Shevanar!

[Andreas Breitschopp (ab-tools)]

I've added a new branch and proposed it for merging.

[SirVer (sirver)]

Andreas, thanks for debugging this. But what happens now in this situations? If the pont is full of ships, no other ship can go there. The crash was not ideal, but what happens now after your solution?

[Andreas Breitschopp (ab-tools)]

First this crash does not only occur when there is really no space left anymore at all, but already (see the description of my duplicate bug where I also explained the reason for it) when a ship is requested to move, but it cannot, because in every direction there is another ship already.

Therefore in my test save game there was not yet every space filled up with ships as my sea is quite large.

But just for fun I've tested it now after applying my patch: after building 40 shipyards the sea was full of ships quite soon. ;-)

It looks funny then, because every ship constantly trys to move around, ships just swapping places with each other all the time, everywhere in the sea.

I think this is expected behavior in this case and with my patch applied there was no crash anymore (I kept it running for about 4 hours at 6x speed).

[SirVer (sirver)]

merged in r6398. Thanks Andreas!

[SirVer (sirver)]

Released in build-18 rc1.