diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2020-08-04 23:01:35 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2020-08-05 08:08:16 +0200 |
| commit | 99d281201cc36dbf1760701343056f8cbed11710 (patch) | |
| tree | 3d0e7e029b0a0ea698b5fc3b9a7a7a29fe0e7068 /canvas | |
| parent | 41723365f680421f12869d5af2e895f94a25d9d5 (diff) | |
Throw detailed IllegalArgumentException regardless of OSL_DEBUG_LEVEL
...assuming that such an exception is rare enough that any performance impact
does not matter. (The code was like this ever since its introduction in
9e2bf1d54a6e92dcfa3076bf4d7cc623ace87cd3 "INTEGRATION: CWS canvas02".)
This nicely avoids loplugin:unusedmember about unused mpStr and mnArgPos when
OSL_DEBUG_LEVEL is zero.
Change-Id: I2ad3e36e98a6811ae255525cf8159db89c78ad7f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100143
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Tested-by: Jenkins
Diffstat (limited to 'canvas')
| -rw-r--r-- | canvas/source/tools/verifyinput.cxx | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/canvas/source/tools/verifyinput.cxx b/canvas/source/tools/verifyinput.cxx index 9d0020c12da9..ae0704d81d57 100644 --- a/canvas/source/tools/verifyinput.cxx +++ b/canvas/source/tools/verifyinput.cxx @@ -394,15 +394,11 @@ namespace canvas::tools { if( !std::isfinite( rVal ) || rVal < 0.0 ) { -#if OSL_DEBUG_LEVEL > 0 throw lang::IllegalArgumentException( OUString::createFromAscii(mpStr) + ": verifyInput(): one of stroke attributes' DashArray value out of range (is " + OUString::number(rVal) + ")", mrIf, mnArgPos ); -#else - throw lang::IllegalArgumentException(); -#endif } } |