From 5e193d854596db714ca786c2a676b5ac9684d5eb Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Fri, 27 Jan 2023 15:46:28 +0200 Subject: [PATCH] JobURL does not need to use the SolarMutex it is only used and accessed as a local stack variable inside two methods Change-Id: I651dad0912dd47d3670e7b341d444d17e200efc2 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146260 Tested-by: Jenkins Reviewed-by: Noel Grandin --- framework/source/jobs/joburl.cxx | 7 ------- 1 file changed, 7 deletions(-) diff --git a/framework/source/jobs/joburl.cxx b/framework/source/jobs/joburl.cxx index efff201a8401..fc1424867d99 100644 --- a/framework/source/jobs/joburl.cxx +++ b/framework/source/jobs/joburl.cxx @@ -99,7 +99,6 @@ JobURL::JobURL( /*IN*/ const OUString& sURL ) */ bool JobURL::isValid() const { - SolarMutexGuard g; return (m_eRequest!=E_UNKNOWN); } @@ -121,8 +120,6 @@ bool JobURL::isValid() const */ bool JobURL::getEvent( /*OUT*/ OUString& sEvent ) const { - SolarMutexGuard g; - sEvent.clear(); bool bSet = ((m_eRequest & E_EVENT) == E_EVENT); if (bSet) @@ -149,8 +146,6 @@ bool JobURL::getEvent( /*OUT*/ OUString& sEvent ) const */ bool JobURL::getAlias( /*OUT*/ OUString& sAlias ) const { - SolarMutexGuard g; - sAlias.clear(); bool bSet = ((m_eRequest & E_ALIAS) == E_ALIAS); if (bSet) @@ -177,8 +172,6 @@ bool JobURL::getAlias( /*OUT*/ OUString& sAlias ) const */ bool JobURL::getService( /*OUT*/ OUString& sService ) const { - SolarMutexGuard g; - sService.clear(); bool bSet = ((m_eRequest & E_SERVICE) == E_SERVICE); if (bSet)