This is an archive of the discontinued LLVM Phabricator instance.

[clang-format] Mark constexpr lambdas as lambda
ClosedPublic

Authored by HazardyKnusperkeks on Jul 16 2022, 2:51 PM.

Details

Summary

Otherwise the brace was detected as a function brace, not wrong per se, but when directly calling the lambda the calling parens were put on the next line.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptJul 16 2022, 2:51 PM
HazardyKnusperkeks requested review of this revision.Jul 16 2022, 2:51 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 16 2022, 2:51 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
owenpan accepted this revision.Jul 16 2022, 4:52 PM
owenpan added inline comments.
clang/lib/Format/UnwrappedLineParser.cpp
2131

Maybe move it up so that it's grouped with kw_const?

This revision is now accepted and ready to land.Jul 16 2022, 4:52 PM
curdeius accepted this revision.Jul 17 2022, 12:50 AM
clang/lib/Format/UnwrappedLineParser.cpp
2131

Will do before submit.

This revision was landed with ongoing or failed builds.Jul 18 2022, 12:42 PM
This revision was automatically updated to reflect the committed changes.