[PATCH] D13871: Add modernize-use-default check to clang-tidy.

Angel Garcia via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 20 01:33:52 PDT 2015


angelgarcia updated this revision to Diff 37833.
angelgarcia added a comment.

Global variable -> static, and a few other additions.

> Will be good idea to add handling of cases where default constructor is empty and only call base class(es) default constructor/members default constructors


I agree, but checking all members and base classes is the main difficulty of the copy-constructor/copy-assignment operator. When I implement those I will probably update this.


http://reviews.llvm.org/D13871

Files:
  clang-tidy/modernize/CMakeLists.txt
  clang-tidy/modernize/ModernizeTidyModule.cpp
  clang-tidy/modernize/UseDefaultCheck.cpp
  clang-tidy/modernize/UseDefaultCheck.h
  docs/clang-tidy/checks/list.rst
  docs/clang-tidy/checks/modernize-use-default.rst
  test/clang-tidy/modernize-use-default.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13871.37833.patch
Type: text/x-patch
Size: 11030 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151020/c54d9271/attachment.bin>


More information about the cfe-commits mailing list