[PATCH] D113319: [clang-format] Improve require and concept handling

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 22 05:13:44 PST 2022


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



More information about the cfe-commits mailing list