[PATCH v4 2/4] jscript: Lookup host global object on demand instead of storing it in script context.
Gabriel Ivăncescu
gabrielopcode at gmail.com
Thu Mar 5 10:14:49 CST 2020
Hi Jacek,
On 05/03/2020 17:37, Jacek Caban wrote:
> Hi Gabriel,
>
> On 02.03.2020 14:38, Gabriel Ivăncescu wrote:
>> Signed-off-by: Gabriel Ivăncescu<gabrielopcode at gmail.com>
>> ---
>> dlls/jscript/engine.c | 21 +++++++++++++++++----
>> dlls/jscript/engine.h | 1 +
>> dlls/jscript/function.c | 4 +---
>> dlls/jscript/jscript.c | 10 ----------
>> dlls/jscript/jscript.h | 2 --
>> 5 files changed, 19 insertions(+), 19 deletions(-)
>
>
> How about storing this as NULL for global code and then do the lookup in
> interp_this() when this_obj is NULL?
>
>
> Thanks,
>
> Jacek
>
It seems to be used in multiple places other than interp_this:
interp_ret, pop_call_frame. And also for eval in global.c. Wouldn't that
require to look it up in each of them every time?
I'm not sure if it's better since it adds more code, too. What do you think?
Thanks,
Gabriel
More information about the wine-devel
mailing list