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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to