widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~peter.waller/widelands/clang-warning-fixes into lp:widelands

To: mp+103975@xxxxxxxxxxxxxxxxxx
From: Peter Waller <peter.waller@xxxxxxxxx>
Date: Sat, 28 Apr 2012 23:37:17 -0000
In-reply-to: <20120428215335.27839.79854.codereview@gac.canonical.com>
Reply-to: mp+103975@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

> Why do you comment the names out?

This is to avoid "unused variable" warnings, which can be quite prolific because the header files are included in many places and generate the warnings for in each and every file they are included in.
-- 
https://code.launchpad.net/~peter.waller/widelands/clang-warning-fixes/+merge/103975
Your team Widelands Developers is requested to review the proposed merge of lp:~peter.waller/widelands/clang-warning-fixes into lp:widelands.

References

Re: [Merge] lp:~peter.waller/widelands/clang-warning-fixes into lp:widelands
From: David Allwicher, 2012-04-28