<br><br><div class="gmail_quote">On Tue, Oct 4, 2011 at 10:09 AM, Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com">kremenek@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div style="word-wrap:break-word"><div class="im"><div>On Oct 4, 2011, at 9:27 AM, David Blaikie wrote:</div><div><div><br><blockquote type="cite"><span style="border-collapse:separate;font-family:Helvetica;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:-webkit-auto;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;font-size:medium">Could/should we support -Wno-* by default anyway? I mean if we don't<br>
have the warning it is certainly not on.<br>(on the other hand, as a user I wouldn't mind knowing if I'm passing<br>useless flags. Except when I'm trying to use multiple compilers and<br>turn something off in one which isn't present in the other. Minor<br>
convenience to have that silently(ish) accepted on the no supporting<br>compiler)</span></blockquote></div><br></div></div><div><div>Hi David,</div><div><br></div>Numerous people have requested that all warnings can be placed under a -W flag, and allow them to selectively turn them on or off.  I'll try and summarize why I think this is a critical feature.</div>
</div></blockquote><div><br></div><div>Oh, sorry - I think I wasn't clear. I believe I understand & agree with the motivation (& if I get a chance I'll have a go & providing some patches to group the ungrouped warnings to help get that 304 down to zero). My point was (& it's possible this is the existing behavior, I don't have access to clang right now to test) to counter Bob's point (that he wouldn't want to break users who were disabling (-Wno-blah) the warning when the warning was removed):<br>
<br>Could we just silently allow users to pass -Wno-foo & if we don't have a warning called 'foo' we could just silently swallow this flag & ignore it? Since we don't support the 'foo' warning anyway, we can trivially support -Wno-foo.</div>
<div><br></div><div>- David</div></div>