[all-commits] [llvm/llvm-project] da55af: [clang-tidy] Enable modernize-concat-nested-namesp...

[Dmitry Polukhin via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: da55af7f1d348c133774d8e8117d60462363fef5
      https://github.com/llvm/llvm-project/commit/da55af7f1d348c133774d8e8117d60462363fef5
  Author: Dmitry Polukhin <dmitry.polukhin at gmail.com>
  Date:   2021-03-15 (Mon, 15 Mar 2021)

  Changed paths:
    M clang-tools-extra/clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp
    A clang-tools-extra/test/clang-tidy/checkers/Inputs/modernize-concat-nested-namespaces/modernize-concat-nested-namespaces.h
    M clang-tools-extra/test/clang-tidy/checkers/modernize-concat-nested-namespaces.cpp

  Log Message:
  -----------
  [clang-tidy] Enable modernize-concat-nested-namespaces also on headers

For some reason the initial implementation of the check had an explicit check
for the main file to avoid being applied in headers. This diff removes this
check and add a test for the check on a header.

Similar approach was proposed in D61989 but review there got stuck.

Test Plan: added new test case

Differential Revision: https://reviews.llvm.org/D97563