github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff origin/main HEAD --extensions cpp,h --
clang/include/clang/Format/Format.h clang/lib/Format/ContinuationIndenter.cpp
clang/lib/Format/Format.cpp clang/lib/Format/FormatToken.cpp
clang/lib/Format/TokenAnnotator.cpp
clang/unittests/Format/AlignBracketsTest.cpp
clang/unittests/Format/ConfigParseTest.cpp
clang/unittests/Format/FormatTest.cpp
clang/unittests/Format/FormatTestComments.cpp
clang/unittests/Format/FormatTestObjC.cpp --diff_from_common_commit
``````````
:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/unittests/Format/AlignBracketsTest.cpp
b/clang/unittests/Format/AlignBracketsTest.cpp
index d61ce1eb4..cd3143057 100644
--- a/clang/unittests/Format/AlignBracketsTest.cpp
+++ b/clang/unittests/Format/AlignBracketsTest.cpp
@@ -685,8 +685,8 @@ TEST_F(AlignBracketsTest, FormatsDeclarationBreakAlways) {
" int cccccccccccccccccccccccc);",
BreakAlways);
- // Ensure AlignAfterOpenBracket interacts correctly with
PackParameters.BinPack set
- // to BPPS_AlwaysOnePerLine.
+ // Ensure AlignAfterOpenBracket interacts correctly with
+ // PackParameters.BinPack set to BPPS_AlwaysOnePerLine.
BreakAlways.BreakAfterOpenBracketFunction = true;
verifyFormat(
"void someLongFunctionName(\n"
@@ -712,8 +712,8 @@ TEST_F(AlignBracketsTest, FormatsDefinitionBreakAlways) {
"}",
BreakAlways);
- // Ensure BinPackArguments interact correctly when PackParameters.BinPack is
set to
- // BPPS_AlwaysOnePerLine.
+ // Ensure BinPackArguments interact correctly when PackParameters.BinPack is
+ // set to BPPS_AlwaysOnePerLine.
verifyFormat("void f(int aaaaaaaaaaaaaaaaaaaaaaaaaa,\n"
" int bbbbbbbbbbbbbbbbbbbbbbbbb,\n"
" int cccccccccccccccccccccccc) {\n"
@@ -742,8 +742,8 @@ TEST_F(AlignBracketsTest, FormatsDefinitionBreakAlways) {
BreakAlways);
BreakAlways.BreakFunctionDefinitionParameters = false;
- // Ensure AlignAfterOpenBracket interacts correctly with
PackParameters.BinPack set
- // to BPPS_AlwaysOnePerLine.
+ // Ensure AlignAfterOpenBracket interacts correctly with
+ // PackParameters.BinPack set to BPPS_AlwaysOnePerLine.
BreakAlways.BreakAfterOpenBracketFunction = true;
verifyFormat(
"void someLongFunctionName(\n"
diff --git a/clang/unittests/Format/ConfigParseTest.cpp
b/clang/unittests/Format/ConfigParseTest.cpp
index 6cf6566a9..a82a77798 100644
--- a/clang/unittests/Format/ConfigParseTest.cpp
+++ b/clang/unittests/Format/ConfigParseTest.cpp
@@ -538,8 +538,8 @@ TEST(ConfigParseTest, ParsesConfiguration) {
FormatStyle::BPAS_OnePerLine);
CHECK_PARSE_NESTED_VALUE("BinPack: UseBreakAfter", PackArguments, BinPack,
FormatStyle::BPAS_UseBreakAfter);
- CHECK_PARSE_NESTED_VALUE("BreakAfter: 1234", PackArguments,
- BreakAfter, 1234u);
+ CHECK_PARSE_NESTED_VALUE("BreakAfter: 1234", PackArguments, BreakAfter,
+ 1234u);
// For backward compatibility:
CHECK_PARSE_NESTED_VALUE("BinPack: true", PackArguments, BinPack,
FormatStyle::BPAS_BinPack);
@@ -547,7 +547,7 @@ TEST(ConfigParseTest, ParsesConfiguration) {
FormatStyle::BPAS_OnePerLine);
CHECK_PARSE("BinPackArguments: true", PackArguments.BinPack,
FormatStyle::BPAS_BinPack);
-
+
Style.PackConstructorInitializers = FormatStyle::PCIS_BinPack;
CHECK_PARSE("PackConstructorInitializers: Never",
PackConstructorInitializers,
FormatStyle::PCIS_Never);
@@ -585,8 +585,8 @@ TEST(ConfigParseTest, ParsesConfiguration) {
FormatStyle::BPPS_AlwaysOnePerLine);
CHECK_PARSE_NESTED_VALUE("BinPack: UseBreakAfter", PackParameters, BinPack,
FormatStyle::BPPS_UseBreakAfter);
- CHECK_PARSE_NESTED_VALUE("BreakAfter: 1234", PackParameters,
- BreakAfter, 1234u);
+ CHECK_PARSE_NESTED_VALUE("BreakAfter: 1234", PackParameters, BreakAfter,
+ 1234u);
// For backward compatibility:
CHECK_PARSE_NESTED_VALUE("BinPack: true", PackParameters, BinPack,
FormatStyle::BPPS_BinPack);
diff --git a/clang/unittests/Format/FormatTest.cpp
b/clang/unittests/Format/FormatTest.cpp
index 98ac99d58..98a31bbde 100644
--- a/clang/unittests/Format/FormatTest.cpp
+++ b/clang/unittests/Format/FormatTest.cpp
@@ -24048,7 +24048,8 @@ TEST_F(FormatTest, FormatsLambdas) {
" LambdaBodyMustBeBreak);\n"
"};",
LLVMWithBeforeLambdaBody);
- LLVMWithBeforeLambdaBody.PackParameters.BinPack =
FormatStyle::BPPS_OnePerLine;
+ LLVMWithBeforeLambdaBody.PackParameters.BinPack =
+ FormatStyle::BPPS_OnePerLine;
verifyFormat("FctAllOnSameLine_SLS_All([]() { return S; }, Fst, Second);",
LLVMWithBeforeLambdaBody);
verifyFormat(
``````````
</details>
https://github.com/llvm/llvm-project/pull/181281
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
