[PATCH] D21279: Fix some issues in clang-format's AlignConsecutive modes

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 22 09:06:43 PDT 2016


djasper added a comment.

I think the IndentLevel in WhitespaceManager (and the nested Change) is a horrible mess and should be cleaned up. It gets set either to 0 or to the "Level" of the AnnotatedLine. To me only the latter makes sense as the line defines the indent level. Everything else, including recomputing this and introducing a ScopeLevel makes the current situation worse. I can try to do this in advance of this patch, if you prefer.


https://reviews.llvm.org/D21279





More information about the cfe-commits mailing list