diff options
| author | Martin Probst <martin@probst.io> | 2017-11-25 09:19:42 +0000 |
|---|---|---|
| committer | Martin Probst <martin@probst.io> | 2017-11-25 09:19:42 +0000 |
| commit | 7e0f25b28d56ee2c568c7bcdc3fcc90182800eab (patch) | |
| tree | c94db3bf2591ba1f63e333e90beff0561b5cb6ab /llvm/lib/Support/ThreadPool.cpp | |
| parent | 1d8fa7ff3f05234a4995ce0af0c8d072c3c9753a (diff) | |
| download | llvm-7e0f25b28d56ee2c568c7bcdc3fcc90182800eab.zip llvm-7e0f25b28d56ee2c568c7bcdc3fcc90182800eab.tar.gz llvm-7e0f25b28d56ee2c568c7bcdc3fcc90182800eab.tar.bz2 | |
clang-format: [JS] disable ASI on decorators.
Summary:
Automatic Semicolon Insertion in clang-format tries to guess if a line
wrap should insert an implicit semicolong. The previous heuristic would
not trigger ASI if a token was immediately preceded by an `@` sign:
function foo(@Bar // <-- does not trigger due to preceding @
baz) {}
However decorators can have arbitrary parameters:
function foo(@Bar(param, param, param) // <-- precending @ missed
baz) {}
While it would be possible to precisely find the matching `@`, just
conversatively disabling ASI for the entire line is simpler, while also
not regressing ASI substatially.
Reviewers: djasper
Subscribers: klimek, cfe-commits
Differential Revision: https://reviews.llvm.org/D40410
llvm-svn: 318973
Diffstat (limited to 'llvm/lib/Support/ThreadPool.cpp')
0 files changed, 0 insertions, 0 deletions
