[PATCH] D29634: clang-format: [JS] exclaim preceding regex literals.

2017-02-07 Thread Martin Probst via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL294304: clang-format: [JS] exclaim preceding regex literals. (authored by mprobst). Changed prior to commit: https://reviews.llvm.org/D29634?vs=87402=87417#toc Repository: rL LLVM

[PATCH] D29634: clang-format: [JS] exclaim preceding regex literals.

2017-02-07 Thread Daniel Jasper via Phabricator via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Looks good. https://reviews.llvm.org/D29634 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D29634: clang-format: [JS] exclaim preceding regex literals.

2017-02-07 Thread Martin Probst via Phabricator via cfe-commits
mprobst created this revision. Herald added a subscriber: klimek. Regex detection would incorrectly classify a trailing `!` operator (nullability cast) followed by a `/` as the start of a regular expression literal. This fixes code such as: var foo = x()! / 10; Which would previously parse a