djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Looks good, thank you.


================
Comment at: unittests/Format/FormatTest.cpp:8657
@@ +8656,3 @@
+               "};\n"
+               "int i  = 0;\n"
+               "auto v = type{\n"
----------------
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 ;-).


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