MyDeveloperDay requested changes to this revision.
MyDeveloperDay added inline comments.
This revision now requires changes to proceed.
================
Comment at: clang/docs/ClangFormatStyleOptions.rst:1372
+.. _AlwaysBreakBeforeFunctionParameters:
+
+**AlwaysBreakBeforeFunctionParameters** (``Boolean``)
:versionbadge:`clang-format 16.0`
----------------
Regererate
================
Comment at: clang/docs/ClangFormatStyleOptions.rst:1376
+
+ This flag is meant to align function parameters starting on the line
following
+ a function declaration or definition. Thus, it will only take effect if a
function
----------------
Ok you wrote this by hand it’s auto generated from format.h
================
Comment at: clang/lib/Format/Format.cpp:872
IO.mapOptional("AlwaysBreakBeforeMultilineStrings",
Style.AlwaysBreakBeforeMultilineStrings);
+ IO.mapOptional("AlwaysBreakBeforeFunctionParameters",
----------------
Needs a parse test
================
Comment at: clang/unittests/Format/FormatTest.cpp:25357
+ // verify that there is no break by default
+ verifyFormat("int function1(int param1, int param2, int param3);\n"
+ "int function2();\n",
----------------
Please write it out long form
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125171/new/
https://reviews.llvm.org/D125171
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
