diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-16 15:23:12 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-17 08:25:47 +0200 |
| commit | ccb2a1f650bc505f8a4f1abebf8ce4f9396562a8 (patch) | |
| tree | 2ee2fd4f300ae95cf23bade1f242e02f9b276c07 /writerfilter | |
| parent | da5c3a1ee43dd1a07cbd1b8005488bf05432593d (diff) | |
clang-tidy readability-redundant-smartptr-get
redundant get() call on smart pointer
Change-Id: Icb5a03bbc15e79a30d3d135a507d22914d15c2bd
Reviewed-on: https://gerrit.libreoffice.org/61837
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'writerfilter')
| -rw-r--r-- | writerfilter/source/dmapper/DomainMapper.cxx | 4 | ||||
| -rw-r--r-- | writerfilter/source/dmapper/DomainMapperTableHandler.cxx | 2 | ||||
| -rw-r--r-- | writerfilter/source/dmapper/NumberingManager.cxx | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/writerfilter/source/dmapper/DomainMapper.cxx b/writerfilter/source/dmapper/DomainMapper.cxx index 3a0140f2646e..5310d933ea00 100644 --- a/writerfilter/source/dmapper/DomainMapper.cxx +++ b/writerfilter/source/dmapper/DomainMapper.cxx @@ -3699,14 +3699,14 @@ StyleSheetTablePtr const & DomainMapper::GetStyleSheetTable( ) GraphicZOrderHelper* DomainMapper::graphicZOrderHelper() { - if( zOrderHelper.get() == nullptr ) + if (zOrderHelper == nullptr) zOrderHelper.reset( new GraphicZOrderHelper ); return zOrderHelper.get(); } GraphicNamingHelper& DomainMapper::GetGraphicNamingHelper() { - if (m_pGraphicNamingHelper.get() == nullptr) + if (m_pGraphicNamingHelper == nullptr) m_pGraphicNamingHelper.reset(new GraphicNamingHelper()); return *m_pGraphicNamingHelper; } diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx index cf3d5e62df01..01c7f59fc314 100644 --- a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx +++ b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx @@ -951,7 +951,7 @@ css::uno::Sequence<css::beans::PropertyValues> DomainMapperTableHandler::endTabl #ifdef DEBUG_WRITERFILTER TagLogger::getInstance().startElement("rowProps.row"); #endif - if( rRow.get() ) + if (rRow) { //set default to 'break across pages" rRow->Insert( PROP_IS_SPLIT_ALLOWED, uno::makeAny(true ), false ); diff --git a/writerfilter/source/dmapper/NumberingManager.cxx b/writerfilter/source/dmapper/NumberingManager.cxx index fb02feccd3cd..2d5605a271dc 100644 --- a/writerfilter/source/dmapper/NumberingManager.cxx +++ b/writerfilter/source/dmapper/NumberingManager.cxx @@ -197,7 +197,7 @@ sal_Int16 ListLevel::GetParentNumbering( const OUString& sText, sal_Int16 nLevel uno::Sequence<beans::PropertyValue> ListLevel::GetProperties(bool bDefaults) { uno::Sequence<beans::PropertyValue> aLevelProps = GetLevelProperties(bDefaults); - if ( m_pParaStyle.get( ) ) + if (m_pParaStyle) AddParaProperties( &aLevelProps ); return aLevelProps; } |