The International Simutrans Forum

Simutrans Extended => Simutrans-Extended bug reports => Simutrans-Extended development => Simutrans-Extended closed bug reports => Topic started by: Vladki on March 11, 2018, 01:31:12 PM

Title: privatecar.tab changes
Post by: Vladki on March 11, 2018, 01:31:12 PM
Hello,

I have noticed that the test game with swedish pakset does not start any more. After running with debug it complained about privatecar.tab. So I have checked, and found that british pakset is using new format of privatecar.tab introduced here:
https://forum.simutrans.com/index.php?topic=16979.msg164240#msg164240

swedish pakset has no privatecar.tab at all, so I have copied the new file to ~/simutrans/config/ (i.e. the pakset independent config), overwriting the old one, but it did not help. Only after putting the file to ~/simutrans/pak128.sweden/config/ it worked again.

Is that intended? Should not this file work from the common config? Should it be uploaded to simutrans-extended repository?
Title: Re: privatecar.tab changes
Post by: jamespetts on March 11, 2018, 02:56:38 PM
Hmm - that is odd: loading Pak128.Sweden-Ex does not give me an error. Can you tell me what error message(s) that you got and at what point during the loading process? It should be possible to have a pakset with no privatecar.tab, which will simply use a generic default value.
Title: Re: privatecar.tab changes
Post by: Vladki on March 11, 2018, 08:19:18 PM
Quote from: jamespetts on March 11, 2018, 02:56:38 PM
Hmm - that is odd: loading Pak128.Sweden-Ex does not give me an error. Can you tell me what error message(s) that you got and at what point during the loading process? It should be possible to have a pakset with no privatecar.tab, which will simply use a generic default value.

It says this:

Reading menu configuration ...
Warning: tool_t::init_menu():   wrong cursor (14) given for general_tool[26]
Warning: tool_t::init_menu():   wrong cursor (15) given for general_tool[29]
Warning: tool_t::read_menu():   toolbar[11][5]: replaced way-builder(id=14) with default param=cityroad by cityroad builder(id=36)
Reading private car ownership configuration ...
Message: stadt_t::privatecar_init():    Error opening config/privatecar.tab.
Will use default values.
Midi disabled ...
Calculating textures ...done
Message: karte_t::load():       Prepare for loading
Message: karte_t::load():       Time is now: 114
World destroyed.
Warning: karte_t::load: Fileversion: 120004
Message: nwc_auth_player_t::init_player_lock_server:    new = 32767
Segmentation fault


I have tried removing privatecar.tab completely, and it gives the same error.
Title: Re: privatecar.tab changes
Post by: jamespetts on March 11, 2018, 11:10:00 PM
The message,


Message: stadt_t::privatecar_init():    Error opening config/privatecar.tab.
Will use default values.
[/quote]

is consistent with the intended handling of missing private car data. The segmentation fault occurs at a later time in the loading process (although if you find that adding privatecar.tab fixes this, then this may well be related to private cars). Can you upload the saved game in which the segmentation fault can be reproduced on loading (and also the pakset version in which this occurs)?
Title: Re: privatecar.tab changes
Post by: Vladki on March 13, 2018, 07:41:20 AM
It is the Swedish online game, with nightly Swedish pakset.
Title: Re: privatecar.tab changes
Post by: jamespetts on March 14, 2018, 09:08:11 PM
I have downloaded the latest nightly Swedish pakset, however, I have not been able to connect to the Swedish server as it is not responding. I have not been able to reproduce the crash (confirming that privatecar.tab is absent from ~/config) on the saved games that I have loaded.
Title: Re: privatecar.tab changes
Post by: Vladki on March 14, 2018, 09:49:30 PM
Try this: http://list.extended.simutrans.org/~vladki/debug-saves/autosave-Pak128.Sweden-Ex.sve
Title: Re: privatecar.tab changes
Post by: jamespetts on March 14, 2018, 09:59:34 PM
Splendid, thank you for that. I think that I have now fixed this. I should be grateful if you could re-test with the next nightly build to confirm.
Title: Re: privatecar.tab changes
Post by: Vladki on April 08, 2018, 07:43:58 PM
Sorry for delay in testing, but it does not work. This is what I get if I remove privatecar.tab from pakset:

Reading private car ownership configuration ...
Message: stadt_t::privatecar_init():    Error opening config/privatecar.tab.
Will use default values.
Midi disabled ...
Calculating textures ...done
Message: karte_t::load():       Prepare for loading
Message: karte_t::load():       Time is now: 182
World destroyed.
Warning: karte_t::load: Fileversion: 120004
Message: nwc_auth_player_t::init_player_lock_server:    new = 32767
Segmentation fault


And the same I get if I remove the privatecar.tab from pakset independent config.
Perhaps it affects server binary only

Title: Re: privatecar.tab changes
Post by: jamespetts on April 08, 2018, 08:24:46 PM
I am afraid that I can no longer reproduce this with the above saved game - can you upload a saved game in which this can reliably be reproduced?
Title: Re: privatecar.tab changes
Post by: Vladki on April 09, 2018, 07:58:51 PM
You are right, the old save loads OK. Here is a new save that does not:
http://list.extended.simutrans.org/~vladki/debug-saves/autosave-Pak128.Sweden-Ex.sve
Title: Re: privatecar.tab changes
Post by: Ves on April 09, 2018, 08:15:58 PM
Do I need to do something to pak.sweden? Should I copy the privatecar.tab from britain permanently? Reasons I have not done that is I simply have not considered private traffic at all yet.

btw, I cannot connect to any of the two swedish savegames online. It says the games wont respond. Not even connecting manually works.
Title: Re: privatecar.tab changes
Post by: jamespetts on April 09, 2018, 08:58:06 PM
Thank you for the report - I think that I have now fixed this. I should be grateful if you could re-test with the latest version.
Title: Re: privatecar.tab changes
Post by: Vladki on April 12, 2018, 07:49:22 PM
Now it seems to be fixed. Thank you James.

The other swedish server was running just temporarily so that I did not have to stop the main one.