[PATCH] D102706: [clang-format] Add new LambdaBodyIndentation option

Vitali Lovich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 18 11:28:16 PDT 2021


vlovich added a comment.

I'm not 100% certain if the implementation is absolutely correct. For example, the `#define A` test case is broken (the closing brace is on the wrong line). Additionally, the `pop_back` feels weird but seems to fix all the other cases to make the closing brace aligned properly. Maybe some/all of the implementation belongs in ContinuationIndenter.cpp instead but I couldn't really follow where the best place to do that might be. I can also just leave the preprocessor macro case as a known issue for now.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102706/new/

https://reviews.llvm.org/D102706



More information about the cfe-commits mailing list