[PATCH] D25675: clang-format: [JS] Fix template string ASI.

2016-10-20 Thread Martin Probst via cfe-commits
This revision was automatically updated to reflect the committed changes. mprobst marked an inline comment as done. Closed by commit rL284807: clang-format: [JS] Fix template string ASI. (authored by mprobst). Changed prior to commit: https://reviews.llvm.org/D25675?vs=74837&id=75390#toc Repos

[PATCH] D25675: clang-format: [JS] Fix template string ASI.

2016-10-20 Thread Martin Probst via cfe-commits
mprobst marked an inline comment as done. mprobst added inline comments. Comment at: unittests/Format/FormatTestJS.cpp:1280 +TEST_F(FormatTestJS, TemplateStringASI) { + verifyFormat("var x = `hello${world}`;", "var x = `hello${\n" + "

[PATCH] D25675: clang-format: [JS] Fix template string ASI.

2016-10-20 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added inline comments. This revision is now accepted and ready to land. Comment at: unittests/Format/FormatTestJS.cpp:1280 +TEST_F(FormatTestJS, TemplateStringASI) { + verifyFormat("var x = `hello${world}`;", "var x = `hello${\n" +

[PATCH] D25675: clang-format: [JS] Fix template string ASI.

2016-10-17 Thread Martin Probst via cfe-commits
mprobst created this revision. mprobst added a reviewer: djasper. mprobst added a subscriber: cfe-commits. Herald added a subscriber: klimek. Previously, automatic semicolon insertion would add an unwrapped line when a template string contained a line break. var x = `foo${ bar}`; Would b