[all-commits] [llvm/llvm-project] 92910a: [clang-tidy] fix concat-nest-namespace fix hint re...

Congcong Cai via All-commits all-commits at lists.llvm.org
Fri Apr 7 21:26:16 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 92910a51b9043550cec6c9e63a5b92a15d42c665
      https://github.com/llvm/llvm-project/commit/92910a51b9043550cec6c9e63a5b92a15d42c665
  Author: Congcong Cai <congcong.ca.cai at bmw.com>
  Date:   2023-04-08 (Sat, 08 Apr 2023)

  Changed paths:
    M clang-tools-extra/clang-tidy/modernize/ConcatNestedNamespacesCheck.cpp
    M clang-tools-extra/clang-tidy/modernize/ConcatNestedNamespacesCheck.h
    M clang-tools-extra/clang-tidy/utils/LexerUtils.cpp
    M clang-tools-extra/clang-tidy/utils/LexerUtils.h
    M clang-tools-extra/docs/ReleaseNotes.rst
    M clang-tools-extra/test/clang-tidy/checkers/modernize/Inputs/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] fix concat-nest-namespace fix hint remove the macro

Partially fixed [#60035](https://github.com/llvm/llvm-project/issues/60035)
This patch refactor the FixHint for concat-nest-namespace.
1. remove each namespace except the last non-nest namespace.
2. replace the last non-nest namespace with the new name.

It can remain the comment / pragma / macro between namespace and update the close comment.

Reviewed By: PiotrZSL

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




More information about the All-commits mailing list