Compile fix: use fabs() instead of abs() for doubles
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Aug 2012 10:42:09 +0000 (10:42 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Aug 2012 10:42:09 +0000 (10:42 +0000)
https://bugs.webkit.org/show_bug.cgi?id=95184

Patch by Kevin Funk <kevin.funk@kdab.com> on 2012-08-28
Reviewed by Kentaro Hara.

Simple compilation fix.

* platform/ScrollAnimatorNone.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@126863 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/ScrollAnimatorNone.cpp

index ad7333d..0d41f5a 100644 (file)
@@ -1,3 +1,14 @@
+2012-08-28  Kevin Funk  <kevin.funk@kdab.com>
+
+        Compile fix: use fabs() instead of abs() for doubles
+        https://bugs.webkit.org/show_bug.cgi?id=95184
+
+        Reviewed by Kentaro Hara.
+
+        Simple compilation fix.
+
+        * platform/ScrollAnimatorNone.cpp:
+
 2012-08-28  Christophe Dumez  <christophe.dumez@intel.com>
 
         REGRESSION (r124512): Failures in MathML Presentation tests on GTK and EFL
index 34a6f77..6d1f3df 100644 (file)
@@ -299,9 +299,9 @@ bool ScrollAnimatorNone::PerAxisData::updateDataFromParameters(float step, float
         // This needs to be as minimal as possible while not being intrusive to page up/down.
         double minCoastDelta = m_visibleLength;
 
-        if (abs(remainingDelta) > minCoastDelta) {
+        if (fabs(remainingDelta) > minCoastDelta) {
             double maxCoastDelta = parameters->m_maximumCoastTime * targetMaxCoastVelocity;
-            double coastFactor = min(1., (abs(remainingDelta) - minCoastDelta) / (maxCoastDelta - minCoastDelta));
+            double coastFactor = min(1., (fabs(remainingDelta) - minCoastDelta) / (maxCoastDelta - minCoastDelta));
 
             // We could play with the curve here - linear seems a little soft. Initial testing makes me want to feed into the sustain time more aggressively.
             double coastMinTimeLeft = min(parameters->m_maximumCoastTime, minTimeLeft + coastCurve(parameters->m_coastTimeCurve, coastFactor) * (parameters->m_maximumCoastTime - minTimeLeft));