[PATCH] D18943: clang-format: [JS] do not insert semicolons after wrapped annotations.

Martin Probst via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 10 22:43:54 PDT 2016


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





More information about the cfe-commits mailing list