[PATCH] D29635: clang-format: [JS] handle parenthesized class expressions.

2017-02-07 Thread Daniel Jasper via Phabricator via cfe-commits
djasper added a comment. Sorry.. Should have caught this in the initial review. Still looks good. https://reviews.llvm.org/D29635 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D29635: clang-format: [JS] handle parenthesized class expressions.

2017-02-07 Thread Martin Probst via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL294302: clang-format: [JS] handle parenthesized class expressions. (authored by mprobst). Changed prior to commit: https://reviews.llvm.org/D29635?vs=87412=87416#toc Repository: rL LLVM

[PATCH] D29635: clang-format: [JS] handle parenthesized class expressions.

2017-02-07 Thread Martin Probst via Phabricator via cfe-commits
mprobst updated this revision to Diff 87412. mprobst added a comment. - Parse nested classes as child expressions. https://reviews.llvm.org/D29635 Files: lib/Format/UnwrappedLineParser.cpp lib/Format/UnwrappedLineParser.h unittests/Format/FormatTestJS.cpp Index:

[PATCH] D29635: clang-format: [JS] handle parenthesized class expressions.

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. Thanks https://reviews.llvm.org/D29635 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D29635: clang-format: [JS] handle parenthesized class expressions.

2017-02-07 Thread Martin Probst via Phabricator via cfe-commits
mprobst created this revision. Herald added a subscriber: klimek. In JavaScript, classes are expressions, so they can appear e.g. in argument lists. var C = foo(class { bar() { return 1; } }; https://reviews.llvm.org/D29635 Files: lib/Format/UnwrappedLineParser.cpp