<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 18, 2013 at 2:58 PM, Ismail Pazarbasi <span dir="ltr"><<a href="mailto:ismail.pazarbasi@gmail.com" target="_blank">ismail.pazarbasi@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thank you for feedback. I have updated the patch, so that it adds<br>
fix-it test into the existing test/FixIt/fixit.cpp, and entire patch<br>
is in single file.<br></blockquote><div><br></div><div style>(you may've forgotten to attach the latest iteration?)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5"><br>
On Mon, Feb 18, 2013 at 10:50 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
><br>
><br>
><br>
> On Mon, Feb 18, 2013 at 1:40 PM, Ismail Pazarbasi<br>
> <<a href="mailto:ismail.pazarbasi@gmail.com">ismail.pazarbasi@gmail.com</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> I have made this a while ago, but didn't have a chance to submit it.<br>
>><br>
>> When an invalid destructor name is found within a class scope, a<br>
>> fix-it hint will be issued to correct the destructor name so that it<br>
>> matches the class name.<br>
><br>
><br>
> Looks pretty good. Rather than adding another test file - perhaps you could<br>
> just add this fixit case to the general test/FixIt/fixit.cpp or similarly<br>
> general test case (avoiding extra process invocations keeps the regression<br>
> test execution time down)<br>
><br>
> For future reference: providing a single patch that includes both production<br>
> changes & tests is fine/easy/good.<br>
</div></div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">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>
</div></div></blockquote></div><br></div></div>