Changeset 43121 in webkit for trunk/JavaScriptCore/runtime/JSValue.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/JSValue.h (modified) (3 diffs)

trunk/JavaScriptCore/runtime/JSValue.h

@@ -62 +62 @@
         
     public:
-        JSValuePtr()
-            : m_ptr(0)
-        {
-        }
-
-        JSValuePtr(JSCell* ptr)
-            : m_ptr(ptr)
-        {
-        }
-
-        JSValuePtr(const JSCell* ptr)
-            : m_ptr(const_cast<JSCell*>(ptr))
-        {
-        }
-
-        operator bool() const
-        {
-            return m_ptr;
-        }
-
-        bool operator==(const JSValuePtr other) const
-        {
-            return m_ptr == other.m_ptr;
-        }
-
-        bool operator!=(const JSValuePtr other) const
-        {
-            return m_ptr != other.m_ptr;
-        }
-
-        static EncodedJSValuePtr encode(JSValuePtr value)
-        {
-            return reinterpret_cast<EncodedJSValuePtr>(value.m_ptr);
-        }
-
-        static JSValuePtr decode(EncodedJSValuePtr ptr)
-        {
-            return JSValuePtr(reinterpret_cast<JSCell*>(ptr));
-        }
+        enum ImpossibleValueTag { ImpossibleValue };
+        enum JSNullTag { JSNull };
+        enum JSUndefinedTag { JSUndefined };
+        enum JSTrueTag { JSTrue };
+        enum JSFalseTag { JSFalse };
+
+        static EncodedJSValuePtr encode(JSValuePtr value);
+        static JSValuePtr decode(EncodedJSValuePtr ptr);
+
+        JSValuePtr();
+        JSValuePtr(ImpossibleValueTag);
+        JSValuePtr(JSNullTag);
+        JSValuePtr(JSUndefinedTag);
+        JSValuePtr(JSTrueTag);
+        JSValuePtr(JSFalseTag);
+        JSValuePtr(JSCell* ptr);
+        JSValuePtr(const JSCell* ptr);
+
+        operator bool() const;
+        bool operator==(const JSValuePtr other) const;
+        bool operator!=(const JSValuePtr other) const;
 
         // Querying the type.
@@ -207 +189 @@
     };
 
+    // Stand-alone helper functions.
     inline JSValuePtr noValue()
     {
         return JSValuePtr();
+    }
+
+    inline JSValuePtr jsImpossibleValue()
+    {
+        return JSValuePtr(JSValuePtr::ImpossibleValue);
+    }
+
+    inline JSValuePtr jsNull()
+    {
+        return JSValuePtr(JSValuePtr::JSNull);
+    }
+
+    inline JSValuePtr jsUndefined()
+    {
+        return JSValuePtr(JSValuePtr::JSUndefined);
+    }
+
+    inline JSValuePtr jsBoolean(bool b)
+    {
+        return b ? JSValuePtr(JSValuePtr::JSTrue) : JSValuePtr(JSValuePtr::JSFalse);
     }
 
@@ -218 +221 @@
     inline bool operator!=(const JSCell* a, const JSValuePtr b) { return JSValuePtr(a) != b; }
 
+    // JSValuePtr member functions.
+    inline EncodedJSValuePtr JSValuePtr::encode(JSValuePtr value)
+    {
+        return reinterpret_cast<EncodedJSValuePtr>(value.m_ptr);
+    }
+
+    inline JSValuePtr JSValuePtr::decode(EncodedJSValuePtr ptr)
+    {
+        return JSValuePtr(reinterpret_cast<JSCell*>(ptr));
+    }
+
+    inline JSValuePtr::JSValuePtr()
+        : m_ptr(0)
+    {
+    }
+
+    inline JSValuePtr::JSValuePtr(JSCell* ptr)
+        : m_ptr(ptr)
+    {
+    }
+
+    inline JSValuePtr::JSValuePtr(const JSCell* ptr)
+        : m_ptr(const_cast<JSCell*>(ptr))
+    {
+    }
+
+    inline JSValuePtr::operator bool() const
+    {
+        return m_ptr;
+    }
+
+    inline bool JSValuePtr::operator==(const JSValuePtr other) const
+    {
+        return m_ptr == other.m_ptr;
+    }
+
+    inline bool JSValuePtr::operator!=(const JSValuePtr other) const
+    {
+        return m_ptr != other.m_ptr;
+    }
+
+    inline bool JSValuePtr::isUndefined() const
+    {
+        return asValue() == jsUndefined();
+    }
+
+    inline bool JSValuePtr::isNull() const
+    {
+        return asValue() == jsNull();
+    }
+
 } // namespace JSC