I think selecting info tool in EditorInteractive::cleanup_for_load is 
unnecessary and does not fix the problem at its root. I propose:
 - changing 'current_player_ = 0;' to 'current_player_ = 1;' in 
EditorSetStartingPosTool's constructor
 - replacing 'select_tool(tools()->info, EditorTool::First);' with 
'mutable_field_overlay_manager()->register_overlay_callback_function(nullptr);' 
in EditorInteractive::cleanup_for_load
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1573968-new-map-crash/+merge/292712
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/bug-1573968-new-map-crash.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to