Review: Approve diff, test

Change is looking good and seems to be working as expected.
Am I right that this is only relevant for loading multiplayer games, since in 
single player you are unable to change positions?

Independent of your change, I am a bit puzzled about this part:

} catch (...) {
  throw;
}

Isn't it completely useless and couldn't the whole try{} block be removed? Or 
am I missing something?
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1629691-make-empty-ai-empty/+merge/349389
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/bug-1629691-make-empty-ai-empty.

_______________________________________________
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