Re: [PATCH] D24804: clang-format: [JS] reserved words in method names.

2016-09-22 Thread Martin Probst via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL282138: clang-format: [JS] reserved words in method names. (authored by mprobst). Changed prior to commit: https://reviews.llvm.org/D24804?vs=72061&id=72140#toc Repository: rL LLVM https://reviews.l

Re: [PATCH] D24804: clang-format: [JS] reserved words in method names.

2016-09-21 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Looks good. Thank you. https://reviews.llvm.org/D24804 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/ma

Re: [PATCH] D24804: clang-format: [JS] reserved words in method names.

2016-09-21 Thread Martin Probst via cfe-commits
mprobst updated this revision to Diff 72061. mprobst added a comment. - move up to support let in methods. https://reviews.llvm.org/D24804 Files: lib/Format/TokenAnnotator.cpp unittests/Format/FormatTestJS.cpp Index: unittests/Format/FormatTestJS.cpp ===

[PATCH] D24804: clang-format: [JS] reserved words in method names.

2016-09-21 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. Before: class X { delete () { ... } } After: class X { delete() { ... } } https://reviews.llvm.org