[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