<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Feb 16, 2016 at 12:22 PM, Craig, Ben <span dir="ltr"><<a href="mailto:ben.craig@codeaurora.org" target="_blank">ben.craig@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
On 2/16/2016 1:18 PM, Nico Weber via cfe-dev wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Won't this line of reasoning lead to all useful warnings being in -Werror eventually? Say, forgetting a return statement in a function is also "just" a warning...<br>
</blockquote>
<br></span>
Not all of them :)<br>
<br>
Visual Studio groups warnings into big warning level buckets.  Level 1 has the most important / severe (obvious use of uninitialized value), level 4 has fairly minor warnings (unused parameter), and /Weverything will tell you about really useless stuff (warning! you just used __declspec(align)! ).  I could imagine a world where the "Level 1", and maybe "Level 2" warnings were errors by default.<br></blockquote><div><br></div><div>We have this too: on-by-default warnings, -Wall, -Wextra, -Weverything. I don't think we should turn on-by-default warnings into errors.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
We should make it harder to compile broken code, and easier to write correct code.  We can't change it all at once without angering the world though :)</blockquote><div><br></div><div>People who don't like writing broken code don't ignore warnings. People who do like writing broken code will just pass -Wno-error. I don't think this proposal helps either party.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
-- <br>
Employee of Qualcomm Innovation Center, Inc.<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project<br>
<br>
</div></div></blockquote></div><br></div></div>