berenm added inline comments.

================
Comment at: unittests/Format/FormatTest.cpp:8657
@@ +8656,3 @@
+               "};\n"
+               "int i  = 0;\n"
+               "auto v = type{\n"
----------------
djasper wrote:
> So, it is kind of interesting that we (would) align with a lambda on the next 
> line here, but not with the lambda declaration a few lines above.
> 
> I guess this makes sense, but I would also vager that somebody is going to 
> file a bug report about it ;-).
Yes, this would require to handle nested levels of alignment sequences and push 
the current one when a new scope is detected. It's just not implemented yet :)


http://reviews.llvm.org/D13513



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

Reply via email to