mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

update couple of loplugins for move constructors

Browse Source 
Change-Id: I82140569a3e09225065d721e4aeac5c5fe93bc45
This commit is contained in:
Noel Grandin
2016-11-16 14:15:29 +02:00
parent ac4f846335
commit 92da39d5f9
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
compilerplugins/clang/expandablemethods.cxx
View File

@@ -306,7 +306,9 @@ bool ExpandableMethods::isCalleeFunctionInteresting(const FunctionDecl* function
if (functionDecl->isDeleted() || functionDecl->isDefaulted()) { if (functionDecl->isDeleted() || functionDecl->isDefaulted()) {
return false; return false;
} }
if (isa<CXXConstructorDecl>(functionDecl) && dyn_cast<CXXConstructorDecl>(functionDecl)->isCopyConstructor()) { if (isa<CXXConstructorDecl>(functionDecl)
&& dyn_cast<CXXConstructorDecl>(functionDecl)->isCopyOrMoveConstructor())
{
return false; return false;
} }
if (!functionDecl->getLocation().isValid() || ignoreLocation(functionDecl)) { if (!functionDecl->getLocation().isValid() || ignoreLocation(functionDecl)) {

4
compilerplugins/clang/unusedmethods.cxx
View File

@@ -296,7 +296,9 @@ bool UnusedMethods::VisitFunctionDecl( const FunctionDecl* functionDecl )
if (functionDecl->isDeleted() || functionDecl->isDefaulted()) { if (functionDecl->isDeleted() || functionDecl->isDefaulted()) {
return true; return true;
} }
if (isa<CXXConstructorDecl>(functionDecl) && dyn_cast<CXXConstructorDecl>(functionDecl)->isCopyConstructor()) { if (isa<CXXConstructorDecl>(functionDecl)
&& dyn_cast<CXXConstructorDecl>(functionDecl)->isCopyOrMoveConstructor())
{
return true; return true;
} }
if (!canonicalFunctionDecl->getLocation().isValid() || ignoreLocation(canonicalFunctionDecl)) { if (!canonicalFunctionDecl->getLocation().isValid() || ignoreLocation(canonicalFunctionDecl)) {