[PATCH] D139937: [clang-format] fix typo in doc for SLS_Inline

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 13 09:21:38 PST 2022


MyDeveloperDay requested changes to this revision.
MyDeveloperDay added a comment.
This revision now requires changes to proceed.

I'm not sure if that is correct, as that doesn't fit the example (unless the example is wrong)

It feels like its saying merge if its an argument of a function (sort in this case)

sort(a.begin(), a.end(), **[]() { return x < y; }**);

The lambda is the argument, not that the lamda has no arguments?  either way something isn't correct by making that change (either the description was wrong as well as the example or now the description doesn't match the example)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139937/new/

https://reviews.llvm.org/D139937



More information about the cfe-commits mailing list