<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Sep 12, 2012, at 3:48 PM, Richard Trieu <<a href="mailto:rtrieu@google.com">rtrieu@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">On Wed, Sep 12, 2012 at 3:35 PM, Ted Kremenek <span dir="ltr"><<a href="mailto:kremenek@apple.com" target="_blank">kremenek@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 style="word-wrap:break-word"><div class="im"><div><div>On Sep 12, 2012, at 3:27 PM, Richard Trieu <<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>> wrote:</div><br><blockquote type="cite">
<span style="font-family:Helvetica;font-size:medium;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;display:inline!important;float:none">This warning caught around 5 bugs. Roughly twice that amount in false positives needed to be silenced.</span></blockquote>
</div><div><br></div></div>Thanks Richard.<div><br><div>IMO, that's a pretty dreadful signal-to-noise ratio. That's a 66% false positive rate. That's the kind of false positive rate I'd consider a warning being DefaultIgnore, not be on by default. Do the other false positives look like cases the warning logic could be improved to handle? Less than 10% false positive rate seems required to me for this warning to be on by default.</div>
</div></div></blockquote></div><br><div>I have no problems with switching this to DefaultIgnore. Looking back, through the false positives, it looks like the most common case would be when the enum is simply there to hold a collection of constants. I think checking if there is a variable of the enum type would be the best way to cut down on false positives, but I am not sure how to go about implementing that.</div></blockquote><br></div><div>I don't think this warning belongs in Clang at all, even under DefaultIgnore. The false positive rate is far too high for it to be useful for the majority of users, and it isn't the kind of warning that maps well to DefaultIgnore, because it doesn't correspond to some stylistic decision or usage pattern…. it's just too noisy for the majority of users.</div><div><br></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>- Doug</div><br></body></html>