[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