diff options
author | homeboy445 <akshitsan13@gmail.com> | 2020-12-23 19:46:09 +0530 |
---|---|---|
committer | Michael Stahl <michael.stahl@allotropia.de> | 2021-01-07 16:03:34 +0100 |
commit | 7bd5c31472804624d740d8c831d3b043f2d3a0be (patch) | |
tree | dd8f543ffd4d73ab7806653456610b37c8839cd8 | |
parent | b8d9334b9d4cc03a9b7d1e570a35e0ac6ca42338 (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.cxx | 4 | ||||
-rw-r--r-- | editeng/source/uno/UnoForbiddenCharsTable.cxx | 6 | ||||
-rw-r--r-- | embeddedobj/source/commonembedding/embedobj.cxx | 8 |
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 ) |