Changeset 26899 in webkit for trunk/JavaScriptCore/kjs/value.h

Timestamp:

Oct 22, 2007, 4:36:36 PM (18 years ago)

Author:

darin

Message:

JavaScriptCore:

Reviewed by Geoff.

• fix ​http://bugs.webkit.org/show_bug.cgi?id=15632 js1_5/Array/array-001.js test failing

One of the JavaScriptCore tests was failing; it failed because of
my change to NumberImp::getUInt32. The incorrect code I copied was
from JSImmediate::getUInt32, and was a pre-existing bug.

This patch fixes correctness, but will surely slow down SunSpider.
We may be able to code this tighter and get the speed back.

• kjs/JSImmediate.h: (KJS::JSImmediate::getInt32): Renamed from toInt32 to more accurately reflect the fact that this function only returns true if the value is accurate (no fractional part, etc.). Changed code so that it returns false when the value has a fraction. (KJS::JSImmediate::getUInt32): Ditto.

• kjs/internal.cpp: (KJS::NumberImp::getInt32): Changed code so that it returns false when the value has a fraction. Restores the old behavior. (KJS::NumberImp::getUInt32): Ditto.

• kjs/value.h: (KJS::JSValue::getInt32): Updated for name change. (KJS::JSValue::getUInt32): Ditto. (KJS::JSValue::toInt32): Ditto. (KJS::JSValue::toUInt32): Ditto.

LayoutTests:

Reviewed by Geoff.

• tests for ​http://bugs.webkit.org/show_bug.cgi?id=15632

Added tests for cases where you use something that looks like an array
index, but it has a fractional part.

• fast/js/kde/resources/Array.js: Added tests.
• fast/js/kde/Array-expected.txt: Updated.

File:

• trunk/JavaScriptCore/kjs/value.h (modified) (5 diffs)

trunk/JavaScriptCore/kjs/value.h

@@ -335 +335 @@
 inline bool JSValue::getInt32(int32_t& v) const
 {
-    return JSImmediate::isImmediate(this) ? JSImmediate::toInt32(this, v) : asCell()->getInt32(v);
+    return JSImmediate::isImmediate(this) ? JSImmediate::getInt32(this, v) : asCell()->getInt32(v);
 }
 
 inline bool JSValue::getUInt32(uint32_t& v) const
 {
-    return JSImmediate::isImmediate(this) ? JSImmediate::toUInt32(this, v) : asCell()->getUInt32(v);
+    return JSImmediate::isImmediate(this) ? JSImmediate::getUInt32(this, v) : asCell()->getUInt32(v);
 }
 
@@ -387 +387 @@
 {
     int32_t i;
-    if (JSImmediate::isImmediate(this) && JSImmediate::toInt32(this, i))
+    if (JSImmediate::isImmediate(this) && JSImmediate::getInt32(this, i))
         return i;
     bool ok;
@@ -396 +396 @@
 {
     uint32_t i;
-    if (JSImmediate::isImmediate(this) && JSImmediate::toUInt32(this, i))
+    if (JSImmediate::isImmediate(this) && JSImmediate::getUInt32(this, i))
         return i;
     bool ok;
@@ -405 +405 @@
 {
     int32_t i;
-    if (JSImmediate::isImmediate(this) && JSImmediate::toInt32(this, i)) {
+    if (JSImmediate::isImmediate(this) && JSImmediate::getInt32(this, i)) {
         ok = true;
         return i;
@@ -415 +415 @@
 {
     uint32_t i;
-    if (JSImmediate::isImmediate(this) && JSImmediate::toUInt32(this, i)) {
+    if (JSImmediate::isImmediate(this) && JSImmediate::getUInt32(this, i)) {
         ok = true;
         return i;