[PATCH v5 4/7] vbscript: Append to the global lists when the code is executed, rather than compiled.

[Gabriel Ivăncescu]

On 11/7/19 8:28 PM, Jacek Caban wrote:
> Hi Gabriel,
> 
> On 11/7/19 2:28 PM, Gabriel Ivăncescu wrote:
>> diff --git a/dlls/vbscript/vbscript.h b/dlls/vbscript/vbscript.h
>> index 04a9b83..efc3585 100644
>> --- a/dlls/vbscript/vbscript.h
>> +++ b/dlls/vbscript/vbscript.h
>> @@ -350,6 +350,9 @@ struct _vbscode_t {
>>       unsigned bstr_cnt;
>>       heap_pool_t heap;
>> +    dynamic_var_t *global_vars;
> 
> 
> It seems to me that we don't need it. Now that we post-process variables 
> to append to global variable set anyway, we could just get rid of 
> special-case for global code in compiler and use variables stored in 
> vbscode_t::main_code when we need to add them to global variables set.
> 
> 
> Jacek
> 

I've been looking at the code a bit and I'm afraid I'm missing 
something. What exactly do you mean by post-process variables?

As it is now, the code is wrong and will not pass the tests I wrote. 
When the code is executed (and it can be executed several times) it 
should add its vars/functions to the global lists so they get DISPIDs 
and so on.

Or do you have a better place to add its functions and variables to the 
global lists? It has to be a place where they get added if the code gets 
re-executed (pending code), though. And only after the script is started.

Sorry for the confusion.