Unfortunately, this interacts non-trivially with 
http://reviews.llvm.org/rL232320. I can fix the merge, but I am not sure what 
the proposed behavior is. Could you outline what you would want the flag to do 
in these cases? Or does the other commit possibly fix your use case?


================
Comment at: unittests/Format/FormatTestJS.cpp:161-162
@@ -146,1 +160,4 @@
   verifyFormat("f({a: 1, b: 2, c: 3});");
+  verifyFormat("f({a: 1, b: 2, c: 3});",
+               singleLineDictLiteralsStyle(FormatStyle::SLS_All));
+  verifyFormat("f({\n"
----------------
We probably should be testing 'nested' here, too.

http://reviews.llvm.org/D7200

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