On Fri, Jul 6, 2012 at 4:12 PM, 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_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
On Jul 6, 2012, at 16:07 , Ted Kremenek <<a href="mailto:kremenek@apple.com">kremenek@apple.com</a>> wrote:<br>
<br>
> This patch also significantly reduces the number of warnings not under flags from 229<br>
> to 158 (all extension warnings).  That's a 31% reduction.<br>
<br>
</div>…but it's arguable whether that really counts, since it's likely many of these warnings should have actual flags anyway…<br></blockquote><div><br></div><div>... and indeed we no longer have any protection against extension warnings being added with no explicit flag (other than -Wpedantic, which doesn't seem fine-grained enough to be useful). Would it be worthwhile to teach diagtool to additionally list diagnostics which are contained directly in the 'pedantic' group and include that in our CHECKs, with the intention of eventually emptying that list too?</div>
</div>