summaryrefslogtreecommitdiff
path: root/sw/source/filter/xml
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2020-09-30 10:10:08 +0100
committerCaolán McNamara <caolanm@redhat.com>2020-09-30 14:27:41 +0200
commit93099409f3e9894b7a40182f775d76757fa2fb5b (patch)
tree0ef85877beba2acc6a0764258f9719e3142858eb /sw/source/filter/xml
parentdaa058c714804ffbd2aa1ff20096b4c79bebea0c (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.cxx2
-rw-r--r--sw/source/filter/xml/xmlimp.cxx2
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" );