djasper added inline comments.

================
Comment at: lib/Format/TokenAnnotator.cpp:789
@@ -788,3 +788,3 @@
     if (Style.Language == FormatStyle::LK_JavaScript &&
-        CurrentToken->is(Keywords.kw_import))
+        CurrentToken->is(Keywords.kw_import) && !Style.JavaScriptWrapImports)
       return LT_ImportStatement;
----------------
I think it is a bit dodgy to not mark the line as import statement based on 
this style option. Would it be a lot of work to pull the decision of whether or 
not to format this out a level higher?


http://reviews.llvm.org/D21273



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to