curdeius added a comment.

In D113319#3337485 <https://reviews.llvm.org/D113319#3337485>, @krasimir wrote:

> 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).

Addressed in https://reviews.llvm.org/D120324.


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