diff options
author | Caolán McNamara <caolanm@redhat.com> | 2011-08-15 22:15:57 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2011-08-16 09:04:08 +0100 |
commit | 7684c90b6a40f9975e48cd86700676feb6f310df (patch) | |
tree | f23e04fe70adf74e959a1fab5b4845f6361236ac /reportdesign | |
parent | 2ef7e20436221f0adc8f8f1b95ac2a7ba734b2af (diff) |
catch by const reference
Diffstat (limited to 'reportdesign')
-rw-r--r-- | reportdesign/source/core/api/ReportDefinition.cxx | 26 |
1 files changed, 12 insertions, 14 deletions
diff --git a/reportdesign/source/core/api/ReportDefinition.cxx b/reportdesign/source/core/api/ReportDefinition.cxx index 724664aeb51f..a2d5bad42ddf 100644 --- a/reportdesign/source/core/api/ReportDefinition.cxx +++ b/reportdesign/source/core/api/ReportDefinition.cxx @@ -241,7 +241,7 @@ void lcl_extractAndStartStatusIndicator( const ::comphelper::MediaDescriptor& _r _rCallArgs[ nLength ] <<= _rxStatusIndicator; } } - catch( const uno::Exception& ) + catch (const uno::Exception&) { OSL_FAIL( "lcl_extractAndStartStatusIndicator: caught an exception!" ); } @@ -542,7 +542,7 @@ namespace ::comphelper::disposeComponent(xModel); } } - catch(uno::Exception& ex) + catch (const uno::Exception&) { DBG_UNHANDLED_EXCEPTION(); } @@ -777,7 +777,7 @@ void OReportDefinition::init() } m_pImpl->m_pObjectContainer.reset( new comphelper::EmbeddedObjectContainer(m_pImpl->m_xStorage , static_cast<cppu::OWeakObject*>(this) ) ); } - catch ( const uno::Exception& ) + catch (const uno::Exception&) { DBG_UNHANDLED_EXCEPTION(); } @@ -1257,8 +1257,8 @@ void SAL_CALL OReportDefinition::close( ::sal_Bool _bDeliverOwnership ) throw (u if ( xFrame.is() ) xFrame->close( _bDeliverOwnership ); } - catch( const util::CloseVetoException& ) { throw; } - catch( const uno::Exception& ) + catch (const util::CloseVetoException&) { throw; } + catch (const uno::Exception&) { OSL_FAIL( "ODatabaseDocument::impl_closeControllerFrames: caught an unexpected exception!" ); } @@ -1288,7 +1288,7 @@ void SAL_CALL OReportDefinition::close( ::sal_Bool _bDeliverOwnership ) throw (u fillArgs(aDescriptor); m_pImpl->m_pReportModel->SetModified(sal_False); } - catch ( ... ) + catch (...) { m_pImpl->m_pUndoManager->GetSfxUndoManager().EnableUndo( true ); throw; @@ -1619,7 +1619,7 @@ void SAL_CALL OReportDefinition::storeToStorage( const uno::Reference< embed::XS if ( xTransact.is() ) xTransact->commit(); } - catch(uno::Exception) + catch (const uno::Exception&) { OSL_FAIL("Exception Caught: Could not commit report storage!"); throw io::IOException(); @@ -1730,9 +1730,8 @@ sal_Bool OReportDefinition::WriteThroughComponent( // finally, commit stream. return bRet; } - catch(const uno::Exception& e) + catch (const uno::Exception&) { - (void)e; // helper to know what e contains throw; } } @@ -1870,7 +1869,7 @@ void SAL_CALL OReportDefinition::load( const uno::Sequence< beans::PropertyValue { xDocumentStorage.set( xStorageFactory->createInstanceWithArguments( aStorageCreationArgs ), uno::UNO_QUERY_THROW ); } - catch( const uno::Exception& ) + catch (const uno::Exception&) { if ( i == nLastOpenMode ) throw lang::WrappedTargetException( @@ -2028,7 +2027,7 @@ void OReportDefinition::notifyEvent(const ::rtl::OUString& _sEventName) aGuard.clear(); m_pImpl->m_aDocEventListeners.notifyEach(&document::XEventListener::notifyEvent,aEvt); } - catch(uno::Exception&) + catch (const uno::Exception&) { } } @@ -2077,7 +2076,7 @@ uno::Reference< container::XIndexAccess > SAL_CALL OReportDefinition::getViewDat { xContainer->insertByIndex(xContainer->getCount(),(*aIter)->getViewData()); } - catch(uno::Exception&) + catch (const uno::Exception&) { } } @@ -2876,10 +2875,9 @@ uno::Any SAL_CALL OReportDefinition::getTransferData( const datatransfer::DataFl { aResult <<= getPreferredVisualRepresentation(0).Data; } - catch( uno::Exception & ex ) + catch (const uno::Exception &) { DBG_UNHANDLED_EXCEPTION(); - (void)ex; } } else |