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

dante (via logerrit) logerrit at kemper.freedesktop.org
Sat May 8 07:03:41 UTC 2021


 sc/source/core/tool/interpr2.cxx |    2 +-
 sc/source/core/tool/interpr3.cxx |    4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit e5750bacf680177447134ca6409967ed4b00e434
Author:     dante <dante19031999 at gmail.com>
AuthorDate: Fri May 7 13:19:51 2021 +0200
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Sat May 8 09:03:05 2021 +0200

    Review of the code affected by KahanSum
    
    Change-Id: I96fd479eeae287ef5c7a8615856631d4ba9b02f5
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115238
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/sc/source/core/tool/interpr2.cxx b/sc/source/core/tool/interpr2.cxx
index 8f2e4ed9a257..dc30df4be07e 100644
--- a/sc/source/core/tool/interpr2.cxx
+++ b/sc/source/core/tool/interpr2.cxx
@@ -1735,7 +1735,7 @@ void ScInterpreter::ScDB()
             fSumOffRate += fDb;
         }
         if (fPeriod > fLife)
-            fDb = (-(fSumOffRate - fCost).get() * fOffRate * (12.0 - fMonths)) / 12.0;
+            fDb = -(fSumOffRate - fCost).get() * fOffRate * (12.0 - fMonths) / 12.0;
     }
     PushDouble(fDb);
 }
diff --git a/sc/source/core/tool/interpr3.cxx b/sc/source/core/tool/interpr3.cxx
index bc4265be0b67..3354488a88e4 100644
--- a/sc/source/core/tool/interpr3.cxx
+++ b/sc/source/core/tool/interpr3.cxx
@@ -4609,8 +4609,8 @@ void ScInterpreter::CalculatePearsonCovar( bool _bPearson, bool _bStexy, bool _b
                 || (!_bStexy && fSumSqrDeltaY < ::std::numeric_limits<double>::min()))
                 PushError( FormulaError::DivisionByZero);
             else if ( _bStexy )
-                PushDouble( sqrt( ( fSumSqrDeltaY - fSumDeltaXDeltaY.get() *
-                            fSumDeltaXDeltaY.get() / fSumSqrDeltaX.get() ).get() / (fCount-2)));
+                PushDouble( sqrt( ( fSumSqrDeltaY - fSumDeltaXDeltaY *
+                            fSumDeltaXDeltaY / fSumSqrDeltaX ).get() / (fCount-2)));
             else
                 PushDouble( fSumDeltaXDeltaY.get() / sqrt( fSumSqrDeltaX.get() * fSumSqrDeltaY.get() ));
         }


More information about the Libreoffice-commits mailing list