alexshap added inline comments.

================
Comment at: change-namespace/tool/ClangChangeNamespace.cpp:48
@@ +47,3 @@
+
+cl::opt<std::string> OldNamespace("old_namespace", cl::desc("Old namespace."),
+                                  cl::cat(ChangeNamespaceCategory));
----------------
probably you need to add cl::Required
   cl::opt<std::string> OldNamespace("old_namespace", cl::Required, 
cl::desc("Old namespace."),
                                                            
cl::cat(ChangeNamespaceCategory));

and the same for NewNamespace.
 


https://reviews.llvm.org/D24183



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

Reply via email to