Eugene.Zelenko added inline comments.
================
Comment at: clang-tools-extra/clang-doc/tool/ClangDocMain.cpp:205
+ llvm::cl::OptionCategory &Category) {
+ auto OptionsParser =
+ CommonOptionsParser::create(argc, argv, Category, llvm::cl::ZeroOrMore);
----------------
Please don't use auto when type is not spelled in same statement.
================
Comment at: clang-tools-extra/clang-doc/tool/ClangDocMain.cpp:209
+ return OptionsParser.takeError();
+ for (auto I = ToolExecutorPluginRegistry::begin(),
+ E = ToolExecutorPluginRegistry::end();
----------------
Please use range loop.
https://reviews.llvm.org/D53170
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
