Changeset 177736 in webkit

Timestamp:

Dec 25, 2014, 10:48:41 AM (10 years ago)

Author:

ap@apple.com

Message:

Simplify saturated integer add/sub
​https://bugs.webkit.org/show_bug.cgi?id=139854

Address Darin's post-commit feedback to fix API tests.

• wtf/SaturatedArithmetic.h: (saturatedSubtraction):

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/SaturatedArithmetic.h (modified) (1 diff)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2014-12-25  Alexey Proskuryakov  <ap@apple.com>
+
+        Simplify saturated integer add/sub
+        https://bugs.webkit.org/show_bug.cgi?id=139854
+
+        Address Darin's post-commit feedback to fix API tests.
+
+        * wtf/SaturatedArithmetic.h: (saturatedSubtraction):
+
 2014-12-24  Benjamin Poulain  <benjamin@webkit.org>
 

trunk/Source/WTF/wtf/SaturatedArithmetic.h

@@ -97 +97 @@
 #else
     if (signedSubtractOverflows(a, b, result))
-        result = std::numeric_limits<uint32_t>::max() + (static_cast<uint32_t>(a) >> 31);
+        result = std::numeric_limits<int32_t>::max() + (static_cast<uint32_t>(a) >> 31);
 #endif
     return result;