Re: [PATCH] D12369: [clang-format] Fix alignConsecutiveAssignments not working properly

2015-09-08 Thread Matt Oakes via cfe-commits
matto1990 accepted this revision. matto1990 added a comment. This revision is now accepted and ready to land. Looks good to me now! http://reviews.llvm.org/D12369 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/

Re: [PATCH] D12369: [clang-format] Fix alignConsecutiveAssignments not working properly

2015-09-02 Thread Matt Oakes via cfe-commits
matto1990 requested changes to this revision. matto1990 added a comment. This revision now requires changes to proceed. This looks much better than the code I wrote. If all the unit tests are passing, the one change I can see to make is a misspelling in one of the comments. The rest looks excell