The International Simutrans Forum

Development => Patches & Projects => Incorporated Patches and Solved Bug Reports => Topic started by: Dwachs on July 30, 2010, 04:37:46 PM

Title: [patch] bug fix code clean up in wegbauer
Post by: Dwachs on July 30, 2010, 04:37:46 PM
This patch should fix the bug reported here:

http://forum.simutrans.com/index.php?topic=5352.0

Moreover, it removes a lot of code duplication from wegbauer_t::is_allowed_step.

And it fixes a bug for building canals: there the check for right owner was never executed, which means one could build over other players canals (similar to roads).

Should this behavior be kept? Or fixed?

Please test and comment.
Title: Re: [patch] bug fix code clean up in wegbauer
Post by: prissi on July 30, 2010, 07:49:43 PM
Player only seems a very good idea. However usually such changes have lots of subtle bugs, so maybe just include it and wait for bug reports?

EDIT: I can lay tram tracks into a road depot. That needs to be forbidden (and vice versa) otherwise I can move trams to road depots where they are lost forever.

Road vehicle in tram depot will crash simutrans even ...
Title: Re: [patch] bug fix code clean up in wegbauer
Post by: Dwachs on August 03, 2010, 07:15:52 PM
in rev 3608. longest log message ever :P