diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-09-11 16:43:19 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-09-12 08:03:09 +0200 |
| commit | 65e293f524437a548b6c099bbe6099b56957063c (patch) | |
| tree | f058b4f61a6f046e71c45814e1a8c0fb749162a9 /package | |
| parent | baa5f32c21185025b64c01fc589c69961528b0c5 (diff) | |
clang-tidy modernize-use-emplace in package..sax
Change-Id: Ibc0258fd79df987a0e295552bcc5bf89ee12007a
Reviewed-on: https://gerrit.libreoffice.org/42173
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'package')
| -rw-r--r-- | package/source/manifest/ManifestImport.cxx | 4 | ||||
| -rw-r--r-- | package/source/xstor/ohierarchyholder.cxx | 2 | ||||
| -rw-r--r-- | package/source/xstor/xstorage.cxx | 4 |
3 files changed, 5 insertions, 5 deletions
diff --git a/package/source/manifest/ManifestImport.cxx b/package/source/manifest/ManifestImport.cxx index bfde6aa3cb2c..3868be460c41 100644 --- a/package/source/manifest/ManifestImport.cxx +++ b/package/source/manifest/ManifestImport.cxx @@ -351,7 +351,7 @@ OUString ManifestImport::PushNameAndNamespaces( const OUString& aName, const uno aNamespaces[aNsName] = aAttrValue; } else { // this is no namespace declaration - aAttribsStrs.push_back( pair< OUString, OUString >( aAttrName, aAttrValue ) ); + aAttribsStrs.emplace_back( aAttrName, aAttrValue ); } } } @@ -360,7 +360,7 @@ OUString ManifestImport::PushNameAndNamespaces( const OUString& aName, const uno if ( !aConvertedName.getLength() ) aConvertedName = ConvertName( aName ); - aStack.push_back( ManifestScopeEntry( aConvertedName, aNamespaces ) ); + aStack.emplace_back( aConvertedName, aNamespaces ); for (const std::pair<OUString,OUString> & rAttribsStr : aAttribsStrs) { // convert the attribute names on filling diff --git a/package/source/xstor/ohierarchyholder.cxx b/package/source/xstor/ohierarchyholder.cxx index 96a079e5ccd5..c07c7740c502 100644 --- a/package/source/xstor/ohierarchyholder.cxx +++ b/package/source/xstor/ohierarchyholder.cxx @@ -123,7 +123,7 @@ uno::Reference< embed::XExtendedStorageStream > OHierarchyElement_Impl::GetStrea xStreamComp->addEventListener( static_cast< lang::XEventListener* >( this ) ); } - m_aOpenStreams.push_back( uno::WeakReference< embed::XExtendedStorageStream >( xResult ) ); + m_aOpenStreams.emplace_back( xResult ); } else { diff --git a/package/source/xstor/xstorage.cxx b/package/source/xstor/xstorage.cxx index 79b239588c77..06c5ec31ae4e 100644 --- a/package/source/xstor/xstorage.cxx +++ b/package/source/xstor/xstorage.cxx @@ -375,7 +375,7 @@ void OStorage_Impl::SetReadOnlyWrap( OStorage& aStorage ) { // Weak reference is used inside the holder so the refcount must not be zero at this point OSL_ENSURE( aStorage.GetRefCount_Impl(), "There must be a reference alive to use this method!" ); - m_aReadOnlyWrapVector.push_back( StorageHolder_Impl( &aStorage ) ); + m_aReadOnlyWrapVector.emplace_back( &aStorage ); } void OStorage_Impl::RemoveReadOnlyWrap( OStorage& aStorage ) @@ -2042,7 +2042,7 @@ void OStorage::MakeLinkToSubComponent_Impl( const uno::Reference< lang::XCompone xComponent->addEventListener( uno::Reference< lang::XEventListener >( static_cast< ::cppu::OWeakObject* >(m_pData->m_pSubElDispListener.get()), uno::UNO_QUERY)); - m_pData->m_aOpenSubComponentsVector.push_back( xComponent ); + m_pData->m_aOpenSubComponentsVector.emplace_back(xComponent ); } // XInterface |