Review: Needs Fixing
Generally the changes look good (beside of the already elsewhere mentioned
copyright year in the header :) ).
However I tested the change with all my savegames and quite some savegames
failed to load. I even got a segfault:
Reading Building Data ...
Program received signal SIGSEGV, Segmentation fault.
0x087401f4 in Widelands::Player::tribe (this=0x0) at
/home/drehatlas/widelands/widelands/src/logic/player.h:116
116 const Tribe_Descr & tribe() const throw () {return m_tribe;}
(gdb) bt
#0 0x087401f4 in Widelands::Player::tribe (this=0x0) at
/home/drehatlas/widelands/widelands/src/logic/player.h:116
#1 0x0890fb1a in Widelands::DismantleSite::DismantleSite (this=0xb362150,
gdescr=..., egbase=..., c=..., plr=..., loading=true,
former_buildings=std::vector of length 1, capacity 1 = {...}) at
/home/drehatlas/widelands/widelands/src/logic/dismantlesite.cc:77
#2 0x089084cf in Widelands::Editor_Game_Base::warp_dismantlesite
(this=0xbfffc3a4, c=..., owner=1 '\001', loading=true,
former_buildings=std::vector of length 1, capacity 1 = {...}) at
/home/drehatlas/widelands/widelands/src/logic/editor_game_base.cc:349
#3 0x08a2945b in Widelands::Map_Building_Data_Packet::Read (this=0xbfffc0dc,
fs=..., egbase=..., skip=false, mol=...)
at
/home/drehatlas/widelands/widelands/src/map_io/widelands_map_building_data_packet.cc:91
#4 0x08a1e7a0 in Widelands::WL_Map_Loader::load_map_complete (this=0x956f888,
egbase=..., scenario=true)
at
/home/drehatlas/widelands/widelands/src/map_io/widelands_map_loader.cc:246
#5 0x08a739a7 in Widelands::Game_Map_Data_Packet::Read_Complete
(this=0xbfffc218, game=...) at
/home/drehatlas/widelands/widelands/src/game_io/game_map_data_packet.cc:56
#6 0x087ee732 in Widelands::Game_Loader::load_game (this=0xbfffc340,
multiplayer=false) at
/home/drehatlas/widelands/widelands/src/game_io/game_loader.cc:82
#7 0x0892dc13 in Widelands::Game::run_load_game (this=0xbfffc3a4,
filename="save/Katy.wgf") at
/home/drehatlas/widelands/widelands/src/logic/game.cc:415
#8 0x086e6cf6 in WLApplication::load_game (this=0x920a838) at
/home/drehatlas/widelands/widelands/src/wlapplication.cc:2063
#9 0x086e5fd9 in WLApplication::mainmenu_singleplayer (this=0x920a838) at
/home/drehatlas/widelands/widelands/src/wlapplication.cc:1650
#10 0x086e5b94 in WLApplication::mainmenu (this=0x920a838) at
/home/drehatlas/widelands/widelands/src/wlapplication.cc:1567
#11 0x086e11b2 in WLApplication::run (this=0x920a838) at
/home/drehatlas/widelands/widelands/src/wlapplication.cc:469
#12 0x086df695 in main (argc=1, argv=0xbfffed64) at
/home/drehatlas/widelands/widelands/src/main.cc:103
I will send you the savegame I triggered this crash with per mail
--
https://code.launchpad.net/~widelands-dev/widelands/formerbuildings_index/+merge/179570
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/formerbuildings_index.
_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help : https://help.launchpad.net/ListHelp
