[PATCH] D33722: [clang-tidy] Add checker for undelegated copy of base classes

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 3 07:49:21 PDT 2017


xazax.hun updated this revision to Diff 121477.
xazax.hun added a comment.

- Dominic said he no longer have time to continue with this patch, so I commandeered this revision
- Skip template instantiations
- Do not attempt fix macro expansions
- Do not attempt fix type aliases and typedef types
- Do not attempt to fix cases where the parameter is unnamed
- Use a more conservative approach and only warn for default constructors invoked by copy ctors. This could be refined in a followup patch.
- Skip the check for empty base classes
- Do not use matchers in the check callback
- Add tests for relevant changes
- Fix style issues in tests and docs
- Rebased for latest ToT


https://reviews.llvm.org/D33722

Files:
  clang-tidy/misc/CMakeLists.txt
  clang-tidy/misc/CopyConstructorInitCheck.cpp
  clang-tidy/misc/CopyConstructorInitCheck.h
  clang-tidy/misc/MiscTidyModule.cpp
  docs/ReleaseNotes.rst
  docs/clang-tidy/checks/list.rst
  docs/clang-tidy/checks/misc-copy-constructor-init.rst
  test/clang-tidy/misc-copy-constructor-init.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33722.121477.patch
Type: text/x-patch
Size: 13771 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171103/28af6a43/attachment.bin>


More information about the cfe-commits mailing list