This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Finish cppcoreguidelines-avoid-capturing-lambda-coroutines check
ClosedPublic

Authored by PiotrZSL on Mar 9 2023, 12:40 PM.

Details

Summary

This commit implements check for CP.51 C++ Core Guidelines

Depends on D137514

Diff Detail

Event Timeline

PiotrZSL created this revision.Mar 9 2023, 12:40 PM
PiotrZSL requested review of this revision.Mar 9 2023, 12:40 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 9 2023, 12:40 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
PiotrZSL updated this revision to Diff 503889.Mar 9 2023, 12:48 PM

Change base-line

Eugene.Zelenko added inline comments.
clang-tools-extra/docs/ReleaseNotes.rst
126

Just Flags.

clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines/avoid-capturing-lambda-coroutines.rst
6

Ditto.

PiotrZSL marked 2 inline comments as done.Mar 9 2023, 1:08 PM
PiotrZSL updated this revision to Diff 503907.Mar 9 2023, 1:08 PM

Improve documentation (review comments)

PiotrZSL updated this revision to Diff 503940.Mar 9 2023, 2:47 PM

Change baseline

ChuanqiXu accepted this revision.Mar 9 2023, 6:09 PM

LGTM. Thanks!

This revision is now accepted and ready to land.Mar 9 2023, 6:09 PM