<div>On Tue, May 22, 2012 at 11:36 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div class="im"><br></div>For what it's worth, -fixit mode only applies fixits on the warning<br>itself. Fixits on notes are never automatically applied (as far as I<br>know/understand) so that's where semantic-changing fixits can be<br>

found, in any order.<br><br>Very few warnings have fixits on them directly for this reason.</blockquote></div><div><br></div>Sorry for not replying sooner. This warning is definitely missing some notes, but if I understand you correctly fixits are also fine? They won't be applied but they still offer some useful information about the exact code transformations that are needed to make the warning go away.<div>

<br></div><div>One more thing, the text <span style="white-space:pre-wrap">"replace parentheses with an initializer to declare a variable" is copied from </span><span style="white-space:pre-wrap">DiagnosticSemaKinds.td verbatim. It's needed in the ParseTentative.cpp which knows nothing about definitions in Sema, so I'm guessing that there's no better way to solve this than to have the same wording in two places?</span><br>

</div>