cid#1471553 establish that pViewData will never be null

seeing as these are created from CreateScSubTotalDlg and that
is always called with a aArgSet containing a ScSubTotalItem
with a non-null ScViewData set

Change-Id: I1e3d243e4b00d54f9f5015a543556c39d6053f07
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109069
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>

1 files changed, 8 insertions, 11 deletions

diff --git a/sc/source/ui/dbgui/tpsubt.cxx b/sc/source/ui/dbgui/tpsubt.cxx
index a12eff3ba6e9..b5acbc6cdb60 100644
--- a/sc/source/ui/dbgui/tpsubt.cxx
+++ b/sc/source/ui/dbgui/tpsubt.cxx
@@ -73,10 +73,10 @@ void ScTpSubTotalGroup::Init()
     const ScSubTotalItem& rSubTotalItem = static_cast<const ScSubTotalItem&>(
                                           GetItemSet().Get( nWhichSubTotals ));
 
-    pViewData   = rSubTotalItem.GetViewData();
-    pDoc        = pViewData ? &pViewData->GetDocument() : nullptr;
-
-    OSL_ENSURE( pViewData && pDoc, "ViewData or Document not found :-(" );
+    pViewData = rSubTotalItem.GetViewData();
+    assert(pViewData && "CreateScSubTotalDlg aArgSet must contain a ScSubTotalItem with ViewData set");
+    pDoc = &pViewData->GetDocument();
+    assert(pDoc && "Document not found :-(");
 
     mxLbGroup->connect_changed( LINK( this, ScTpSubTotalGroup, SelectListBoxHdl ) );
     mxLbColumns->connect_changed( LINK( this, ScTpSubTotalGroup, SelectTreeListBoxHdl ) );
@@ -244,10 +244,7 @@ bool ScTpSubTotalGroup::DoFillItemSet( sal_uInt16       nGroupNo,
 
 void ScTpSubTotalGroup::FillListBoxes()
 {
-    OSL_ENSURE( pViewData && pDoc, "ViewData or Document not found :-/" );
-
-    if ( !(pViewData && pDoc) )
-        return;
+    assert(pViewData && pDoc && "CreateScSubTotalDlg aArgSet must contain a ScSubTotalItem with ViewData set");
 
     SCCOL   nFirstCol   = rSubTotalData.nCol1;
     SCROW   nFirstRow   = rSubTotalData.nRow1;
@@ -469,9 +466,9 @@ void ScTpSubTotalOptions::Init()
                                           GetItemSet().Get( nWhichSubTotals ));
 
     pViewData   = rSubTotalItem.GetViewData();
-    pDoc        = pViewData ? &pViewData->GetDocument() : nullptr;
-
-    OSL_ENSURE( pViewData && pDoc, "ViewData or Document not found!" );
+    assert(pViewData && "CreateScSubTotalDlg aArgSet must contain a ScSubTotalItem with ViewData set");
+    pDoc = &pViewData->GetDocument();
+    assert(pDoc && "Document not found!");
 
     m_xBtnSort->connect_clicked( LINK( this, ScTpSubTotalOptions, CheckHdl ) );
     m_xBtnUserDef->connect_clicked( LINK( this, ScTpSubTotalOptions, CheckHdl ) );