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.

_______________________________________________
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

Reply via email to