<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 18, 2013 at 1:40 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">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.</blockquote><div><br></div><div style>Looks pretty good. Rather than adding another test file - perhaps you could just add this fixit case to the general test/FixIt/fixit.cpp or similarly general test case (avoiding extra process invocations keeps the regression test execution time down)<br>
<br>For future reference: providing a single patch that includes both production changes & tests is fine/easy/good. </div></div></div></div>