[PATCH] D139834: [clang-format] AllowShortCompoundRequirementOnASingleLine
Emilia Kond via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon May 15 14:30:29 PDT 2023
rymiel accepted this revision.
rymiel added a comment.
This revision is now accepted and ready to land.
I see nothing wrong with this patch alone as it currently stands, since it's a quite simple change to the LineJoiner, and I see it as one of the final stopgaps in clang-format's support for requires clauses, and I think it definitely beats the current buggy behavior where behaviour is conflated with BraceWrapping.AfterFunction.
But I'd like other reviewers to maybe have another look?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139834/new/
https://reviews.llvm.org/D139834
More information about the cfe-commits
mailing list