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/inc/bigrange.hxx | |
| 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/inc/bigrange.hxx')
| -rw-r--r-- | sc/inc/bigrange.hxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sc/inc/bigrange.hxx b/sc/inc/bigrange.hxx index 136c6e054c5a..056c0d4ae304 100644 --- a/sc/inc/bigrange.hxx +++ b/sc/inc/bigrange.hxx @@ -60,7 +60,7 @@ public: void GetVars( sal_Int32& nColP, sal_Int32& nRowP, sal_Int32& nTabP ) const { nColP = nCol; nRowP = nRow; nTabP = nTab; } - bool IsValid( const ScDocument* pDoc ) const; + bool IsValid( const ScDocument& rDoc ) const; inline ScAddress MakeAddress() const; ScBigAddress& operator=( const ScBigAddress& r ) @@ -132,8 +132,8 @@ public: { aStart.GetVars( nCol1, nRow1, nTab1 ); aEnd.GetVars( nCol2, nRow2, nTab2 ); } - bool IsValid( const ScDocument* pDoc ) const - { return aStart.IsValid( pDoc ) && aEnd.IsValid( pDoc ); } + bool IsValid( const ScDocument& rDoc ) const + { return aStart.IsValid( rDoc ) && aEnd.IsValid( rDoc ); } ScRange MakeRange() const { return ScRange( aStart.MakeAddress(), aEnd.MakeAddress() ); } |