[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