Changeset 43121 in webkit for trunk/JavaScriptCore/runtime/JSImmediate.h

Timestamp:

May 1, 2009, 2:20:11 PM (16 years ago)

Author:

[email protected]

Message:

2009-05-01 Geoffrey Garen <​[email protected]>

Reviewed by Sam "That doesn't look like what I thought it looks like" Weinig.

Beefed up the JSValuePtr class and removed some non-JSValuePtr dependencies
on JSImmediate, in prepapration for making JSImmediate an implementation
detail of JSValuePtr.

SunSpider reports no change.

• interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute):
• jit/JIT.cpp: (JSC::JIT::privateCompileMainPass):
• jit/JITArithmetic.cpp: (JSC::JIT::compileFastArith_op_mod):
• runtime/JSGlobalObjectFunctions.cpp: (JSC::globalFuncParseInt): Updated for interface changes.

• runtime/JSImmediate.h: (JSC::JSValuePtr::JSValuePtr):
• runtime/JSValue.h: (JSC::JSValuePtr::): (JSC::jsImpossibleValue): (JSC::jsNull): (JSC::jsUndefined): (JSC::jsBoolean): (JSC::JSValuePtr::encode): (JSC::JSValuePtr::decode): (JSC::JSValuePtr::JSValuePtr): (JSC::JSValuePtr::operator bool): (JSC::JSValuePtr::operator==): (JSC::JSValuePtr::operator!=): (JSC::JSValuePtr::isUndefined): (JSC::JSValuePtr::isNull): Changed jsImpossibleValue(), jsNull(), jsUndefined(), and jsBoolean() to operate in terms of JSValuePtr instead of JSImmediate.

• wtf/StdLibExtras.h: (WTF::bitwise_cast): Fixed up for clarity.

File:

• trunk/JavaScriptCore/runtime/JSImmediate.h (modified) (4 diffs)

trunk/JavaScriptCore/runtime/JSImmediate.h

@@ -43 +43 @@
     class UString;
 
-    JSValuePtr js0();
-    JSValuePtr jsNull();
-    JSValuePtr jsBoolean(bool b);
-    JSValuePtr jsUndefined();
-    JSValuePtr jsImpossibleValue();
     JSValuePtr jsNumber(ExecState* exec, double d);
     JSValuePtr jsNumber(ExecState*, char i);
@@ -161 +156 @@
         friend class JSValuePtr;
         friend class JSFastMath;
-        friend JSValuePtr js0();
-        friend JSValuePtr jsNull();
-        friend JSValuePtr jsBoolean(bool b);
-        friend JSValuePtr jsUndefined();
-        friend JSValuePtr jsImpossibleValue();
         friend JSValuePtr jsNumber(ExecState* exec, double d);
         friend JSValuePtr jsNumber(ExecState*, char i);
@@ -578 +568 @@
     }
 
-    inline JSValuePtr js0()
-    {
-        return JSImmediate::zeroImmediate();
-    }
-
-    inline JSValuePtr jsNull()
-    {
-        return JSImmediate::nullImmediate();
-    }
-
-    inline JSValuePtr jsBoolean(bool b)
-    {
-        return b ? JSImmediate::trueImmediate() : JSImmediate::falseImmediate();
-    }
-
-    inline JSValuePtr jsUndefined()
-    {
-        return JSImmediate::undefinedImmediate();
-    }
-
-    inline JSValuePtr jsImpossibleValue()
-    {
-        return JSImmediate::impossibleValue();
-    }
-
     // These are identical logic to the JSValue functions above, and faster than jsNumber(number).toInt32().
     int32_t toInt32(double);
@@ -609 +574 @@
     uint32_t toUInt32SlowCase(double, bool& ok);
 
-    inline bool JSValuePtr::isUndefined() const
-    {
-        return asValue() == jsUndefined();
-    }
-
-    inline bool JSValuePtr::isNull() const
-    {
-        return asValue() == jsNull();
+    inline JSValuePtr::JSValuePtr(ImpossibleValueTag)
+    {
+        *this = JSImmediate::impossibleValue();
+    }
+
+    inline JSValuePtr::JSValuePtr(JSNullTag)
+    {
+        *this = JSImmediate::nullImmediate();
+    }
+    
+    inline JSValuePtr::JSValuePtr(JSUndefinedTag)
+    {
+        *this = JSImmediate::undefinedImmediate();
+    }
+
+    inline JSValuePtr::JSValuePtr(JSTrueTag)
+    {
+        *this = JSImmediate::trueImmediate();
+    }
+
+    inline JSValuePtr::JSValuePtr(JSFalseTag)
+    {
+        *this = JSImmediate::falseImmediate();
     }