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 /codemaker | |
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 'codemaker')
-rw-r--r-- | codemaker/source/codemaker/options.cxx | 1 | ||||
-rw-r--r-- | codemaker/source/cppumaker/cppuoptions.cxx | 1 | ||||
-rw-r--r-- | codemaker/source/cppumaker/cppuoptions.hxx | 3 | ||||
-rw-r--r-- | codemaker/source/javamaker/javaoptions.cxx | 1 | ||||
-rw-r--r-- | codemaker/source/javamaker/javaoptions.hxx | 3 |
5 files changed, 2 insertions, 7 deletions
diff --git a/codemaker/source/codemaker/options.cxx b/codemaker/source/codemaker/options.cxx index 83d0b2fd5633..53560800fc1b 100644 --- a/codemaker/source/codemaker/options.cxx +++ b/codemaker/source/codemaker/options.cxx @@ -38,7 +38,6 @@ bool Options::isValid(const OString& option) const } const OString& Options::getOption(const OString& option) const - throw( IllegalArgument ) { OptionMap::const_iterator i(m_options.find(option)); if (i == m_options.end()) { diff --git a/codemaker/source/cppumaker/cppuoptions.cxx b/codemaker/source/cppumaker/cppuoptions.cxx index 2f7724b6290a..3a0bdb5d0cd4 100644 --- a/codemaker/source/cppumaker/cppuoptions.cxx +++ b/codemaker/source/cppumaker/cppuoptions.cxx @@ -33,7 +33,6 @@ using ::rtl::OString; #endif bool CppuOptions::initOptions(int ac, char* av[], bool bCmdFile) - throw( IllegalArgument ) { bool ret = true; sal_uInt16 i=0; diff --git a/codemaker/source/cppumaker/cppuoptions.hxx b/codemaker/source/cppumaker/cppuoptions.hxx index 9d5addf326e9..497fbd96402b 100644 --- a/codemaker/source/cppumaker/cppuoptions.hxx +++ b/codemaker/source/cppumaker/cppuoptions.hxx @@ -28,8 +28,7 @@ public: CppuOptions() : Options() {} - bool initOptions(int ac, char* av[], bool bCmdFile=false) - throw( IllegalArgument ) override; + bool initOptions(int ac, char* av[], bool bCmdFile=false) override; OString prepareHelp() override; diff --git a/codemaker/source/javamaker/javaoptions.cxx b/codemaker/source/javamaker/javaoptions.cxx index bdfa4ebd78d1..97eef36b0e30 100644 --- a/codemaker/source/javamaker/javaoptions.cxx +++ b/codemaker/source/javamaker/javaoptions.cxx @@ -31,7 +31,6 @@ #endif bool JavaOptions::initOptions(int ac, char* av[], bool bCmdFile) - throw( IllegalArgument ) { bool ret = true; sal_uInt16 i=0; diff --git a/codemaker/source/javamaker/javaoptions.hxx b/codemaker/source/javamaker/javaoptions.hxx index 817d8805e123..e4eed6883849 100644 --- a/codemaker/source/javamaker/javaoptions.hxx +++ b/codemaker/source/javamaker/javaoptions.hxx @@ -28,8 +28,7 @@ public: JavaOptions() : Options() {} - bool initOptions(int ac, char* av[], bool bCmdFile=false) - throw( IllegalArgument ) override; + bool initOptions(int ac, char* av[], bool bCmdFile=false) override; OString prepareHelp() override; |