tdf#89329: use unique_ptr for pImpl in dataaccessdescriptor

Change-Id: I0d04c23b8760ce2e7df9523d99a1fd862e501038
Reviewed-on: https://gerrit.libreoffice.org/25938
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>

1 files changed, 1 insertions, 32 deletions

diff --git a/svx/source/form/dataaccessdescriptor.cxx b/svx/source/form/dataaccessdescriptor.cxx
index 935a564f0ff0..29915af33312 100644
--- a/svx/source/form/dataaccessdescriptor.cxx
+++ b/svx/source/form/dataaccessdescriptor.cxx
@@ -26,11 +26,8 @@
 #include <com/sun/star/beans/PropertyAttribute.hpp>
 #include <tools/urlobj.hxx>
 
-
 namespace svx
 {
-
-
     using namespace ::com::sun::star::uno;
     using namespace ::com::sun::star::sdbc;
     using namespace ::com::sun::star::beans;
@@ -77,14 +74,12 @@ namespace svx
         static PropertyMapEntry const *         getPropertyMapEntry( const DescriptorValues::const_iterator& _rPos );
     };
 
-
     ODADescriptorImpl::ODADescriptorImpl()
         :m_bSetOutOfDate(true)
         ,m_bSequenceOutOfDate(true)
     {
     }
 
-
     ODADescriptorImpl::ODADescriptorImpl(const ODADescriptorImpl& _rSource)
         :m_bSetOutOfDate( _rSource.m_bSetOutOfDate )
         ,m_bSequenceOutOfDate( _rSource.m_bSequenceOutOfDate )
@@ -96,7 +91,6 @@ namespace svx
             m_aAsSequence = _rSource.m_aAsSequence;
     }
 
-
     bool ODADescriptorImpl::buildFrom( const Sequence< PropertyValue >& _rValues )
     {
         const MapString2PropertyEntry& rProperties = getPropertyMap();
@@ -130,7 +124,6 @@ namespace svx
         return bValidPropsOnly;
     }
 
-
     bool ODADescriptorImpl::buildFrom( const Reference< XPropertySet >& _rxValues )
     {
         Reference< XPropertySetInfo > xPropInfo;
@@ -168,14 +161,12 @@ namespace svx
         return bValidPropsOnly;
     }
 
-
     void ODADescriptorImpl::invalidateExternRepresentations()
     {
         m_bSetOutOfDate = true;
         m_bSequenceOutOfDate = true;
     }
 
-
     const ODADescriptorImpl::MapString2PropertyEntry& ODADescriptorImpl::getPropertyMap( )
     {
         // the properties we know
@@ -212,7 +203,6 @@ namespace svx
         return s_aProperties;
     }
 
-
     PropertyMapEntry const * ODADescriptorImpl::getPropertyMapEntry( const DescriptorValues::const_iterator& _rPos )
     {
         const MapString2PropertyEntry& rProperties = getPropertyMap();
@@ -230,7 +220,6 @@ namespace svx
         throw RuntimeException();
     }
 
-
     PropertyValue ODADescriptorImpl::buildPropertyValue( const DescriptorValues::const_iterator& _rPos )
     {
         // the map entry
@@ -247,7 +236,6 @@ namespace svx
         return aReturn;
     }
 
-
     void ODADescriptorImpl::updateSequence()
     {
         if (!m_bSequenceOutOfDate)
@@ -274,28 +262,23 @@ namespace svx
     {
     }
 
-
     ODataAccessDescriptor::ODataAccessDescriptor( const ODataAccessDescriptor& _rSource )
         :m_pImpl(new ODADescriptorImpl(*_rSource.m_pImpl))
     {
     }
 
-
     ODataAccessDescriptor& ODataAccessDescriptor::operator=(const ODataAccessDescriptor& _rSource)
     {
-        delete m_pImpl;
-        m_pImpl = new ODADescriptorImpl(*_rSource.m_pImpl);
+        m_pImpl.reset(new ODADescriptorImpl(*_rSource.m_pImpl));
         return *this;
     }
 
-
     ODataAccessDescriptor::ODataAccessDescriptor( const Reference< XPropertySet >& _rValues )
         :m_pImpl(new ODADescriptorImpl)
     {
         m_pImpl->buildFrom(_rValues);
     }
 
-
     ODataAccessDescriptor::ODataAccessDescriptor( const Any& _rValues )
         :m_pImpl(new ODADescriptorImpl)
     {
@@ -308,26 +291,21 @@ namespace svx
             m_pImpl->buildFrom( xValues );
     }
 
-
     ODataAccessDescriptor::ODataAccessDescriptor( const Sequence< PropertyValue >& _rValues )
         :m_pImpl(new ODADescriptorImpl)
     {
         m_pImpl->buildFrom(_rValues);
     }
 
-
     ODataAccessDescriptor::~ODataAccessDescriptor()
     {
-        delete m_pImpl;
     }
 
-
     void ODataAccessDescriptor::clear()
     {
         m_pImpl->m_aValues.clear();
     }
 
-
     void ODataAccessDescriptor::erase(DataAccessDescriptorProperty _eWhich)
     {
         OSL_ENSURE(has(_eWhich), "ODataAccessDescriptor::erase: invalid call!");
@@ -335,13 +313,11 @@ namespace svx
             m_pImpl->m_aValues.erase(_eWhich);
     }
 
-
     bool ODataAccessDescriptor::has(DataAccessDescriptorProperty _eWhich) const
     {
         return m_pImpl->m_aValues.find(_eWhich) != m_pImpl->m_aValues.end();
     }
 
-
     const Any& ODataAccessDescriptor::operator [] ( DataAccessDescriptorProperty _eWhich ) const
     {
         if (!has(_eWhich))
@@ -354,28 +330,24 @@ namespace svx
         return m_pImpl->m_aValues[_eWhich];
     }
 
-
     Any& ODataAccessDescriptor::operator[] ( DataAccessDescriptorProperty _eWhich )
     {
         m_pImpl->invalidateExternRepresentations();
         return m_pImpl->m_aValues[_eWhich];
     }
 
-
     void ODataAccessDescriptor::initializeFrom(const Sequence< PropertyValue >& _rValues)
     {
         clear();
         m_pImpl->buildFrom(_rValues);
     }
 
-
     Sequence< PropertyValue > ODataAccessDescriptor::createPropertyValueSequence()
     {
         m_pImpl->updateSequence();
         return m_pImpl->m_aAsSequence;
     }
 
-
     OUString ODataAccessDescriptor::getDataSource() const
     {
         OUString sDataSourceName;
@@ -396,9 +368,6 @@ namespace svx
         else
             (*this)[ daDataSource ] <<= OUString();
     }
-
-
 }
 
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */