From 99508583e0cb3d25dfff2ab7dea5c6e8ca0119e7 Mon Sep 17 00:00:00 2001 From: Stephan Bergmann <sbergman@redhat.com> Date: Tue, 18 Mar 2014 15:45:36 +0100 Subject: Use SolarMutexGuard directly Change-Id: I674b247531d36e68e95f3e95893973eac6d61e01 --- framework/inc/jobs/jobresult.hxx | 3 +-- framework/source/jobs/jobresult.cxx | 25 ++++++------------------- 2 files changed, 7 insertions(+), 21 deletions(-) diff --git a/framework/inc/jobs/jobresult.hxx b/framework/inc/jobs/jobresult.hxx index d1d190209659..d5790a3f5eaf 100644 --- a/framework/inc/jobs/jobresult.hxx +++ b/framework/inc/jobs/jobresult.hxx @@ -20,7 +20,6 @@ #ifndef INCLUDED_FRAMEWORK_INC_JOBS_JOBRESULT_HXX #define INCLUDED_FRAMEWORK_INC_JOBS_JOBRESULT_HXX -#include <threadhelp/threadhelpbase.hxx> #include <stdtypes.h> #include <general.h> @@ -40,7 +39,7 @@ namespace framework{ data from the code place where the job was finished to the outside code, where e.g. listener must be notified. */ -class JobResult : private ThreadHelpBase +class JobResult { // types diff --git a/framework/source/jobs/jobresult.cxx b/framework/source/jobs/jobresult.cxx index 749f03367a40..731cdf176c93 100644 --- a/framework/source/jobs/jobresult.cxx +++ b/framework/source/jobs/jobresult.cxx @@ -19,7 +19,6 @@ #include <jobs/jobresult.hxx> #include <jobs/jobconst.hxx> -#include <threadhelp/guard.hxx> #include <general.h> #include <services.h> @@ -35,8 +34,7 @@ namespace framework{ but it marks this new instance as non valid! */ JobResult::JobResult() - : ThreadHelpBase(&Application::GetSolarMutex()) - , m_bDeactivate(false) + : m_bDeactivate(false) { // reset the flag mask! // It will reset the accessible state of this object. @@ -89,8 +87,7 @@ JobResult::JobResult() the job result */ JobResult::JobResult( /*IN*/ const css::uno::Any& aResult ) - : ThreadHelpBase(&Application::GetSolarMutex()) - , m_bDeactivate(false) + : m_bDeactivate(false) { // safe the pure result // May someone need it later ... @@ -136,7 +133,6 @@ JobResult::JobResult( /*IN*/ const css::uno::Any& aResult ) @descr - */ JobResult::JobResult( const JobResult& rCopy ) - : ThreadHelpBase(&Application::GetSolarMutex()) { m_aPureResult = rCopy.m_aPureResult ; m_eParts = rCopy.m_eParts ; @@ -165,15 +161,12 @@ JobResult::~JobResult() */ void JobResult::operator=( const JobResult& rCopy ) { - /* SAFE { */ - Guard aWriteLock(m_aLock); + SolarMutexGuard g; m_aPureResult = rCopy.m_aPureResult ; m_eParts = rCopy.m_eParts ; m_lArguments = rCopy.m_lArguments ; m_bDeactivate = rCopy.m_bDeactivate ; m_aDispatchResult = rCopy.m_aDispatchResult ; - aWriteLock.unlock(); - /* } SAFE */ } @@ -191,10 +184,8 @@ void JobResult::operator=( const JobResult& rCopy ) */ sal_Bool JobResult::existPart( sal_uInt32 eParts ) const { - /* SAFE { */ - Guard aReadLock(m_aLock); + SolarMutexGuard g; return ((m_eParts & eParts) == eParts); - /* } SAFE */ } @@ -208,20 +199,16 @@ sal_Bool JobResult::existPart( sal_uInt32 eParts ) const */ css::uno::Sequence< css::beans::NamedValue > JobResult::getArguments() const { - /* SAFE { */ - Guard aReadLock(m_aLock); + SolarMutexGuard g; return m_lArguments; - /* } SAFE */ } css::frame::DispatchResultEvent JobResult::getDispatchResult() const { - /* SAFE { */ - Guard aReadLock(m_aLock); + SolarMutexGuard g; return m_aDispatchResult; - /* } SAFE */ } } // namespace framework -- cgit