[clang-tools-extra] [clang-tidy] Option to ignore anonymous namespaces in avoid-non-const-global-variables (PR #93827)

Carlos Galvez via cfe-commits cfe-commits at lists.llvm.org
Thu May 30 09:23:07 PDT 2024


carlosgalvezp wrote:

I have a bit the feeling that users should use NOLINT* here instead of having a global option that is not seen in the code. Anonymous namespaces do not solve many of the remaining issues. A NOLINT in the code can grab the attention of a future developer and prompt a refactoring to avoid the pattern.

I don't have a strong opinion so if this is really wanted I won't object.

https://github.com/llvm/llvm-project/pull/93827


More information about the cfe-commits mailing list