llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-tools-extra Author: Baranov Victor (vbvictor) <details> <summary>Changes</summary> Fixes https://github.com/llvm/llvm-project/issues/52859. --- Full diff: https://github.com/llvm/llvm-project/pull/221428.diff 4 Files Affected: - (modified) clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp (+3-4) - (modified) clang-tools-extra/docs/ReleaseNotes.md (+4) - (modified) clang-tools-extra/unittests/clang-tidy/IncludeInserterTest.cpp (-9) - (modified) clang-tools-extra/unittests/clang-tidy/TransformerClangTidyCheckTest.cpp (-2) ``````````diff diff --git a/clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp b/clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp index b7c313693781d..826c3d2676e88 100644 --- a/clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp +++ b/clang-tools-extra/clang-tidy/utils/IncludeSorter.cpp @@ -199,19 +199,18 @@ IncludeSorter::createIncludeInsertion(StringRef FileName, bool IsAngled) { return std::nullopt; if (NonEmptyKind < IncludeKind) { - // Create a block after. + // Insert directly after the nearest preceding group. + // This keeps clang-format able to fix up the ordering within group. const std::string &LastInclude = IncludeBucket[NonEmptyKind].back(); const SourceRange LastIncludeLocation = IncludeLocations[LastInclude].back(); - IncludeStmt.insert(0, LineEnding); return FixItHint::CreateInsertion(LastIncludeLocation.getEnd(), IncludeStmt); } - // Create a block before. + // Insert directly before the nearest group for the same reason as above. const std::string &FirstInclude = IncludeBucket[NonEmptyKind][0]; const SourceRange FirstIncludeLocation = IncludeLocations[FirstInclude].back(); - IncludeStmt.append(LineEnding); return FixItHint::CreateInsertion(FirstIncludeLocation.getBegin(), IncludeStmt); } diff --git a/clang-tools-extra/docs/ReleaseNotes.md b/clang-tools-extra/docs/ReleaseNotes.md index b32b8a51e0606..c0ff5b0cc096b 100644 --- a/clang-tools-extra/docs/ReleaseNotes.md +++ b/clang-tools-extra/docs/ReleaseNotes.md @@ -261,6 +261,10 @@ infrastructure are described first, followed by tool-specific sections. #### Miscellaneous +- Fixed clang-tidy adding an unwanted blank line when automatically inserting + a missing `#include`, which stopped `clang-format` and `llvm-include-order` + from being able to sort it correctly afterward. + ### Improvements to include-fixer ### Improvements to clang-include-fixer diff --git a/clang-tools-extra/unittests/clang-tidy/IncludeInserterTest.cpp b/clang-tools-extra/unittests/clang-tidy/IncludeInserterTest.cpp index d01ba9814b8a1..b01bcc8e69d63 100644 --- a/clang-tools-extra/unittests/clang-tidy/IncludeInserterTest.cpp +++ b/clang-tools-extra/unittests/clang-tidy/IncludeInserterTest.cpp @@ -392,7 +392,6 @@ void foo() { #include <list> #include <map> - #include "path/to/header.h" void foo() { @@ -413,7 +412,6 @@ void foo() { })"; const char *PostCode = R"( #include "clang_tidy/tests/insert_includes_test_header.h" - #include "path/to/header.h" void foo() { @@ -524,7 +522,6 @@ void foo() { })"; const char *PostCode = R"( #include "clang_tidy/tests/insert_includes_test_header.h" - #include <set> #include "path/to/a/header.h" @@ -553,7 +550,6 @@ void foo() { #include "clang_tidy/tests/insert_includes_test_header.h" #include <stdlib.h> - #include <set> #include "path/to/a/header.h" @@ -576,7 +572,6 @@ void foo() { })"; const char *PostCode = R"( #include <set> - #include "path/to/a/header.h" void foo() { @@ -600,7 +595,6 @@ void foo() { })"; const char *PostCode = R"( #include <stdlib.h> - #include <set> #include "path/to/a/header.h" @@ -718,7 +712,6 @@ void foo() { })"; const char *PostCode = R"( #include "clang_tidy/tests/insert_includes_test_header.h" - #include <c.h> #include <d> @@ -747,7 +740,6 @@ void foo() { })"; const char *PostCode = R"( #import "clang_tidy/tests/insert_includes_test_header.h" - #import "a/header.h" void foo() { @@ -798,7 +790,6 @@ void foo() { #include <map> #include "path/to/a/header.h" - #import "clang_tidy/tests/generated_file.proto.h" void foo() { diff --git a/clang-tools-extra/unittests/clang-tidy/TransformerClangTidyCheckTest.cpp b/clang-tools-extra/unittests/clang-tidy/TransformerClangTidyCheckTest.cpp index fc43f2579e8e2..612229a4f1eb1 100644 --- a/clang-tools-extra/unittests/clang-tidy/TransformerClangTidyCheckTest.cpp +++ b/clang-tools-extra/unittests/clang-tidy/TransformerClangTidyCheckTest.cpp @@ -335,7 +335,6 @@ TEST(TransformerClangTidyCheckTest, AddIncludeObeysSortStyleLocalOption) { int h(int x) { return 3; })cc"; std::string TreatsAsLibraryHeader = R"cc(#include "input.h" - #include "bar.h" int h(int x) { return 5; })cc"; @@ -367,7 +366,6 @@ TEST(TransformerClangTidyCheckTest, AddIncludeObeysSortStyleGlobalOption) { int h(int x) { return 3; })cc"; std::string TreatsAsLibraryHeader = R"cc(#include "input.h" - #include "bar.h" int h(int x) { return 5; })cc"; `````````` </details> https://github.com/llvm/llvm-project/pull/221428 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
