diff options
author | Felix Berger <flx@google.com> | 2016-07-01 20:12:15 +0000 |
---|---|---|
committer | Felix Berger <flx@google.com> | 2016-07-01 20:12:15 +0000 |
commit | 7f8827576cf6ab35e0e19aeeaede34289737115e (patch) | |
tree | d074940e6c0bc271b474561764d5cb96e1ee8b3c /lldb/packages/Python/lldbsuite/test/python_api | |
parent | 2aac720adc9ab5131416362ad6b1d22478967ea0 (diff) | |
download | bcm5719-llvm-7f8827576cf6ab35e0e19aeeaede34289737115e.tar.gz bcm5719-llvm-7f8827576cf6ab35e0e19aeeaede34289737115e.zip |
[clang-tidy] UnnecessaryValueParamCheck - suggest std::move() if non-const value parameter can be moved.
Summary:
Make check more useful in the following two cases:
The parameter is passed by non-const value, has a non-deleted move constructor and is only referenced once in the function as argument to the type's copy constructor.
The parameter is passed by non-const value, has a non-deleted move assignment operator and is only referenced once in the function as argument of the the type's copy assignment operator.
In this case suggest a fix to move the parameter which avoids the unnecessary copy and is closest to what the user might have intended.
Reviewers: alexfh, sbenza
Subscribers: cfe-commits, Prazek
Differential Revision: http://reviews.llvm.org/D20277
llvm-svn: 274380
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api')
0 files changed, 0 insertions, 0 deletions