ioeric added inline comments.
================
Comment at: change-namespace/tool/ClangChangeNamespace.cpp:68
+cl::opt<bool>
+ DumpYAML("dump_yaml",
+ cl::desc("Dump new file content in YAML, if specified."),
----------------
hokein wrote:
> hokein wrote:
> > `dump_result` maybe a clearer name, which also is consistent with
> > clang-move's.
> Nit: Also change the variable name to `DumpResult`.
Since I am also dumping output as plain text, I named this `DumpYAML` for
clarity.
https://reviews.llvm.org/D29893
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
