[PATCH] D145642: [clang-format] Annotate lambdas with requires clauses.
Björn Schäpers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 23 12:33:27 PDT 2023
HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added a comment.
I think this is okay. A followup could handle single line lambdas. But I have a personal struggle with them, especially the name `AllowShortLambdasOnASingleLine` no it is not //allow// if turned on it is //force//. And in combination with `ColumnLimit` 0 there seem to be some mismatches (never bothered enough to investigate, or if it is just a feeling).
The question would be if something with a requires clause should be merges into one line, then I think it should put everything in one line. But what if that now is in conflict with `RequiresClausePosition`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145642/new/
https://reviews.llvm.org/D145642
More information about the cfe-commits
mailing list