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

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 1 07:12:19 PST 2018


djasper added a comment.

Are you sure that you are even addressing an important case? I have done some research on our codebase and this is something that happens incredibly rarely. The reason is that you have to have a very specific combination of line length, where the last parameter does not fit on one line if indented back to align with the open paren while it does fit on multiple lines if indented right of the comma. In all of LLVM/Clang, there seem to be only about 50 cases. How certain are you that people actually care?


Repository:
  rC Clang

https://reviews.llvm.org/D42787





More information about the cfe-commits mailing list