[Libreoffice-commits] core.git: 2 commits - sw/source
Libreoffice Gerrit user
logerrit at kemper.freedesktop.org
Tue Sep 18 07:06:19 UTC 2018
sw/source/filter/ascii/parasc.cxx | 12 ++++++------
sw/source/filter/html/swhtml.cxx | 8 ++++----
2 files changed, 10 insertions(+), 10 deletions(-)
New commits:
commit ce30c6316f7c7ffc727b2d852b6b42c8fefa52bd
Author: Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Fri Sep 14 14:05:12 2018 +0200
Commit: Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Tue Sep 18 09:06:03 2018 +0200
loplugin:useuniqueptr in SwHTMLParser::SetAttr_
Change-Id: I8acc0df204417846d275dbba290177d0f166f3d9
Reviewed-on: https://gerrit.libreoffice.org/60605
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/sw/source/filter/html/swhtml.cxx b/sw/source/filter/html/swhtml.cxx
index 70c9d7cb99c4..0e00318fe7c3 100644
--- a/sw/source/filter/html/swhtml.cxx
+++ b/sw/source/filter/html/swhtml.cxx
@@ -2689,7 +2689,7 @@ void SwHTMLParser::SetAttr_( bool bChkEnd, bool bBeforeTable,
HTMLAttr* pAttr;
SwContentNode* pCNd;
- HTMLAttrs aFields;
+ std::vector<std::unique_ptr<HTMLAttr>> aFields;
for( auto n = m_aSetAttrTab.size(); n; )
{
@@ -2884,7 +2884,7 @@ void SwHTMLParser::SetAttr_( bool bChkEnd, bool bBeforeTable,
}
else
{
- aFields.push_back( pAttr);
+ aFields.emplace_back( pAttr);
}
}
pAttrPam->DeleteMark();
@@ -2992,7 +2992,7 @@ void SwHTMLParser::SetAttr_( bool bChkEnd, bool bBeforeTable,
m_aMoveFlyCnts.erase( m_aMoveFlyCnts.begin() + n );
}
}
- for (auto const& field : aFields)
+ for (auto & field : aFields)
{
pCNd = field->nSttPara.GetNode().GetContentNode();
pAttrPam->GetPoint()->nNode = field->nSttPara;
@@ -3010,7 +3010,7 @@ void SwHTMLParser::SetAttr_( bool bChkEnd, bool bBeforeTable,
m_xDoc->getIDocumentContentOperations().InsertPoolItem( *pAttrPam, *field->pItem );
- delete field;
+ field.reset();
}
aFields.clear();
}
commit 6649cd063832f8571d08ed264599d589ed6d60cc
Author: Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Fri Sep 14 13:54:33 2018 +0200
Commit: Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Tue Sep 18 09:05:56 2018 +0200
loplugin:useuniqueptr in AsciiReader::Read
Change-Id: Id8701c91d272850368db80b1ebabf11ea2a028f5
Reviewed-on: https://gerrit.libreoffice.org/60604
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/sw/source/filter/ascii/parasc.cxx b/sw/source/filter/ascii/parasc.cxx
index a23bdfc77b41..f4962f61692d 100644
--- a/sw/source/filter/ascii/parasc.cxx
+++ b/sw/source/filter/ascii/parasc.cxx
@@ -83,11 +83,11 @@ ErrCode AsciiReader::Read( SwDoc &rDoc, const OUString&, SwPaM &rPam, const OUSt
return ERR_SWG_READ_ERROR;
}
- SwASCIIParser* pParser = new SwASCIIParser( &rDoc, rPam, *m_pStream,
- !m_bInsertMode, m_aOption.GetASCIIOpts() );
+ std::unique_ptr<SwASCIIParser> pParser(new SwASCIIParser( &rDoc, rPam, *m_pStream,
+ !m_bInsertMode, m_aOption.GetASCIIOpts() ));
ErrCode nRet = pParser->CallParser();
- delete pParser;
+ pParser.reset();
// after Read reset the options
m_aOption.ResetASCIIOpts();
return nRet;
@@ -143,12 +143,12 @@ ErrCode SwASCIIParser::CallParser()
::StartProgress( STR_STATSTR_W4WREAD, 0, nFileSize, pDoc->GetDocShell() );
- SwPaM* pInsPam = nullptr;
+ std::unique_ptr<SwPaM> pInsPam;
sal_Int32 nSttContent = 0;
if (!bNewDoc)
{
const SwNodeIndex& rTmp = pPam->GetPoint()->nNode;
- pInsPam = new SwPaM( rTmp, rTmp, 0, -1 );
+ pInsPam.reset(new SwPaM( rTmp, rTmp, 0, -1 ));
nSttContent = pPam->GetPoint()->nContent.GetIndex();
}
@@ -237,7 +237,7 @@ ErrCode SwASCIIParser::CallParser()
pItemSet.reset();
}
- delete pInsPam;
+ pInsPam.reset();
::EndProgress( pDoc->GetDocShell() );
return nError;
More information about the Libreoffice-commits
mailing list