[PATCH] D50230: clang-format: [JS] don't break comments before any '{'

2018-08-03 Thread Martin Probst via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC338837: clang-format: [JS] don't break comments before any '{' (authored by mprobst, committed by ). Changed prior to commit: https://reviews.llvm.org/D50230?vs=158946&id=158957#toc Repository: rC Cl

[PATCH] D50230: clang-format: [JS] don't break comments before any '{'

2018-08-03 Thread Krasimir Georgiev via Phabricator via cfe-commits
krasimir accepted this revision. krasimir added a comment. This revision is now accepted and ready to land. Great! Thanks! Repository: rC Clang https://reviews.llvm.org/D50230 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

[PATCH] D50230: clang-format: [JS] don't break comments before any '{'

2018-08-03 Thread Martin Probst via Phabricator via cfe-commits
mprobst updated this revision to Diff 158946. mprobst added a comment. Also handle multiple numbered list tokens. Repository: rC Clang https://reviews.llvm.org/D50230 Files: lib/Format/BreakableToken.cpp unittests/Format/FormatTestJS.cpp Index: unittests/Format/FormatTestJS.cpp ===

[PATCH] D50230: clang-format: [JS] don't break comments before any '{'

2018-08-03 Thread Martin Probst via Phabricator via cfe-commits
mprobst created this revision. mprobst added a reviewer: krasimir. Previously, clang-format would avoid breaking before the first `{` found, but then happily break before subsequent '{'s on the line. This change fixes that by looking for the first location that has no opening curly, if any. Repo