[PATCH] D21279: Fix some issues in clang-format's AlignConsecutive modes
Ben Harper via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 26 00:52:53 PDT 2016
bmharper updated this revision to Diff 72444.
bmharper added a comment.
This revision merges NestingLevel and IndentLevel into an std::pair, as suggested by @djasper.
IMO this makes the code slightly harder to read, because you lose the unique variable names "IndentLevel" and "NestingLevel". Those are now replaced by .first and .second, respectively. In addition, NestingLevel is no longer equal to the original NestingLevel that gets passed in, because it needs to be tweaked slightly to better work with the recursive technique we use for alignment scope.
However, the approach presented here certainly does work, and it's actually not too much of a code change from the previous patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 25487 bytes
Desc: not available
More information about the cfe-commits