[PATCH] D140290: [clang-tidy] Add misc-static-declaration-in-header check

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 3 01:13:18 PST 2023


carlosgalvezp added a comment.

Friendly ping. I realized @aaron.ballman you probably have very good insights into this use case, since (AFAICT) you wrote a guideline about anonymous namespaces in headers, which is very closely related to this:
https://wiki.sei.cmu.edu/confluence/display/cplusplus/DCL59-CPP.+Do+not+define+an+unnamed+namespace+in+a+header+file

Would appreciate your feedback!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140290/new/

https://reviews.llvm.org/D140290



More information about the cfe-commits mailing list