Jacek Caban : jscript: Get rid of no longer needed VARIANT helpers.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 5fecc71d34e583aa216f3e8eeb1b05fe123bed35
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=5fecc71d34e583aa216f3e8eeb1b05fe123bed35

Author: Jacek Caban <jacek at codeweavers.com>
Date:   Mon Sep 17 15:19:44 2012 +0200

jscript: Get rid of no longer needed VARIANT helpers.

---

 dlls/jscript/jscript.h |   33 ---------------------------------
 dlls/jscript/jsutils.c |   14 ++++++++++++--
 2 files changed, 12 insertions(+), 35 deletions(-)

diff --git a/dlls/jscript/jscript.h b/dlls/jscript/jscript.h
index bbac902..bc67426 100644
--- a/dlls/jscript/jscript.h
+++ b/dlls/jscript/jscript.h
@@ -387,16 +387,6 @@ static inline BOOL is_vclass(vdisp_t *vdisp, jsclass_t class)
     return is_jsdisp(vdisp) && is_class(vdisp->u.jsdisp, class);
 }
 
-static inline BOOL is_num_vt(enum VARENUM vt)
-{
-    return vt == VT_I4 || vt == VT_R8;
-}
-
-static inline DOUBLE num_val(const VARIANT *v)
-{
-    return V_VT(v) == VT_I4 ? V_I4(v) : V_R8(v);
-}
-
 #ifndef INT32_MIN
 #define INT32_MIN (-2147483647-1)
 #endif
@@ -410,29 +400,6 @@ static inline BOOL is_int32(double d)
     return INT32_MIN <= d && d <= INT32_MAX && (double)(int)d == d;
 }
 
-static inline void num_set_int(VARIANT *v, INT i)
-{
-    V_VT(v) = VT_I4;
-    V_I4(v) = i;
-}
-
-static inline void num_set_val(VARIANT *v, DOUBLE d)
-{
-    if(is_int32(d)) {
-        V_VT(v) = VT_I4;
-        V_I4(v) = d;
-    }else {
-        V_VT(v) = VT_R8;
-        V_R8(v) = d;
-    }
-}
-
-static inline void var_set_jsdisp(VARIANT *v, jsdisp_t *jsdisp)
-{
-    V_VT(v) = VT_DISPATCH;
-    V_DISPATCH(v) = to_disp(jsdisp);
-}
-
 static inline DWORD make_grfdex(script_ctx_t *ctx, DWORD flags)
 {
     return (ctx->version << 28) | flags;
diff --git a/dlls/jscript/jsutils.c b/dlls/jscript/jsutils.c
index 1801f86..540d814 100644
--- a/dlls/jscript/jsutils.c
+++ b/dlls/jscript/jsutils.c
@@ -352,9 +352,19 @@ HRESULT jsval_to_variant(jsval_t val, VARIANT *retv)
             V_BSTR(retv) = NULL;
         }
         return S_OK;
-    case JSV_NUMBER:
-        num_set_val(retv, val.u.n);
+    case JSV_NUMBER: {
+        double n = get_number(val);
+
+        if(is_int32(n)) {
+            V_VT(retv) = VT_I4;
+            V_I4(retv) = n;
+        }else {
+            V_VT(retv) = VT_R8;
+            V_R8(retv) = n;
+        }
+
         return S_OK;
+    }
     case JSV_BOOL:
         V_VT(retv) = VT_BOOL;
         V_BOOL(retv) = val.u.b ? VARIANT_TRUE : VARIANT_FALSE;