[PATCH] D109557: Adds a BreakBeforeClosingParen option

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay added a comment.

FYI, this is a very aggressive change, I highly recommend you run this over a large code base before landing. to double check, here is one slight oddity which I cannot determine if its correct or not.

  void foo() {
      if (quitelongarg != (alsolongarg - 1)) { // ABC is a very longgggggggggggg comment
        return;
      }
  }

becomes

  void foo() {
    if (quitelongarg != (alsolongarg - 1)
    ) { // ABC is a very longgggggggggggg comment
      return;
    }
  }



  BasedOnStyle: LLVM
  BreakBeforeClosingParen: true

That might be what you expect but I wasn't quite sure


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109557