News:

Simutrans Sites
Know our official sites. Find tools and resources for Simutrans.

[bug] intercity road timeline - r4396

Started by VS, April 16, 2011, 11:10:39 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

VS

This is an interesting bug...

The code first looks for old-style entry without indexing, and if not present, fills in "asphalt_road".
If the old-style entry loaded something, it overwrites the list with this one.
Then the indexed entries are checked, and if present they are loaded and overwrite the list.

This happens for each simuconf.tab loaded.

The logic error is between the first and second step - the fallback ensures that there is always something, and each config thus trashes the list. Even an empty file :D This needs user directory simuconf though, because for singleuser pakset simuconf is last one.

Patch attached - copypaste of cityroad code, which does work correctly, it seems.


Thanks a lot for DirrrrtyDirk's help when debugging! We thought it was a pakset bug at first :P The fallbacks and timeline effects somehow ensured that pak128 always ended up with plausibly looking stuff built.

Dwachs


prissi

This was the same as for city_roads. When fixing this, I aparently overlooked that intercityroads can have a timeline.