[PATCH] D42787: clang-format: do not add extra indent when wrapping last parameter

Francois Ferrand via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 28 06:56:47 PST 2018


Typz added a comment.

In https://reviews.llvm.org/D42787#994781, @djasper wrote:

> What I mean is, users will find it surprising if whether or not a parameter gets wrapped leads to a different indentation internal to that parameter. I have not heard of a single user that would be surprised by this extra indentation.


well you have now...
I configured the tool to align AlignOperands, to I expected the operands of my expressions to be aligned, even if they are in a function call.

(Btw, I don't think the comma used to separate function parameters are actually considered "operators" by the C++ standard, so technically this is not a case of multiple precedences, and users may not expect it to be handled this way...)


Repository:
  rC Clang

https://reviews.llvm.org/D42787





More information about the cfe-commits mailing list