[PATCH] D21279: Fix some issues in clang-format's AlignConsecutive modes
Beren Minor via cfe-commits
cfe-commits at lists.llvm.org
Mon Jun 13 11:00:10 PDT 2016
berenm added a comment.
I think it's better to update the diff with full context (git diff -U999999) rather than attach the file, in order to have it integrated into phabricator diff view. Or use the arcanist command-line tool, which should do it for you.
Except from that, from a quick reading it looks good to me and I'm happy that someone fixed theses cases, I see them from time to time actually but never had time to work on a proper fix.
Repository:
rL LLVM
http://reviews.llvm.org/D21279
More information about the cfe-commits
mailing list