krasimir added a comment.

It appears that after this patch clang-format started breaking up `requires` in 
javascript, e.g.:

  // before
  function f() {
    var requires = {};
  }
  // after
  function f() {
    var
      requires
    = {};
  }

Could we restrict the requires logic only to (obj)C++? (normally I'd go and 
come up with a patch for cases like this, but this patch is quite large and 
it's not immediately clear where's the best place for this restriction).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113319

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

Reply via email to