<div class="gmail_quote">On Fri, Mar 2, 2012 at 2:15 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">> Thanks, this looks fine to commit.<br>
<br>
</div>Thanks, r151943.<br>
<div class="im"><br>
>> > Other than that, a test for the fix-it would be great (see test/FixIt<br>
>> > for<br>
>> > examples).<br>
>><br>
>> Wouldn't that require the fixit to be on the warning instead of the<br>
>> note? (If so, maybe there's no need for the test yet?)<br>
><br>
><br>
> You can test fixits on notes with -fdiagnostics-parseable-fixits +<br>
> FileCheck. See test/FixIt/fixit-vexing-parse.cpp for an example of that. I'm<br>
> happy for that to be a subsequent commit.<br>
<br>
</div>Attached. I added only a test for the first fixit since having lots of<br>
line numbers hardcoded in the test would probably make the test more<br>
annoying than useful. Is this what you had in mind?<br></blockquote></div><br><div>This looks great, thanks.</div>