<div dir="ltr"><div class="gmail_quote"><div>cc Daniel, who might have more memory of that time :)</div><div dir="ltr"><br></div><div dir="ltr">On Wed, Feb 1, 2017 at 3:01 AM Julian Bangert via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">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" class="gmail_msg">Hello,<div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">how to make RefactoringCallback and RefactoringTool play nicely together? </div><div class="gmail_msg">Each RefactoringCallback produces its own Replacements, but RefactoringTool wants FrontendActions that populate the Replacements returned by RefactoringTool::getReplacements(). </div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">I could write an adapter that does this (consume many callbacks, at the end of each translation unit move from replacements and clear the old ones), but that seems pretty inelegant. Do we care about API stability for RefactoringCallback? It might be more elegant to change RefactoringCallback to contain a reference to a map<string, Replacements> and add directly to that. </div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Would you be willing to accept a patch that does this? </div></div></blockquote><div><br></div><div>a) we don't care about API stability here - if we find a better way for the API, we go for it</div><div>b) I don't think RefactoringCallback is really in use - we built this waaaay back before there was RefactoringTool; are you sure you need it (as opposed to us just deleting it and introducing an abstraction that makes more sense)?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Regards,</div><div class="gmail_msg">Julian </div></div>
_______________________________________________<br class="gmail_msg">
cfe-dev mailing list<br class="gmail_msg">
<a href="mailto:cfe-dev@lists.llvm.org" class="gmail_msg" target="_blank">cfe-dev@lists.llvm.org</a><br class="gmail_msg">
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br class="gmail_msg">
</blockquote></div></div>