I have started with the code review - can you please have a look at the logic changes? The meaning should still be the same.
I also did some formatting on Scenario 1, can you do the same for Scenario 2? Also a NOCOM comment about a variable name - abbreviations make the code harder to read. -- https://code.launchpad.net/~widelands-dev/widelands/frisians/+merge/336910 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/frisians. _______________________________________________ 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
