This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Fix bugprone-copy-constructor-init documentation
ClosedPublic

Authored by PiotrZSL on Feb 22 2023, 2:53 PM.

Diff Detail

Event Timeline

PiotrZSL created this revision.Feb 22 2023, 2:53 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: xazax.hun. · View Herald Transcript
PiotrZSL requested review of this revision.Feb 22 2023, 2:53 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 22 2023, 2:53 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Eugene.Zelenko added inline comments.
clang-tools-extra/docs/clang-tidy/checks/bugprone/copy-constructor-init.rst
37

English is not my native tongue, but may be the check is correct?

PiotrZSL updated this revision to Diff 499732.Feb 22 2023, 11:03 PM

Update doc

PiotrZSL marked an inline comment as done.Feb 22 2023, 11:03 PM
PiotrZSL updated this revision to Diff 499733.Feb 22 2023, 11:07 PM

Update doc

carlosgalvezp accepted this revision.Feb 26 2023, 12:37 AM

Great improvement, thanks!

This revision is now accepted and ready to land.Feb 26 2023, 12:37 AM