diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2016-09-13 13:09:01 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2016-09-13 13:19:22 +0200 |
commit | 91dd2db17bd6cb9b357d1d69b187174e31eabef0 (patch) | |
tree | d634de3a1a6820904b5699c2136b79b1a5a807c7 /package | |
parent | 6f8ea7e89ea190b9462c945d55a3ad8777b2f3ef (diff) |
loplugin:override: No more need for the "MSVC dtor override" workaround
The issue of 362d4f0cd4e50111edfae9d30c90602c37ed65a2 "Explicitly mark
overriding destructors as 'virtual'" appears to no longer be a problem with
MSVC 2013.
(The little change in the rewriting code of compilerplugins/clang/override.cxx
was necessary to prevent an endless loop when adding "override" to
OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager();
in chart2/source/inc/LifeTime.hxx, getting stuck in the leading
OOO_DLLPUBLIC_CHARTTOOLS macro. Can't remember what that
isAtEndOfImmediateMacroExpansion thing was originally necessary for, anyway.)
Change-Id: I534c634504d7216b9bb632c2775c04eaf27e927e
Diffstat (limited to 'package')
23 files changed, 23 insertions, 23 deletions
diff --git a/package/inc/ZipPackage.hxx b/package/inc/ZipPackage.hxx index 12e1602178ac..378876962b0d 100644 --- a/package/inc/ZipPackage.hxx +++ b/package/inc/ZipPackage.hxx @@ -117,7 +117,7 @@ protected: public: ZipPackage( const css::uno::Reference < css::uno::XComponentContext > &xContext ); - virtual ~ZipPackage(); + virtual ~ZipPackage() override; ZipFile& getZipFile() { return *m_pZipFile;} sal_Int32 getFormat() const { return m_nFormat; } diff --git a/package/inc/ZipPackageBuffer.hxx b/package/inc/ZipPackageBuffer.hxx index 6873d7b45065..daac93d7c17f 100644 --- a/package/inc/ZipPackageBuffer.hxx +++ b/package/inc/ZipPackageBuffer.hxx @@ -37,7 +37,7 @@ protected: bool m_bMustInitBuffer; public: ZipPackageBuffer(); - virtual ~ZipPackageBuffer(); + virtual ~ZipPackageBuffer() override; inline void realloc ( sal_Int32 nSize ) { m_aBuffer.realloc ( nSize ); } const css::uno::Sequence < sal_Int8>& getSequence () const { return m_aBuffer; } diff --git a/package/inc/ZipPackageEntry.hxx b/package/inc/ZipPackageEntry.hxx index 6d7381e280b7..1dc3539e234d 100644 --- a/package/inc/ZipPackageEntry.hxx +++ b/package/inc/ZipPackageEntry.hxx @@ -57,7 +57,7 @@ protected: public: ZipEntry aEntry; ZipPackageEntry(); - virtual ~ZipPackageEntry(); + virtual ~ZipPackageEntry() override; const OUString& GetMediaType () const { return msMediaType; } void SetMediaType ( const OUString & sNewType) { msMediaType = sNewType; } diff --git a/package/inc/ZipPackageFolder.hxx b/package/inc/ZipPackageFolder.hxx index b024d2ccea04..916c82477d9d 100644 --- a/package/inc/ZipPackageFolder.hxx +++ b/package/inc/ZipPackageFolder.hxx @@ -47,7 +47,7 @@ public: ZipPackageFolder( const css::uno::Reference < css::uno::XComponentContext >& xContext, sal_Int32 nFormat, bool bAllowRemoveOnInsert ); - virtual ~ZipPackageFolder(); + virtual ~ZipPackageFolder() override; const OUString& GetVersion() const { return m_sVersion; } void SetVersion( const OUString& aVersion ) { m_sVersion = aVersion; } diff --git a/package/inc/ZipPackageStream.hxx b/package/inc/ZipPackageStream.hxx index 67f536816a0a..bd1367bab7fa 100644 --- a/package/inc/ZipPackageStream.hxx +++ b/package/inc/ZipPackageStream.hxx @@ -124,7 +124,7 @@ public: const css::uno::Reference < css::uno::XComponentContext >& xContext, sal_Int32 nFormat, bool bAllowRemoveOnInsert ); - virtual ~ZipPackageStream(); + virtual ~ZipPackageStream() override; css::uno::Reference< css::io::XInputStream > GetRawEncrStreamNoHeaderCopy(); css::uno::Reference< css::io::XInputStream > TryToGetRawFromDataStream(bool bAddHeaderForEncr ); diff --git a/package/inc/zipfileaccess.hxx b/package/inc/zipfileaccess.hxx index d324b0a4476c..8160447e33b0 100644 --- a/package/inc/zipfileaccess.hxx +++ b/package/inc/zipfileaccess.hxx @@ -53,7 +53,7 @@ class OZipFileAccess : public ::cppu::WeakImplHelper< public: OZipFileAccess( const css::uno::Reference< css::uno::XComponentContext >& rxContext ); - virtual ~OZipFileAccess(); + virtual ~OZipFileAccess() override; static css::uno::Sequence< OUString > GetPatternsFromString_Impl( const OUString& aString ); diff --git a/package/source/manifest/ManifestImport.hxx b/package/source/manifest/ManifestImport.hxx index 4e25499f5629..9cace18ee251 100644 --- a/package/source/manifest/ManifestImport.hxx +++ b/package/source/manifest/ManifestImport.hxx @@ -119,7 +119,7 @@ protected: public: ManifestImport( std::vector < css::uno::Sequence < css::beans::PropertyValue > > & rNewVector ); - virtual ~ManifestImport(); + virtual ~ManifestImport() override; virtual void SAL_CALL startDocument( ) throw(css::xml::sax::SAXException, css::uno::RuntimeException, std::exception) override; virtual void SAL_CALL endDocument( ) diff --git a/package/source/manifest/ManifestReader.hxx b/package/source/manifest/ManifestReader.hxx index 0e0b4b0fa0aa..ba9221d5a77e 100644 --- a/package/source/manifest/ManifestReader.hxx +++ b/package/source/manifest/ManifestReader.hxx @@ -40,7 +40,7 @@ private: css::uno::Reference< css::uno::XComponentContext > m_xContext; public: ManifestReader( const css::uno::Reference< css::uno::XComponentContext > & xContext ); - virtual ~ManifestReader(); + virtual ~ManifestReader() override; // XManifestReader virtual css::uno::Sequence< css::uno::Sequence< css::beans::PropertyValue > > SAL_CALL readManifestSequence( const css::uno::Reference< css::io::XInputStream >& rStream ) diff --git a/package/source/manifest/ManifestWriter.hxx b/package/source/manifest/ManifestWriter.hxx index fd20e33252ce..0a63befb89f5 100644 --- a/package/source/manifest/ManifestWriter.hxx +++ b/package/source/manifest/ManifestWriter.hxx @@ -40,7 +40,7 @@ private: css::uno::Reference< css::uno::XComponentContext > m_xContext; public: ManifestWriter( const css::uno::Reference< css::uno::XComponentContext > & xContext ); - virtual ~ManifestWriter(); + virtual ~ManifestWriter() override; // XManifestWriter virtual void SAL_CALL writeManifestSequence( const css::uno::Reference< css::io::XOutputStream >& rStream, const css::uno::Sequence< css::uno::Sequence< css::beans::PropertyValue > >& rSequence ) diff --git a/package/source/xstor/disposelistener.hxx b/package/source/xstor/disposelistener.hxx index 21b9b31d05d1..75d81c8913c3 100644 --- a/package/source/xstor/disposelistener.hxx +++ b/package/source/xstor/disposelistener.hxx @@ -32,7 +32,7 @@ class OChildDispListener_Impl : public ::cppu::WeakImplHelper < css::lang::XEven public: explicit OChildDispListener_Impl( OStorage& aStorage ); - virtual ~OChildDispListener_Impl(); + virtual ~OChildDispListener_Impl() override; void OwnerIsDisposed(); diff --git a/package/source/xstor/ocompinstream.hxx b/package/source/xstor/ocompinstream.hxx index 451f2201fb5a..36c91a54c190 100644 --- a/package/source/xstor/ocompinstream.hxx +++ b/package/source/xstor/ocompinstream.hxx @@ -58,7 +58,7 @@ public: const css::uno::Sequence< css::beans::PropertyValue >& aProps, sal_Int32 nStorageType ); - virtual ~OInputCompStream(); + virtual ~OInputCompStream() override; void InternalDispose(); diff --git a/package/source/xstor/oseekinstream.hxx b/package/source/xstor/oseekinstream.hxx index dcadc763e98e..803f4c7fc783 100644 --- a/package/source/xstor/oseekinstream.hxx +++ b/package/source/xstor/oseekinstream.hxx @@ -40,7 +40,7 @@ public: const css::uno::Sequence< css::beans::PropertyValue >& aProps, sal_Int32 nStorageType ); - virtual ~OInputSeekStream(); + virtual ~OInputSeekStream() override; virtual css::uno::Sequence< css::uno::Type > SAL_CALL getTypes() throw (css::uno::RuntimeException, std::exception) override; diff --git a/package/source/xstor/owriteablestream.hxx b/package/source/xstor/owriteablestream.hxx index 63c7fccee258..130d5fe1c532 100644 --- a/package/source/xstor/owriteablestream.hxx +++ b/package/source/xstor/owriteablestream.hxx @@ -272,7 +272,7 @@ protected: public: - virtual ~OWriteStream(); + virtual ~OWriteStream() override; void CheckInitOnDemand(); void DeInit(); diff --git a/package/source/xstor/selfterminatefilestream.hxx b/package/source/xstor/selfterminatefilestream.hxx index 4cd4dbdbe81d..abd60c4676d1 100644 --- a/package/source/xstor/selfterminatefilestream.hxx +++ b/package/source/xstor/selfterminatefilestream.hxx @@ -41,7 +41,7 @@ protected: public: OSelfTerminateFileStream( const css::uno::Reference< css::uno::XComponentContext >& xContext, const OUString& aURL ); - virtual ~OSelfTerminateFileStream(); + virtual ~OSelfTerminateFileStream() override; void CloseStreamDeleteFile(); diff --git a/package/source/xstor/switchpersistencestream.hxx b/package/source/xstor/switchpersistencestream.hxx index 713a7cfdbd01..dacb28ad805d 100644 --- a/package/source/xstor/switchpersistencestream.hxx +++ b/package/source/xstor/switchpersistencestream.hxx @@ -68,7 +68,7 @@ public: const css::uno::Reference< css::uno::XComponentContext >& xContext, const css::uno::Reference< css::io::XInputStream >& xInStream ); - virtual ~SwitchablePersistenceStream(); + virtual ~SwitchablePersistenceStream() override; void SwitchPersistenceTo( const css::uno::Reference< css::io::XStream >& xStream ); diff --git a/package/source/xstor/xstorage.hxx b/package/source/xstor/xstorage.hxx index f7f82c086126..a0f2b3b74c8f 100644 --- a/package/source/xstor/xstorage.hxx +++ b/package/source/xstor/xstorage.hxx @@ -318,7 +318,7 @@ public: OStorage( OStorage_Impl* pImpl, bool bReadOnlyWrap ); - virtual ~OStorage(); + virtual ~OStorage() override; void SAL_CALL InternalDispose( bool bNotifyImpl ); diff --git a/package/source/zipapi/XUnbufferedStream.hxx b/package/source/zipapi/XUnbufferedStream.hxx index 1d34c2c158e6..a22124a5cd02 100644 --- a/package/source/zipapi/XUnbufferedStream.hxx +++ b/package/source/zipapi/XUnbufferedStream.hxx @@ -80,7 +80,7 @@ public: const css::uno::Reference < css::io::XInputStream >& xRawStream, const ::rtl::Reference< EncryptionData >& rData ); - virtual ~XUnbufferedStream(); + virtual ~XUnbufferedStream() override; // XInputStream virtual sal_Int32 SAL_CALL readBytes( css::uno::Sequence< sal_Int8 >& aData, sal_Int32 nBytesToRead ) diff --git a/package/source/zipapi/blowfishcontext.hxx b/package/source/zipapi/blowfishcontext.hxx index 47c4adf6c263..9f368a5cc117 100644 --- a/package/source/zipapi/blowfishcontext.hxx +++ b/package/source/zipapi/blowfishcontext.hxx @@ -37,7 +37,7 @@ class BlowfishCFB8CipherContext : public cppu::WeakImplHelper< css::xml::crypto: public: - virtual ~BlowfishCFB8CipherContext(); + virtual ~BlowfishCFB8CipherContext() override; static css::uno::Reference< css::xml::crypto::XCipherContext > Create( const css::uno::Sequence< sal_Int8 >& aDerivedKey, const css::uno::Sequence< sal_Int8 >& aInitVector, bool bEncrypt ); diff --git a/package/source/zipapi/sha1context.hxx b/package/source/zipapi/sha1context.hxx index 5298d3bfea67..3be983d31913 100644 --- a/package/source/zipapi/sha1context.hxx +++ b/package/source/zipapi/sha1context.hxx @@ -35,7 +35,7 @@ class SHA1DigestContext : public cppu::WeakImplHelper< css::xml::crypto::XDigest public: - virtual ~SHA1DigestContext(); + virtual ~SHA1DigestContext() override; static css::uno::Reference< css::xml::crypto::XDigestContext > Create(); diff --git a/package/source/zippackage/ContentInfo.hxx b/package/source/zippackage/ContentInfo.hxx index 1fd3a95159a0..890368744bbe 100644 --- a/package/source/zippackage/ContentInfo.hxx +++ b/package/source/zippackage/ContentInfo.hxx @@ -47,7 +47,7 @@ public: , pFolder ( pNewFolder ) { } - virtual ~ContentInfo () + virtual ~ContentInfo () override { if ( bFolder ) pFolder->clearParent(); diff --git a/package/source/zippackage/ZipPackageFolderEnumeration.hxx b/package/source/zippackage/ZipPackageFolderEnumeration.hxx index eee73c5db362..6befe8c6f18f 100644 --- a/package/source/zippackage/ZipPackageFolderEnumeration.hxx +++ b/package/source/zippackage/ZipPackageFolderEnumeration.hxx @@ -36,7 +36,7 @@ protected: public: //ZipPackageFolderEnumeration (unordered_map < OUString, css::uno::Reference < css::container::XNamed >, hashFunc, eqFunc > &rInput); ZipPackageFolderEnumeration (ContentHash &rInput); - virtual ~ZipPackageFolderEnumeration(); + virtual ~ZipPackageFolderEnumeration() override; // XEnumeration virtual sal_Bool SAL_CALL hasMoreElements( ) diff --git a/package/source/zippackage/ZipPackageSink.hxx b/package/source/zippackage/ZipPackageSink.hxx index 1f141d2eb989..6ece3eb528f7 100644 --- a/package/source/zippackage/ZipPackageSink.hxx +++ b/package/source/zippackage/ZipPackageSink.hxx @@ -28,7 +28,7 @@ protected: css::uno::Reference < css::io::XInputStream > xStream; public: ZipPackageSink(); - virtual ~ZipPackageSink(); + virtual ~ZipPackageSink() override; virtual void SAL_CALL setInputStream( const css::uno::Reference< css::io::XInputStream >& aStream ) throw(css::uno::RuntimeException, std::exception) override; virtual css::uno::Reference< css::io::XInputStream > SAL_CALL getInputStream( ) diff --git a/package/source/zippackage/wrapstreamforshare.hxx b/package/source/zippackage/wrapstreamforshare.hxx index 7e5950a39467..8fc51001755e 100644 --- a/package/source/zippackage/wrapstreamforshare.hxx +++ b/package/source/zippackage/wrapstreamforshare.hxx @@ -39,7 +39,7 @@ protected: public: WrapStreamForShare( const css::uno::Reference< css::io::XInputStream >& xInStream, const rtl::Reference<SotMutexHolder>& rMutexRef ); - virtual ~WrapStreamForShare(); + virtual ~WrapStreamForShare() override; // XInputStream virtual sal_Int32 SAL_CALL readBytes( css::uno::Sequence< sal_Int8 >& aData, sal_Int32 nBytesToRead ) |