Ezlanding1 wrote:

@EtienneBasilik great, thanks for the feedback! I've removed the functionality 
to break lines with >1 params*, and allowed function calls to be affected by 
`BreakParametersAfter` (and I've renamed the option as well, as the old name 
wasn't fitting for the new functionality).

Note that all parameter-separated parenthesized expressions are now affected by 
`BreakParametersAfter`, including function definitions/declarations, function 
calls, comma expressions, C# tuples, etc. Let me know if there are 
expression(s) you want me to remove or add (e.g. adding initializer lists, etc).

For reviewers, I've also added unit tests and cleaned up formatting, so this 
commit is ready for review!

*if someone does have a use case for this, let me know. It can easily be added 
to the current design right now, or later as a separate PR.

https://github.com/llvm/llvm-project/pull/181281
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to