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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to