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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits