summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorhomeboy445 <akshitsan13@gmail.com>2020-12-23 19:46:09 +0530
committerMichael Stahl <michael.stahl@allotropia.de>2021-01-07 16:03:34 +0100
commit7bd5c31472804624d740d8c831d3b043f2d3a0be (patch)
treedd8f543ffd4d73ab7806653456610b37c8839cd8
parentb8d9334b9d4cc03a9b7d1e570a35e0ac6ca42338 (diff)
tdf#42982 Improved UNO API error reporting
Change-Id: I450ea0b1a19381c47370633d124c2ba906415987 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108356 Reviewed-by: Michael Stahl <michael.stahl@allotropia.de> Tested-by: Jenkins
-rw-r--r--chart2/source/tools/Scaling.cxx4
-rw-r--r--editeng/source/uno/UnoForbiddenCharsTable.cxx6
-rw-r--r--embeddedobj/source/commonembedding/embedobj.cxx8
3 files changed, 9 insertions, 9 deletions
diff --git a/chart2/source/tools/Scaling.cxx b/chart2/source/tools/Scaling.cxx
index 212eadba54ee..b61886c643e8 100644
--- a/chart2/source/tools/Scaling.cxx
+++ b/chart2/source/tools/Scaling.cxx
@@ -167,7 +167,7 @@ uno::Reference< XScaling > SAL_CALL
{
// ToDo: ApproxEqual ?
if( m_fSlope == 0 )
- throw uno::RuntimeException();
+ throw uno::RuntimeException("Divide by zero exception");
return new LinearScaling( 1.0 / m_fSlope, m_fOffset / m_fSlope );
}
@@ -218,7 +218,7 @@ uno::Reference< XScaling > SAL_CALL
{
// ToDo: ApproxEqual ?
if( m_fExponent == 0 )
- throw uno::RuntimeException();
+ throw uno::RuntimeException("Divide by zero exception");
return new PowerScaling( 1.0 / m_fExponent );
}
diff --git a/editeng/source/uno/UnoForbiddenCharsTable.cxx b/editeng/source/uno/UnoForbiddenCharsTable.cxx
index 8bb83f65583b..474180f2585d 100644
--- a/editeng/source/uno/UnoForbiddenCharsTable.cxx
+++ b/editeng/source/uno/UnoForbiddenCharsTable.cxx
@@ -50,7 +50,7 @@ ForbiddenCharacters SvxUnoForbiddenCharsTable::getForbiddenCharacters( const lan
SolarMutexGuard aGuard;
if (!mxForbiddenChars)
- throw RuntimeException();
+ throw RuntimeException("No Forbidden Characters present");
const LanguageType eLang = LanguageTag::convertToLanguageType( rLocale );
const ForbiddenCharacters* pForbidden = mxForbiddenChars->GetForbiddenCharacters( eLang, false );
@@ -78,7 +78,7 @@ void SvxUnoForbiddenCharsTable::setForbiddenCharacters(const lang::Locale& rLoca
SolarMutexGuard aGuard;
if (!mxForbiddenChars)
- throw RuntimeException();
+ throw RuntimeException("No Forbidden Characters present");
const LanguageType eLang = LanguageTag::convertToLanguageType( rLocale );
mxForbiddenChars->SetForbiddenCharacters( eLang, rForbiddenCharacters );
@@ -91,7 +91,7 @@ void SvxUnoForbiddenCharsTable::removeForbiddenCharacters( const lang::Locale& r
SolarMutexGuard aGuard;
if (!mxForbiddenChars)
- throw RuntimeException();
+ throw RuntimeException("No Forbidden Characters present");
const LanguageType eLang = LanguageTag::convertToLanguageType( rLocale );
mxForbiddenChars->ClearForbiddenCharacters( eLang );
diff --git a/embeddedobj/source/commonembedding/embedobj.cxx b/embeddedobj/source/commonembedding/embedobj.cxx
index 669ba884b055..dcba248ca4f7 100644
--- a/embeddedobj/source/commonembedding/embedobj.cxx
+++ b/embeddedobj/source/commonembedding/embedobj.cxx
@@ -209,7 +209,7 @@ void OCommonEmbeddedObject::SwitchStateTo_Impl( sal_Int32 nNextState )
else
{
SAL_WARN( "embeddedobj.common", "Unacceptable state switch!" );
- throw uno::RuntimeException(); // TODO
+ throw uno::RuntimeException("invalid next state, only RUNNING state allowed"); // TODO
}
}
else if ( m_nObjectState == embed::EmbedStates::RUNNING )
@@ -271,7 +271,7 @@ void OCommonEmbeddedObject::SwitchStateTo_Impl( sal_Int32 nNextState )
else
{
SAL_WARN( "embeddedobj.common", "Unacceptable state switch!" );
- throw uno::RuntimeException(); // TODO
+ throw uno::RuntimeException("invalid next state,only LOADED/INPLACE_ACTIVE/ACTIVE allowed"); // TODO
}
}
}
@@ -339,7 +339,7 @@ void OCommonEmbeddedObject::SwitchStateTo_Impl( sal_Int32 nNextState )
else
{
SAL_WARN( "embeddedobj.common", "Unacceptable state switch!" );
- throw uno::RuntimeException(); // TODO
+ throw uno::RuntimeException("invalid next state,only RUNNING/UI_ACTIVE allowed"); // TODO
}
}
else if ( m_nObjectState == embed::EmbedStates::ACTIVE )
@@ -352,7 +352,7 @@ void OCommonEmbeddedObject::SwitchStateTo_Impl( sal_Int32 nNextState )
else
{
SAL_WARN( "embeddedobj.common", "Unacceptable state switch!" );
- throw uno::RuntimeException(); // TODO
+ throw uno::RuntimeException("invalid next state, only RUNNING state allowed"); // TODO
}
}
else if ( m_nObjectState == embed::EmbedStates::UI_ACTIVE )