Commit 96990cfa authored by Jacek Caban's avatar Jacek Caban Committed by Alexandre Julliard

jscript: Use generic property for Error.description.

parent 2761b0de
...@@ -29,7 +29,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(jscript); ...@@ -29,7 +29,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(jscript);
typedef struct { typedef struct {
DispatchEx dispex; DispatchEx dispex;
VARIANT description;
VARIANT message; VARIANT message;
} ErrorInstance; } ErrorInstance;
...@@ -49,24 +48,6 @@ static inline ErrorInstance *error_this(vdisp_t *jsthis) ...@@ -49,24 +48,6 @@ static inline ErrorInstance *error_this(vdisp_t *jsthis)
return is_vclass(jsthis, JSCLASS_ERROR) ? error_from_vdisp(jsthis) : NULL; return is_vclass(jsthis, JSCLASS_ERROR) ? error_from_vdisp(jsthis) : NULL;
} }
static HRESULT Error_description(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags,
DISPPARAMS *dp, VARIANT *retv, jsexcept_t *ei, IServiceProvider *sp)
{
ErrorInstance *This = error_from_vdisp(jsthis);
TRACE("\n");
switch(flags) {
case DISPATCH_PROPERTYGET:
return VariantCopy(retv, &This->description);
case DISPATCH_PROPERTYPUT:
return VariantCopy(&This->description, get_arg(dp, 0));
default:
FIXME("unimplemented flags %x\n", flags);
return E_NOTIMPL;
}
}
/* ECMA-262 3rd Edition 15.11.4.3 */ /* ECMA-262 3rd Edition 15.11.4.3 */
static HRESULT Error_message(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, static HRESULT Error_message(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags,
DISPPARAMS *dp, VARIANT *retv, jsexcept_t *ei, IServiceProvider *sp) DISPPARAMS *dp, VARIANT *retv, jsexcept_t *ei, IServiceProvider *sp)
...@@ -184,13 +165,11 @@ static void Error_destructor(DispatchEx *dispex) ...@@ -184,13 +165,11 @@ static void Error_destructor(DispatchEx *dispex)
{ {
ErrorInstance *This = (ErrorInstance*)dispex; ErrorInstance *This = (ErrorInstance*)dispex;
VariantClear(&This->description);
VariantClear(&This->message); VariantClear(&This->message);
heap_free(This); heap_free(This);
} }
static const builtin_prop_t Error_props[] = { static const builtin_prop_t Error_props[] = {
{descriptionW, Error_description, 0},
{messageW, Error_message, 0}, {messageW, Error_message, 0},
{toStringW, Error_toString, PROPF_METHOD} {toStringW, Error_toString, PROPF_METHOD}
}; };
...@@ -205,7 +184,6 @@ static const builtin_info_t Error_info = { ...@@ -205,7 +184,6 @@ static const builtin_info_t Error_info = {
}; };
static const builtin_prop_t ErrorInst_props[] = { static const builtin_prop_t ErrorInst_props[] = {
{descriptionW, Error_description, 0},
{messageW, Error_message, 0}, {messageW, Error_message, 0},
}; };
...@@ -264,14 +242,17 @@ static HRESULT create_error(script_ctx_t *ctx, DispatchEx *constr, ...@@ -264,14 +242,17 @@ static HRESULT create_error(script_ctx_t *ctx, DispatchEx *constr,
V_VT(&err->message) = VT_BSTR; V_VT(&err->message) = VT_BSTR;
if(msg) V_BSTR(&err->message) = SysAllocString(msg); if(msg) V_BSTR(&err->message) = SysAllocString(msg);
else V_BSTR(&err->message) = SysAllocStringLen(NULL, 0); else V_BSTR(&err->message) = SysAllocStringLen(NULL, 0);
VariantCopy(&err->description, &err->message);
if(!V_BSTR(&err->message)) { if(!V_BSTR(&err->message)) {
heap_free(err); heap_free(err);
return E_OUTOFMEMORY; return E_OUTOFMEMORY;
} }
hres = jsdisp_propput_name(&err->dispex, descriptionW, &err->message, NULL/*FIXME*/, NULL/*FIXME*/);
if(FAILED(hres)) {
jsdisp_release(&err->dispex);
return hres;
}
*ret = &err->dispex; *ret = &err->dispex;
return S_OK; return S_OK;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment