[Libreoffice-commits] core.git: sw/inc sw/source

Caolán McNamara (via logerrit) logerrit at kemper.freedesktop.org
Wed Sep 30 12:29:08 UTC 2020


 sw/inc/pam.hxx                                          |    2 -
 sw/source/core/access/accpara.cxx                       |    2 -
 sw/source/core/crsr/annotationmark.cxx                  |    5 +--
 sw/source/core/crsr/bookmrk.cxx                         |   25 ++++++----------
 sw/source/core/crsr/pam.cxx                             |    4 +-
 sw/source/core/doc/DocumentContentOperationsManager.cxx |    8 ++---
 sw/source/core/doc/docbm.cxx                            |    6 +--
 sw/source/core/undo/SwUndoField.cxx                     |    4 +-
 sw/source/core/undo/unattr.cxx                          |    4 +-
 sw/source/core/undo/unbkmk.cxx                          |   10 +++---
 sw/source/core/undo/undobj.cxx                          |    4 +-
 sw/source/core/undo/unnum.cxx                           |    4 +-
 sw/source/core/undo/untbl.cxx                           |    2 -
 sw/source/core/unocore/unobkm.cxx                       |    4 +-
 sw/source/core/unocore/unotbl.cxx                       |    7 +---
 sw/source/filter/ww8/writerhelper.cxx                   |    2 -
 16 files changed, 42 insertions(+), 51 deletions(-)

New commits:
commit 947e89cf6755d15b57ee3c5fa2661e7a1c09402b
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Wed Sep 30 10:20:07 2020 +0100
Commit:     Caolán McNamara <caolanm at redhat.com>
CommitDate: Wed Sep 30 14:28:20 2020 +0200

    SwPosition::GetDoc can return a reference
    
    and remove discovered redundant null checks
    
    Change-Id: I90e2a84b260bfaf283b787db055cd1ed54dab4ee
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103676
    Tested-by: Jenkins
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/inc/pam.hxx b/sw/inc/pam.hxx
index 0f80752a276a..8c0a8abe6de9 100644
--- a/sw/inc/pam.hxx
+++ b/sw/inc/pam.hxx
@@ -47,7 +47,7 @@ struct SAL_WARN_UNUSED SW_DLLPUBLIC SwPosition
 
        @return the document this position is in.
     */
-    SwDoc * GetDoc() const;
+    SwDoc& GetDoc() const;
 
     bool operator < (const SwPosition &) const;
     bool operator > (const SwPosition &) const;
diff --git a/sw/source/core/access/accpara.cxx b/sw/source/core/access/accpara.cxx
index 001c32ade91b..4f60b2b9dff0 100644
--- a/sw/source/core/access/accpara.cxx
+++ b/sw/source/core/access/accpara.cxx
@@ -536,7 +536,7 @@ const SwRangeRedline* SwAccessibleParagraph::GetRedlineAtIndex()
     if ( pCrSr )
     {
         SwPosition* pStart = pCrSr->Start();
-        pRedline = pStart->GetDoc()->getIDocumentRedlineAccess().GetRedline(*pStart, nullptr);
+        pRedline = pStart->GetDoc().getIDocumentRedlineAccess().GetRedline(*pStart, nullptr);
     }
 
     return pRedline;
diff --git a/sw/source/core/crsr/annotationmark.cxx b/sw/source/core/crsr/annotationmark.cxx
index 887333525390..061d88a08323 100644
--- a/sw/source/core/crsr/annotationmark.cxx
+++ b/sw/source/core/crsr/annotationmark.cxx
@@ -78,11 +78,10 @@ namespace sw::mark
 
     const SwFormatField* AnnotationMark::GetAnnotationFormatField() const
     {
-        SwDoc* pDoc = GetMarkPos().GetDoc();
-        assert(pDoc != nullptr);
+        SwDoc& rDoc = GetMarkPos().GetDoc();
 
         const auto sName = GetName();
-        SwFieldType* pType = pDoc->getIDocumentFieldsAccess().GetFieldType( SwFieldIds::Postit, OUString(), false );
+        SwFieldType* pType = rDoc.getIDocumentFieldsAccess().GetFieldType( SwFieldIds::Postit, OUString(), false );
         std::vector<SwFormatField*> vFields;
         pType->GatherFields(vFields);
         auto ppFound = std::find_if(vFields.begin(), vFields.end(), [&sName](SwFormatField* pF)
diff --git a/sw/source/core/crsr/bookmrk.cxx b/sw/source/core/crsr/bookmrk.cxx
index 4e43a49bf007..c918ec28917e 100644
--- a/sw/source/core/crsr/bookmrk.cxx
+++ b/sw/source/core/crsr/bookmrk.cxx
@@ -455,16 +455,14 @@ namespace sw::mark
 
     ::sfx2::IXmlIdRegistry& Bookmark::GetRegistry()
     {
-        SwDoc *const pDoc( GetMarkPos().GetDoc() );
-        assert(pDoc);
-        return pDoc->GetXmlIdRegistry();
+        SwDoc& rDoc( GetMarkPos().GetDoc() );
+        return rDoc.GetXmlIdRegistry();
     }
 
     bool Bookmark::IsInClipboard() const
     {
-        SwDoc *const pDoc( GetMarkPos().GetDoc() );
-        assert(pDoc);
-        return pDoc->IsClipBoard();
+        SwDoc& rDoc( GetMarkPos().GetDoc() );
+        return rDoc.IsClipBoard();
     }
 
     bool Bookmark::IsInUndo() const
@@ -474,17 +472,15 @@ namespace sw::mark
 
     bool Bookmark::IsInContent() const
     {
-        SwDoc *const pDoc( GetMarkPos().GetDoc() );
-        assert(pDoc);
-        return !pDoc->IsInHeaderFooter( GetMarkPos().nNode );
+        SwDoc& rDoc( GetMarkPos().GetDoc() );
+        return !rDoc.IsInHeaderFooter( GetMarkPos().nNode );
     }
 
     uno::Reference< rdf::XMetadatable > Bookmark::MakeUnoObject()
     {
-        SwDoc *const pDoc( GetMarkPos().GetDoc() );
-        assert(pDoc);
+        SwDoc& rDoc( GetMarkPos().GetDoc() );
         const uno::Reference< rdf::XMetadatable> xMeta(
-                SwXBookmark::CreateXBookmark(*pDoc, this), uno::UNO_QUERY);
+                SwXBookmark::CreateXBookmark(rDoc, this), uno::UNO_QUERY);
         return xMeta;
     }
 
@@ -626,9 +622,8 @@ namespace sw::mark
         {
             (*GetParameters())[OUString(ODF_FORMCHECKBOX_RESULT)] <<= checked;
             // mark document as modified
-            SwDoc *const pDoc( GetMarkPos().GetDoc() );
-            if ( pDoc )
-                pDoc->getIDocumentState().SetModified();
+            SwDoc& rDoc( GetMarkPos().GetDoc() );
+            rDoc.getIDocumentState().SetModified();
         }
     }
 
diff --git a/sw/source/core/crsr/pam.cxx b/sw/source/core/crsr/pam.cxx
index e73885ff6008..aae189370d0a 100644
--- a/sw/source/core/crsr/pam.cxx
+++ b/sw/source/core/crsr/pam.cxx
@@ -175,9 +175,9 @@ bool SwPosition::operator!=(const SwPosition &rPos) const
         || (nContent != rPos.nContent);
 }
 
-SwDoc * SwPosition::GetDoc() const
+SwDoc& SwPosition::GetDoc() const
 {
-    return &nNode.GetNode().GetDoc();
+    return nNode.GetNode().GetDoc();
 }
 
 void SwPosition::dumpAsXml(xmlTextWriterPtr pWriter) const
diff --git a/sw/source/core/doc/DocumentContentOperationsManager.cxx b/sw/source/core/doc/DocumentContentOperationsManager.cxx
index fd7dd06a6cfa..4ca1c1cb894d 100644
--- a/sw/source/core/doc/DocumentContentOperationsManager.cxx
+++ b/sw/source/core/doc/DocumentContentOperationsManager.cxx
@@ -234,9 +234,9 @@ namespace sw
     void CopyBookmarks(const SwPaM& rPam, SwPosition& rCpyPam)
     {
         const SwDoc& rSrcDoc = rPam.GetDoc();
-        SwDoc* pDestDoc =  rCpyPam.GetDoc();
+        SwDoc& rDestDoc =  rCpyPam.GetDoc();
         const IDocumentMarkAccess* const pSrcMarkAccess = rSrcDoc.getIDocumentMarkAccess();
-        ::sw::UndoGuard const undoGuard(pDestDoc->GetIDocumentUndoRedo());
+        ::sw::UndoGuard const undoGuard(rDestDoc.GetIDocumentUndoRedo());
 
         const SwPosition &rStt = *rPam.Start(), &rEnd = *rPam.End();
         SwPosition const*const pCpyStt = &rCpyPam;
@@ -283,14 +283,14 @@ namespace sw
                 lcl_SetCpyPos(pMark->GetOtherMarkPos(), rStt, *pCpyStt, *aTmpPam.GetMark(), nDelCount);
             }
 
-            ::sw::mark::IMark* const pNewMark = pDestDoc->getIDocumentMarkAccess()->makeMark(
+            ::sw::mark::IMark* const pNewMark = rDestDoc.getIDocumentMarkAccess()->makeMark(
                 aTmpPam,
                 pMark->GetName(),
                 IDocumentMarkAccess::GetType(*pMark),
                 ::sw::mark::InsertMode::CopyText);
             // Explicitly try to get exactly the same name as in the source
             // because NavigatorReminders, DdeBookmarks etc. ignore the proposed name
-            pDestDoc->getIDocumentMarkAccess()->renameMark(pNewMark, pMark->GetName());
+            rDestDoc.getIDocumentMarkAccess()->renameMark(pNewMark, pMark->GetName());
 
             // copying additional attributes for bookmarks or fieldmarks
             ::sw::mark::IBookmark* const pNewBookmark =
diff --git a/sw/source/core/doc/docbm.cxx b/sw/source/core/doc/docbm.cxx
index 749c473d8c56..271ad9bcc0fc 100644
--- a/sw/source/core/doc/docbm.cxx
+++ b/sw/source/core/doc/docbm.cxx
@@ -812,7 +812,7 @@ namespace sw::mark
         ::sw::mark::IMark* const io_pMark,
         const SwPaM& rPaM)
     {
-        assert(io_pMark->GetMarkPos().GetDoc() == &m_rDoc &&
+        assert(&io_pMark->GetMarkPos().GetDoc() == &m_rDoc &&
             "<MarkManager::repositionMark(..)>"
             " - Mark is not in my doc.");
         MarkBase* const pMarkBase = dynamic_cast< MarkBase* >(io_pMark);
@@ -839,7 +839,7 @@ namespace sw::mark
         ::sw::mark::IMark* io_pMark,
         const OUString& rNewName )
     {
-        assert(io_pMark->GetMarkPos().GetDoc() == &m_rDoc &&
+        assert(&io_pMark->GetMarkPos().GetDoc() == &m_rDoc &&
             "<MarkManager::renameMark(..)>"
             " - Mark is not in my doc.");
         if ( io_pMark->GetName() == rNewName )
@@ -1288,7 +1288,7 @@ namespace sw::mark
 
     void MarkManager::deleteMark(const IMark* const pMark)
     {
-        assert(pMark->GetMarkPos().GetDoc() == &m_rDoc &&
+        assert(&pMark->GetMarkPos().GetDoc() == &m_rDoc &&
             "<MarkManager::deleteMark(..)>"
             " - Mark is not in my doc.");
         // finds the last Mark that is starting before pMark
diff --git a/sw/source/core/undo/SwUndoField.cxx b/sw/source/core/undo/SwUndoField.cxx
index 3ab16ebe4ff3..353070852588 100644
--- a/sw/source/core/undo/SwUndoField.cxx
+++ b/sw/source/core/undo/SwUndoField.cxx
@@ -33,11 +33,11 @@
 using namespace ::com::sun::star::uno;
 
 SwUndoField::SwUndoField(const SwPosition & rPos )
-    : SwUndo(SwUndoId::FIELD, rPos.GetDoc())
+    : SwUndo(SwUndoId::FIELD, &rPos.GetDoc())
 {
     m_nNodeIndex = rPos.nNode.GetIndex();
     m_nOffset = rPos.nContent.GetIndex();
-    m_pDoc = rPos.GetDoc();
+    m_pDoc = &rPos.GetDoc();
 }
 
 SwUndoField::~SwUndoField()
diff --git a/sw/source/core/undo/unattr.cxx b/sw/source/core/undo/unattr.cxx
index 68fb64f903bd..3cbdebd6a989 100644
--- a/sw/source/core/undo/unattr.cxx
+++ b/sw/source/core/undo/unattr.cxx
@@ -538,7 +538,7 @@ SwUndoResetAttr::SwUndoResetAttr( const SwPaM& rRange, sal_uInt16 nFormatId )
 }
 
 SwUndoResetAttr::SwUndoResetAttr( const SwPosition& rPos, sal_uInt16 nFormatId )
-    : SwUndo( SwUndoId::RESETATTR, rPos.GetDoc() )
+    : SwUndo( SwUndoId::RESETATTR, &rPos.GetDoc() )
     , m_pHistory( new SwHistory )
     , m_nFormatId( nFormatId )
 {
@@ -1017,7 +1017,7 @@ void SwUndoEndNoteInfo::RedoImpl(::sw::UndoRedoContext & rContext)
 }
 
 SwUndoDontExpandFormat::SwUndoDontExpandFormat( const SwPosition& rPos )
-    : SwUndo( SwUndoId::DONTEXPAND, rPos.GetDoc() )
+    : SwUndo( SwUndoId::DONTEXPAND, &rPos.GetDoc() )
     , m_nNodeIndex( rPos.nNode.GetIndex() )
     , m_nContentIndex( rPos.nContent.GetIndex() )
 {
diff --git a/sw/source/core/undo/unbkmk.cxx b/sw/source/core/undo/unbkmk.cxx
index 16d45a7b705c..cabcd02a5170 100644
--- a/sw/source/core/undo/unbkmk.cxx
+++ b/sw/source/core/undo/unbkmk.cxx
@@ -32,7 +32,7 @@
 
 SwUndoBookmark::SwUndoBookmark( SwUndoId nUndoId,
             const ::sw::mark::IMark& rBkmk )
-    : SwUndo( nUndoId, rBkmk.GetMarkPos().GetDoc() )
+    : SwUndo( nUndoId, &rBkmk.GetMarkPos().GetDoc() )
     , m_pHistoryBookmark(new SwHistoryBookmark(rBkmk, true, rBkmk.IsExpanded()))
 {
 }
@@ -148,7 +148,7 @@ void SwUndoRenameBookmark::RedoImpl(::sw::UndoRedoContext & rContext)
 }
 
 SwUndoInsNoTextFieldmark::SwUndoInsNoTextFieldmark(const ::sw::mark::IFieldmark& rFieldmark)
-    : SwUndo(SwUndoId::INSERT, rFieldmark.GetMarkPos().GetDoc())
+    : SwUndo(SwUndoId::INSERT, &rFieldmark.GetMarkPos().GetDoc())
     , m_pHistoryNoTextFieldmark(new SwHistoryNoTextFieldmark(rFieldmark))
 {
 }
@@ -164,7 +164,7 @@ void SwUndoInsNoTextFieldmark::RedoImpl(::sw::UndoRedoContext & rContext)
 }
 
 SwUndoDelNoTextFieldmark::SwUndoDelNoTextFieldmark(const ::sw::mark::IFieldmark& rFieldmark)
-    : SwUndo(SwUndoId::DELETE, rFieldmark.GetMarkPos().GetDoc())
+    : SwUndo(SwUndoId::DELETE, &rFieldmark.GetMarkPos().GetDoc())
     , m_pHistoryNoTextFieldmark(new SwHistoryNoTextFieldmark(rFieldmark))
 {
 }
@@ -182,7 +182,7 @@ void SwUndoDelNoTextFieldmark::RedoImpl(::sw::UndoRedoContext & rContext)
 }
 
 SwUndoInsTextFieldmark::SwUndoInsTextFieldmark(const ::sw::mark::IFieldmark& rFieldmark)
-    : SwUndo(SwUndoId::INSERT, rFieldmark.GetMarkPos().GetDoc())
+    : SwUndo(SwUndoId::INSERT, &rFieldmark.GetMarkPos().GetDoc())
     , m_pHistoryTextFieldmark(new SwHistoryTextFieldmark(rFieldmark))
 {
 }
@@ -198,7 +198,7 @@ void SwUndoInsTextFieldmark::RedoImpl(::sw::UndoRedoContext & rContext)
 }
 
 SwUndoDelTextFieldmark::SwUndoDelTextFieldmark(const ::sw::mark::IFieldmark& rFieldmark)
-    : SwUndo(SwUndoId::DELETE, rFieldmark.GetMarkPos().GetDoc())
+    : SwUndo(SwUndoId::DELETE, &rFieldmark.GetMarkPos().GetDoc())
     , m_pHistoryTextFieldmark(new SwHistoryTextFieldmark(rFieldmark))
 {
 }
diff --git a/sw/source/core/undo/undobj.cxx b/sw/source/core/undo/undobj.cxx
index 323de872c3cf..0ef36bd5f6e7 100644
--- a/sw/source/core/undo/undobj.cxx
+++ b/sw/source/core/undo/undobj.cxx
@@ -1597,7 +1597,7 @@ bool IsDestroyFrameAnchoredAtChar(SwPosition const & rAnchorPos,
     }
 
     if ((nDelContentType & DelContentType::WriterfilterHack)
-        && rAnchorPos.GetDoc()->IsInWriterfilterImport())
+        && rAnchorPos.GetDoc().IsInWriterfilterImport())
     {   // FIXME hack for writerfilter RemoveLastParagraph() and MakeFlyAndMove(); can't test file format more specific?
         return (rStart < rAnchorPos) && (rAnchorPos < rEnd);
     }
@@ -1638,7 +1638,7 @@ bool IsSelectFrameAnchoredAtPara(SwPosition const & rAnchorPos,
     }
 
     if ((nDelContentType & DelContentType::WriterfilterHack)
-        && rAnchorPos.GetDoc()->IsInWriterfilterImport())
+        && rAnchorPos.GetDoc().IsInWriterfilterImport())
     {   // FIXME hack for writerfilter RemoveLastParagraph() and MakeFlyAndMove(); can't test file format more specific?
         // but it MUST NOT be done during the SetRedlineFlags at the end of ODF
         // import, where the IsInXMLImport() cannot be checked because the
diff --git a/sw/source/core/undo/unnum.cxx b/sw/source/core/undo/unnum.cxx
index 603567544bff..d7d517a5f37f 100644
--- a/sw/source/core/undo/unnum.cxx
+++ b/sw/source/core/undo/unnum.cxx
@@ -323,14 +323,14 @@ void SwUndoNumOrNoNum::RepeatImpl(::sw::RepeatContext & rContext)
 }
 
 SwUndoNumRuleStart::SwUndoNumRuleStart( const SwPosition& rPos, bool bFlg )
-    : SwUndo( SwUndoId::SETNUMRULESTART, rPos.GetDoc() ),
+    : SwUndo( SwUndoId::SETNUMRULESTART, &rPos.GetDoc() ),
     m_nIndex( rPos.nNode.GetIndex() ), m_nOldStart( USHRT_MAX ),
     m_nNewStart( USHRT_MAX ), m_bSetStartValue( false ), m_bFlag( bFlg )
 {
 }
 
 SwUndoNumRuleStart::SwUndoNumRuleStart( const SwPosition& rPos, sal_uInt16 nStt )
-    : SwUndo(SwUndoId::SETNUMRULESTART, rPos.GetDoc())
+    : SwUndo(SwUndoId::SETNUMRULESTART, &rPos.GetDoc())
     , m_nIndex(rPos.nNode.GetIndex())
     , m_nOldStart(USHRT_MAX)
     , m_nNewStart(nStt)
diff --git a/sw/source/core/undo/untbl.cxx b/sw/source/core/undo/untbl.cxx
index f5f853503174..0df50f847820 100644
--- a/sw/source/core/undo/untbl.cxx
+++ b/sw/source/core/undo/untbl.cxx
@@ -223,7 +223,7 @@ SwUndoInsTable::SwUndoInsTable( const SwPosition& rPos, sal_uInt16 nCl, sal_uInt
                             const SwTableAutoFormat* pTAFormat,
                             const std::vector<sal_uInt16> *pColArr,
                             const OUString & rName)
-    : SwUndo( SwUndoId::INSTABLE, rPos.GetDoc() ),
+    : SwUndo( SwUndoId::INSTABLE, &rPos.GetDoc() ),
     m_aInsTableOptions( rInsTableOpts ),
     m_nStartNode( rPos.nNode.GetIndex() ), m_nRows( nRw ), m_nColumns( nCl ), m_nAdjust( nAdj )
 {
diff --git a/sw/source/core/unocore/unobkm.cxx b/sw/source/core/unocore/unobkm.cxx
index 97550fec41da..233d14b23421 100644
--- a/sw/source/core/unocore/unobkm.cxx
+++ b/sw/source/core/unocore/unobkm.cxx
@@ -101,8 +101,8 @@ void SwXBookmark::Impl::registerInMark(SwXBookmark& rThis,
         {
             pMarkBase->SetXBookmark(xBookmark);
         }
-        assert(m_pDoc == nullptr || m_pDoc == pBkmk->GetMarkPos().GetDoc());
-        m_pDoc = pBkmk->GetMarkPos().GetDoc();
+        assert(m_pDoc == nullptr || m_pDoc == &pBkmk->GetMarkPos().GetDoc());
+        m_pDoc = &pBkmk->GetMarkPos().GetDoc();
     }
     else if (m_pRegisteredBookmark)
     {
diff --git a/sw/source/core/unocore/unotbl.cxx b/sw/source/core/unocore/unotbl.cxx
index 86b0b7ca5446..f58ee3dad7e0 100644
--- a/sw/source/core/unocore/unotbl.cxx
+++ b/sw/source/core/unocore/unotbl.cxx
@@ -1074,11 +1074,8 @@ uno::Any SwXCell::getPropertyValue(const OUString& rPropertyName)
                     return uno::Any();
 
                 SwPosition aPos(*pTableNode);
-                SwDoc* pDoc = aPos.GetDoc();
-                if (!pDoc)
-                    return uno::Any();
-
-                m_xParentText = sw::CreateParentXText(*pDoc, aPos);
+                SwDoc& rDoc = aPos.GetDoc();
+                m_xParentText = sw::CreateParentXText(rDoc, aPos);
             }
 
             return uno::makeAny(m_xParentText);
diff --git a/sw/source/filter/ww8/writerhelper.cxx b/sw/source/filter/ww8/writerhelper.cxx
index af79be052618..bed5a4eeb23d 100644
--- a/sw/source/filter/ww8/writerhelper.cxx
+++ b/sw/source/filter/ww8/writerhelper.cxx
@@ -736,7 +736,7 @@ namespace sw
                     SwPosition const end(*rPos.nNode.GetNode().GetTextNode(),
                                          nIndex - 1);
                     sw::mark::IFieldmark *const pFieldMark(
-                        rPos.GetDoc()->getIDocumentMarkAccess()->getFieldmarkAt(end));
+                        rPos.GetDoc().getIDocumentMarkAccess()->getFieldmarkAt(end));
                     SAL_WARN_IF(!pFieldMark, "sw.ww8", "expected a field mark");
                     if (pFieldMark && pFieldMark->GetMarkPos().nNode.GetIndex() == (*aResult)->m_aMkPos.m_nNode.GetIndex()+1
                         && pFieldMark->GetMarkPos().nContent.GetIndex() < (*aResult)->m_aMkPos.m_nContent)


More information about the Libreoffice-commits mailing list