[PATCH] D91949: [clang-format] Add BeforeStructInitialization option in BraceWrapping configuration

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay added a comment.

Are we happy with the way this behaves? I realise some of this needs semantic information but I think the fact it only handles 1 of these 4 cases leaves me feeling a little cold. (I just don't want to be having to defend this as to why it doesn't work in all cases.)

  ---
  Language: Cpp
  BasedOnStyle: LLVM
  BreakBeforeBraces: Custom
  BraceWrapping:
      BeforeClassStructInit: true



  struct S2 {
    int x, y;
  };
  
  void foo() {
    struct S2 a1 =
        {1, 2};
    struct S2 a2[3] = {{1, 2}, {3, 4}, 5, 6};
  
    S2 a3 = {1, 2};
    S2 a4[3] = {{1, 2}, {3, 4}, 5, 6};
  }
  
  int main(int argc, char **argv) { return 0; }


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

https://reviews.llvm.org/D91949