Changeset 231864 in webkit

Timestamp:

May 16, 2018 12:46:22 PM (6 years ago)

Author:

Yusuke Suzuki

Message:

Unreviewed, follow-up after r231762
​https://bugs.webkit.org/show_bug.cgi?id=185589

Addresses darin's suggestions to improve timeClip.

• wtf/DateMath.cpp:

(WTF::timeClip):

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/DateMath.cpp (modified) (1 diff)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2018-05-16  Yusuke Suzuki  <utatane.tea@gmail.com>
+
+        Unreviewed, follow-up after r231762
+        https://bugs.webkit.org/show_bug.cgi?id=185589
+
+        Addresses darin's suggestions to improve timeClip.
+
+        * wtf/DateMath.cpp:
+        (WTF::timeClip):
+
 2018-05-14  Yusuke Suzuki  <utatane.tea@gmail.com>
 

trunk/Source/WTF/wtf/DateMath.cpp

@@ -1156 +1156 @@
 double timeClip(double t)
 {
-    if (!std::isfinite(t))
-        return std::numeric_limits<double>::quiet_NaN();
-    if (fabs(t) > maxECMAScriptTime)
-        return std::numeric_limits<double>::quiet_NaN();
-    return trunc(t) + 0.0;
+    if (std::abs(t) > maxECMAScriptTime)
+        return std::numeric_limits<double>::quiet_NaN();
+    return std::trunc(t) + 0.0;
 }