[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