[PATCH] D36142: clang-format: [JS] do not insert whitespace in call positions.

[Daniel Jasper via Phabricator via cfe-commits]

djasper added inline comments.


================
Comment at: lib/Format/TokenAnnotator.cpp:2355
+        (Left.Tok.getIdentifierInfo() ||
+         Left.isOneOf(tok::kw_switch, tok::kw_case, tok::kw_delete)))
+      return false;
----------------
Why is instanceof not required in this list?


================
Comment at: unittests/Format/FormatTestJS.cpp:237
+  verifyFormat("x.switch() = 1;");
+  verifyFormat("x.case() = 1;");
   verifyFormat("x = {\n"
----------------
no test for delete?


https://reviews.llvm.org/D36142



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits