[PATCH] D75034: [clang-format] use spaces for alignment with UT_ForContinuationAndIndentation

Maximilian Fickert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 17 02:17:35 PDT 2020


fickert added a comment.

>From the original commit, it does seem that this behavior was desired.

However, from the name of the option and description in the documentation ("Use tabs only for line continuation and indentation"), I would expect it to use tabs only for indentation and continuation, but spaces for alignment. The current behavior uses spaces for some types of alignment that appear in the middle of a line (e.g. consecutive assignments/declarations), but not for others that occur at the beginning of a line (e.g. function parameters). The bug report seems to indicate that other people also expected a different behavior given the description.

Does anyone else want to comment? Otherwise I can move the changed behavior into a new option; in that case, I'd be happy about naming suggestions.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D75034/new/

https://reviews.llvm.org/D75034





More information about the cfe-commits mailing list