[PATCH] D36132: clang-format: [JS] support default imports.

2017-08-01 Thread Martin Probst via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL309697: clang-format: [JS] support default imports. (authored by mprobst). Changed prior to commit: https://reviews.llvm.org/D36132?vs=109059=109136#toc Repository: rL LLVM

[PATCH] D36132: clang-format: [JS] support default imports.

2017-08-01 Thread Daniel Jasper via Phabricator via cfe-commits
djasper accepted this revision. djasper added inline comments. This revision is now accepted and ready to land. Comment at: lib/Format/SortJavaScriptImports.cpp:416 break; - if (Current->isNot(tok::identifier)) + if (Current->isNot(tok::identifier) &&

[PATCH] D36132: clang-format: [JS] support default imports.

2017-08-01 Thread Martin Probst via Phabricator via cfe-commits
mprobst created this revision. Herald added a subscriber: klimek. Formerly, `import {default as X} from y;` would not be recognized as an import. https://reviews.llvm.org/D36132 Files: lib/Format/SortJavaScriptImports.cpp unittests/Format/SortImportsTestJS.cpp Index: