[PATCH] D146042: [clang-format] Fix numerous issues with "LambdaBodyIndentation: OuterScope" option

[Owen Pan via Phabricator via cfe-commits]

owenpan added a comment.

Looks like this patch doesn't put the opening brace at the outerscope for some of the examples from the linked github issues:

  $ cat .clang-format
  AllowShortLambdasOnASingleLine: None
  BraceWrapping:
    BeforeLambdaBody: true
  BreakBeforeBraces: Custom
  LambdaBodyIndentation: OuterScope
  $ cat test.cpp
  aaaaaaaaaaaaaaaaaaaa(1,
  	             b(
  	             	[]
                       {
                         return 0;
                       }));
  
  some_function(a_really_long_name, another_long_name, a_third_long_name,
                [&](LineCallback line_callback)
  {
    int a;
    int b;
  });
  $ clang-format test.cpp
  aaaaaaaaaaaaaaaaaaaa(1, b(
                              []
                              {
                                return 0;
                              }));
  
  some_function(a_really_long_name, another_long_name, a_third_long_name,
                [&](LineCallback line_callback)
                {
                  int a;
                  int b;
                });
  $ 

Shouldn't the expected output be the same as the input?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146042