Reviving an old patch. Is this patch OK to commit? <br><br>Cheers<br>Michael<br><br><div class="gmail_quote">On Thu, Dec 6, 2012 at 9:39 AM, Michael Han <span dir="ltr"><<a href="mailto:fragmentshaders@gmail.com" target="_blank">fragmentshaders@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Richard, <div><br></div><div>Do you also have a chance to look at this patch? Please let me know if it's OK to commit it. Thanks!<span class="HOEnZb"><font color="#888888"><br>
<div><br></div><div>Michael</div></font></span><div><div class="h5"><div><div><div><br><div class="gmail_quote">On Mon, Dec 3, 2012 at 11:00 AM, Michael Han <span dir="ltr"><<a href="mailto:fragmentshaders@gmail.com" target="_blank">fragmentshaders@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I see. Thanks for the explanation!<span><font color="#888888"><br><br>Michael</font></span><div>
<div><br><div class="gmail_quote">On Mon, Dec 3, 2012 at 9:38 AM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br>
<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>Some of them are primarily testing the caret fixits, but you're right that the parseable fixits don't actually <i>prevent</i> you from testing the caret fixits. On the other hand, until recently FileCheck didn't have a way to specify relative line numbers, meaning that adding a new line at the top of the file would change all of the parseable fixits (unless matched with wildcards).</div>


<div><br></div><div>Going forward we should probably endeavour to test the parseable fixits for every test where we're not <i>just</i> testing caret fixit emission.</div><div><br></div><div>Jordan</div><div><br></div>


<br><div><div><div><div>On Dec 1, 2012, at 21:05 , Michael Han <<a href="mailto:fragmentshaders@gmail.com" target="_blank">fragmentshaders@gmail.com</a>> wrote:</div><br></div></div><blockquote type="cite">
<div><div>Hi Dmitri,<br><br>Thanks! Attach updated patch.<br><br>I noticed that not all test cases under clang/test/FixIt use "-fdiagnostics-parseable-fixits". Is there a reason why not all FixIt test cases use this option?<br>




<br>Cheers<br>Michael<br><br><div class="gmail_quote">On Sat, Dec 1, 2012 at 5:21 AM, Dmitri Gribenko <span dir="ltr"><<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>On Sat, Dec 1, 2012 at 6:30 AM, Michael Han <<a href="mailto:fragmentshaders@gmail.com" target="_blank">fragmentshaders@gmail.com</a>> wrote:<br>
> This patch adds two FixIts to parser when parsing C++11 attributes appertain<br>
> to class specifier. It is a following up patch of a previous patch [1].<br>
><br>
> Please review, thanks!<br>
<br>
</div>Hello Michael,<br>
<br>
FixIts can be tested with -fdiagnostics-parseable-fixits (search the<br>
testsuite for this option to find examples).<br>
<span><font color="#888888"><br>
Dmitri<br>
<br>
--<br>
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if<br>
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>>*/<br>
</font></span></blockquote></div><br>
</div></div><span><fixit.patch></span>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>


</blockquote></div><br></div></blockquote></div><br>
</div></div></blockquote></div><br></div></div></div></div></div></div>
</blockquote></div><br>