← Back to team overview

widelands-dev team mailing list archive

  1. widelands-dev team
  2. Mailing list archive
  3. Message #12732

Re: [Merge] lp:~widelands-dev/widelands/frisians into lp:widelands

  Thread Previous • Date Previous • • Thread Next 
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.

References

  Thread Previous • Date Previous • • Thread Next