[Libreoffice-commits] core.git: Branch 'libreoffice-5-1' - sax/source

Caolán McNamara caolanm at redhat.com
Fri Feb 5 14:53:18 UTC 2016


 sax/source/tools/converter.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 3d1638e9deded1b10a38720ef61077ebb8881d47
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Wed Feb 3 10:30:43 2016 +0000

    fix gcc build error
    
    error: invalid operands of types '__gnu_cxx::__enable_if<true, double>::__type
    {aka double}' and 'int' to binary 'operator%'
    
    apparently there's a template returning double for abs for the
    non-int/long/float/double argument case. So promote earlier to int so the
    abs<int> is called
    
    Change-Id: I882a27c5ec349f894c1c9f4857687360a46b55ae
    (cherry picked from commit b9fa2963cf12e6987b3a0acc219c4fa591e41be8)
    Reviewed-on: https://gerrit.libreoffice.org/22071
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sax/source/tools/converter.cxx b/sax/source/tools/converter.cxx
index b4eb276..3e2cbc9 100644
--- a/sax/source/tools/converter.cxx
+++ b/sax/source/tools/converter.cxx
@@ -1250,7 +1250,7 @@ bool Converter::convertDuration(util::Duration& rDuration,
 
 
 static void
-lcl_AppendTimezone(OUStringBuffer & i_rBuffer, sal_Int16 const nOffset)
+lcl_AppendTimezone(OUStringBuffer & i_rBuffer, int const nOffset)
 {
     if (0 == nOffset)
     {
@@ -1435,7 +1435,7 @@ lcl_MaxDaysPerMonth(const sal_Int32 nMonth, const sal_Int32 nYear)
 static void lcl_ConvertToUTC(
         sal_Int16 & o_rYear, sal_uInt16 & o_rMonth, sal_uInt16 & o_rDay,
         sal_uInt16 & o_rHours, sal_uInt16 & o_rMinutes,
-        sal_Int16 const nSourceOffset)
+        int const nSourceOffset)
 {
     sal_Int16 nOffsetHours(abs(nSourceOffset) / 60);
     sal_Int16 const nOffsetMinutes(abs(nSourceOffset) % 60);


More information about the Libreoffice-commits mailing list