widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #02406
Re: [Merge] lp:~widelands-dev/widelands/bug-1341081 into lp:widelands
I added a bunch of NOCOM(#codereview). You do not implement some methods (see the codereview comments). The code I pushed still doesn't compile, but it should give you an idea of what is missing.
Also, your editor formats source code quite differently to the codebase, so there are a lot of style checker warnings. Don't you get style warnings during compile?
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1341081/+merge/227107
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1341081.
Follow ups
References