diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2017-05-16 23:50:46 +0200 |
|---|---|---|
| committer | Andras Timar <andras.timar@collabora.com> | 2017-05-22 10:00:16 +0200 |
| commit | b547e4f32c63c066f752e86f73deae9e13c38c3f (patch) | |
| tree | 8c32c0e182024df18369d65c3894cb682c50d5ea /sc | |
| parent | 47fd47bf8c206cf9d60f9b872f2224a680e887e1 (diff) | |
tdf#107892: fix negative values case with maxifs
std::numeric_limits<double>::min() returns positive value for float/double
so use std::numeric_limits<double>::lowest()
(see http://stackoverflow.com/questions/17070351/why-does-numeric-limitsmin-return-a-negative-value-for-int-but-positive-values)
Change-Id: I0afce2d38c6936aeff26923182bcafd2f0008d9e
Reviewed-on: https://gerrit.libreoffice.org/37693
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
(cherry picked from commit 20d3a60b41bda2513723c145d919f584840b1056)
Reviewed-on: https://gerrit.libreoffice.org/37730
(cherry picked from commit f6d7bd1f525d3930c48b61abb2e91418f2c66ca5)
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/core/inc/interpre.hxx | 2 | ||||
| -rw-r--r-- | sc/source/core/tool/interpr1.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/sc/source/core/inc/interpre.hxx b/sc/source/core/inc/interpre.hxx index fe860eba7120..636f2ecd62d1 100644 --- a/sc/source/core/inc/interpre.hxx +++ b/sc/source/core/inc/interpre.hxx @@ -69,7 +69,7 @@ struct ParamIfsResult double mfMem = 0.0; double mfCount = 0.0; double mfMin = std::numeric_limits<double>::max(); - double mfMax = std::numeric_limits<double>::min(); + double mfMax = std::numeric_limits<double>::lowest(); }; } diff --git a/sc/source/core/tool/interpr1.cxx b/sc/source/core/tool/interpr1.cxx index 9a02b624df22..f005b26e4635 100644 --- a/sc/source/core/tool/interpr1.cxx +++ b/sc/source/core/tool/interpr1.cxx @@ -5780,7 +5780,7 @@ void ScInterpreter::ScMaxIfs_MS() sc::ParamIfsResult aRes; IterateParametersIfs(aRes); - PushDouble((aRes.mfMax > std::numeric_limits<double>::min()) ? aRes.mfMax : 0.0); + PushDouble((aRes.mfMax > std::numeric_limits<double>::lowest()) ? aRes.mfMax : 0.0); } void ScInterpreter::ScLookup() |