loplugin:passstuffbyref also for {css::uno,rtl}::Reference

Change-Id: Ifc0f5788c906b2c014f277b2d290c33806c357ab
This commit is contained in:
Stephan Bergmann
2016-07-07 15:59:09 +02:00
parent 7a0a0babf6
commit 2bcafc519e
4 changed files with 12 additions and 12 deletions

View File

@@ -221,7 +221,7 @@ friend class UCBStorage;
protected: protected:
virtual ~UCBStorageStream(); virtual ~UCBStorageStream();
public: public:
UCBStorageStream( const OUString& rName, StreamMode nMode, bool bDirect, const OString* pKey, bool bRepair, css::uno::Reference< css::ucb::XProgressHandler > xProgress ); UCBStorageStream( const OUString& rName, StreamMode nMode, bool bDirect, const OString* pKey, bool bRepair, css::uno::Reference< css::ucb::XProgressHandler > const & xProgress );
UCBStorageStream( UCBStorageStream_Impl* ); UCBStorageStream( UCBStorageStream_Impl* );
virtual sal_uLong Read( void * pData, sal_uLong nSize ) override; virtual sal_uLong Read( void * pData, sal_uLong nSize ) override;
@@ -269,7 +269,7 @@ public:
bool bDirect, bool bDirect,
bool bIsRoot, bool bIsRoot,
bool bIsRepair, bool bIsRepair,
css::uno::Reference< css::ucb::XProgressHandler > css::uno::Reference< css::ucb::XProgressHandler > const &
xProgressHandler ); xProgressHandler );
UCBStorage( UCBStorage_Impl* ); UCBStorage( UCBStorage_Impl* );

View File

@@ -437,7 +437,7 @@ public:
bool m_bIsOLEStorage;// an OLEStorage on a UCBStorageStream makes this an Autocommit-stream bool m_bIsOLEStorage;// an OLEStorage on a UCBStorageStream makes this an Autocommit-stream
UCBStorageStream_Impl( const OUString&, StreamMode, UCBStorageStream*, bool, const OString* pKey=nullptr, UCBStorageStream_Impl( const OUString&, StreamMode, UCBStorageStream*, bool, const OString* pKey=nullptr,
bool bRepair = false, Reference< XProgressHandler > xProgress = Reference< XProgressHandler >() ); bool bRepair = false, Reference< XProgressHandler > const & xProgress = Reference< XProgressHandler >() );
void Free(); void Free();
bool Init(); bool Init();
@@ -498,9 +498,9 @@ public:
Reference< XProgressHandler > m_xProgressHandler; Reference< XProgressHandler > m_xProgressHandler;
UCBStorage_Impl( const ::ucbhelper::Content&, const OUString&, StreamMode, UCBStorage*, bool, UCBStorage_Impl( const ::ucbhelper::Content&, const OUString&, StreamMode, UCBStorage*, bool,
bool, bool = false, Reference< XProgressHandler > = Reference< XProgressHandler >() ); bool, bool = false, Reference< XProgressHandler > const & = Reference< XProgressHandler >() );
UCBStorage_Impl( const OUString&, StreamMode, UCBStorage*, bool, bool, UCBStorage_Impl( const OUString&, StreamMode, UCBStorage*, bool, bool,
bool = false, Reference< XProgressHandler > = Reference< XProgressHandler >() ); bool = false, Reference< XProgressHandler > const & = Reference< XProgressHandler >() );
UCBStorage_Impl( SvStream&, UCBStorage*, bool ); UCBStorage_Impl( SvStream&, UCBStorage*, bool );
void Init(); void Init();
sal_Int16 Commit(); sal_Int16 Commit();
@@ -634,7 +634,7 @@ bool UCBStorageElement_Impl::IsModified()
return bModified; return bModified;
} }
UCBStorageStream_Impl::UCBStorageStream_Impl( const OUString& rName, StreamMode nMode, UCBStorageStream* pStream, bool bDirect, const OString* pKey, bool bRepair, Reference< XProgressHandler > xProgress ) UCBStorageStream_Impl::UCBStorageStream_Impl( const OUString& rName, StreamMode nMode, UCBStorageStream* pStream, bool bDirect, const OString* pKey, bool bRepair, Reference< XProgressHandler > const & xProgress )
: m_pAntiImpl( pStream ) : m_pAntiImpl( pStream )
, m_aURL( rName ) , m_aURL( rName )
, m_pContent( nullptr ) , m_pContent( nullptr )
@@ -1241,7 +1241,7 @@ void UCBStorageStream_Impl::PrepareCachedForReopen( StreamMode nMode )
} }
} }
UCBStorageStream::UCBStorageStream( const OUString& rName, StreamMode nMode, bool bDirect, const OString* pKey, bool bRepair, Reference< XProgressHandler > xProgress ) UCBStorageStream::UCBStorageStream( const OUString& rName, StreamMode nMode, bool bDirect, const OString* pKey, bool bRepair, Reference< XProgressHandler > const & xProgress )
{ {
// pImp must be initialized in the body, because otherwise the vtable of the stream is not initialized // pImp must be initialized in the body, because otherwise the vtable of the stream is not initialized
// to class UCBStorageStream ! // to class UCBStorageStream !
@@ -1451,7 +1451,7 @@ UCBStorage::UCBStorage( const ::ucbhelper::Content& rContent, const OUString& rN
StorageBase::m_nMode = pImp->m_nMode; StorageBase::m_nMode = pImp->m_nMode;
} }
UCBStorage::UCBStorage( const OUString& rName, StreamMode nMode, bool bDirect, bool bIsRoot, bool bIsRepair, Reference< XProgressHandler > xProgressHandler ) UCBStorage::UCBStorage( const OUString& rName, StreamMode nMode, bool bDirect, bool bIsRoot, bool bIsRepair, Reference< XProgressHandler > const & xProgressHandler )
{ {
// pImp must be initialized in the body, because otherwise the vtable of the stream is not initialized // pImp must be initialized in the body, because otherwise the vtable of the stream is not initialized
// to class UCBStorage ! // to class UCBStorage !
@@ -1490,7 +1490,7 @@ UCBStorage::~UCBStorage()
pImp->ReleaseRef(); pImp->ReleaseRef();
} }
UCBStorage_Impl::UCBStorage_Impl( const ::ucbhelper::Content& rContent, const OUString& rName, StreamMode nMode, UCBStorage* pStorage, bool bDirect, bool bIsRoot, bool bIsRepair, Reference< XProgressHandler > xProgressHandler ) UCBStorage_Impl::UCBStorage_Impl( const ::ucbhelper::Content& rContent, const OUString& rName, StreamMode nMode, UCBStorage* pStorage, bool bDirect, bool bIsRoot, bool bIsRepair, Reference< XProgressHandler > const & xProgressHandler )
: m_pAntiImpl( pStorage ) : m_pAntiImpl( pStorage )
, m_pContent( new ::ucbhelper::Content( rContent ) ) , m_pContent( new ::ucbhelper::Content( rContent ) )
, m_pTempFile( nullptr ) , m_pTempFile( nullptr )
@@ -1522,7 +1522,7 @@ UCBStorage_Impl::UCBStorage_Impl( const ::ucbhelper::Content& rContent, const OU
m_aURL = rName; m_aURL = rName;
} }
UCBStorage_Impl::UCBStorage_Impl( const OUString& rName, StreamMode nMode, UCBStorage* pStorage, bool bDirect, bool bIsRoot, bool bIsRepair, Reference< XProgressHandler > xProgressHandler ) UCBStorage_Impl::UCBStorage_Impl( const OUString& rName, StreamMode nMode, UCBStorage* pStorage, bool bDirect, bool bIsRoot, bool bIsRepair, Reference< XProgressHandler > const & xProgressHandler )
: m_pAntiImpl( pStorage ) : m_pAntiImpl( pStorage )
, m_pContent( nullptr ) , m_pContent( nullptr )
, m_pTempFile( nullptr ) , m_pTempFile( nullptr )

View File

@@ -40,7 +40,7 @@ const sal_Int32 nBytesCount = 32000;
OLESimpleStorage::OLESimpleStorage( OLESimpleStorage::OLESimpleStorage(
css::uno::Reference<css::uno::XComponentContext> xContext, css::uno::Reference<css::uno::XComponentContext> const & xContext,
css::uno::Sequence<css::uno::Any> const &aArguments) css::uno::Sequence<css::uno::Any> const &aArguments)
: m_bDisposed( false ) : m_bDisposed( false )
, m_pStream( nullptr ) , m_pStream( nullptr )

View File

@@ -62,7 +62,7 @@ class OLESimpleStorage : public cppu::WeakImplHelper<css::embed::XOLESimpleStora
public: public:
OLESimpleStorage(css::uno::Reference<css::uno::XComponentContext> xContext, OLESimpleStorage(css::uno::Reference<css::uno::XComponentContext> const & xContext,
css::uno::Sequence<css::uno::Any> const &arguments); css::uno::Sequence<css::uno::Any> const &arguments);
virtual ~OLESimpleStorage(); virtual ~OLESimpleStorage();