[PATCH] D125171: Add a new clang-format option AlwaysBreakBeforeFunctionParameters
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 30 03:31:25 PDT 2023
MyDeveloperDay added a comment.
Can you show your example from a implementation case and not just a declaration i.e. what happens with
`int function1(int param1) {}`
================
Comment at: clang/lib/Format/TokenAnnotator.cpp:4789
+ if (Left.is(tok::l_paren) && Style.AlwaysBreakBeforeFunctionParameters &&
+ !Right.is(tok::r_paren) && Left.Previous &&
+ Left.Previous->is(TT_FunctionDeclarationName)) {
----------------
can you test the `()` case in your tests please with `AlwaysBreakBeforeFunctionParameters` set to true
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125171/new/
https://reviews.llvm.org/D125171
More information about the cfe-commits
mailing list