widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/bug-1341674_codecheck into lp:widelands

To: mp+227936@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Fri, 25 Jul 2014 11:18:23 -0000
In-reply-to: <20140723145324.13185.411.launchpad@ackee.canonical.com>
Reply-to: mp+227936@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

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.

References

[Merge] lp:~widelands-dev/widelands/bug-1341674_codecheck into lp:widelands
From: GunChleoc, 2014-07-23