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

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 7 03:08:06 PST 2017


djasper accepted this revision.
djasper added a comment.

This looks very nice now :-D. Thanks for working on this!!



================
Comment at: lib/Format/WhitespaceManager.cpp:196
+
+  // ScopeStack keeps track of the current scope depth.
+  // We only run the "Matches" function on tokens from the outer-most scope.
----------------
Maybe add: "It contains indices of the first token on each scope."


https://reviews.llvm.org/D21279





More information about the cfe-commits mailing list