Changeset 52292 in webkit

Timestamp:

Dec 17, 2009 2:37:17 PM (14 years ago)

Author:

eric@webkit.org

Message:

2009-12-17 Steve Block <​steveblock@google.com>

Reviewed by Eric Seidel.

Fixes HTMLInputElement::stepMismatch() to avoid ambiguous overload of pow().
​https://bugs.webkit.org/show_bug.cgi?id=32675

Build fix only, no new tests.

• html/HTMLInputElement.cpp: Modified. (WebCore::HTMLInputElement::stepMismatch): Cast first argument of pow to double to force 'double pow(double, int)'

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/HTMLInputElement.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-12-17  Steve Block  <steveblock@google.com>
+
+        Reviewed by Eric Seidel.
+
+        Fixes HTMLInputElement::stepMismatch() to avoid ambiguous overload of pow().
+        https://bugs.webkit.org/show_bug.cgi?id=32675
+
+        Build fix only, no new tests.
+
+        * html/HTMLInputElement.cpp: Modified.
+        (WebCore::HTMLInputElement::stepMismatch): Cast first argument of pow to double to force 'double pow(double, int)'
+
 2009-12-17  Marc-Antoine Ruel  <maruel@chromium.org>
 

trunk/WebCore/html/HTMLInputElement.cpp

@@ -338 +338 @@
         // value is greater than step*2^DBL_MANT_DIG, the following fmod() makes
         // no sense.
-        if (doubleValue / pow(2, DBL_MANT_DIG) > step)
+        if (doubleValue / pow(static_cast<double>(2), DBL_MANT_DIG) > step)
             return false;
         double remainder = fmod(doubleValue, step);
         // Accepts errors in lower 7-bit.
-        double acceptableError = step / pow(2, DBL_MANT_DIG - 7);
+        double acceptableError = step / pow(static_cast<double>(2), DBL_MANT_DIG - 7);
         return acceptableError < remainder && remainder < (step - acceptableError);
     }