Jacek Caban : jscript: Code clean up.

Alexandre Julliard julliard at winehq.org
Tue Sep 1 11:05:57 CDT 2009


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

Author: Jacek Caban <jacek at codeweavers.com>
Date:   Tue Sep  1 13:26:34 2009 +0200

jscript: Code clean up.

---

 dlls/jscript/array.c    |    2 +-
 dlls/jscript/function.c |    5 ++---
 dlls/jscript/string.c   |   12 ++++++------
 3 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/dlls/jscript/array.c b/dlls/jscript/array.c
index 04f3dde..1c2b95a 100644
--- a/dlls/jscript/array.c
+++ b/dlls/jscript/array.c
@@ -315,7 +315,7 @@ static HRESULT Array_join(DispatchEx *dispex, LCID lcid, WORD flags, DISPPARAMS
     if(arg_cnt(dp)) {
         BSTR sep;
 
-        hres = to_string(dispex->ctx, dp->rgvarg + dp->cArgs-1, ei, &sep);
+        hres = to_string(dispex->ctx, get_arg(dp,0), ei, &sep);
         if(FAILED(hres))
             return hres;
 
diff --git a/dlls/jscript/function.c b/dlls/jscript/function.c
index a1d0987..772c29a 100644
--- a/dlls/jscript/function.c
+++ b/dlls/jscript/function.c
@@ -70,12 +70,11 @@ static HRESULT init_parameters(DispatchEx *var_disp, FunctionInstance *function,
     HRESULT hres;
 
     V_VT(&var_empty) = VT_EMPTY;
-    cargs = dp->cArgs - dp->cNamedArgs;
+    cargs = arg_cnt(dp);
 
     for(param = function->parameters; param; param = param->next) {
         hres = jsdisp_propput_name(var_disp, param->identifier, lcid,
-                i < cargs ? dp->rgvarg + dp->cArgs-1 - i : &var_empty,
-                ei, caller);
+                i < cargs ? get_arg(dp,i) : &var_empty, ei, caller);
         if(FAILED(hres))
             return hres;
 
diff --git a/dlls/jscript/string.c b/dlls/jscript/string.c
index 2350f56..3b0ad59 100644
--- a/dlls/jscript/string.c
+++ b/dlls/jscript/string.c
@@ -1066,7 +1066,7 @@ static HRESULT String_slice(DispatchEx *dispex, LCID lcid, WORD flags, DISPPARAM
     }
 
     if(arg_cnt(dp)) {
-        hres = to_integer(dispex->ctx, dp->rgvarg + dp->cArgs-1, ei, &v);
+        hres = to_integer(dispex->ctx, get_arg(dp,0), ei, &v);
         if(FAILED(hres)) {
             SysFreeString(val_str);
             return hres;
@@ -1089,7 +1089,7 @@ static HRESULT String_slice(DispatchEx *dispex, LCID lcid, WORD flags, DISPPARAM
     }
 
     if(arg_cnt(dp) >= 2) {
-        hres = to_integer(dispex->ctx, dp->rgvarg + dp->cArgs-2, ei, &v);
+        hres = to_integer(dispex->ctx, get_arg(dp,1), ei, &v);
         if(FAILED(hres)) {
             SysFreeString(val_str);
             return hres;
@@ -1325,7 +1325,7 @@ static HRESULT String_substring(DispatchEx *dispex, LCID lcid, WORD flags, DISPP
     }
 
     if(arg_cnt(dp) >= 1) {
-        hres = to_integer(dispex->ctx, dp->rgvarg + dp->cArgs-1, ei, &v);
+        hres = to_integer(dispex->ctx, get_arg(dp,0), ei, &v);
         if(FAILED(hres)) {
             SysFreeString(val_str);
             return hres;
@@ -1343,7 +1343,7 @@ static HRESULT String_substring(DispatchEx *dispex, LCID lcid, WORD flags, DISPP
     }
 
     if(arg_cnt(dp) >= 2) {
-        hres = to_integer(dispex->ctx, dp->rgvarg + dp->cArgs-2, ei, &v);
+        hres = to_integer(dispex->ctx, get_arg(dp,1), ei, &v);
         if(FAILED(hres)) {
             SysFreeString(val_str);
             return hres;
@@ -1412,7 +1412,7 @@ static HRESULT String_substr(DispatchEx *dispex, LCID lcid, WORD flags, DISPPARA
     }
 
     if(arg_cnt(dp) >= 1) {
-        hres = to_integer(dispex->ctx, dp->rgvarg + dp->cArgs-1, ei, &v);
+        hres = to_integer(dispex->ctx, get_arg(dp,0), ei, &v);
         if(FAILED(hres)) {
             SysFreeString(val_str);
             return hres;
@@ -1430,7 +1430,7 @@ static HRESULT String_substr(DispatchEx *dispex, LCID lcid, WORD flags, DISPPARA
     }
 
     if(arg_cnt(dp) >= 2) {
-        hres = to_integer(dispex->ctx, dp->rgvarg + dp->cArgs-2, ei, &v);
+        hres = to_integer(dispex->ctx, get_arg(dp,1), ei, &v);
         if(FAILED(hres)) {
             SysFreeString(val_str);
             return hres;




More information about the wine-cvs mailing list