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