djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.

Looks good.


================
Comment at: unittests/Format/FormatTestJS.cpp:689
@@ -688,1 +688,3 @@
                                       "String");
+  verifyFormat("function f(@Foo bar) {}", "function f(@Foo\n"
+                                          "  bar) {}");
----------------
Are there any other valid use cases of "@" in JS? If so, should we add a test 
for at least one of those?


http://reviews.llvm.org/D18943



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

Reply via email to