diff options
| author | Lionel Elie Mamane <lionel@mamane.lu> | 2013-08-02 23:35:20 +0200 |
|---|---|---|
| committer | Lionel Elie Mamane <lionel@mamane.lu> | 2013-08-02 23:36:40 +0200 |
| commit | a43a18edb0023b2a9533e719c8cf3dd2f894dad7 (patch) | |
| tree | c522cf75d94886e14d7af42460182a7eb7e2337f /basic | |
| parent | fc9080a0c60f263d00eb71111fcda72b3c0a2ebb (diff) | |
do *not* silently ignore errors when saving libraries
In case of error, it leads to *data* *loss*.
Change-Id: I80d806ef10a3364174eced3095ebf1ea217d75b4
Diffstat (limited to 'basic')
| -rw-r--r-- | basic/source/uno/namecont.cxx | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/basic/source/uno/namecont.cxx b/basic/source/uno/namecont.cxx index e49a84642636..62577316f22d 100644 --- a/basic/source/uno/namecont.cxx +++ b/basic/source/uno/namecont.cxx @@ -1924,7 +1924,7 @@ void SfxLibraryContainer::storeLibraries_Impl( const uno::Reference< embed::XSto << rLib.aName << "\". Exception: " << comphelper::anyToString(aError)); #endif - return; + throw; } } @@ -1954,6 +1954,7 @@ void SfxLibraryContainer::storeLibraries_Impl( const uno::Reference< embed::XSto { DBG_UNHANDLED_EXCEPTION(); // TODO: error handling + throw; } } } @@ -2033,6 +2034,7 @@ void SfxLibraryContainer::storeLibraries_Impl( const uno::Reference< embed::XSto catch( const Exception& ) { DBG_UNHANDLED_EXCEPTION(); + throw; } } @@ -2384,7 +2386,7 @@ void SAL_CALL SfxLibraryContainer::loadLibrary( const OUString& Name ) << "\". Exception: " << comphelper::anyToString(aError)); #endif - return; + throw; } } @@ -2418,12 +2420,7 @@ void SAL_CALL SfxLibraryContainer::loadLibrary( const OUString& Name ) aFile = aElementName; aFile += "."; aFile += maLibElementFileExtension; - try - { - xElementStream = xLibraryStor->openStreamElement( aFile, embed::ElementModes::READ ); - } - catch(const uno::Exception& ) - {} + xElementStream = xLibraryStor->openStreamElement( aFile, embed::ElementModes::READ ); } if ( xElementStream.is() ) @@ -2436,7 +2433,7 @@ void SAL_CALL SfxLibraryContainer::loadLibrary( const OUString& Name ) "basic", "couldn't open library element stream - attempted to" " open library \"" << Name << '"'); - return; + throw RuntimeException("couln't open library element stream", *this); } } else |