[clang-tools-extra] [clang-tidy] Update llvmlibc-implementation-in-namespace to new rules (PR #66504)

Guillaume Chatelet via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 21 05:38:38 PDT 2023


gchatelet wrote:

> As a single small change to make hardcoded namespaces configurable looks fine. I added some comments related to overall issues in this check. Fell free to fix them or ignore them.

Thx for the review! I had to restructure the code a bit to accommodate for anonymous namespaces (they generate an implict UsingDirectiveDecl that was triggering the warning twice).

Let me know what you think.

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


More information about the cfe-commits mailing list