summaryrefslogtreecommitdiff
path: root/unotools/source/streaming
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2017-01-26 12:28:58 +0100
committerStephan Bergmann <sbergman@redhat.com>2017-01-26 12:54:43 +0000
commite57ca02849c3d87142ff5ff9099a212e72b8139c (patch)
treebcce66b27261553c308779f3e8663a269ed3a671 /unotools/source/streaming
parent8802ebd5172ec4bc412a59d136c82b77ab452281 (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 'unotools/source/streaming')
-rw-r--r--unotools/source/streaming/streamhelper.cxx11
-rw-r--r--unotools/source/streaming/streamwrap.cxx41
2 files changed, 23 insertions, 29 deletions
diff --git a/unotools/source/streaming/streamhelper.cxx b/unotools/source/streaming/streamhelper.cxx
index e2881b7b9a55..d3f41ad5e286 100644
--- a/unotools/source/streaming/streamhelper.cxx
+++ b/unotools/source/streaming/streamhelper.cxx
@@ -23,7 +23,6 @@ namespace utl
{
sal_Int32 SAL_CALL OInputStreamHelper::readBytes(css::uno::Sequence< sal_Int8 >& aData, sal_Int32 nBytesToRead)
- throw(css::io::NotConnectedException, css::io::BufferSizeExceededException, css::io::IOException, css::uno::RuntimeException, std::exception)
{
if (!m_xLockBytes.Is())
throw css::io::NotConnectedException(OUString(), static_cast<css::uno::XWeak*>(this));
@@ -49,18 +48,18 @@ sal_Int32 SAL_CALL OInputStreamHelper::readBytes(css::uno::Sequence< sal_Int8 >&
return nRead;
}
-void SAL_CALL OInputStreamHelper::seek( sal_Int64 location ) throw(css::lang::IllegalArgumentException, css::io::IOException, css::uno::RuntimeException, std::exception)
+void SAL_CALL OInputStreamHelper::seek( sal_Int64 location )
{
::osl::MutexGuard aGuard( m_aMutex );
m_nActPos = location;
}
-sal_Int64 SAL_CALL OInputStreamHelper::getPosition( ) throw(css::io::IOException, css::uno::RuntimeException, std::exception)
+sal_Int64 SAL_CALL OInputStreamHelper::getPosition( )
{
return m_nActPos;
}
-sal_Int64 SAL_CALL OInputStreamHelper::getLength( ) throw(css::io::IOException, css::uno::RuntimeException, std::exception)
+sal_Int64 SAL_CALL OInputStreamHelper::getLength( )
{
if (!m_xLockBytes.Is())
return 0;
@@ -73,14 +72,12 @@ sal_Int64 SAL_CALL OInputStreamHelper::getLength( ) throw(css::io::IOException,
sal_Int32 SAL_CALL OInputStreamHelper::readSomeBytes(css::uno::Sequence< sal_Int8 >& aData,
sal_Int32 nMaxBytesToRead)
- throw (css::io::NotConnectedException, css::io::BufferSizeExceededException, css::io::IOException, css::uno::RuntimeException, std::exception)
{
// read all data desired
return readBytes(aData, nMaxBytesToRead);
}
void SAL_CALL OInputStreamHelper::skipBytes(sal_Int32 nBytesToSkip)
- throw (css::io::NotConnectedException, css::io::BufferSizeExceededException, css::io::IOException, css::uno::RuntimeException, std::exception)
{
::osl::MutexGuard aGuard( m_aMutex );
if (!m_xLockBytes.Is())
@@ -93,7 +90,6 @@ void SAL_CALL OInputStreamHelper::skipBytes(sal_Int32 nBytesToSkip)
}
sal_Int32 SAL_CALL OInputStreamHelper::available()
- throw (css::io::NotConnectedException, css::io::IOException, css::uno::RuntimeException, std::exception)
{
::osl::MutexGuard aGuard( m_aMutex );
if (!m_xLockBytes.Is())
@@ -103,7 +99,6 @@ sal_Int32 SAL_CALL OInputStreamHelper::available()
}
void SAL_CALL OInputStreamHelper::closeInput()
- throw (css::io::NotConnectedException, css::io::IOException, css::uno::RuntimeException, std::exception)
{
::osl::MutexGuard aGuard( m_aMutex );
if (!m_xLockBytes.Is())
diff --git a/unotools/source/streaming/streamwrap.cxx b/unotools/source/streaming/streamwrap.cxx
index 59d751d3758e..fea638ed26a4 100644
--- a/unotools/source/streaming/streamwrap.cxx
+++ b/unotools/source/streaming/streamwrap.cxx
@@ -46,7 +46,6 @@ OInputStreamWrapper::~OInputStreamWrapper()
}
sal_Int32 SAL_CALL OInputStreamWrapper::readBytes(css::uno::Sequence< sal_Int8 >& aData, sal_Int32 nBytesToRead)
- throw( css::io::NotConnectedException, css::io::BufferSizeExceededException, css::uno::RuntimeException, std::exception )
{
checkConnected();
@@ -68,7 +67,7 @@ sal_Int32 SAL_CALL OInputStreamWrapper::readBytes(css::uno::Sequence< sal_Int8 >
return nRead;
}
-sal_Int32 SAL_CALL OInputStreamWrapper::readSomeBytes(css::uno::Sequence< sal_Int8 >& aData, sal_Int32 nMaxBytesToRead) throw( css::io::NotConnectedException, css::io::BufferSizeExceededException, css::uno::RuntimeException, std::exception )
+sal_Int32 SAL_CALL OInputStreamWrapper::readSomeBytes(css::uno::Sequence< sal_Int8 >& aData, sal_Int32 nMaxBytesToRead)
{
checkError();
@@ -84,7 +83,7 @@ sal_Int32 SAL_CALL OInputStreamWrapper::readSomeBytes(css::uno::Sequence< sal_In
return readBytes(aData, nMaxBytesToRead);
}
-void SAL_CALL OInputStreamWrapper::skipBytes(sal_Int32 nBytesToSkip) throw( css::io::NotConnectedException, css::io::BufferSizeExceededException, css::uno::RuntimeException, std::exception )
+void SAL_CALL OInputStreamWrapper::skipBytes(sal_Int32 nBytesToSkip)
{
::osl::MutexGuard aGuard( m_aMutex );
checkError();
@@ -93,7 +92,7 @@ void SAL_CALL OInputStreamWrapper::skipBytes(sal_Int32 nBytesToSkip) throw( css:
checkError();
}
-sal_Int32 SAL_CALL OInputStreamWrapper::available() throw( css::io::NotConnectedException, css::uno::RuntimeException, std::exception )
+sal_Int32 SAL_CALL OInputStreamWrapper::available()
{
::osl::MutexGuard aGuard( m_aMutex );
checkConnected();
@@ -111,7 +110,7 @@ sal_Int32 SAL_CALL OInputStreamWrapper::available() throw( css::io::NotConnected
return nAvailable;
}
-void SAL_CALL OInputStreamWrapper::closeInput() throw( css::io::NotConnectedException, css::uno::RuntimeException, std::exception )
+void SAL_CALL OInputStreamWrapper::closeInput()
{
::osl::MutexGuard aGuard( m_aMutex );
checkConnected();
@@ -151,7 +150,7 @@ OSeekableInputStreamWrapper::OSeekableInputStreamWrapper(SvStream* _pStream, boo
SetStream( _pStream, _bOwner );
}
-void SAL_CALL OSeekableInputStreamWrapper::seek( sal_Int64 _nLocation ) throw (IllegalArgumentException, IOException, RuntimeException, std::exception)
+void SAL_CALL OSeekableInputStreamWrapper::seek( sal_Int64 _nLocation )
{
::osl::MutexGuard aGuard( m_aMutex );
checkConnected();
@@ -160,7 +159,7 @@ void SAL_CALL OSeekableInputStreamWrapper::seek( sal_Int64 _nLocation ) throw (I
checkError();
}
-sal_Int64 SAL_CALL OSeekableInputStreamWrapper::getPosition( ) throw (IOException, RuntimeException, std::exception)
+sal_Int64 SAL_CALL OSeekableInputStreamWrapper::getPosition( )
{
::osl::MutexGuard aGuard( m_aMutex );
checkConnected();
@@ -170,7 +169,7 @@ sal_Int64 SAL_CALL OSeekableInputStreamWrapper::getPosition( ) throw (IOExcepti
return (sal_Int64)nPos;
}
-sal_Int64 SAL_CALL OSeekableInputStreamWrapper::getLength( ) throw (IOException, RuntimeException, std::exception)
+sal_Int64 SAL_CALL OSeekableInputStreamWrapper::getLength( )
{
::osl::MutexGuard aGuard( m_aMutex );
checkConnected();
@@ -195,7 +194,7 @@ OOutputStreamWrapper::OOutputStreamWrapper(SvStream& _rStream):
OOutputStreamWrapper::~OOutputStreamWrapper() {}
-void SAL_CALL OOutputStreamWrapper::writeBytes(const css::uno::Sequence< sal_Int8 >& aData) throw( css::io::NotConnectedException, css::io::BufferSizeExceededException, css::uno::RuntimeException, std::exception )
+void SAL_CALL OOutputStreamWrapper::writeBytes(const css::uno::Sequence< sal_Int8 >& aData)
{
sal_uInt32 nWritten = rStream.WriteBytes(aData.getConstArray(), aData.getLength());
ErrCode err = rStream.GetError();
@@ -207,13 +206,13 @@ void SAL_CALL OOutputStreamWrapper::writeBytes(const css::uno::Sequence< sal_Int
}
}
-void SAL_CALL OOutputStreamWrapper::flush() throw( css::io::NotConnectedException, css::io::BufferSizeExceededException, css::uno::RuntimeException, std::exception )
+void SAL_CALL OOutputStreamWrapper::flush()
{
rStream.Flush();
checkError();
}
-void SAL_CALL OOutputStreamWrapper::closeOutput() throw( css::io::NotConnectedException, css::io::BufferSizeExceededException, css::uno::RuntimeException, std::exception )
+void SAL_CALL OOutputStreamWrapper::closeOutput()
{
}
@@ -233,7 +232,7 @@ OSeekableOutputStreamWrapper::OSeekableOutputStreamWrapper(SvStream& _rStream)
OSeekableOutputStreamWrapper::~OSeekableOutputStreamWrapper() {}
-Any SAL_CALL OSeekableOutputStreamWrapper::queryInterface( const Type& _rType ) throw (RuntimeException, std::exception)
+Any SAL_CALL OSeekableOutputStreamWrapper::queryInterface( const Type& _rType )
{
Any aReturn = OOutputStreamWrapper::queryInterface(_rType);
if (!aReturn.hasValue())
@@ -251,20 +250,20 @@ void SAL_CALL OSeekableOutputStreamWrapper::release( ) throw ()
OOutputStreamWrapper::release();
}
-void SAL_CALL OSeekableOutputStreamWrapper::seek( sal_Int64 _nLocation ) throw (IllegalArgumentException, IOException, RuntimeException, std::exception)
+void SAL_CALL OSeekableOutputStreamWrapper::seek( sal_Int64 _nLocation )
{
rStream.Seek((sal_uInt32)_nLocation);
checkError();
}
-sal_Int64 SAL_CALL OSeekableOutputStreamWrapper::getPosition( ) throw (IOException, RuntimeException, std::exception)
+sal_Int64 SAL_CALL OSeekableOutputStreamWrapper::getPosition( )
{
sal_uInt32 nPos = rStream.Tell();
checkError();
return (sal_Int64)nPos;
}
-sal_Int64 SAL_CALL OSeekableOutputStreamWrapper::getLength( ) throw (IOException, RuntimeException, std::exception)
+sal_Int64 SAL_CALL OSeekableOutputStreamWrapper::getLength( )
{
sal_uInt32 nCurrentPos = rStream.Tell();
checkError();
@@ -285,17 +284,17 @@ OStreamWrapper::OStreamWrapper(SvStream& _rStream)
SetStream( &_rStream, false );
}
-css::uno::Reference< css::io::XInputStream > SAL_CALL OStreamWrapper::getInputStream( ) throw (css::uno::RuntimeException, std::exception)
+css::uno::Reference< css::io::XInputStream > SAL_CALL OStreamWrapper::getInputStream( )
{
return this;
}
-css::uno::Reference< css::io::XOutputStream > SAL_CALL OStreamWrapper::getOutputStream( ) throw (css::uno::RuntimeException, std::exception)
+css::uno::Reference< css::io::XOutputStream > SAL_CALL OStreamWrapper::getOutputStream( )
{
return this;
}
-void SAL_CALL OStreamWrapper::writeBytes(const css::uno::Sequence< sal_Int8 >& aData) throw(css::io::NotConnectedException, css::io::BufferSizeExceededException, css::uno::RuntimeException, std::exception)
+void SAL_CALL OStreamWrapper::writeBytes(const css::uno::Sequence< sal_Int8 >& aData)
{
sal_uInt32 nWritten = m_pSvStream->WriteBytes(aData.getConstArray(), aData.getLength());
ErrCode err = m_pSvStream->GetError();
@@ -307,18 +306,18 @@ void SAL_CALL OStreamWrapper::writeBytes(const css::uno::Sequence< sal_Int8 >& a
}
}
-void SAL_CALL OStreamWrapper::flush() throw(css::io::NotConnectedException, css::io::BufferSizeExceededException, css::uno::RuntimeException, std::exception)
+void SAL_CALL OStreamWrapper::flush()
{
m_pSvStream->Flush();
if (m_pSvStream->GetError() != ERRCODE_NONE)
throw css::io::NotConnectedException(OUString(),static_cast<css::uno::XWeak*>(this));
}
-void SAL_CALL OStreamWrapper::closeOutput() throw(css::io::NotConnectedException, css::io::BufferSizeExceededException, css::uno::RuntimeException, std::exception)
+void SAL_CALL OStreamWrapper::closeOutput()
{
}
-void SAL_CALL OStreamWrapper::truncate() throw(css::io::IOException, css::uno::RuntimeException, std::exception)
+void SAL_CALL OStreamWrapper::truncate()
{
m_pSvStream->SetStreamSize(0);
}