[Libreoffice-commits] core.git: 2 commits - framework/source include/vcl

Michael Stahl mstahl at redhat.com
Tue Mar 29 19:55:34 UTC 2016


 framework/source/fwi/classes/propertysethelper.cxx |    5 +----
 include/vcl/svapp.hxx                              |    2 +-
 2 files changed, 2 insertions(+), 5 deletions(-)

New commits:
commit dc6106f989f814d2e5c21943484598d456d0044a
Author: Michael Stahl <mstahl at redhat.com>
Date:   Tue Mar 29 21:50:10 2016 +0200

    framework: cid#1357163 fix the fix d5e4fe23
    
    The branch was never taken in the old code, and re-acquiring the mutex
    without releasing it first is pointless.
    
    Change-Id: Ic865dccf2c442d2625436e175aef91be59d05325

diff --git a/framework/source/fwi/classes/propertysethelper.cxx b/framework/source/fwi/classes/propertysethelper.cxx
index 1e2e120..6a42b67 100644
--- a/framework/source/fwi/classes/propertysethelper.cxx
+++ b/framework/source/fwi/classes/propertysethelper.cxx
@@ -151,7 +151,7 @@ void SAL_CALL PropertySetHelper::setPropertyValue(const OUString& sProperty,
     TransactionGuard aTransaction(m_rTransactionManager, E_HARDEXCEPTIONS);
 
     // SAFE ->
-    SolarMutexResettableGuard aWriteLock;
+    SolarMutexGuard g;
 
     PropertySetHelper::TPropInfoHash::const_iterator pIt = m_lProps.find(sProperty);
     if (pIt == m_lProps.end())
@@ -161,9 +161,6 @@ void SAL_CALL PropertySetHelper::setPropertyValue(const OUString& sProperty,
 
     css::uno::Any aCurrentValue = impl_getPropertyValue(aPropInfo.Name, aPropInfo.Handle);
 
-    // SAFE ->
-    aWriteLock.reset();
-
     bool bWillBeChanged = (aCurrentValue != aValue);
     if (! bWillBeChanged)
         return;
commit 973dc1ebe03b0f1f0b0e167d64bac8b42c6f241c
Author: Michael Stahl <mstahl at redhat.com>
Date:   Tue Mar 29 21:49:15 2016 +0200

    vcl: misleading copypasta in documentation
    
    Change-Id: I673940fde7484500cbf30d4a4b760a8bf4448e33

diff --git a/include/vcl/svapp.hxx b/include/vcl/svapp.hxx
index 608a1b9..c946b35 100644
--- a/include/vcl/svapp.hxx
+++ b/include/vcl/svapp.hxx
@@ -1555,7 +1555,7 @@ public:
         }
     }
 
-    /** Releases mutex. */
+    /** Re-acquires mutex. */
     void SAL_CALL reset()
     {
         if( m_bCleared)


More information about the Libreoffice-commits mailing list