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
9e2bf1d54a "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
This commit is contained in:
Stephan Bergmann 2020-08-04 23:01:35 +02:00
parent 41723365f6
commit 99d281201c

View file

@ -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
}
}