coverity#1362680 Pointer to local outside scope

this doesn't seem to make sense anymore since...

commit 4d49c960
Author: Stephan Bergmann <sbergman@redhat.com>
Date:   Wed Jun 8 17:14:34 2016 +0200

    Let loplugin:passstuffbyref also look at fn defn not preceded by any decl

so lets revert that to be safe

Change-Id: I78fe8ae48dc4c4f3d5786c7232d7e791ff50d9b4
Reviewed-on: https://gerrit.libreoffice.org/26189Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

dbaccess/source/ui/uno/copytablewizard.cxx

@@ -966,7 +966,7 @@ namespace
     class ValueTransfer
     {
     public:
-        ValueTransfer( sal_Int32 _rSourcePos, sal_Int32 _rDestPos, const ::std::vector< sal_Int32 >& _rColTypes,
+        ValueTransfer( const sal_Int32& _rSourcePos, const sal_Int32& _rDestPos, const ::std::vector< sal_Int32 >& _rColTypes,
             const Reference< XRow >& _rxSource, const Reference< XParameters >& _rxDest )
             :m_rSourcePos( _rSourcePos )
             ,m_rDestPos( _rDestPos )