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 /io/source/acceptor/acc_socket.cxx | |
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 'io/source/acceptor/acc_socket.cxx')
-rw-r--r-- | io/source/acceptor/acc_socket.cxx | 38 |
1 files changed, 9 insertions, 29 deletions
diff --git a/io/source/acceptor/acc_socket.cxx b/io/source/acceptor/acc_socket.cxx index 75764022c907..6d6a7b17bf5e 100644 --- a/io/source/acceptor/acc_socket.cxx +++ b/io/source/acceptor/acc_socket.cxx @@ -72,26 +72,15 @@ namespace io_acceptor { explicit SocketConnection( const OUString & sConnectionDescription ); virtual sal_Int32 SAL_CALL read( css::uno::Sequence< sal_Int8 >& aReadBytes, - sal_Int32 nBytesToRead ) - throw(css::io::IOException, - css::uno::RuntimeException, std::exception) override; - virtual void SAL_CALL write( const css::uno::Sequence< sal_Int8 >& aData ) - throw(css::io::IOException, - css::uno::RuntimeException, std::exception) override; - virtual void SAL_CALL flush( ) throw( - css::io::IOException, - css::uno::RuntimeException, std::exception) override; - virtual void SAL_CALL close( ) - throw(css::io::IOException, - css::uno::RuntimeException, std::exception) override; - virtual OUString SAL_CALL getDescription( ) - throw(css::uno::RuntimeException, std::exception) override; + sal_Int32 nBytesToRead ) override; + virtual void SAL_CALL write( const css::uno::Sequence< sal_Int8 >& aData ) override; + virtual void SAL_CALL flush( ) override; + virtual void SAL_CALL close( ) override; + virtual OUString SAL_CALL getDescription( ) override; // XConnectionBroadcaster - virtual void SAL_CALL addStreamListener(const css::uno::Reference< css::io::XStreamListener>& aListener) - throw(css::uno::RuntimeException, std::exception) override; - virtual void SAL_CALL removeStreamListener(const css::uno::Reference< css::io::XStreamListener>& aListener) - throw(css::uno::RuntimeException, std::exception) override; + virtual void SAL_CALL addStreamListener(const css::uno::Reference< css::io::XStreamListener>& aListener) override; + virtual void SAL_CALL removeStreamListener(const css::uno::Reference< css::io::XStreamListener>& aListener) override; public: void completeConnectionString(); @@ -186,8 +175,6 @@ namespace io_acceptor { } sal_Int32 SocketConnection::read( Sequence < sal_Int8 > & aReadBytes , sal_Int32 nBytesToRead ) - throw(css::io::IOException, - css::uno::RuntimeException, std::exception) { if( ! m_nStatus ) { @@ -234,8 +221,6 @@ namespace io_acceptor { } void SocketConnection::write( const Sequence < sal_Int8 > &seq ) - throw(css::io::IOException, - css::uno::RuntimeException, std::exception) { if( ! m_nStatus ) { @@ -270,15 +255,11 @@ namespace io_acceptor { } void SocketConnection::flush( ) - throw(css::io::IOException, - css::uno::RuntimeException, std::exception) { } void SocketConnection::close() - throw(css::io::IOException, - css::uno::RuntimeException, std::exception) { // ensure close is called only once if( 1 == osl_atomic_increment( (&m_nStatus) ) ) @@ -289,21 +270,20 @@ namespace io_acceptor { } OUString SocketConnection::getDescription() - throw( css::uno::RuntimeException, std::exception) { return m_sDescription; } // XConnectionBroadcaster - void SAL_CALL SocketConnection::addStreamListener(const Reference<XStreamListener> & aListener) throw(RuntimeException, std::exception) + void SAL_CALL SocketConnection::addStreamListener(const Reference<XStreamListener> & aListener) { MutexGuard guard(_mutex); _listeners.insert(aListener); } - void SAL_CALL SocketConnection::removeStreamListener(const Reference<XStreamListener> & aListener) throw(RuntimeException, std::exception) + void SAL_CALL SocketConnection::removeStreamListener(const Reference<XStreamListener> & aListener) { MutexGuard guard(_mutex); |