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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to