[PATCH] D18810: [Clang-tidy] Fix readability-static-definition-in-anonymous-namespace warnings; other minor fixes.

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 7 11:33:34 PDT 2016


alexfh added a comment.

Actually, removal of `static` is not the right thing for LLVM:

> Because of this, we have a simple guideline: make anonymous namespaces as small as possible, and only use them for class declarations.


http://llvm.org/docs/CodingStandards.html#anonymous-namespaces


Repository:
  rL LLVM

http://reviews.llvm.org/D18810





More information about the cfe-commits mailing list