diff options
author | Eike Rathke <erack@redhat.com> | 2016-09-13 15:26:09 +0200 |
---|---|---|
committer | Eike Rathke <erack@redhat.com> | 2016-09-13 15:29:08 +0200 |
commit | 4ef10fce39575ec0bd3793b5fdf731c0b9af25a3 (patch) | |
tree | bc6cadd450d233c8f33ddf9580a5f9b89cc8f3f0 /include/formula | |
parent | fix out of bounds error (diff) | |
download | core-4ef10fce39575ec0bd3793b5fdf731c0b9af25a3.tar.gz core-4ef10fce39575ec0bd3793b5fdf731c0b9af25a3.zip |
recognize NaN with no bits set in lower word as error
Which can happen for example for -nan(0x8000000000000) as a result of
calculating with -inf. This was displayed as NaN instead of a proper error
value, now #NUM!
Example test case:
=FORECAST.ETS.ADD(50, {-1,-2,-3,-4}, {10,20,30,40})
Change-Id: I1e1d95e1f188e0036b72be37dd20039c9a9a13f6
Diffstat (limited to 'include/formula')
-rw-r--r-- | include/formula/errorcodes.hxx | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/include/formula/errorcodes.hxx b/include/formula/errorcodes.hxx index 115741504082..e4732b3aed24 100644 --- a/include/formula/errorcodes.hxx +++ b/include/formula/errorcodes.hxx @@ -105,11 +105,14 @@ inline sal_uInt16 GetDoubleErrorValue( double fVal ) return 0; if ( ::rtl::math::isInf( fVal ) ) return errIllegalFPOperation; // normal INF - sal_uInt32 nErr = reinterpret_cast< sal_math_Double * >( - &fVal)->nan_parts.fraction_lo; + sal_uInt32 nErr = reinterpret_cast< sal_math_Double * >( &fVal)->nan_parts.fraction_lo; if ( nErr & 0xffff0000 ) return errNoValue; // just a normal NAN - return (sal_uInt16)(nErr & 0x0000ffff); // any other error + if (!nErr) + // Another NAN, e.g. -nan(0x8000000000000) from calculating with -inf + return errIllegalFPOperation; + // Any other error known to us as error code. + return (sal_uInt16)(nErr & 0x0000ffff); } } // namespace formula |