diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2023-03-31 14:38:56 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2023-03-31 14:34:08 +0000 |
| commit | 96a47eeedfe4c8adcf2c2903d72fe6e4e12a65e3 (patch) | |
| tree | d8120e1cb51d1ad7fc4702293b5fa7cc941061c8 /comphelper | |
| parent | 589cd835436de8ada027280f927fa2ae28420317 (diff) | |
simplify subclasses of comphelper::OPropertySetHelper
they should deal in terms of the fast property IDS, they should not need
to override setPropertyValueImpl, so just remove this, and standardise
on the subclasses overriding setFastPropertyValueImpl
Change-Id: I190955ccc4ae3e07dd481a1044f3c074dd1a40aa
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149836
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'comphelper')
| -rw-r--r-- | comphelper/source/property/propshlp.cxx | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/comphelper/source/property/propshlp.cxx b/comphelper/source/property/propshlp.cxx index 3071dfd4a32d..af6e3f398734 100644 --- a/comphelper/source/property/propshlp.cxx +++ b/comphelper/source/property/propshlp.cxx @@ -162,19 +162,12 @@ OPropertySetHelper::createPropertySetInfo(IPropertyArrayHelper& rProperties) // XPropertySet void OPropertySetHelper::setPropertyValue(const OUString& rPropertyName, const Any& rValue) { - std::unique_lock aGuard(m_aMutex); - setPropertyValueImpl(aGuard, rPropertyName, rValue); -} - -void OPropertySetHelper::setPropertyValueImpl(std::unique_lock<std::mutex>& rGuard, - const OUString& rPropertyName, const Any& rValue) -{ // get the map table IPropertyArrayHelper& rPH = getInfoHelper(); // map the name to the handle sal_Int32 nHandle = rPH.getHandleByName(rPropertyName); - // call the method of the XFastPropertySet interface - setFastPropertyValueImpl(rGuard, nHandle, rValue); + std::unique_lock aGuard(m_aMutex); + setFastPropertyValueImpl(aGuard, nHandle, rValue); } // XPropertySet |