diff options
author | Caolán McNamara <caolanm@redhat.com> | 2020-09-30 10:10:08 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2020-09-30 14:27:41 +0200 |
commit | 93099409f3e9894b7a40182f775d76757fa2fb5b (patch) | |
tree | 0ef85877beba2acc6a0764258f9719e3142858eb /sw/source/filter/xml | |
parent | daa058c714804ffbd2aa1ff20096b4c79bebea0c (diff) |
SwPaM::GetDoc can return a reference instead
and remove discovered redundant null checks
Change-Id: Iac8ad7821d9acfcc9550a96402c02ac248f16f2d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103672
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'sw/source/filter/xml')
-rw-r--r-- | sw/source/filter/xml/swxml.cxx | 2 | ||||
-rw-r--r-- | sw/source/filter/xml/xmlimp.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/sw/source/filter/xml/swxml.cxx b/sw/source/filter/xml/swxml.cxx index 24c8e8c7c749..4683b6ecb1c7 100644 --- a/sw/source/filter/xml/swxml.cxx +++ b/sw/source/filter/xml/swxml.cxx @@ -110,7 +110,7 @@ static void lcl_EnsureValidPam( SwPaM& rPam ) // point is not valid, so move it into the first content rPam.DeleteMark(); rPam.GetPoint()->nNode = - *rPam.GetDoc()->GetNodes().GetEndOfContent().StartOfSectionNode(); + *rPam.GetDoc().GetNodes().GetEndOfContent().StartOfSectionNode(); ++ rPam.GetPoint()->nNode; rPam.Move( fnMoveForward, GoInContent ); // go into content } diff --git a/sw/source/filter/xml/xmlimp.cxx b/sw/source/filter/xml/xmlimp.cxx index c5a9bc5b7b18..00c141332a15 100644 --- a/sw/source/filter/xml/xmlimp.cxx +++ b/sw/source/filter/xml/xmlimp.cxx @@ -797,7 +797,7 @@ void SwXMLImport::endDocument() { SwTextNode* pCurrNd; sal_uLong nNodeIdx = pPos->nNode.GetIndex(); - pDoc = pPaM->GetDoc(); + pDoc = &pPaM->GetDoc(); OSL_ENSURE( pPos->nNode.GetNode().IsContentNode(), "insert position is not a content node" ); |