<div dir="ltr">On Tue, Mar 19, 2013 at 9:13 AM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  IMHO this is the wrong thing to do. Warnings that are errors by default are errors for a good reason: it is usually //very// hard to find a legitimate use case. I see that -Wstatic-float-init is actually something reasonable to do, but the warning text just suggests adding `constexpr`; is that so hard to do? (Is that incompatible with other C++11 compilers?)<br>
</blockquote><div><br></div><div style>Fixing something like this across a 100MLOC code base that has parts that require C++03 compatibility is unfortunately a non-trivial amount of work.</div><div style><br></div><div style>
In addition, it's unexpected that not all warnings are switched off by -w. We particularly ran into that problem when running tools on our code base with -w, which led to different behavior than switching off the warning on its own.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  Put this way: if -Wstatic-float-init were a hard error with a fixit, wouldn't you just accept the fixit and move on, even though Clang can perfectly well tell what you want to do?<br></blockquote><div><br></div><div style>
If it was an error, we would just pay the cost of fixing the code base (and we do so for all warnings that catch enough bugs to be worth it, too).</div></div></div></div>