[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