[PATCH] D61222: [clang-format] Fix a bug in AlignConsecutiveDeclarations

2019-05-01 Thread Owen Pan via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL359711: [clang-format] Fix a bug in AlignConsecutiveDeclarations. (authored by owenpan, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: ht

[PATCH] D61222: [clang-format] Fix a bug in AlignConsecutiveDeclarations

2019-05-01 Thread Owen Pan via Phabricator via cfe-commits
owenpan updated this revision to Diff 197566. owenpan added a comment. Removed a redundant test case. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61222/new/ https://reviews.llvm.org/D61222 Files: clang/lib/Format/WhitespaceManager.cpp clang/unittests/Forma

[PATCH] D61222: [clang-format] Fix a bug in AlignConsecutiveDeclarations

2019-05-01 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. LGTM Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61222/new/ https://reviews.llvm.org/D61222 __

[PATCH] D61222: [clang-format] Fix a bug in AlignConsecutiveDeclarations

2019-04-27 Thread Owen Pan via Phabricator via cfe-commits
owenpan updated this revision to Diff 196994. owenpan retitled this revision from "[clang-format] Fix bug in determineTokenType() for TT_StartOfName" to "[clang-format] Fix a bug in AlignConsecutiveDeclarations". owenpan added a comment. This revision is now accepted and ready to land. Fix it in