diff options
| author | Eike Rathke <erack@redhat.com> | 2017-07-18 19:43:05 +0200 |
|---|---|---|
| committer | Eike Rathke <erack@redhat.com> | 2017-07-18 19:43:35 +0200 |
| commit | 038063a352f59bad1ecdeeb63016e37f6945a420 (patch) | |
| tree | b64769d58f9bc001dbf235f420038ba436344a33 /editeng | |
| parent | 71c3113e9dbf61693fd947262370cc61cdf31ac7 (diff) | |
Change nonsense non-const Date* GetNullDate() to const Date&
* first, a non-const Date* may leave the impression that one could change the
NullDate through the pointer, which is only partly successful; luckily no one
did that
* second, there is always a NullDate so checking for nullptr is superfluous
* third, the pointer was dereferenced (maybe after a check) everywhere to
obtain the NullDate, luckily..
Change-Id: I3c3a788ba0336596ac6bde4c96e77a0cdb7a4a95
Diffstat (limited to 'editeng')
| -rw-r--r-- | editeng/source/items/flditem.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/editeng/source/items/flditem.cxx b/editeng/source/items/flditem.cxx index cb2c69f3abb1..deff10eb19b6 100644 --- a/editeng/source/items/flditem.cxx +++ b/editeng/source/items/flditem.cxx @@ -479,7 +479,7 @@ OUString SvxDateField::GetFormatted( Date& aDate, SvxDateFormat eFormat, SvNumbe nFormatKey = rFormatter.GetStandardFormat( css::util::NumberFormat::DATE, eLang ); } - double fDiffDate = aDate - *(rFormatter.GetNullDate()); + double fDiffDate = aDate - rFormatter.GetNullDate(); OUString aStr; Color* pColor = nullptr; rFormatter.GetOutputString( fDiffDate, nFormatKey, aStr, &pColor ); |