[Libreoffice-commits] core.git: chart2/source

Laurent Balland-Poirier laurent.balland-poirier at laposte.net
Thu Aug 13 02:50:21 PDT 2015


 chart2/source/view/main/VDataSeries.cxx |    8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

New commits:
commit b3abc191e24bd2e812836257024d14557a6238ce
Author: Laurent Balland-Poirier <laurent.balland-poirier at laposte.net>
Date:   Wed Jun 17 15:15:07 2015 +0200

    tdf#76649 Skip NaN initial values for min and max
    
    min and max were initiated as aValuesX[0] which could be NaN
    
    Change-Id: I229f4c8f8fda54684e1c817ea7da06fd87eb79b9
    Reviewed-on: https://gerrit.libreoffice.org/17343
    Reviewed-by: Jan Holesovsky <kendy at collabora.com>
    Tested-by: Jan Holesovsky <kendy at collabora.com>

diff --git a/chart2/source/view/main/VDataSeries.cxx b/chart2/source/view/main/VDataSeries.cxx
index 3efa477..6667cbb 100644
--- a/chart2/source/view/main/VDataSeries.cxx
+++ b/chart2/source/view/main/VDataSeries.cxx
@@ -520,9 +520,13 @@ void VDataSeries::getMinMaxXValue(double& fMin, double& fMax) const
 
     if(aValuesX.getLength() > 0)
     {
-        fMax = fMin = aValuesX[0];
+        sal_Int32 i = 0;
+        while ( ::rtl::math::isNan( aValuesX[i] ) && i < aValuesX.getLength() )
+            i++;
+        if ( i < aValuesX.getLength() )
+            fMax = fMin = aValuesX[i++];
 
-        for (sal_Int32 i = 1; i < aValuesX.getLength(); i++)
+        for ( ; i < aValuesX.getLength(); i++)
         {
             const double aValue = aValuesX[i];
             if ( aValue > fMax)


More information about the Libreoffice-commits mailing list