← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/bug-1573968-new-map-crash into lp:widelands

 

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.


References