Use more SwPosition::Assign

to keep the internal fields of SwPosition in sync.

Change-Id: I49cf97b3beaa3ff79489ba23106ac9ede9475765
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138760
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

5 files changed, 12 insertions, 13 deletions

diff --git a/sw/source/filter/ww8/WW8TableInfo.cxx b/sw/source/filter/ww8/WW8TableInfo.cxx
index f5c318b6df23..e087d515fc5f 100644
--- a/sw/source/filter/ww8/WW8TableInfo.cxx
+++ b/sw/source/filter/ww8/WW8TableInfo.cxx
@@ -589,7 +589,7 @@ WW8TableInfo::processSwTableByLayout(const SwTable * pTable, RowEndInners_t &rLa
                         bDone = true;
                 }
 
-                aPam.GetPoint()->nNode++;
+                aPam.GetPoint()->Adjust(SwNodeOffset(1));
             }
             while (!bDone);
         }
@@ -708,7 +708,7 @@ WW8TableInfo::processTableBoxLines(const SwTableBox * pBox,
             if (aPaM.GetPoint()->GetNode() == aEndPaM.GetPoint()->GetNode())
                 bDone = true;
             else
-                aPaM.GetPoint()->nNode++;
+                aPaM.GetPoint()->Adjust(SwNodeOffset(1));
         }
     }
 
@@ -804,7 +804,7 @@ WW8TableInfo::processTableBox(const SwTable * pTable,
                     bDone = true;
             }
 
-            aPaM.GetPoint()->nNode++;
+            aPaM.GetPoint()->Adjust(SwNodeOffset(1));
         }
         while (!bDone);
 
diff --git a/sw/source/filter/ww8/docxexport.cxx b/sw/source/filter/ww8/docxexport.cxx
index 4894400011f1..52090fc74891 100644
--- a/sw/source/filter/ww8/docxexport.cxx
+++ b/sw/source/filter/ww8/docxexport.cxx
@@ -1872,7 +1872,7 @@ void DocxExport::WriteMainText()
     // body
     m_pDocumentFS->startElementNS(XML_w, XML_body);
 
-    m_pCurPam->GetPoint()->nNode = m_rDoc.GetNodes().GetEndOfContent().StartOfSectionNode()->GetIndex();
+    m_pCurPam->GetPoint()->Assign(*m_rDoc.GetNodes().GetEndOfContent().StartOfSectionNode());
 
     // the text
     WriteText();
diff --git a/sw/source/filter/ww8/rtfexport.cxx b/sw/source/filter/ww8/rtfexport.cxx
index 7a7d110ca835..a48e70c8efe1 100644
--- a/sw/source/filter/ww8/rtfexport.cxx
+++ b/sw/source/filter/ww8/rtfexport.cxx
@@ -500,13 +500,12 @@ void RtfExport::WriteMainText()
     SwTableNode* pTableNode = m_pCurPam->GetPointNode().FindTableNode();
     if (m_pWriter && m_pWriter->m_bWriteOnlyFirstTable && pTableNode != nullptr)
     {
-        m_pCurPam->GetPoint()->nNode = *pTableNode;
-        m_pCurPam->GetMark()->nNode = *(pTableNode->EndOfSectionNode());
+        m_pCurPam->GetPoint()->Assign(*pTableNode);
+        m_pCurPam->GetMark()->Assign(*pTableNode->EndOfSectionNode());
     }
     else
     {
-        m_pCurPam->GetPoint()->nNode
-            = m_rDoc.GetNodes().GetEndOfContent().StartOfSectionNode()->GetIndex();
+        m_pCurPam->GetPoint()->Assign(*m_rDoc.GetNodes().GetEndOfContent().StartOfSectionNode());
     }
 
     WriteText();
diff --git a/sw/source/filter/ww8/wrtw8sty.cxx b/sw/source/filter/ww8/wrtw8sty.cxx
index ff5af1a68785..06c930ba339d 100644
--- a/sw/source/filter/ww8/wrtw8sty.cxx
+++ b/sw/source/filter/ww8/wrtw8sty.cxx
@@ -1013,7 +1013,7 @@ MSWordSections::MSWordSections( MSWordExportBase& rExport )
         if ( SectionType::ToxContent == pSectNd->GetSection().GetType() )
         {
             pNd = pSectNd;
-            rExport.m_pCurPam->GetPoint()->nNode = *pNd;
+            rExport.m_pCurPam->GetPoint()->Assign(*pNd);
         }
 
         if ( SectionType::Content == pSectNd->GetSection().GetType() )
diff --git a/sw/source/filter/ww8/wrtww8.cxx b/sw/source/filter/ww8/wrtww8.cxx
index d2eb7a99e95c..331b0038cdb9 100644
--- a/sw/source/filter/ww8/wrtww8.cxx
+++ b/sw/source/filter/ww8/wrtww8.cxx
@@ -1883,7 +1883,7 @@ void MSWordExportBase::SetCurPam(SwNodeOffset nStt, SwNodeOffset nEnd)
     if ( nStt != m_pCurPam->GetMark()->GetNodeIndex() &&
          m_rDoc.GetNodes()[ nStt ]->IsTableNode() )
     {
-        m_pCurPam->GetMark()->nNode = nStt;
+        m_pCurPam->GetMark()->Assign(nStt);
     }
 
     m_pOrigPam = m_pCurPam.get(); // ???
@@ -2919,7 +2919,7 @@ void MSWordExportBase::WriteText()
         if (pNextNode != nullptr)
             m_pCurPam->GetPoint()->Assign(*pNextNode);
         else
-            ++m_pCurPam->GetPoint()->nNode;
+            m_pCurPam->GetPoint()->Adjust(SwNodeOffset(1));
 
         SwNodeOffset nPos = m_pCurPam->GetPoint()->GetNodeIndex();
         ::SetProgressState( sal_Int32(nPos), m_pCurPam->GetDoc().GetDocShell() );
@@ -2934,7 +2934,7 @@ void WW8Export::WriteMainText()
 
     m_pFib->m_fcMin = Strm().Tell();
 
-    m_pCurPam->GetPoint()->nNode = m_rDoc.GetNodes().GetEndOfContent().StartOfSectionNode()->GetIndex();
+    m_pCurPam->GetPoint()->Assign(*m_rDoc.GetNodes().GetEndOfContent().StartOfSectionNode());
 
     WriteText();
 
@@ -3818,7 +3818,7 @@ ErrCode SwWW8Writer::WriteStorageImpl()
         SwTableNode* pTNd = m_pCurrentPam->GetPointNode().FindTableNode();
         if( pTNd && m_bWriteAll )
             // start with the table node !!
-            m_pCurrentPam->GetPoint()->nNode = *pTNd;
+            m_pCurrentPam->GetPoint()->Assign(*pTNd);
     }
 
     // Do the actual export