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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
