[PATCH] Clang Rename Tool

Matthew Plant mplant at google.com
Thu Aug 7 10:47:59 PDT 2014


I'll get rid of the operator renaming code and take a look at unit tests.

================
Comment at: clang-rename/ClangRename.cpp:69-70
@@ +68,4 @@
+newFrontendActionFactory(ActionT *Action) {
+  // Renaming CreateASTConsumer to createASTConsumer doesn't work, so we have to
+  // create our own custom factory.
+  struct SimpleFactory : public tooling::FrontendActionFactory {
----------------
Manuel Klimek wrote:
> 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 :)
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?

http://reviews.llvm.org/D4739






More information about the cfe-commits mailing list