[PATCH] D13513: [clang-format] Stop alignment sequences on open braces and parens when aligning assignments.
Beren Minor via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 7 08:06:33 PDT 2015
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
More information about the cfe-commits
mailing list