[clang] [clang] Ignore inline namespace for `hasName` (PR #109147)

Alejandro Álvarez Ayllón via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 10 03:24:59 PDT 2024


alejandro-alvarez-sonarsource wrote:

> > Or even SuppressInlineNamespace should be changed from bool to some enum like None, Redundant, All.
> 
> I think this is the suggestion I'd prefer; it's cleaner than adding a second boolean option.

Agreed. And changes. 

> Make sure that all current test (specially from clang-tidy) still pass.

I have run both `check-clang` and `check-clang-tools` (locally) without failures.


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


More information about the cfe-commits mailing list