diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2017-01-26 12:28:58 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2017-01-26 12:54:43 +0000 |
commit | e57ca02849c3d87142ff5ff9099a212e72b8139c (patch) | |
tree | bcce66b27261553c308779f3e8663a269ed3a671 /framework/source/recording | |
parent | 8802ebd5172ec4bc412a59d136c82b77ab452281 (diff) |
Remove dynamic exception specifications
...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at
<https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html>
"Dynamic Exception Specifications" for details.
Most changes have been done automatically by the rewriting loplugin:dynexcspec
(after enabling the rewriting mode, to be committed shortly). The way it only
removes exception specs from declarations if it also sees a definition, it
identified some dead declarations-w/o-definitions (that have been removed
manually) and some cases where a definition appeared in multiple include files
(which have also been cleaned up manually). There's also been cases of macro
paramters (that were used to abstract over exception specs) that have become
unused now (and been removed).
Furthermore, some code needed to be cleaned up manually
(avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no
configurations available that would actually build that code. Missing @throws
documentation has not been applied in such manual clean-up.
Change-Id: I3408691256c9b0c12bc5332de976743626e13960
Reviewed-on: https://gerrit.libreoffice.org/33574
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'framework/source/recording')
-rw-r--r-- | framework/source/recording/dispatchrecorder.cxx | 20 | ||||
-rw-r--r-- | framework/source/recording/dispatchrecordersupplier.cxx | 6 |
2 files changed, 13 insertions, 13 deletions
diff --git a/framework/source/recording/dispatchrecorder.cxx b/framework/source/recording/dispatchrecorder.cxx index aca5335d6164..fad2881fb32f 100644 --- a/framework/source/recording/dispatchrecorder.cxx +++ b/framework/source/recording/dispatchrecorder.cxx @@ -100,14 +100,14 @@ DispatchRecorder::~DispatchRecorder() } // generate header -void SAL_CALL DispatchRecorder::startRecording( const css::uno::Reference< css::frame::XFrame >& ) throw( css::uno::RuntimeException, std::exception ) +void SAL_CALL DispatchRecorder::startRecording( const css::uno::Reference< css::frame::XFrame >& ) { /* SAFE{ */ /* } */ } void SAL_CALL DispatchRecorder::recordDispatch( const css::util::URL& aURL, - const css::uno::Sequence< css::beans::PropertyValue >& lArguments ) throw( css::uno::RuntimeException, std::exception ) + const css::uno::Sequence< css::beans::PropertyValue >& lArguments ) { OUString aTarget; @@ -116,7 +116,7 @@ void SAL_CALL DispatchRecorder::recordDispatch( const css::util::URL& aURL, } void SAL_CALL DispatchRecorder::recordDispatchAsComment( const css::util::URL& aURL, - const css::uno::Sequence< css::beans::PropertyValue >& lArguments ) throw( css::uno::RuntimeException, std::exception ) + const css::uno::Sequence< css::beans::PropertyValue >& lArguments ) { OUString aTarget; @@ -125,13 +125,13 @@ void SAL_CALL DispatchRecorder::recordDispatchAsComment( const css::util::URL& m_aStatements.push_back( aStatement ); } -void SAL_CALL DispatchRecorder::endRecording() throw( css::uno::RuntimeException, std::exception ) +void SAL_CALL DispatchRecorder::endRecording() { SolarMutexGuard g; m_aStatements.clear(); } -OUString SAL_CALL DispatchRecorder::getRecordedMacro() throw( css::uno::RuntimeException, std::exception ) +OUString SAL_CALL DispatchRecorder::getRecordedMacro() { SolarMutexGuard g; @@ -374,22 +374,22 @@ void SAL_CALL DispatchRecorder::implts_recordMacro( const OUString& aURL, /* } */ } -css::uno::Type SAL_CALL DispatchRecorder::getElementType() throw (css::uno::RuntimeException, std::exception) +css::uno::Type SAL_CALL DispatchRecorder::getElementType() { return cppu::UnoType<css::frame::DispatchStatement>::get(); } -sal_Bool SAL_CALL DispatchRecorder::hasElements() throw (css::uno::RuntimeException, std::exception) +sal_Bool SAL_CALL DispatchRecorder::hasElements() { return (! m_aStatements.empty()); } -sal_Int32 SAL_CALL DispatchRecorder::getCount() throw (css::uno::RuntimeException, std::exception) +sal_Int32 SAL_CALL DispatchRecorder::getCount() { return m_aStatements.size(); } -css::uno::Any SAL_CALL DispatchRecorder::getByIndex(sal_Int32 idx) throw (css::lang::IndexOutOfBoundsException, css::lang::WrappedTargetException, css::uno::RuntimeException, std::exception) +css::uno::Any SAL_CALL DispatchRecorder::getByIndex(sal_Int32 idx) { if (idx >= (sal_Int32)m_aStatements.size()) { throw css::lang::IndexOutOfBoundsException( "Dispatch recorder out of bounds" ); @@ -401,7 +401,7 @@ css::uno::Any SAL_CALL DispatchRecorder::getByIndex(sal_Int32 idx) throw (css:: return element; } -void SAL_CALL DispatchRecorder::replaceByIndex(sal_Int32 idx, const css::uno::Any& element) throw (css::lang::IllegalArgumentException, css::lang::IndexOutOfBoundsException, css::lang::WrappedTargetException, css::uno::RuntimeException, std::exception) +void SAL_CALL DispatchRecorder::replaceByIndex(sal_Int32 idx, const css::uno::Any& element) { if (element.getValueType() != cppu::UnoType<css::frame::DispatchStatement>::get()) { diff --git a/framework/source/recording/dispatchrecordersupplier.cxx b/framework/source/recording/dispatchrecordersupplier.cxx index e98b6ee6ccec..d78ca5440aea 100644 --- a/framework/source/recording/dispatchrecordersupplier.cxx +++ b/framework/source/recording/dispatchrecordersupplier.cxx @@ -83,7 +83,7 @@ DispatchRecorderSupplier::~DispatchRecorderSupplier() @change 09.04.2002 by Andreas Schluens */ -void SAL_CALL DispatchRecorderSupplier::setDispatchRecorder( const css::uno::Reference< css::frame::XDispatchRecorder >& xRecorder ) throw (css::uno::RuntimeException, std::exception) +void SAL_CALL DispatchRecorderSupplier::setDispatchRecorder( const css::uno::Reference< css::frame::XDispatchRecorder >& xRecorder ) { SolarMutexGuard g; m_xDispatchRecorder=xRecorder; @@ -103,7 +103,7 @@ void SAL_CALL DispatchRecorderSupplier::setDispatchRecorder( const css::uno::Ref @change 09.04.2002 by Andreas Schluens */ -css::uno::Reference< css::frame::XDispatchRecorder > SAL_CALL DispatchRecorderSupplier::getDispatchRecorder() throw (css::uno::RuntimeException, std::exception) +css::uno::Reference< css::frame::XDispatchRecorder > SAL_CALL DispatchRecorderSupplier::getDispatchRecorder() { SolarMutexGuard g; return m_xDispatchRecorder; @@ -124,7 +124,7 @@ css::uno::Reference< css::frame::XDispatchRecorder > SAL_CALL DispatchRecorderSu */ void SAL_CALL DispatchRecorderSupplier::dispatchAndRecord( const css::util::URL& aURL , const css::uno::Sequence< css::beans::PropertyValue >& lArguments , - const css::uno::Reference< css::frame::XDispatch >& xDispatcher ) throw (css::uno::RuntimeException, std::exception) + const css::uno::Reference< css::frame::XDispatch >& xDispatcher ) { SolarMutexClearableGuard aReadLock; css::uno::Reference< css::frame::XDispatchRecorder > xRecorder = m_xDispatchRecorder; |