diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2015-06-30 14:33:43 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2015-06-30 14:41:47 +0200 |
commit | daf06757009e09e3ecb4a3e91f2d068ca9612fa7 (patch) | |
tree | 61ee9ccf53dfdc78b0176e01754b3dd1f3182370 /ucb | |
parent | 07cd14f1e3ee718e56d063db8c331bec173f4408 (diff) |
Coverity 1308468 etc.: Exception specifications
While these functions do not throw any UNO exceptions, no need to specify them
as nothrow, and do allow std exceptions to pass through.
Also covers Coverity 1308470, 1308476, 1308482, 1308495, 1308496, 1308502,
1308506, and 1308575.
Change-Id: I00306f606f3f20462016177eebbd26ed5af76506
Diffstat (limited to 'ucb')
-rw-r--r-- | ucb/source/ucp/file/bc.cxx | 5 | ||||
-rw-r--r-- | ucb/source/ucp/file/bc.hxx | 15 | ||||
-rw-r--r-- | ucb/source/ucp/file/shell.cxx | 16 | ||||
-rw-r--r-- | ucb/source/ucp/file/shell.hxx | 51 |
4 files changed, 22 insertions, 65 deletions
diff --git a/ucb/source/ucp/file/bc.cxx b/ucb/source/ucp/file/bc.cxx index 3920d9ebb2db..c1e12b6cb418 100644 --- a/ucb/source/ucp/file/bc.cxx +++ b/ucb/source/ucp/file/bc.cxx @@ -834,7 +834,6 @@ Sequence< Any > SAL_CALL BaseContent::setPropertyValues( sal_Int32 nMyCommandIdentifier, const Sequence< beans::PropertyValue >& Values ) - throw() { if( m_nState & Deleted ) { // To do @@ -971,7 +970,6 @@ Reference< XDynamicResultSet > SAL_CALL BaseContent::open( sal_Int32 nMyCommandIdentifier, const OpenCommandArgument2& aCommandArgument ) - throw() { Reference< XDynamicResultSet > retValue( 0 ); @@ -1047,7 +1045,6 @@ BaseContent::open( void SAL_CALL BaseContent::deleteContent( sal_Int32 nMyCommandIdentifier ) - throw() { if( m_nState & Deleted ) return; @@ -1064,7 +1061,6 @@ BaseContent::deleteContent( sal_Int32 nMyCommandIdentifier ) void SAL_CALL BaseContent::transfer( sal_Int32 nMyCommandIdentifier, const TransferInfo& aTransferInfo ) - throw() { if( m_nState & Deleted ) return; @@ -1136,7 +1132,6 @@ BaseContent::transfer( sal_Int32 nMyCommandIdentifier, void SAL_CALL BaseContent::insert( sal_Int32 nMyCommandIdentifier, const InsertCommandArgument& aInsertArgument ) - throw() { if( m_nState & FullFeatured ) { diff --git a/ucb/source/ucp/file/bc.hxx b/ucb/source/ucp/file/bc.hxx index 21b6acad5913..381880d57751 100644 --- a/ucb/source/ucp/file/bc.hxx +++ b/ucb/source/ucp/file/bc.hxx @@ -308,29 +308,24 @@ namespace fileaccess { com::sun::star::uno::Sequence< com::sun::star::uno::Any > SAL_CALL setPropertyValues( sal_Int32 nMyCommandIdentifier, - const com::sun::star::uno::Sequence< com::sun::star::beans::PropertyValue >& Values ) - throw( ); + const com::sun::star::uno::Sequence< com::sun::star::beans::PropertyValue >& Values ); com::sun::star::uno::Reference< com::sun::star::ucb::XDynamicResultSet > SAL_CALL open( sal_Int32 nMyCommandIdentifier, - const com::sun::star::ucb::OpenCommandArgument2& aCommandArgument ) - throw(); + const com::sun::star::ucb::OpenCommandArgument2& aCommandArgument ); void SAL_CALL - deleteContent( sal_Int32 nMyCommandIdentifier ) - throw(); + deleteContent( sal_Int32 nMyCommandIdentifier ); void SAL_CALL transfer( sal_Int32 nMyCommandIdentifier, - const com::sun::star::ucb::TransferInfo& aTransferInfo ) - throw(); + const com::sun::star::ucb::TransferInfo& aTransferInfo ); void SAL_CALL insert( sal_Int32 nMyCommandIdentifier, - const com::sun::star::ucb::InsertCommandArgument& aInsertArgument ) - throw(); + const com::sun::star::ucb::InsertCommandArgument& aInsertArgument ); void SAL_CALL endTask( sal_Int32 CommandId ); diff --git a/ucb/source/ucp/file/shell.cxx b/ucb/source/ucp/file/shell.cxx index 1f352d53657a..6b0ad355491c 100644 --- a/ucb/source/ucp/file/shell.cxx +++ b/ucb/source/ucp/file/shell.cxx @@ -570,7 +570,6 @@ shell::deassociate( const OUString& aUnqPath, void SAL_CALL shell::page( sal_Int32 CommandId, const OUString& aUnqPath, const uno::Reference< io::XOutputStream >& xOutputStream ) - throw() { uno::Reference< XContentProvider > xProvider( m_pProvider ); osl::File aFile( aUnqPath ); @@ -661,7 +660,6 @@ uno::Reference< io::XInputStream > SAL_CALL shell::open( sal_Int32 CommandId, const OUString& aUnqPath, bool bLock ) - throw() { XInputStream_impl* xInputStream = new XInputStream_impl( this, aUnqPath, bLock ); // from filinpstr.hxx @@ -698,7 +696,6 @@ uno::Reference< io::XStream > SAL_CALL shell::open_rw( sal_Int32 CommandId, const OUString& aUnqPath, bool bLock ) - throw() { XStream_impl* xStream = new XStream_impl( this, aUnqPath, bLock ); // from filstr.hxx @@ -735,7 +732,6 @@ shell::ls( sal_Int32 CommandId, const sal_Int32 OpenMode, const uno::Sequence< beans::Property >& seq, const uno::Sequence< NumberedSortingInfo >& seqSort ) - throw() { XResultSet_impl* p = new XResultSet_impl( this,aUnqPath,OpenMode,seq,seqSort ); @@ -766,7 +762,6 @@ shell::ls( sal_Int32 CommandId, uno::Reference< XCommandInfo > SAL_CALL shell::info_c() - throw() { XCommandInfo_impl* p = new XCommandInfo_impl( this ); return uno::Reference< XCommandInfo >( p ); @@ -784,7 +779,6 @@ shell::info_c() uno::Reference< beans::XPropertySetInfo > SAL_CALL shell::info_p( const OUString& aUnqPath ) - throw() { osl::MutexGuard aGuard( m_aMutex ); XPropertySetInfo_impl* p = new XPropertySetInfo_impl( this,aUnqPath ); @@ -807,7 +801,6 @@ shell::info_p( const OUString& aUnqPath ) uno::Sequence< uno::Any > SAL_CALL shell::setv( const OUString& aUnqPath, const uno::Sequence< beans::PropertyValue >& values ) - throw() { osl::MutexGuard aGuard( m_aMutex ); @@ -1044,7 +1037,6 @@ uno::Reference< sdbc::XRow > SAL_CALL shell::getv( sal_Int32 CommandId, const OUString& aUnqPath, const uno::Sequence< beans::Property >& properties ) - throw() { uno::Sequence< uno::Any > seq( properties.getLength() ); @@ -1112,7 +1104,6 @@ shell::move( sal_Int32 CommandId, const OUString& srcUnqPath, const OUString& dstUnqPathIn, const sal_Int32 NameClash ) - throw() { // --> #i88446# Method notifyContentExchanged( getContentExchangedEventListeners( srcUnqPath,dstUnqPath,!isDocument ) ); crashes if // srcUnqPath and dstUnqPathIn are equal @@ -1323,7 +1314,6 @@ shell::copy( const OUString& srcUnqPath, const OUString& dstUnqPathIn, sal_Int32 NameClash ) - throw() { osl::FileBase::RC nError; OUString dstUnqPath( dstUnqPathIn ); @@ -1497,7 +1487,6 @@ shell::remove( sal_Int32 CommandId, const OUString& aUnqPath, sal_Int32 IsWhat, bool MustExist ) - throw() { sal_Int32 nMask = osl_FileStatus_Mask_Type | osl_FileStatus_Mask_FileURL; @@ -1660,7 +1649,6 @@ bool SAL_CALL shell::mkdir( sal_Int32 CommandId, const OUString& rUnqPath, bool OverWrite ) - throw() { OUString aUnqPath; @@ -1722,7 +1710,6 @@ shell::mkfil( sal_Int32 CommandId, const OUString& aUnqPath, bool Overwrite, const uno::Reference< io::XInputStream >& aInputStream ) - throw() { // return value unimportant bool bSuccess = write( CommandId, @@ -1754,7 +1741,6 @@ shell::write( sal_Int32 CommandId, const OUString& aUnqPath, bool OverWrite, const uno::Reference< io::XInputStream >& aInputStream ) - throw() { if( ! aInputStream.is() ) { @@ -1989,7 +1975,6 @@ shell::copy_recursive( const OUString& srcUnqPath, const OUString& dstUnqPath, sal_Int32 TypeToCopy, bool testExistBeforeCopy ) - throw() { osl::FileBase::RC err = osl::FileBase::E_None; @@ -2061,7 +2046,6 @@ shell::copy_recursive( const OUString& srcUnqPath, bool SAL_CALL shell::ensuredir( sal_Int32 CommandId, const OUString& rUnqPath, sal_Int32 errorCode ) - throw() { OUString aPath; diff --git a/ucb/source/ucp/file/shell.hxx b/ucb/source/ucp/file/shell.hxx index a136bb7a61bb..fc9de680328d 100644 --- a/ucb/source/ucp/file/shell.hxx +++ b/ucb/source/ucp/file/shell.hxx @@ -210,8 +210,7 @@ namespace fileaccess { void SAL_CALL page( sal_Int32 CommandId, const OUString& aUnqPath, - const com::sun::star::uno::Reference< com::sun::star::io::XOutputStream >& xOutputStream ) - throw(); + const com::sun::star::uno::Reference< com::sun::star::io::XOutputStream >& xOutputStream ); /** @@ -221,8 +220,7 @@ namespace fileaccess { com::sun::star::uno::Reference< com::sun::star::io::XInputStream > SAL_CALL open( sal_Int32 CommandId, const OUString& aUnqPath, - bool bLock ) - throw(); + bool bLock ); /** @@ -233,8 +231,7 @@ namespace fileaccess { com::sun::star::uno::Reference< com::sun::star::io::XStream > SAL_CALL open_rw( sal_Int32 CommandId, const OUString& aUnqPath, - bool bLock ) - throw(); + bool bLock ); /** @@ -247,8 +244,7 @@ namespace fileaccess { const OUString& aUnqPath, const sal_Int32 OpenMode, const com::sun::star::uno::Sequence< com::sun::star::beans::Property >& sProperty, - const com::sun::star::uno::Sequence< com::sun::star::ucb::NumberedSortingInfo > & sSortingInfo ) - throw(); + const com::sun::star::uno::Sequence< com::sun::star::ucb::NumberedSortingInfo > & sSortingInfo ); /** @@ -257,13 +253,11 @@ namespace fileaccess { // Info for commands com::sun::star::uno::Reference< com::sun::star::ucb::XCommandInfo > SAL_CALL - info_c() - throw(); + info_c(); // Info for the properties com::sun::star::uno::Reference< com::sun::star::beans::XPropertySetInfo > SAL_CALL - info_p( const OUString& aUnqPath ) - throw(); + info_p( const OUString& aUnqPath ); /** @@ -272,8 +266,7 @@ namespace fileaccess { com::sun::star::uno::Sequence< com::sun::star::uno::Any > SAL_CALL setv( const OUString& aUnqPath, - const com::sun::star::uno::Sequence< com::sun::star::beans::PropertyValue >& values ) - throw(); + const com::sun::star::uno::Sequence< com::sun::star::beans::PropertyValue >& values ); /** @@ -284,8 +277,7 @@ namespace fileaccess { com::sun::star::uno::Reference< com::sun::star::sdbc::XRow > SAL_CALL getv( sal_Int32 CommandId, const OUString& aUnqPath, - const com::sun::star::uno::Sequence< com::sun::star::beans::Property >& properties ) - throw(); + const com::sun::star::uno::Sequence< com::sun::star::beans::Property >& properties ); /********************************************************************************/ @@ -300,8 +292,7 @@ namespace fileaccess { move( sal_Int32 CommandId, const OUString& srcUnqPath, // Full file(folder)-path const OUString& dstUnqPath, // Path to the destination-directory - const sal_Int32 NameClash ) - throw(); + const sal_Int32 NameClash ); /** * Copies the content belonging to fileURL srcUnqPath to fileURL dstUnqPath ( files and directories ) @@ -311,8 +302,7 @@ namespace fileaccess { copy( sal_Int32 CommandId, // See "move" const OUString& srcUnqPath, const OUString& dstUnqPath, - sal_Int32 NameClash ) - throw(); + sal_Int32 NameClash ); #define RemoveFolder 1 #define RemoveFile -1 @@ -326,8 +316,7 @@ namespace fileaccess { remove( sal_Int32 CommandId, const OUString& aUnqPath, sal_Int32 TypeToMove = RemoveUnknown, - bool MustExist = true ) - throw(); + bool MustExist = true ); #undef RemoveUnknown #undef RemoveFile @@ -346,8 +335,7 @@ namespace fileaccess { bool SAL_CALL mkdir( sal_Int32 CommandId, const OUString& aDirectoryName, - bool OverWrite ) - throw(); + bool OverWrite ); /** @@ -360,8 +348,7 @@ namespace fileaccess { mkfil( sal_Int32 CommandId, const OUString& aFileName, bool OverWrite, - const com::sun::star::uno::Reference< com::sun::star::io::XInputStream >& aInputStream ) - throw(); + const com::sun::star::uno::Reference< com::sun::star::io::XInputStream >& aInputStream ); /** @@ -373,8 +360,7 @@ namespace fileaccess { write( sal_Int32 CommandId, const OUString& aUnqPath, bool OverWrite, - const com::sun::star::uno::Reference< com::sun::star::io::XInputStream >& aInputStream ) - throw(); + const com::sun::star::uno::Reference< com::sun::star::io::XInputStream >& aInputStream ); @@ -517,8 +503,7 @@ namespace fileaccess { void SAL_CALL setFileProperties( const com::sun::star::uno::Sequence< com::sun::star::beans::PropertyValue >& values, - sal_Int32 numberOfValues ) - throw(); + sal_Int32 numberOfValues ); // Helper function for public copy @@ -528,8 +513,7 @@ namespace fileaccess { const OUString& srcUnqPath, const OUString& dstUnqPath, sal_Int32 TypeToCopy, - bool testExistence ) - throw(); + bool testExistence ); // Helper function for mkfil,mkdir and write @@ -541,8 +525,7 @@ namespace fileaccess { bool SAL_CALL ensuredir( sal_Int32 CommandId, const OUString& aDirectoryName, - sal_Int32 errorCode ) - throw(); + sal_Int32 errorCode ); // General osl::Mutex m_aMutex; |