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