[PATCH] D38291: clang-format/java: Unbreak genenrics formatting after r299952.

2017-09-27 Thread Nico Weber via Phabricator via cfe-commits
thakis closed this revision. thakis added a comment. r314325, thanks! https://reviews.llvm.org/D38291 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D38291: clang-format/java: Unbreak genenrics formatting after r299952.

2017-09-27 Thread Richard Bradfield via Phabricator via cfe-commits
bradfier accepted this revision. bradfier added a comment. This revision is now accepted and ready to land. If this doesn't break the test case for '>>>' then it looks good, although I'm not sure what has changed in the interim to allow that to work. When I made the patch in r299952 we definitel

[PATCH] D38291: clang-format/java: Unbreak genenrics formatting after r299952.

2017-09-26 Thread Nico Weber via Phabricator via cfe-commits
thakis created this revision. Herald added a subscriber: klimek. https://reviews.llvm.org/rL299952 merged '>>>' tokens into a single JavaRightLogicalShift token. This broke formatting of generics nested more than two deep, e.g. `Foo>>` because the '>>>' now weren't three '>' for parseAngle().