mprobst marked 2 inline comments as done.

================
Comment at: unittests/Format/FormatTestJS.cpp:689
@@ -688,1 +688,3 @@
                                       "String");
+  verifyFormat("function f(@Foo bar) {}", "function f(@Foo\n"
+                                          "  bar) {}");
----------------
djasper wrote:
> Are there any other valid use cases of "@" in JS? If so, should we add a test 
> for at least one of those?
@ can only be used for annotations. Those can appear on classes, fields, 
methods, and parameters. I'll send another review to add tests for those, they 
are currently formatted properly (inheriting the Java support for them).


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