[PATCH] D17910: clang-format: [JS] Handle certain cases of ASI.

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 14 12:12:43 PDT 2016


djasper added a comment.

Looks good.


================
Comment at: lib/Format/UnwrappedLineParser.cpp:667
@@ +666,3 @@
+    return true;
+  // FIXME(martinprobst): This returns true for C/C++ keywords like 'struct'.
+  return FormatTok->is(tok::identifier) &&
----------------
We don't usually put usernames into FIXMEs


http://reviews.llvm.org/D17910





More information about the cfe-commits mailing list