The International Simutrans Forum


Author Topic: [BUG][INSOLVENCY] Duplicate lines after takeover  (Read 226 times)

0 Members and 1 Guest are viewing this topic.

Offline Matthew

  • *
  • Posts: 335
    • Japan Railway Journal
  • Languages: EN, some ZH, DE & SQ
[BUG][INSOLVENCY] Duplicate lines after takeover
« on: February 04, 2020, 10:25:16 PM »
I started testing the insolvency-gui branch. It's amazing that you can take over a company with only two or three clicks!

However, after taking over a company, a line is duplicated. The game crashes if you attempt to load a save with a company in this state (it might be coincidental, but the backtrace suggests to this non-coder that the two things are linked).

Line duplication - steps to reproduce
1. Build Ves' insolvency-gui branch as of #1f78627fb88bada910edfa20d2e2bd7d9f0cdfde
2. Start Simutrans-Extended and load Pak128.Britain-Ex
3. Load this save.
4. Switch player to Berkthorne Tramway. You have only one line:

Mark your company as available for voluntary takeover.
5. Switch player to Worklow Coach Company and take over Berkthorne Tramway.
6. Open the Line Window and you have multiple Berkthorne Tramway lines:

7. Save your game.

Unable to load save - steps to reproduce
1 and 2 as above.
3. Load the game saved in step 7 above or this game.
4. The game will crash:

The log reports the same:
Code: [Select]
Warning: karte_t::load: Fileversion: 120004
FATAL ERROR: quickstone<T>::quickstone_tpl(T*,uint16) - slot (15) already taken
Aborting program execution ...

GDB reported the following after I closed the crashed Sim-Ex:
Code: [Select]
Thread 1 "simutrans-exten" received signal SIGINT, Interrupt.
0x00007ffff7bc7c60 in __GI___nanosleep (requested_time=0x7fffffff2250, remaining=0x7fffffff2240) at ../sysdeps/unix/sysv/linux/nanosleep.c:28
28 ../sysdeps/unix/sysv/linux/nanosleep.c: No such file or directory.
(gdb) backtrace
#0  0x00007ffff7bc7c60 in __GI___nanosleep (requested_time=0x7fffffff2250, remaining=0x7fffffff2240)
    at ../sysdeps/unix/sysv/linux/nanosleep.c:28
#1  0x00007ffff7538c85 in ?? () from /usr/lib/x86_64-linux-gnu/
#2  0x000055555595dc42 in log_t::fatal(char const*, char const*, ...) ()
#3  0x00005555558d542e in quickstone_tpl<simline_t>::quickstone_tpl(simline_t*, unsigned short) ()
#4  0x00005555558d4e4c in simline_t::simline_t(player_t*, simline_t::linetype, loadsave_t*) ()
#5  0x00005555558d68a1 in simlinemgmt_t::rdwr(loadsave_t*, player_t*) ()
#6  0x00005555557bc393 in player_t::rdwr(loadsave_t*) ()
#7  0x0000555555944e41 in karte_t::load(loadsave_t*) ()
#8  0x000055555594798e in karte_t::load(char const*) ()
#9  0x0000555555726058 in loadsave_frame_t::item_action(char const*) ()
#10 0x0000555555741fd9 in savegame_frame_t::action_triggered(gui_action_creator_t*, value_t) ()
#11 0x00005555559aded7 in gui_table_t::infowin_event(event_t const*) ()
#12 0x00005555556b64aa in gui_scrollpane_t::infowin_event(event_t const*) ()
#13 0x00005555556aa09e in gui_container_t::infowin_event(event_t const*) ()
#14 0x00005555556fb14a in gui_frame_t::infowin_event(event_t const*) ()
#15 0x0000555555776305 in check_pos_win(event_t*) ()
#16 0x00005555558cd07a in interaction_t::process_event(event_t&) ()
#17 0x00005555558cd79b in interaction_t::check_events() ()
#18 0x0000555555956d04 in karte_t::interactive(unsigned int) ()
#19 0x00005555558dc4b4 in simu_main(int, char**) ()
#20 0x00005555558f2042 in sysmain(int, char**) ()
#21 0x00007ffff6748b97 in __libc_start_main (main=0x5555555ab400 <main>, argc=5, argv=0x7fffffffdfc8, init=<optimised out>,
    fini=<optimised out>, rtld_fini=<optimised out>, stack_end=0x7fffffffdfb8) at ../csu/libc-start.c:310
#22 0x00005555555ab46a in _start ()

Online Freahk

  • Devotee
  • *
  • Posts: 1074
  • Languages: DE, EN
Re: [BUG][INSOLVENCY] Duplicate lines after takeover
« Reply #1 on: February 13, 2020, 11:03:00 AM »
Not part of the buglisting as it is part of the master branch.
I am not quite sure if I should include such as dev branches tend to change quite often, however, I guess git isssues fit here well.

Offline jamespetts

  • Simutrans-Extended project coordinator
  • Moderator
  • *
  • Posts: 19824
  • Cake baker
    • Bridgewater-Brunel
  • Languages: EN
Re: [BUG][INSOLVENCY] Duplicate lines after takeover
« Reply #2 on: May 16, 2020, 11:01:41 PM »
Thank you for your report. I believe that I have now fixed this on the insolvency branch - I should be grateful if you could re-test and confirm.