[chromium-dev] whitlisting compilers for -Werror

2009-11-21 Thread Evan Martin
On Sat, Nov 21, 2009 at 7:21 AM, rahul rahulsin...@gmail.com wrote: I happen to find this warning very useful, just as I find our policy to make warnings hard errors in our own code helpful. Yes, of course. That's a good practice. But what about the end-users who have to deal with warnings as

Re: [chromium-dev] whitlisting compilers for -Werror

2009-11-21 Thread Peter Kasting
On Sat, Nov 21, 2009 at 10:19 AM, Evan Martin e...@chromium.org wrote: I have been particularly frustrated with gcc warning bugs that have been fixed in newer versions of gcc. In older gccs, the following code produces a variable may be used uninitialized warning depending on your

Re: [chromium-dev] whitlisting compilers for -Werror

2009-11-21 Thread Evan Martin
On Sat, Nov 21, 2009 at 11:59 AM, Peter Kasting pkast...@google.com wrote: There is another fix, which is to disable to warning within the file or globally for GCC versions less than X.  GCC exposes a number of different macros and switches that let you determine the version precisely either

Re: [chromium-dev] whitlisting compilers for -Werror

2009-11-21 Thread Jens Alfke
On Nov 21, 2009, at 10:19 AM, Evan Martin wrote: We should whitelist known compiler versions that build successfully with -Werror, and then turn it off for the rest. I did something in this spirit, though in kludgy form, a few weeks ago. After I fixed all the current GCC warnings in WebCore,

Re: [chromium-dev] whitlisting compilers for -Werror

2009-11-21 Thread Peter Kasting
On Sat, Nov 21, 2009 at 12:06 PM, Evan Martin e...@chromium.org wrote: This works for warnings we know about now, but not warnings that will occur in the future, which is the larger problem. I'd say we break the automated Ubuntu builds every couple of weeks (and get an additional report from

Re: [chromium-dev] whitlisting compilers for -Werror

2009-11-21 Thread Antoine Labour
On Sat, Nov 21, 2009 at 12:33 PM, Peter Kasting pkast...@google.com wrote: On Sat, Nov 21, 2009 at 12:06 PM, Evan Martin e...@chromium.org wrote: This works for warnings we know about now, but not warnings that will occur in the future, which is the larger problem. I'd say we break the

Re: [chromium-dev] whitlisting compilers for -Werror

2009-11-21 Thread Fabien Tassin
On Sat, 2009-11-21 at 12:06 -0800, Evan Martin wrote: I'd say we break the automated Ubuntu builds every couple of weeks (and get an additional report from users at about that same rate). I don't mind when my automated daily builds break once in a while, but when the same error stays there for

Re: [chromium-dev] whitlisting compilers for -Werror

2009-11-21 Thread Antoine Labour
On Sat, Nov 21, 2009 at 5:43 PM, Fabien Tassin f...@sofaraway.org wrote: On Sat, 2009-11-21 at 12:06 -0800, Evan Martin wrote: I'd say we break the automated Ubuntu builds every couple of weeks (and get an additional report from users at about that same rate). I don't mind when my