[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