Author: Owen Pan Date: 2025-03-03T17:40:02-08:00 New Revision: 0ed2945a596991b75e4ca090fe04240abba6012b
URL: https://github.com/llvm/llvm-project/commit/0ed2945a596991b75e4ca090fe04240abba6012b DIFF: https://github.com/llvm/llvm-project/commit/0ed2945a596991b75e4ca090fe04240abba6012b.diff LOG: [clang-format] Don't sort includes for C# (#129369) Fixes #106194 Added: Modified: clang/lib/Format/Format.cpp clang/unittests/Format/SortIncludesTest.cpp Removed: ################################################################################ diff --git a/clang/lib/Format/Format.cpp b/clang/lib/Format/Format.cpp index 92678a031178a..c699a96d3f45e 100644 --- a/clang/lib/Format/Format.cpp +++ b/clang/lib/Format/Format.cpp @@ -3547,7 +3547,8 @@ tooling::Replacements sortIncludes(const FormatStyle &Style, StringRef Code, return sortJavaScriptImports(Style, Code, Ranges, FileName); if (Style.Language == FormatStyle::LanguageKind::LK_Java) return sortJavaImports(Style, Code, Ranges, FileName, Replaces); - sortCppIncludes(Style, Code, Ranges, FileName, Replaces, Cursor); + if (Style.isCpp()) + sortCppIncludes(Style, Code, Ranges, FileName, Replaces, Cursor); return Replaces; } diff --git a/clang/unittests/Format/SortIncludesTest.cpp b/clang/unittests/Format/SortIncludesTest.cpp index cb3f8c73a0487..f20862f5f7461 100644 --- a/clang/unittests/Format/SortIncludesTest.cpp +++ b/clang/unittests/Format/SortIncludesTest.cpp @@ -1085,6 +1085,15 @@ TEST_F(SortIncludesTest, DoNotSortLikelyXml) { "input.h", 0)); } +TEST_F(SortIncludesTest, DoNotSortCSharp) { + constexpr StringRef Code{"const string expectedDataStruct = @\"\n" + " #include <b.h>\n" + " #include <a.h>\n" + " \";"}; + FmtStyle.Language = FormatStyle::LK_CSharp; + EXPECT_TRUE(sortIncludes(FmtStyle, Code, GetCodeRange(Code), "a.cs").empty()); +} + TEST_F(SortIncludesTest, DoNotOutputReplacementsForSortedBlocksWithRegrouping) { Style.IncludeBlocks = Style.IBS_Regroup; StringRef Code = "#include \"b.h\"\n" _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
