[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