Re: [PATCH] D21275: clang-format: [JS] Indent namespaces in JavaScript/TS by default.

2016-06-13 Thread Martin Probst via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL272559: clang-format: [JS] Indent namespaces in JavaScript/TS by default. (authored by mprobst). Changed prior to commit: http://reviews.llvm.org/D21275?vs=60487&id=60553#toc Repository: rL LLVM htt

Re: [PATCH] D21275: clang-format: [JS] Indent namespaces in JavaScript/TS by default.

2016-06-13 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. http://reviews.llvm.org/D21275 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listin

[PATCH] D21275: clang-format: [JS] Indent namespaces in JavaScript/TS by default.

2016-06-12 Thread Martin Probst via cfe-commits
mprobst created this revision. mprobst added a reviewer: djasper. mprobst added subscribers: cfe-commits, alexeagle. Herald added a subscriber: klimek. There's no convention of avoiding the nested indentation. http://reviews.llvm.org/D21275 Files: lib/Format/Format.cpp unittests/Format/Forma