[PATCH] D24183: A clang tool for changing surrouding namespaces of class/function definitions.

Alexander Shaposhnikov via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 7 11:51:18 PDT 2016


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





More information about the cfe-commits mailing list