<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 13, 2012 at 6:58 PM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank" class="cremed">klimek@google.com</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">On Fri, Jul 13, 2012 at 5:53 PM, Daniel Jasper <<a href="mailto:djasper@google.com" class="cremed">djasper@google.com</a>> wrote:<br>

> New patch attached:<br>
> - More tests.<br>
> - Fixed line length.<br>
> - Better integer literal.<br>
<br>
</div>Are you sure you want StringRef as class members here?<br></blockquote><div><br></div><div>Oops. Sorry. Fixed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
+    } else {<br>
+      // If we don't have the appropriate clause, then just delete<br>
the whole if.<br>
+      if (!PickTrueBranch)<br>
+        Replace.insert(replaceStmtWithText(*Result.SourceManager, *Node, ""));<br>
+    }<br>
<br>
I'd make that<br>
} else if (!PickTrueBranch) {<br>
  // If we want to use the 'else'-branch, but it doesn't exist, delete<br>
the whole 'if'.<br></blockquote><div><br></div><div>Done.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+void run(const std::string &Code,<br>
<br>
I'd call that expectRewritten or something similar.<br></blockquote><div><br></div><div>Done.</div><div><br></div><div>New patch attached.</div><div><br>Cheers,<br>Daniel</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Cheers,<br>
/Manuel<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
><br>
> On Fri, Jul 13, 2012 at 5:07 PM, Daniel Jasper <<a href="mailto:djasper@google.com" class="cremed">djasper@google.com</a>> wrote:<br>
>><br>
>> :-D. Will change before submitting ...<br>
>><br>
>><br>
>> On Fri, Jul 13, 2012 at 4:34 PM, Jean-Daniel Dupas<br>
>> <<a href="mailto:devlists@shadowlab.org" class="cremed">devlists@shadowlab.org</a>> wrote:<br>
>>><br>
>>><br>
>>> +//<br>
>>> +//  This will replace all integer literals with "2".<br>
>>> +//<br>
>>><br>
>>> Isn't the universal answer 42 instead of 2 ? ;-)<br>
>>><br>
>>> Le 13 juil. 2012 à 16:22, Daniel Jasper <<a href="mailto:djasper@google.com" class="cremed">djasper@google.com</a>> a écrit :<br>
>>><br>
>>> > With these callbacks, simple refactorings should be quite easy. It is a<br>
>>> > starting point and can be extended/improved in many ways.<br>
>>> > <refactoring-callbacks.patch>_______________________________________________<br>
>>> > cfe-commits mailing list<br>
>>> > <a href="mailto:cfe-commits@cs.uiuc.edu" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
>>> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
>>><br>
>>> -- Jean-Daniel<br>
>>><br>
>>><br>
>>><br>
>>><br>
>><br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</div></div></blockquote></div><br></div>