The International Simutrans Forum

 

Author Topic: [Bug] R3764 - changing settings while the game is running breaks savegames  (Read 1779 times)

0 Members and 1 Guest are viewing this topic.

Offline jamespetts gb

  • Simutrans-Extended project coordinator
  • Devotee
  • *
  • Posts: 18721
  • Cake baker
    • Bridgewater-Brunel
  • Languages: EN
Steps to reproduce:

(1) enable "dialog_tool[27]=,i" in menuconf.tab (it is not essential to use the "i" key, but I find this a good key for this tool;
(2) start a new game and use this tool by pressing the "i" key (or whatever key was assigned);
(3) press "simuconf.tab" to reset all the default values to the simuconf.tab defaults;
(4) save the game;
(5) reload the game - it will fail.

Cause:

when the "simuconf.tab" button is pressed. all the values in einstellulnen_t are reset - including things such as the number of cities and the size of the map, which are essential for correct saving/loading. This should only allow the changing of values that can safely be changed without breaking things.

Edit: Similar problems are caused by editing the climate settings (water level, for instance), although this does not necessarily break the savegame - but it does flood areas that were not previously flooded when the game is re-loaded.
« Last Edit: September 11, 2010, 08:49:43 PM by jamespetts »

Offline prissi

  • Developer
  • Administrator
  • *
  • Posts: 9518
  • Languages: De,EN,JP
Re: [Bug] R3764 - changing settings while the game is running breaks savegames
« Reply #1 on: September 11, 2010, 10:23:25 PM »
Imho the real number of cities should be taken from the map itself, same for the size. Aparently this is a design error, that should be fixed sooner.

Offline jamespetts gb

  • Simutrans-Extended project coordinator
  • Devotee
  • *
  • Posts: 18721
  • Cake baker
    • Bridgewater-Brunel
  • Languages: EN
Re: [Bug] R3764 - changing settings while the game is running breaks savegames
« Reply #2 on: September 11, 2010, 10:33:00 PM »
Ahh, yes, you're right: that would be a better solution.