<div dir="ltr">clang-apply-replacements has some legacy code and needs some refactoring in order to take advantage of the cleanup feature (because of the new tooling::Replacements implementation). I had a plan to refactor it but thought it was a low priority since most refactoring tools are using replacements application functions in tooling::RefactoringTool, and I also have a plan to replace clang-apply-replacements with a better infrastructure that can supports more refactoring actions. </div><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 18, 2016 at 11:46 AM Malcolm Parsons <<a href="mailto:malcolm.parsons@gmail.com">malcolm.parsons@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">malcolm.parsons added a comment.<br class="gmail_msg">
<br class="gmail_msg">
Does clang-apply-replacements need a similar change?<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
Repository:<br class="gmail_msg">
  rL LLVM<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D24572" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D24572</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>