[PATCH] D61989: [clang-tidy] enable modernize-concat-nested-namespaces on header files

Mon Feb 22 05:36:02 PST 2021

njames93 added a comment.

In D61989#2578687 <https://reviews.llvm.org/D61989#2578687>, @DmitryPolukhin wrote:

> Everyone please speak up if there is a good reason to limit this check to the main file only.

The only possible reason I can see to limit to main file only is if the header is included in pre-c++17 and post c++17 translation units. Though that is a very niche use case IMO.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61989/new/

https://reviews.llvm.org/D61989