News:

Want to praise Simutrans?
Your feedback is important for us ;D.

r12207: when bankrupt, roadsigns on train tracks are not removed

Started by Nazalassa, Yesterday at 05:00:51 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Nazalassa

Making paks since October 2023  |  pak48.bitlit | pak32.box | MLM for pak64 | Empire F7 cars | Pneumatic tubes | More pak64 vehicles and industries

Life is like a multi-tasking OS: you know you'll eventually get back to everything, but you don't know when.

victor_18993

Fixed and integrated in r12208.
The issue was that several railway signs are implemented as
roadsign_t, so during company bankruptcy they were being transferred to the public player while the underlying railway was removed, leaving orphaned signs behind.
Nazalassa's patch was correct as-is and has been integrated unchanged. A regression test was also added to cover the affected railway signs, normal road signs and public-player ownership cases.
Full test suite: 293/293 passed.
Thanks again, Nazalassa, for the report and the patch.
En la vida todo son vivencias y cada una de ellas nos hace mas grandes,¿Como de grande eres tu? :)