[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