Moving a conversation from a review thread to cfe-dev for broader comments.<div><br></div><div>Currently Clang has a consistent practice of emitting a fixit suggestion note which silences the warning (without changing semantics) first, followed by any alternative suggestions. This seems a bit counter intuitive, as in many (most?) cases, the alternative suggestion is more likely what the user intended.</div>
<div><br></div><div>After discussing this on IRC with dgregor, he agreed and raised another perspective that I find compelling for inverting the current policy: if there is a fixit suggestion note which merely silences the compiler, it should be *last*.</div>
<div><br></div><div><div>< dgregor> I do agree that always putting "silence the compiler" last would be better… essentially, one could imagine the user reading each of the notes, shaking his head, and then clicking on the last one "oh, shut up, I know what I'm doing"</div>
</div><div><br></div><div>Thoughts? If there is general agreement with the strategy, I'm willing to revisit my patch, implementing the policy consistently across all of our warnings.</div><div>-Chandler</div><div><br>
<div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Chandler Carruth</b> <span dir="ltr"><<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>></span><br>Date: Tue, Jun 21, 2011 at 11:34 AM<br>
Subject: Re: [cfe-commits] r133526 - /cfe/trunk/lib/Sema/SemaExpr.cpp<br>To: Douglas Gregor <<a href="mailto:dgregor@apple.com">dgregor@apple.com</a>><br>Cc: <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<br><br><div class="im"><div class="gmail_quote">On Tue, Jun 21, 2011 at 11:23 AM, Douglas Gregor <span dir="ltr"><<a href="mailto:dgregor@apple.com" target="_blank">dgregor@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

I'm not a big fan of this change. In general, we've taken the approach that the "silence the warning" option is always first.</blockquote></div><br></div><div>Ahh. Is that documented anywhere? It's not enforced by tests that I can see... Anyways, didn't realize there was any policy.</div>

<div><br></div><div><br></div><div>I actually find this fairly unfortunate across the board. Most of the cases where we have an alternative to silencing the warning, it's a more likely alternative. It'd seem nice to invert the policy (silence comes last), but maybe that can't be changed at this late stage.</div>

</div><br></div>