This revision was automatically updated to reflect the committed changes.
Closed by commit rGe168964ce1fb: [clang-tidy] Fix
bugprone-copy-constructor-init documentation (authored by PiotrZSL).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144594/new/
carlosgalvezp accepted this revision.
carlosgalvezp added a comment.
This revision is now accepted and ready to land.
Great improvement, thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144594/new/
https://reviews.llvm.org/D144594
___
PiotrZSL updated this revision to Diff 499733.
PiotrZSL added a comment.
Update doc
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144594/new/
https://reviews.llvm.org/D144594
Files:
clang-tools-extra/docs/clang-tidy/checks/bugprone/copy-construc
PiotrZSL updated this revision to Diff 499732.
PiotrZSL added a comment.
Update doc
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D144594/new/
https://reviews.llvm.org/D144594
Files:
clang-tools-extra/docs/clang-tidy/checks/bugprone/copy-construc
Eugene.Zelenko added inline comments.
Comment at:
clang-tools-extra/docs/clang-tidy/checks/bugprone/copy-constructor-init.rst:37
+
+In summary check detects cases where the copy constructor of a derived class
+doesn't properly call or initialize the copy constructor of the base
PiotrZSL created this revision.
Herald added a subscriber: xazax.hun.
Herald added a reviewer: njames93.
Herald added a project: All.
PiotrZSL requested review of this revision.
Herald added a project: clang-tools-extra.
Herald added a subscriber: cfe-commits.
Correct example, and add information
