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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
