Please upload patch with full context in the future. Phabricator can't/doesn't 
access the existing repository to get the context.


================
Comment at: lib/Format/TokenAnnotator.cpp:1760
@@ -1759,1 +1759,3 @@
     return false;
+  if (Left.is(TT_TemplateCloser) &&
+      Style.Language == FormatStyle::LK_JavaScript &&
----------------
So, there is a of spaceRequiredBefore and spaceRequiredBetween. It is entirely 
historic, not logical, essentially those two are just in sequence. However, I'd 
prefer to move this to spaceRequiredBefore, which has a JavaScript-specific 
section at the top.

http://reviews.llvm.org/D8333

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to