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

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 10 22:20:22 PDT 2016


djasper added inline comments.

================
Comment at: lib/Format/UnwrappedLineParser.cpp:708
@@ +707,3 @@
+  if (Line && Line->Tokens.size() > 1) {
+    // If the token before the previous one is an '@', the previous token is an
+    // annotation and can precede another identifier/value.
----------------
How about:
* Moving this down a bit
* Checking this only if PreviousMustBeValue
* return directly

Like this, it is a bit hard to understand all the combinations of what's going on.


http://reviews.llvm.org/D18943





More information about the cfe-commits mailing list