[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