[Libreoffice-commits] core.git: compilerplugins/clang

[Stephan Bergmann]

 compilerplugins/clang/passstuffbyref.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b998313d9d1ec511eff5076029e62608c8c5ca0e
Author: Stephan Bergmann <sberg...@redhat.com>
Date:   Fri Dec 16 15:14:23 2016 +0100

    Make move detection in loplugin:passstuffbyref work with MSVCRT
    
    ...where an ImplicitCastExpr happens to appear between CXXConstructExpr and
    CallExpr
    
    Change-Id: I62226cc89d87bd3d9c03743b650f10c32c18f9be

diff --git a/compilerplugins/clang/passstuffbyref.cxx 
b/compilerplugins/clang/passstuffbyref.cxx
index 8695ae6..fca6b73 100644
--- a/compilerplugins/clang/passstuffbyref.cxx
+++ b/compilerplugins/clang/passstuffbyref.cxx
@@ -204,7 +204,7 @@ void PassStuffByRef::checkParams(const FunctionDecl * 
functionDecl) {
                     auto cxxConstructExpr = 
dyn_cast<CXXConstructExpr>(cxxCtorInitializer->getInit());
                     if (cxxConstructExpr && cxxConstructExpr->getNumArgs() == 
1)
                     {
-                        if (auto callExpr = 
dyn_cast<CallExpr>(cxxConstructExpr->getArg(0))) {
+                        if (auto callExpr = 
dyn_cast<CallExpr>(cxxConstructExpr->getArg(0)->IgnoreParenImpCasts())) {
                             if 
(loplugin::DeclCheck(callExpr->getCalleeDecl()).Function("move").StdNamespace())
 {
                                 bFoundMove = true;
                                 break;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits