<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Aug 7, 2014 at 7:47 PM, Matthew Plant <span dir="ltr"><<a href="mailto:mplant@google.com" target="_blank">mplant@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">I'll get rid of the operator renaming code and take a look at unit tests.<br>

<div class=""><br>
================<br>
Comment at: clang-rename/ClangRename.cpp:69-70<br>
@@ +68,4 @@<br>
+newFrontendActionFactory(ActionT *Action) {<br>
+  // Renaming CreateASTConsumer to createASTConsumer doesn't work, so we have to<br>
+  // create our own custom factory.<br>
+  struct SimpleFactory : public tooling::FrontendActionFactory {<br>
----------------<br>
</div><div class="">Manuel Klimek wrote:<br>
> Can you explain what exactly didn't work? Perhaps give me the compile error? But it's no biggie - I can also get rid of this once it's checked in - the template error messages are sometimes hard to get through :)<br>

</div>It spat out a virtual method not implemented error. I've grepped through the code, and I couldn't find one interface with a virtual method createASTConsumer. Did you mean some method with a different name?<br>
</blockquote><div><br></div><div>Nope, I meant the newFrontendActionFactory function from Tooling.h that is templated to take anything that has a newASTConsumer method. Ah, I think it's new vs. create here, sorry for the typo...</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
<a href="http://reviews.llvm.org/D4739" target="_blank">http://reviews.llvm.org/D4739</a><br>
<br>
<br>
</blockquote></div><br></div></div>