https://github.com/andergnet created https://github.com/llvm/llvm-project/pull/122282
This PR fixes the issue https://github.com/llvm/llvm-project/issues/106228 were the C++ ">>" operator got a different formatting than the "<<" one when using BreakBinaryOperations. This is my first PR on the project don't be too harsh ;) >From e2780f01d47518bb61a5c01c9ad4d9daddb5044a Mon Sep 17 00:00:00 2001 From: W123011 <ander.ge...@worldline.com> Date: Thu, 9 Jan 2025 15:04:26 +0100 Subject: [PATCH 1/2] Fix >> behavior on continuation intenter --- clang/lib/Format/ContinuationIndenter.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/clang/lib/Format/ContinuationIndenter.cpp b/clang/lib/Format/ContinuationIndenter.cpp index 554b55fa75c926..5e5c93e2a5286f 100644 --- a/clang/lib/Format/ContinuationIndenter.cpp +++ b/clang/lib/Format/ContinuationIndenter.cpp @@ -147,7 +147,8 @@ static bool startsNextOperand(const FormatToken &Current) { // Returns \c true if \c Current is a binary operation that must break. static bool mustBreakBinaryOperation(const FormatToken &Current, const FormatStyle &Style) { - return Style.BreakBinaryOperations != FormatStyle::BBO_Never && + return Current.CanBreakBefore && + Style.BreakBinaryOperations != FormatStyle::BBO_Never && (Style.BreakBeforeBinaryOperators == FormatStyle::BOS_None ? startsNextOperand : isAlignableBinaryOperator)(Current); >From 4a5a1368b8a2ae577cb6e142022b563a3c77d583 Mon Sep 17 00:00:00 2001 From: W123011 <ander.ge...@worldline.com> Date: Thu, 9 Jan 2025 15:47:08 +0100 Subject: [PATCH 2/2] Unit test --- clang/unittests/Format/FormatTest.cpp | 72 +++++++++++++++------------ 1 file changed, 41 insertions(+), 31 deletions(-) diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp index 4d48bcacddead8..c9cf03bff6acce 100644 --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -22301,16 +22301,16 @@ TEST_F(FormatTest, HandlesUTF8BOM) { #if !defined(_MSC_VER) TEST_F(FormatTest, CountsUTF8CharactersProperly) { - verifyFormat("\"Однажды в студёную зимнюю пору...\"", + verifyFormat("\"Однажды в ѝтудёную зимнюю пору...\"", getLLVMStyleWithColumns(35)); - verifyFormat("\"一 二 三 四 五 六 七 八 九 十\"", + verifyFormat("\"一 二 三 四 五 六 七 八 九 坝\"", getLLVMStyleWithColumns(31)); - verifyFormat("// Однажды в студёную зимнюю пору...", + verifyFormat("// Однажды в ѝтудёную зимнюю пору...", getLLVMStyleWithColumns(36)); - verifyFormat("// 一 二 三 四 五 六 七 八 九 十", getLLVMStyleWithColumns(32)); - verifyFormat("/* Однажды в студёную зимнюю пору... */", + verifyFormat("// 一 二 三 四 五 六 七 八 九 坝", getLLVMStyleWithColumns(32)); + verifyFormat("/* Однажды в ѝтудёную зимнюю пору... */", getLLVMStyleWithColumns(39)); - verifyFormat("/* 一 二 三 四 五 六 七 八 九 十 */", + verifyFormat("/* 一 二 三 四 五 六 七 八 九 坝 */", getLLVMStyleWithColumns(35)); } @@ -22329,18 +22329,18 @@ TEST_F(FormatTest, SplitsUTF8Strings) { format("\"aaaaaaaÄ\xc2\x8d\";", getLLVMStyleWithColumns(10))); // FIXME: unstable test case EXPECT_EQ("\"Однажды, в \"\n" - "\"студёную \"\n" + "\"ѝтудёную \"\n" "\"зимнюю \"\n" "\"пору,\"", - format("\"Однажды, в студёную зимнюю пору,\"", + format("\"Однажды, в ѝтудёную зимнюю пору,\"", getLLVMStyleWithColumns(13))); // FIXME: unstable test case EXPECT_EQ( "\"一 二 三 \"\n" "\"四 五六 \"\n" "\"七 八 九 \"\n" - "\"十\"", - format("\"一 二 三 四 五六 七 八 九 十\"", getLLVMStyleWithColumns(11))); + "\"坝\"", + format("\"一 二 三 四 五六 七 八 九 坝\"", getLLVMStyleWithColumns(11))); // FIXME: unstable test case EXPECT_EQ("\"一\t\"\n" "\"二 \t\"\n" @@ -22348,8 +22348,8 @@ TEST_F(FormatTest, SplitsUTF8Strings) { "\"五\t\"\n" "\"六 \t\"\n" "\"七 \"\n" - "\"八九十\tqq\"", - format("\"一\t二 \t三 四 五\t六 \t七 八九十\tqq\"", + "\"八九坝\tqq\"", + format("\"一\t二 \t三 四 五\t六 \t七 八九坝\tqq\"", getLLVMStyleWithColumns(11))); // UTF8 character in an escape sequence. @@ -22362,44 +22362,44 @@ TEST_F(FormatTest, SplitsUTF8Strings) { TEST_F(FormatTest, HandlesDoubleWidthCharsInMultiLineStrings) { verifyFormat("const char *sssss =\n" " \"一二三四五六七八\\\n" - " 九 十\";", + " 九 坝\";", "const char *sssss = \"一二三四五六七八\\\n" - " 九 十\";", + " 九 坝\";", getLLVMStyleWithColumns(30)); } TEST_F(FormatTest, SplitsUTF8LineComments) { verifyFormat("// aaaaÄ\xc2\x8d", getLLVMStyleWithColumns(10)); - verifyFormat("// Я из лесу\n" + verifyFormat("// Я из леѝу\n" "// вышел; был\n" - "// сильный\n" + "// ѝильный\n" "// мороз.", - "// Я из лесу вышел; был сильный мороз.", + "// Я из леѝу вышел; был ѝильный мороз.", getLLVMStyleWithColumns(13)); verifyFormat("// 一二三\n" "// 四五六七\n" "// 八 九\n" - "// 十", - "// 一二三 四五六七 八 九 十", getLLVMStyleWithColumns(9)); + "// 坝", + "// 一二三 四五六七 八 九 坝", getLLVMStyleWithColumns(9)); } TEST_F(FormatTest, SplitsUTF8BlockComments) { - verifyFormat("/* Гляжу,\n" - " * поднимается\n" + verifyFormat("/* Глѝжу,\n" + " * поднимаетѝѝ\n" " * медленно в\n" " * гору\n" " * Лошадка,\n" - " * везущая\n" - " * хворосту\n" + " * везущаѝ\n" + " * хвороѝту\n" " * воз. */", - "/* Гляжу, поднимается медленно в гору\n" - " * Лошадка, везущая хворосту воз. */", + "/* Глѝжу, поднимаетѝѝ медленно в гору\n" + " * Лошадка, везущаѝ хвороѝту воз. */", getLLVMStyleWithColumns(13)); verifyFormat("/* 一二三\n" " * 四五六七\n" " * 八 九\n" - " * 十 */", - "/* 一二三 四五六七 八 九 十 */", getLLVMStyleWithColumns(9)); + " * 坝 */", + "/* 一二三 四五六七 八 九 坝 */", getLLVMStyleWithColumns(9)); verifyFormat("/* 𝓣𝓮𝓼𝓽 𝔣𝔬𝔲𝔯\n" " * 𝕓𝕪𝕥𝕖\n" " * 𝖀𝕿𝕱-𝟠 */", @@ -27908,9 +27908,9 @@ TEST_F(FormatTest, BreakAdjacentStringLiterals) { TEST_F(FormatTest, AlignUTFCommentsAndStringLiterals) { verifyFormat( - "int rus; // А теперь комментарии, например, на русском, 2-байта\n" - "int long_rus; // Верхний коммент еще не превысил границу в 80, однако\n" - " // уже отодвинут. Перенос, при этом, отрабатывает верно"); + "int rus; // Н теперь комментарии, например, на руѝѝком, 2-байта\n" + "int long_rus; // Верхний коммент еще не превыѝил границу в 80, однако\n" + " // уже отодвинут. Переноѝ, при ѝтом, отрабатывает верно"); auto Style = getLLVMStyle(); Style.ColumnLimit = 15; @@ -27940,7 +27940,7 @@ TEST_F(FormatTest, AlignUTFCommentsAndStringLiterals) { verifyFormat("Languages languages = {\n" " Language{{'e', 'n'}, U\"Test English\" },\n" " Language{{'l', 'v'}, U\"Test Latviešu\"},\n" - " Language{{'r', 'u'}, U\"Test Русский\" },\n" + " Language{{'r', 'u'}, U\"Test Руѝѝкий\" },\n" "};", Style); } @@ -28188,6 +28188,16 @@ TEST_F(FormatTest, BreakBinaryOperations) { " | byte_buffer[2] << 16\n" " | byte_buffer[3] << 24;", Style); + + Style.BreakBinaryOperations = FormatStyle::BBO_OnePerLine; + // Check operator >> special case + verifyFormat("std::cout\n" + " << longOperand1\n" + " << longOperand2\n" + " << longOperand3\n" + " << longOperand4\n" + " << longOperand5;", + Style); } TEST_F(FormatTest, RemoveEmptyLinesInUnwrappedLines) { _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
