sc/source/filter/html/htmlimp.cxx |   16 ++++++++--------
 sc/source/filter/inc/htmlimp.hxx  |    2 +-
 2 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit 4ade8461354063b8c1d22eaf65373a6e89f6a14a
Author:     Caolán McNamara <caol...@redhat.com>
AuthorDate: Sun Sep 13 19:32:19 2020 +0100
Commit:     Caolán McNamara <caol...@redhat.com>
CommitDate: Mon Sep 14 09:52:22 2020 +0200

    InsertRangeName always dereferences its ScDocument* arg
    
    Change-Id: I85e10c92c2480c4b1aff36c3c29cc7972f2bed8c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/102599
    Tested-by: Caolán McNamara <caol...@redhat.com>
    Reviewed-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sc/source/filter/html/htmlimp.cxx 
b/sc/source/filter/html/htmlimp.cxx
index 40c0babd73eb..165cbc80b679 100644
--- a/sc/source/filter/html/htmlimp.cxx
+++ b/sc/source/filter/html/htmlimp.cxx
@@ -98,16 +98,16 @@ ScHTMLImport::ScHTMLImport( ScDocument* pDocP, const 
OUString& rBaseURL, const S
         mpParser.reset( new ScHTMLQueryParser( mpEngine.get(), pDocP ));
 }
 
-void ScHTMLImport::InsertRangeName( ScDocument* pDoc, const OUString& rName, 
const ScRange& rRange )
+void ScHTMLImport::InsertRangeName( ScDocument& rDoc, const OUString& rName, 
const ScRange& rRange )
 {
     ScComplexRefData aRefData;
     aRefData.InitRange( rRange );
     aRefData.Ref1.SetFlag3D( true );
     aRefData.Ref2.SetFlag3D( aRefData.Ref2.Tab() != aRefData.Ref1.Tab() );
-    ScTokenArray aTokArray(*pDoc);
+    ScTokenArray aTokArray(rDoc);
     aTokArray.AddDoubleReference( aRefData );
-    ScRangeData* pRangeData = new ScRangeData( pDoc, rName, aTokArray );
-    pDoc->GetRangeName()->insert( pRangeData );
+    ScRangeData* pRangeData = new ScRangeData( &rDoc, rName, aTokArray );
+    rDoc.GetRangeName()->insert( pRangeData );
 }
 
 void ScHTMLImport::WriteToDocument(
@@ -160,10 +160,10 @@ void ScHTMLImport::WriteToDocument(
     ScRange aNewRange( maRange.aStart );
     aNewRange.aEnd.IncCol( static_cast<SCCOL>(pGlobTable->GetDocSize( tdCol )) 
- 1 );
     aNewRange.aEnd.IncRow( pGlobTable->GetDocSize( tdRow ) - 1 );
-    InsertRangeName( mpDoc, ScfTools::GetHTMLDocName(), aNewRange );
+    InsertRangeName( *mpDoc, ScfTools::GetHTMLDocName(), aNewRange );
 
     // 2 - all tables
-    InsertRangeName( mpDoc, ScfTools::GetHTMLTablesName(), ScRange( 
maRange.aStart ) );
+    InsertRangeName( *mpDoc, ScfTools::GetHTMLTablesName(), ScRange( 
maRange.aStart ) );
 
     // 3 - single tables
     SCCOL nColDiff = maRange.aStart.Col();
@@ -181,13 +181,13 @@ void ScHTMLImport::WriteToDocument(
             assert(!"can't move");
         }
         // insert table number as name
-        InsertRangeName( mpDoc, ScfTools::GetNameFromHTMLIndex( nTableId ), 
aNewRange );
+        InsertRangeName( *mpDoc, ScfTools::GetNameFromHTMLIndex( nTableId ), 
aNewRange );
         // insert table id as name
         if (!pTable->GetTableName().isEmpty())
         {
             OUString aName( ScfTools::GetNameFromHTMLName( 
pTable->GetTableName() ) );
             if 
(!mpDoc->GetRangeName()->findByUpperName(ScGlobal::getCharClassPtr()->uppercase(aName)))
-                InsertRangeName( mpDoc, aName, aNewRange );
+                InsertRangeName( *mpDoc, aName, aNewRange );
         }
     }
 }
diff --git a/sc/source/filter/inc/htmlimp.hxx b/sc/source/filter/inc/htmlimp.hxx
index da50cb33c4de..eaddd57cf16f 100644
--- a/sc/source/filter/inc/htmlimp.hxx
+++ b/sc/source/filter/inc/htmlimp.hxx
@@ -25,7 +25,7 @@
 class ScHTMLImport : public ScEEImport
 {
 private:
-    static void         InsertRangeName( ScDocument* pDoc, const OUString& 
rName, const ScRange& rRange );
+    static void         InsertRangeName( ScDocument& rDoc, const OUString& 
rName, const ScRange& rRange );
 
 public:
     ScHTMLImport( ScDocument* pDoc, const OUString& rBaseURL, const ScRange& 
rRange, bool bCalcWidthHeight );
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to