diff options
author | Caolán McNamara <caolanm@redhat.com> | 2020-09-23 12:09:41 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2020-09-24 11:02:05 +0200 |
commit | 2346b09d19d98992e2dae826d7576cfe01538461 (patch) | |
tree | 04857a9ef28258c06c49cf42249d0d51a43ff245 /sc/source/ui/app | |
parent | 87f6ca70a7c1ce94d6a5a32789ab430f6e604db6 (diff) |
ScCellFormat::GetString never called with a null ScDocument*
and similar, allowing a few redundant null checks to be dropped
Change-Id: Ice0d6d57df112a09d3e1af3d34126b3519529daf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103278
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'sc/source/ui/app')
-rw-r--r-- | sc/source/ui/app/scmod.cxx | 2 | ||||
-rw-r--r-- | sc/source/ui/app/transobj.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/sc/source/ui/app/scmod.cxx b/sc/source/ui/app/scmod.cxx index 06196ffd0b34..7f272e131192 100644 --- a/sc/source/ui/app/scmod.cxx +++ b/sc/source/ui/app/scmod.cxx @@ -1230,7 +1230,7 @@ void ScModule::ModifyOptions( const SfxItemSet& rOptSet ) if ( pViewSh ) pViewSh->UpdateCharts( true ); else - ScDBFunc::DoUpdateCharts( ScAddress(), pDoc, true ); + ScDBFunc::DoUpdateCharts( ScAddress(), *pDoc, true ); if (pBindings) pBindings->Invalidate( SID_ATTR_SIZE ); //SvxPosSize StatusControl Update } diff --git a/sc/source/ui/app/transobj.cxx b/sc/source/ui/app/transobj.cxx index 5de58eb6dd00..dae7c230aade 100644 --- a/sc/source/ui/app/transobj.cxx +++ b/sc/source/ui/app/transobj.cxx @@ -319,7 +319,7 @@ bool ScTransferObj::GetData( const datatransfer::DataFlavor& rFlavor, const OUSt sal_uInt32 nNumFmt = pPattern->GetNumberFormat(pFormatter); OUString aText; const Color* pColor; - ScCellFormat::GetString(aCell, nNumFmt, aText, &pColor, *pFormatter, m_pDoc.get()); + ScCellFormat::GetString(aCell, nNumFmt, aText, &pColor, *pFormatter, *m_pDoc); if (!aText.isEmpty()) aEngine.SetTextCurrentDefaults(aText); } |