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 /svl/qa/unit | |
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 'svl/qa/unit')
-rw-r--r-- | svl/qa/unit/test_URIHelper.cxx | 30 |
1 files changed, 9 insertions, 21 deletions
diff --git a/svl/qa/unit/test_URIHelper.cxx b/svl/qa/unit/test_URIHelper.cxx index 1a8e029519ef..70e6fcc69da4 100644 --- a/svl/qa/unit/test_URIHelper.cxx +++ b/svl/qa/unit/test_URIHelper.cxx @@ -73,40 +73,33 @@ public: css::uno::Reference< css::ucb::XContentIdentifier > const & identifier); virtual css::uno::Reference< css::ucb::XContentIdentifier > SAL_CALL - getIdentifier() throw (css::uno::RuntimeException, std::exception) override { + getIdentifier() override { return m_identifier; } - virtual OUString SAL_CALL getContentType() - throw (css::uno::RuntimeException, std::exception) override + virtual OUString SAL_CALL getContentType() override { return OUString(); } virtual void SAL_CALL addContentEventListener( - css::uno::Reference< css::ucb::XContentEventListener > const &) - throw (css::uno::RuntimeException, std::exception) override + css::uno::Reference< css::ucb::XContentEventListener > const &) override {} virtual void SAL_CALL removeContentEventListener( - css::uno::Reference< css::ucb::XContentEventListener > const &) - throw (css::uno::RuntimeException, std::exception) override + css::uno::Reference< css::ucb::XContentEventListener > const &) override {} - virtual sal_Int32 SAL_CALL createCommandIdentifier() - throw (css::uno::RuntimeException, std::exception) override + virtual sal_Int32 SAL_CALL createCommandIdentifier() override { return 0; } virtual css::uno::Any SAL_CALL execute( css::ucb::Command const & command, sal_Int32 commandId, - css::uno::Reference< css::ucb::XCommandEnvironment > const &) - throw ( - css::uno::Exception, css::ucb::CommandAbortedException, - css::uno::RuntimeException, std::exception) override; + css::uno::Reference< css::ucb::XCommandEnvironment > const &) override; - virtual void SAL_CALL abort(sal_Int32) throw (css::uno::RuntimeException, std::exception) override {} + virtual void SAL_CALL abort(sal_Int32) override {} private: static char const m_prefix[]; @@ -132,9 +125,6 @@ Content::Content( css::uno::Any Content::execute( css::ucb::Command const & command, sal_Int32, css::uno::Reference< css::ucb::XCommandEnvironment > const &) - throw ( - css::uno::Exception, css::ucb::CommandAbortedException, - css::uno::RuntimeException, std::exception) { if ( command.Name != "getCasePreservingURL" ) { @@ -171,16 +161,14 @@ css::uno::Any Content::execute( class Provider: public cppu::WeakImplHelper< css::ucb::XContentProvider > { public: virtual css::uno::Reference< css::ucb::XContent > SAL_CALL queryContent( - css::uno::Reference< css::ucb::XContentIdentifier > const & identifier) - throw (css::ucb::IllegalIdentifierException, css::uno::RuntimeException, std::exception) override + css::uno::Reference< css::ucb::XContentIdentifier > const & identifier) override { return new Content(identifier); } virtual sal_Int32 SAL_CALL compareContentIds( css::uno::Reference< css::ucb::XContentIdentifier > const & id1, - css::uno::Reference< css::ucb::XContentIdentifier > const & id2) - throw (css::uno::RuntimeException, std::exception) override + css::uno::Reference< css::ucb::XContentIdentifier > const & id2) override { assert(id1.is() && id2.is()); return |