[PATCH] D24192: [clang-refactor] introducing clang-refactor

[Alex Lorenz via cfe-commits]

arphaman added inline comments.

================
Comment at: clang-refactor/driver/Driver.cpp:36
@@ +35,3 @@
+    argv[1] = (llvm::Twine(argv[0]) + " " + llvm::Twine(argv[1])).str().c_str();
+    Manager.dispatch(Command, argc - 1, argv + 1);
+  } else {
----------------
Is there a particular reason why you don't return a value that's returned by Manager.dispatch here? It seems odd to me that Manager.dispatch returns an integer value that's not actually used here.


https://reviews.llvm.org/D24192