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

Caolán McNamara caolanm at redhat.com
Thu Jul 28 07:36:49 UTC 2016


 sc/source/core/tool/interpr1.cxx |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit e043cce9a48c1e4f4dd232a58de64f03d1d3919f
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Thu Jul 28 08:31:31 2016 +0100

    use div() instead of /0 to content ubsan
    
    for the same reason as...
    
    commit fb2c146752074b132d665e40343a08dcb2e6672e
    Author: Eike Rathke <erack at redhat.com>
    Date:   Mon Jul 25 13:32:17 2016 +0200
    
        use div() instead of /0 to content ubsan
    
        While we do handle double floating point division by 0 at least on
        IEEE754 systems, ubsan builds mock about.
    
    Change-Id: I4cc696e3725beaeb75066e02166bf7bbfdedfde5

diff --git a/sc/source/core/tool/interpr1.cxx b/sc/source/core/tool/interpr1.cxx
index 37afb35..32b17f9 100644
--- a/sc/source/core/tool/interpr1.cxx
+++ b/sc/source/core/tool/interpr1.cxx
@@ -1772,22 +1772,22 @@ void ScInterpreter::ScArcCotHyp()
 
 void ScInterpreter::ScCosecant()
 {
-    PushDouble(1.0 / ::rtl::math::sin(GetDouble()));
+    PushDouble(div(1.0, ::rtl::math::sin(GetDouble())));
 }
 
 void ScInterpreter::ScSecant()
 {
-    PushDouble(1.0 / ::rtl::math::cos(GetDouble()));
+    PushDouble(div(1.0, ::rtl::math::cos(GetDouble())));
 }
 
 void ScInterpreter::ScCosecantHyp()
 {
-    PushDouble(1.0 / sinh(GetDouble()));
+    PushDouble(div(1.0, sinh(GetDouble())));
 }
 
 void ScInterpreter::ScSecantHyp()
 {
-    PushDouble(1.0 / cosh(GetDouble()));
+    PushDouble(div(1.0, cosh(GetDouble())));
 }
 
 void ScInterpreter::ScExp()


More information about the Libreoffice-commits mailing list