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

Daphne Pfister via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 20 21:17:39 PST 2017


daphnediane added a comment.

Rebuilt with the latest patch and got one compile error. See line comment worked okay after fixing it.



================
Comment at: lib/Format/WhitespaceManager.cpp:215
+
+    if (i != Start) {
+      if (Changes[i].nestingAndIndentLevel() >
----------------
These ifs can get merged again, when you merged my changes in it was based on a version before you merged them.


================
Comment at: lib/Format/WhitespaceManager.h:157
+    std::pair<unsigned, unsigned>
+    WhitespaceManager::Change::nestingAndIndentLevel() const {
+      return std::make_pair(Tok->NestingLevel, Tok->IndentLevel);
----------------
Extra WhitespaceManager::Change:: prefix here


https://reviews.llvm.org/D21279





More information about the cfe-commits mailing list