<div dir="ltr">I cleared the replacements in between and it still corrupted the file.  I am spending some time trying to track it down.  Easy work around is to create two tool instances without overlapped lifetimes.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 1, 2015 at 4:58 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">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 dir="ltr"><div class="gmail_quote"><div><div class="h5"><div dir="ltr">On Mon, Aug 31, 2015 at 10:07 PM Daniel Dilts via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>I have a pair of FrontEndActions that I am running against a source file.  The first one modifies the token stream.  The second one modifies the AST.</div><div><br></div><div>I am running them both using the same RefactoringTool instance.  When both actions modify the source it corrupts the file.  Looking at it more closely it appears to be caused by runAndSave() not clearing out RefactoringTool::Replace after applying the replacements.</div><div><br></div><div>Should runAndSave() clear the replacements collection after applying them or is the current behavior correct?</div></div></blockquote><div><br></div></div></div><div>So far I never thought of anybody wanting to call runAndSave multiple times. If we can make it work, you're right, we should clear the replacements in between - feel free to send me a patch to that end :)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" target="_blank" rel="noreferrer">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>
</blockquote></div></div>
</blockquote></div><br></div>