[PATCH] D24487: [Clang] Fix some Clang-tidy modernize-use-bool-literals and Include What You Use warnings in AST; other minor fixes

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 15 10:32:51 PST 2016


Eugene.Zelenko added inline comments.


================
Comment at: lib/ASTMatchers/Dynamic/Parser.cpp:617
 
-}  // namespace dynamic
-}  // namespace ast_matchers
-}  // namespace clang
+} // end namespace dynamic
+} // end namespace ast_matchers
----------------
Prazek wrote:
> I think check should not warn in these cases.
I think it's reasonable to unify such comments. Three different styles are too much for one project :-)


Repository:
  rL LLVM

https://reviews.llvm.org/D24487





More information about the cfe-commits mailing list