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 /lotuswordpro | |
| 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 'lotuswordpro')
| -rw-r--r-- | lotuswordpro/source/filter/lwpfilter.cxx | 2 | ||||
| -rw-r--r-- | lotuswordpro/source/filter/lwplayout.cxx | 6 | ||||
| -rw-r--r-- | lotuswordpro/source/filter/lwptablelayout.cxx | 2 |
3 files changed, 5 insertions, 5 deletions
diff --git a/lotuswordpro/source/filter/lwpfilter.cxx b/lotuswordpro/source/filter/lwpfilter.cxx index afab92eb24ea..5dbaf0f29c03 100644 --- a/lotuswordpro/source/filter/lwpfilter.cxx +++ b/lotuswordpro/source/filter/lwpfilter.cxx @@ -118,7 +118,7 @@ static bool Decompress(SvStream *pCompressed, SvStream * & pOutDecompressed) std::unique_ptr<LtcUtBenValueStream> aWordProData(pBentoContainer->FindValueStreamWithPropertyName("WordProData")); - if (!aWordProData.get()) + if (!aWordProData) return false; // decompressing diff --git a/lotuswordpro/source/filter/lwplayout.cxx b/lotuswordpro/source/filter/lwplayout.cxx index d79217319ff3..ce17fab7970f 100644 --- a/lotuswordpro/source/filter/lwplayout.cxx +++ b/lotuswordpro/source/filter/lwplayout.cxx @@ -951,7 +951,7 @@ sal_uInt16 LwpMiddleLayout::GetScaleMode() return GetLayoutScale()->GetScaleMode(); rtl::Reference<LwpObject> xBase(GetBasedOnStyle()); if (xBase.is()) - return dynamic_cast<LwpMiddleLayout&>(*xBase.get()).GetScaleMode(); + return dynamic_cast<LwpMiddleLayout&>(*xBase).GetScaleMode(); else return (LwpLayoutScale::FIT_IN_FRAME | LwpLayoutScale::MAINTAIN_ASPECT_RATIO); } @@ -963,7 +963,7 @@ sal_uInt16 LwpMiddleLayout::GetScaleTile() ? 1 : 0; rtl::Reference<LwpObject> xBase(GetBasedOnStyle()); if (xBase.is()) - return dynamic_cast<LwpMiddleLayout&>(*xBase.get()).GetScaleTile(); + return dynamic_cast<LwpMiddleLayout&>(*xBase).GetScaleTile(); else return 0; } @@ -985,7 +985,7 @@ sal_uInt16 LwpMiddleLayout::GetScaleCenter() { rtl::Reference<LwpObject> xBase(GetBasedOnStyle()); if (xBase.is()) - nRet = dynamic_cast<LwpMiddleLayout&>(*xBase.get()).GetScaleCenter(); + nRet = dynamic_cast<LwpMiddleLayout&>(*xBase).GetScaleCenter(); } m_bGettingScaleCenter = false; diff --git a/lotuswordpro/source/filter/lwptablelayout.cxx b/lotuswordpro/source/filter/lwptablelayout.cxx index 6c9b71ef5d64..b88a23bcdf1b 100644 --- a/lotuswordpro/source/filter/lwptablelayout.cxx +++ b/lotuswordpro/source/filter/lwptablelayout.cxx @@ -773,7 +773,7 @@ void LwpTableLayout::ParseTable() throw std::runtime_error("missing super table"); } - if (m_pXFTable.get()) + if (m_pXFTable) { throw std::runtime_error("this table is already parsed"); } |