Avoid second null-date check of convertDateToDaysNormalizing()

This is micro-optimization, but..

Change-Id: Ic5d43bc0ce9343cf194e189777109636d3b587c4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/143126
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins

1 files changed, 7 insertions, 1 deletions

diff --git a/tools/source/datetime/tdate.cxx b/tools/source/datetime/tdate.cxx
index 6c33dca13558..4b135fc3008a 100644
--- a/tools/source/datetime/tdate.cxx
+++ b/tools/source/datetime/tdate.cxx
@@ -75,7 +75,13 @@ sal_Int32 Date::GetAsNormalizedDays() const
         assert(DateToDays( GetDay(), GetMonth(), GetYear() ) == 693594);
         return 693594;
     }
-    return DateToDays( GetDay(), GetMonth(), GetYear() );
+    // Not calling comphelper::date::convertDateToDaysNormalizing() here just
+    // avoids a second check on null-date handling like above.
+    sal_uInt16 nDay = GetDay();
+    sal_uInt16 nMonth = GetMonth();
+    sal_Int16 nYear = GetYear();
+    comphelper::date::normalize( nDay, nMonth, nYear);
+    return comphelper::date::convertDateToDays( nDay, nMonth, nYear);
 }
 
 sal_Int32 Date::DateToDays( sal_uInt16 nDay, sal_uInt16 nMonth, sal_Int16 nYear )