diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-04-11 08:45:48 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-04-11 14:18:25 +0200 |
commit | c8e2ae1b915477c76d38352298570af309a36d85 (patch) | |
tree | 607e2f652750ea11aceae68ce436219158979e3a /lotuswordpro/source/filter/lwplayout.cxx | |
parent | b0459b45f431490b8fcf94ecc6b01365efe0a11e (diff) |
loplugin:inlinefields in lotuswordpro
Change-Id: I1ba5bdd59badd6f1b6562c6284baf05bcfc3d78c
Reviewed-on: https://gerrit.libreoffice.org/36402
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'lotuswordpro/source/filter/lwplayout.cxx')
-rw-r--r-- | lotuswordpro/source/filter/lwplayout.cxx | 34 |
1 files changed, 8 insertions, 26 deletions
diff --git a/lotuswordpro/source/filter/lwplayout.cxx b/lotuswordpro/source/filter/lwplayout.cxx index 3c32355e776d..b5f5056ebf50 100644 --- a/lotuswordpro/source/filter/lwplayout.cxx +++ b/lotuswordpro/source/filter/lwplayout.cxx @@ -540,46 +540,32 @@ void LwpLayoutStyle::Read(LwpObjectStream* pStrm) } LwpLayoutMisc::LwpLayoutMisc() : -m_nGridDistance(0), m_nGridType(0), -m_pContentStyle(new LwpAtomHolder) +m_nGridDistance(0), m_nGridType(0) { } LwpLayoutMisc::~LwpLayoutMisc() { - if (m_pContentStyle) - { - delete m_pContentStyle; - } } void LwpLayoutMisc::Read(LwpObjectStream* pStrm) { m_nGridType = pStrm->QuickReaduInt16(); m_nGridDistance = pStrm->QuickReadInt32(); - m_pContentStyle->Read(pStrm); + m_aContentStyle.Read(pStrm); pStrm->SkipExtra(); } LwpMiddleLayout::LwpMiddleLayout( LwpObjectHeader &objHdr, LwpSvStream* pStrm ) : LwpVirtualLayout(objHdr, pStrm) - , m_pStyleStuff(new LwpLayoutStyle) - , m_pMiscStuff(new LwpLayoutMisc) , m_bGettingGeometry(false) { } LwpMiddleLayout::~LwpMiddleLayout() { - if (m_pStyleStuff) - { - delete m_pStyleStuff; - } - if (m_pMiscStuff) - { - delete m_pMiscStuff; - } } + void LwpMiddleLayout::Read() { LwpObjectStream* pStrm = m_pObjStrm.get(); @@ -605,11 +591,11 @@ void LwpMiddleLayout::Read() if (nWhatsItGot & DISK_GOT_STYLE_STUFF) { - m_pStyleStuff->Read(pStrm); + m_aStyleStuff.Read(pStrm); } if (nWhatsItGot & DISK_GOT_MISC_STUFF) { - m_pMiscStuff->Read(pStrm); + m_aMiscStuff.Read(pStrm); } m_LayGeometry.ReadIndexed(pStrm); @@ -1473,15 +1459,11 @@ bool LwpMiddleLayout::HasContent() } LwpLayout::LwpLayout( LwpObjectHeader &objHdr, LwpSvStream* pStrm ) : - LwpMiddleLayout(objHdr, pStrm), m_pUseWhen(new LwpUseWhen) + LwpMiddleLayout(objHdr, pStrm) {} LwpLayout::~LwpLayout() { - if (m_pUseWhen) - { - delete m_pUseWhen; - } } void LwpLayout::Read() @@ -1499,7 +1481,7 @@ void LwpLayout::Read() if (!nSimple) { - m_pUseWhen->Read(pStrm); + m_aUseWhen.Read(pStrm); sal_uInt8 nFlag = pStrm->QuickReaduInt8(); if (nFlag) @@ -1749,7 +1731,7 @@ LwpUseWhen* LwpLayout::VirtualGetUseWhen() { if(m_nOverrideFlag & OVER_PLACEMENT) { - return m_pUseWhen; + return &m_aUseWhen; } else { |