[PATCH v6 4/8] jscript: Lookup and ref the named item's dispatch first, during interpretion.
Jacek Caban
jacek at codeweavers.com
Mon Mar 9 09:05:23 CDT 2020
Hi Gabriel,
On 07.03.2020 14:40, Gabriel Ivăncescu wrote:
> diff --git a/dlls/jscript/dispex.c b/dlls/jscript/dispex.c
> index e986de6..a0c8f38 100644
> --- a/dlls/jscript/dispex.c
> +++ b/dlls/jscript/dispex.c
> @@ -1527,6 +1527,7 @@ static HRESULT WINAPI DispatchEx_InvokeEx(IDispatchEx *iface, DISPID id, LCID lc
> }
>
> enter_script(This->ctx, &ei);
> + ei.script_obj = This;
This doesn't look right. What are you trying to handle here? Note that
IDispatchEx is used by regular JavaScript objects and using such objects
and script_obj doesn't seem right. Even if the object is a script
dispatch, I'm not sure this is right. In fact, I'm not yet convinced
that we need script_obj in ei at all.
Thanks,
Jacek
More information about the wine-devel
mailing list