I think this is done now - except for 1 empty TODO comment that I left around. Maybe someone else will know why there's a TODO there:
src/io/filesystem/zip_filesystem.cc:563: Use "TODO(username): <msg>". Do not add empty TODO comments. I have also made the feedback individual to each case, so the user will know why the rule barfed specifically. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1341674_codecheck/+merge/227936 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1341674_codecheck. _______________________________________________ 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