[PATCH] D12031: Const std::move() argument ClangTidy check
Vadym Doroshenko via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 3 09:15:37 PST 2015
dvadym updated the summary for this revision.
dvadym updated this revision to Diff 39077.
dvadym marked 6 inline comments as done.
dvadym added a comment.
1.Most comments addressed
2.Taking into consideration applying move to trivially copyable objects
3.Different message if move argument variable or expression.
It's not addressed yet case when move argument is depend in some way on template argument. There is comment in code about this.
http://reviews.llvm.org/D12031
Files:
clang-tidy/misc/CMakeLists.txt
clang-tidy/misc/MiscTidyModule.cpp
clang-tidy/misc/MoveConstantArgumentCheck.cpp
clang-tidy/misc/MoveConstantArgumentCheck.h
test/clang-tidy/move-const-arg.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12031.39077.patch
Type: text/x-patch
Size: 6033 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151103/17af9499/attachment.bin>
More information about the cfe-commits
mailing list