[PATCH] D113319: [clang-format] Improve require and concept handling
Marek Kurdej via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 22 06:49:21 PST 2022
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
More information about the cfe-commits
mailing list