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

Piotr Padlewski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 15 10:51:56 PST 2016


Prazek added a comment.

I would say that as long as most of the people don't see big difference between "end namespace X" and "namespace X" or even "X" then we should not be so strict. For me "end" is redundant. 
Even checks added by add_new_check uses "namespace clang" instead of "end namespace clang"


Repository:
  rL LLVM

https://reviews.llvm.org/D24487





More information about the cfe-commits mailing list