November 2019 - Wine-Commits

Rémi Bernon : user32/tests: Flush remaining events before starting a new sequence.
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: 766ee1692e93f94d0ab6320b3450e8cdeca41ce5 URL: https://source.winehq.org/git/wine.git/?a=commit;h=766ee1692e93f94d0ab6320b… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Nov 5 20:44:52 2019 +0100 user32/tests: Flush remaining events before starting a new sequence. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/msg.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 3f11995e33..193a20fd95 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -14375,6 +14375,7 @@ static void test_dbcs_wm_char(void) WS_OVERLAPPEDWINDOW, 100, 100, 200, 200, 0, 0, 0, NULL); ok (hwnd != 0, "Failed to create overlapped window\n"); ok (hwnd2 != 0, "Failed to create overlapped window\n"); + flush_events(); flush_sequence(); memset( wmCharSeq, 0, sizeof(wmCharSeq) );

1 0

Rémi Bernon : user32/tests: Add optional flags for alternative sequence on w1064v1809.
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: ad5ca21de664d384f18f84d861c824e3a6ecc0f1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ad5ca21de664d384f18f84d8… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Nov 5 20:44:51 2019 +0100 user32/tests: Add optional flags for alternative sequence on w1064v1809. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/msg.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 6d8add4e0e..3f11995e33 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -9054,11 +9054,13 @@ static const struct message WmShiftMouseButton[] = { { WM_MOUSEMOVE, sent|wparam|optional, 0, 0 }, { WM_LBUTTONDOWN, wparam, MK_LBUTTON|MK_SHIFT, 0 }, { WM_LBUTTONDOWN, sent|wparam, MK_LBUTTON|MK_SHIFT, 0 }, - { WM_LBUTTONUP, wparam, MK_SHIFT, 0 }, - { WM_LBUTTONUP, sent|wparam, MK_SHIFT, 0 }, + { WM_LBUTTONUP, wparam|optional, MK_SHIFT, 0 }, /* < w1064v1809 */ + { WM_LBUTTONUP, sent|wparam|optional, MK_SHIFT, 0 }, /* < w1064v1809 */ { HCBT_KEYSKIPPED, hook|wparam|lparam|optional, VK_SHIFT, 0xc0000001 }, /* XP */ { WM_KEYUP, wparam|lparam, VK_SHIFT, 0xc0000001 }, { WM_KEYUP, sent|wparam|lparam, VK_SHIFT, 0xc0000001 }, + { WM_LBUTTONUP, wparam|optional, MK_SHIFT, 0 }, /* >= w1064v1809 */ + { WM_LBUTTONUP, sent|wparam|optional, MK_SHIFT, 0 }, /* >= w1064v1809 */ { 0 } }; static const struct message WmF1Seq[] = {

1 0

Gabriel Ivăncescu : jscript: Set the DISPID to DISPID_UNKNOWN if unknown.
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: 15fce35b599998193e1890eb7cf0048e6434f09e URL: https://source.winehq.org/git/wine.git/?a=commit;h=15fce35b599998193e1890eb… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Wed Nov 6 17:27:59 2019 +0200 jscript: Set the DISPID to DISPID_UNKNOWN if unknown. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/dispex.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/jscript/dispex.c b/dlls/jscript/dispex.c index da23d18aa2..3c2733086b 100644 --- a/dlls/jscript/dispex.c +++ b/dlls/jscript/dispex.c @@ -1083,6 +1083,7 @@ HRESULT jsdisp_get_id(jsdisp_t *jsdisp, const WCHAR *name, DWORD flags, DISPID * } TRACE("not found %s\n", debugstr_w(name)); + *id = DISPID_UNKNOWN; return DISP_E_UNKNOWNNAME; }

1 0

Rémi Bernon : user32/tests: Add some wparam mask to message sequences for w1064v1809.
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: 37fe4b0bab2ba567c0170be201ece50bde67d6fb URL: https://source.winehq.org/git/wine.git/?a=commit;h=37fe4b0bab2ba567c0170be2… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Nov 5 20:44:48 2019 +0100 user32/tests: Add some wparam mask to message sequences for w1064v1809. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/msg.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 477661ab62..08a106ee19 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -780,7 +780,7 @@ static const struct message WmCreateInvisibleMaxPopupSeq[] = { static const struct message WmShowMaxPopupResizedSeq[] = { { HCBT_MINMAX, hook|lparam, 0, SW_MAXIMIZE }, { WM_GETMINMAXINFO, sent }, - { WM_WINDOWPOSCHANGING, sent|wparam, SWP_SHOWWINDOW|SWP_FRAMECHANGED }, + { WM_WINDOWPOSCHANGING, sent|wparam, SWP_SHOWWINDOW|SWP_FRAMECHANGED, 0, SWP_STATECHANGED /* w1064v1809 */ }, { WM_NCCALCSIZE, sent|wparam, TRUE }, { EVENT_OBJECT_SHOW, winevent_hook|wparam|lparam, 0, 0 }, { HCBT_ACTIVATE, hook }, @@ -812,7 +812,7 @@ static const struct message WmShowMaxPopupResizedSeq[] = { static const struct message WmShowMaxPopupSeq[] = { { HCBT_MINMAX, hook|lparam, 0, SW_MAXIMIZE }, { WM_GETMINMAXINFO, sent }, - { WM_WINDOWPOSCHANGING, sent|wparam, SWP_SHOWWINDOW|SWP_FRAMECHANGED }, + { WM_WINDOWPOSCHANGING, sent|wparam, SWP_SHOWWINDOW|SWP_FRAMECHANGED, 0, SWP_STATECHANGED /* w1064v1809 */ }, { WM_NCCALCSIZE, sent|wparam, TRUE }, { EVENT_OBJECT_SHOW, winevent_hook|wparam|lparam, 0, 0 }, { HCBT_ACTIVATE, hook }, @@ -836,7 +836,7 @@ static const struct message WmShowMaxPopupSeq[] = { { WM_ERASEBKGND, sent|optional }, { WM_NCPAINT, sent|wparam|defwinproc|optional, 1 }, { WM_ERASEBKGND, sent|defwinproc|optional }, - { WM_WINDOWPOSCHANGED, sent|wparam, SWP_NOCLIENTSIZE|SWP_NOCLIENTMOVE|SWP_SHOWWINDOW|SWP_FRAMECHANGED|SWP_NOMOVE|SWP_NOSIZE }, + { WM_WINDOWPOSCHANGED, sent|wparam, SWP_NOCLIENTSIZE|SWP_NOCLIENTMOVE|SWP_SHOWWINDOW|SWP_FRAMECHANGED|SWP_NOMOVE|SWP_NOSIZE, 0, SWP_STATECHANGED /* w1064v1809 */ }, { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam, 0, 0 }, { WM_SIZE, sent|defwinproc|optional }, { 0 } @@ -3476,7 +3476,7 @@ static const struct message WmMaximizeMDIchildInvisibleSeq[] = { static const struct message WmMaximizeMDIchildInvisibleSeq2[] = { { HCBT_MINMAX, hook|lparam, 0, SW_MAXIMIZE }, { WM_GETMINMAXINFO, sent }, - { WM_WINDOWPOSCHANGING, sent|wparam, SWP_SHOWWINDOW|SWP_FRAMECHANGED }, + { WM_WINDOWPOSCHANGING, sent|wparam, SWP_SHOWWINDOW|SWP_FRAMECHANGED, 0, SWP_STATECHANGED /* w1064v1809 */ }, { WM_GETMINMAXINFO, sent|defwinproc }, { WM_NCCALCSIZE, sent|wparam, 1 }, { EVENT_OBJECT_SHOW, winevent_hook|wparam|lparam, 0, 0 }, @@ -3495,7 +3495,7 @@ static const struct message WmMaximizeMDIchildInvisibleSeq2[] = { { EVENT_OBJECT_FOCUS, winevent_hook|wparam|lparam, OBJID_CLIENT, 0 }, { WM_SETFOCUS, sent|defwinproc|optional }, { WM_MDIACTIVATE, sent|defwinproc|optional }, - { WM_WINDOWPOSCHANGED, sent|wparam, SWP_SHOWWINDOW|SWP_FRAMECHANGED|SWP_NOSIZE|SWP_NOMOVE|SWP_NOCLIENTSIZE|SWP_NOCLIENTMOVE }, + { WM_WINDOWPOSCHANGED, sent|wparam, SWP_SHOWWINDOW|SWP_FRAMECHANGED|SWP_NOSIZE|SWP_NOMOVE|SWP_NOCLIENTSIZE|SWP_NOCLIENTMOVE, 0, SWP_STATECHANGED /* w1064v1809 */ }, { EVENT_OBJECT_LOCATIONCHANGE, winevent_hook|wparam|lparam, 0, 0 }, /* MDI child */ { WM_SIZE, sent|defwinproc|optional }, { 0 }

1 0

Zebediah Figura : ntoskrnl.exe: Stub IRP_MN_QUERY_CAPABILITIES for root-enumerated devices.
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: 481b099345b8afa074a21c34f6b020c5e58de0d0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=481b099345b8afa074a21c34… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Nov 6 00:13:15 2019 -0600 ntoskrnl.exe: Stub IRP_MN_QUERY_CAPABILITIES for root-enumerated devices. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntoskrnl.exe/pnp.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/dlls/ntoskrnl.exe/pnp.c b/dlls/ntoskrnl.exe/pnp.c index 5b5acd2b34..b5c4e6878e 100644 --- a/dlls/ntoskrnl.exe/pnp.c +++ b/dlls/ntoskrnl.exe/pnp.c @@ -827,6 +827,9 @@ static NTSTATUS WINAPI pnp_manager_device_pnp( DEVICE_OBJECT *device, IRP *irp ) /* Nothing to do. */ irp->IoStatus.u.Status = STATUS_SUCCESS; break; + case IRP_MN_QUERY_CAPABILITIES: + irp->IoStatus.u.Status = STATUS_SUCCESS; + break; case IRP_MN_QUERY_ID: { BUS_QUERY_ID_TYPE type = stack->Parameters.QueryId.IdType;

1 0

Rémi Bernon : user32/tests: Update SetActiveWindow(0) test result for w1064v1809.
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: 32b3e502d9752498f54108913346344cf93223e6 URL: https://source.winehq.org/git/wine.git/?a=commit;h=32b3e502d9752498f5410891… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Nov 5 20:44:49 2019 +0100 user32/tests: Update SetActiveWindow(0) test result for w1064v1809. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/tests/msg.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 08a106ee19..43c1859e04 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -14218,8 +14218,8 @@ static void test_SetActiveWindow(void) trace("SetActiveWindow(0)\n"); ret = SetActiveWindow(0); - ok( ret == popup, "Failed to SetActiveWindow(0)\n"); - ok_sequence(SetActiveWindowSeq0, "SetActiveWindow(0)", FALSE); + ok( ret == popup || broken(ret == 0) /* w1064v1809 */, "Failed to SetActiveWindow(0), ret:%p\n", ret); + if (ret == popup) ok_sequence(SetActiveWindowSeq0, "SetActiveWindow(0)", FALSE); flush_sequence(); trace("SetActiveWindow(hwnd), hwnd visible\n");

1 0

Zebediah Figura : ntoskrnl.exe: Trace the call to AddDevice().
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: 781fc47f361953f7411a84e32b066d5011473bd4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=781fc47f361953f7411a84e3… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Nov 6 00:13:17 2019 -0600 ntoskrnl.exe: Trace the call to AddDevice(). Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntoskrnl.exe/pnp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/ntoskrnl.exe/pnp.c b/dlls/ntoskrnl.exe/pnp.c index 89676a06ad..a3dbe772ae 100644 --- a/dlls/ntoskrnl.exe/pnp.c +++ b/dlls/ntoskrnl.exe/pnp.c @@ -226,10 +226,12 @@ static void load_function_driver( DEVICE_OBJECT *device, HDEVINFO set, SP_DEVINF return; } + TRACE("Calling AddDevice routine %p.\n", driver_obj->DriverExtension->AddDevice); if (driver_obj->DriverExtension->AddDevice) status = driver_obj->DriverExtension->AddDevice( driver_obj, device ); else status = STATUS_NOT_IMPLEMENTED; + TRACE("AddDevice routine %p returned %#x.\n", driver_obj->DriverExtension->AddDevice, status); ObDereferenceObject( driver_obj );

1 0

Gabriel Ivăncescu : vbscript: Store global variables in an array.
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: 4699d366565f7a27ac5606c7a1a318e24242ec73 URL: https://source.winehq.org/git/wine.git/?a=commit;h=4699d366565f7a27ac5606c7… Author: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Date: Wed Nov 6 17:35:04 2019 +0100 vbscript: Store global variables in an array. Signed-off-by: Gabriel Ivăncescu <gabrielopcode(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vbscript/compile.c | 30 ++++++++++++++++++++---------- dlls/vbscript/interp.c | 45 ++++++++++++++++++++++++++++++++++++--------- dlls/vbscript/vbdisp.c | 4 ++-- dlls/vbscript/vbscript.c | 9 +++++++-- dlls/vbscript/vbscript.h | 5 ++++- 5 files changed, 69 insertions(+), 24 deletions(-) diff --git a/dlls/vbscript/compile.c b/dlls/vbscript/compile.c index 4ccb164c4a..7b6649ec56 100644 --- a/dlls/vbscript/compile.c +++ b/dlls/vbscript/compile.c @@ -1787,11 +1787,10 @@ static HRESULT compile_class(compile_ctx_t *ctx, class_decl_t *class_decl) static BOOL lookup_script_identifier(script_ctx_t *script, const WCHAR *identifier) { class_desc_t *class; - dynamic_var_t *var; unsigned i; - for(var = script->global_vars; var; var = var->next) { - if(!wcsicmp(var->name, identifier)) + for(i = 0; i < script->global_vars_cnt; i++) { + if(!wcsicmp(script->global_vars[i]->name, identifier)) return TRUE; } @@ -1895,6 +1894,7 @@ HRESULT compile_script(script_ctx_t *script, const WCHAR *src, const WCHAR *deli function_t *new_func, *func_iter; function_decl_t *func_decl; class_decl_t *class_decl; + dynamic_var_t *var_iter; compile_ctx_t ctx; vbscode_t *code; size_t cnt; @@ -1952,13 +1952,19 @@ HRESULT compile_script(script_ctx_t *script, const WCHAR *src, const WCHAR *deli return compile_error(script, hres); } - if(ctx.global_vars) { - dynamic_var_t *var; - - for(var = ctx.global_vars; var->next; var = var->next); - - var->next = script->global_vars; - script->global_vars = ctx.global_vars; + cnt = script->global_vars_cnt; + for(var_iter = ctx.global_vars; var_iter; var_iter = var_iter->next) + cnt++; + if(cnt > script->global_vars_size) { + dynamic_var_t **new_vars; + if(script->global_vars) + new_vars = heap_realloc(script->global_vars, cnt * sizeof(*new_vars)); + else + new_vars = heap_alloc(cnt * sizeof(*new_vars)); + if(!new_vars) + return compile_error(script, E_OUTOFMEMORY); + script->global_vars = new_vars; + script->global_vars_size = cnt; } cnt = script->global_funcs_cnt; @@ -1975,6 +1981,10 @@ HRESULT compile_script(script_ctx_t *script, const WCHAR *src, const WCHAR *deli script->global_funcs = new_funcs; script->global_funcs_size = cnt; } + + for(var_iter = ctx.global_vars; var_iter; var_iter = var_iter->next) + script->global_vars[script->global_vars_cnt++] = var_iter; + for(func_iter = ctx.funcs; func_iter; func_iter = func_iter->next) { unsigned i; for(i = 0; i < script->global_funcs_cnt; i++) { diff --git a/dlls/vbscript/interp.c b/dlls/vbscript/interp.c index e4f982a025..3dfcf1e7f7 100644 --- a/dlls/vbscript/interp.c +++ b/dlls/vbscript/interp.c @@ -94,6 +94,22 @@ static BOOL lookup_dynamic_vars(dynamic_var_t *var, const WCHAR *name, ref_t *re return FALSE; } +static BOOL lookup_global_vars(script_ctx_t *script, const WCHAR *name, ref_t *ref) +{ + dynamic_var_t **vars = script->global_vars; + size_t i, cnt = script->global_vars_cnt; + + for(i = 0; i < cnt; i++) { + if(!wcsicmp(vars[i]->name, name)) { + ref->type = vars[i]->is_const ? REF_CONST : REF_VAR; + ref->u.v = &vars[i]->v; + return TRUE; + } + } + + return FALSE; +} + static HRESULT lookup_identifier(exec_ctx_t *ctx, BSTR name, vbdisp_invoke_type_t invoke_type, ref_t *ref) { named_item_t *item; @@ -159,7 +175,7 @@ static HRESULT lookup_identifier(exec_ctx_t *ctx, BSTR name, vbdisp_invoke_type_ } } - if(lookup_dynamic_vars(ctx->script->global_vars, name, ref)) + if(lookup_global_vars(ctx->script, name, ref)) return S_OK; for(i = 0; i < ctx->script->global_funcs_cnt; i++) { @@ -227,8 +243,19 @@ static HRESULT add_dynamic_var(exec_ctx_t *ctx, const WCHAR *name, V_VT(&new_var->v) = VT_EMPTY; if(ctx->func->type == FUNC_GLOBAL) { - new_var->next = ctx->script->global_vars; - ctx->script->global_vars = new_var; + size_t cnt = ctx->script->global_vars_cnt + 1; + if(cnt > ctx->script->global_vars_size) { + dynamic_var_t **new_vars; + if(ctx->script->global_vars) + new_vars = heap_realloc(ctx->script->global_vars, cnt * 2 * sizeof(*new_vars)); + else + new_vars = heap_alloc(cnt * 2 * sizeof(*new_vars)); + if(!new_vars) + return E_OUTOFMEMORY; + ctx->script->global_vars = new_vars; + ctx->script->global_vars_size = cnt * 2; + } + ctx->script->global_vars[ctx->script->global_vars_cnt++] = new_var; }else { new_var->next = ctx->dynamic_vars; ctx->dynamic_vars = new_var; @@ -1118,14 +1145,14 @@ static HRESULT interp_dim(exec_ctx_t *ctx) assert(array_id < ctx->func->array_cnt); if(ctx->func->type == FUNC_GLOBAL) { - dynamic_var_t *var; - for(var = ctx->script->global_vars; var; var = var->next) { - if(!wcsicmp(var->name, ident)) + unsigned i; + for(i = 0; i < ctx->script->global_vars_cnt; i++) { + if(!wcsicmp(ctx->script->global_vars[i]->name, ident)) break; } - assert(var != NULL); - v = &var->v; - array_ref = &var->array; + assert(i < ctx->script->global_vars_cnt); + v = &ctx->script->global_vars[i]->v; + array_ref = &ctx->script->global_vars[i]->array; }else { ref_t ref; diff --git a/dlls/vbscript/vbdisp.c b/dlls/vbscript/vbdisp.c index a5ae40d098..00e975e40b 100644 --- a/dlls/vbscript/vbdisp.c +++ b/dlls/vbscript/vbdisp.c @@ -675,7 +675,6 @@ static HRESULT WINAPI ScriptDisp_Invoke(IDispatchEx *iface, DISPID dispIdMember, static HRESULT WINAPI ScriptDisp_GetDispID(IDispatchEx *iface, BSTR bstrName, DWORD grfdex, DISPID *pid) { ScriptDisp *This = ScriptDisp_from_IDispatchEx(iface); - dynamic_var_t *var; ident_map_t *ident; unsigned i; @@ -691,7 +690,8 @@ static HRESULT WINAPI ScriptDisp_GetDispID(IDispatchEx *iface, BSTR bstrName, DW } } - for(var = This->ctx->global_vars; var; var = var->next) { + for(i = 0; i < This->ctx->global_vars_cnt; i++) { + dynamic_var_t *var = This->ctx->global_vars[i]; if(!wcsicmp(var->name, bstrName)) { ident = add_ident(This, var->name); if(!ident) diff --git a/dlls/vbscript/vbscript.c b/dlls/vbscript/vbscript.c index 886c9d01af..6d2bb54e52 100644 --- a/dlls/vbscript/vbscript.c +++ b/dlls/vbscript/vbscript.c @@ -131,14 +131,19 @@ IDispatch *lookup_named_item(script_ctx_t *ctx, const WCHAR *name, unsigned flag static void release_script(script_ctx_t *ctx) { class_desc_t *class_desc; + unsigned i; collect_objects(ctx); clear_ei(&ctx->ei); - release_dynamic_vars(ctx->global_vars); - ctx->global_vars = NULL; + for(i = 0; i < ctx->global_vars_cnt; i++) + VariantClear(&ctx->global_vars[i]->v); + heap_free(ctx->global_vars); heap_free(ctx->global_funcs); + ctx->global_vars = NULL; + ctx->global_vars_cnt = 0; + ctx->global_vars_size = 0; ctx->global_funcs = NULL; ctx->global_funcs_cnt = 0; ctx->global_funcs_size = 0; diff --git a/dlls/vbscript/vbscript.h b/dlls/vbscript/vbscript.h index ac5242d94f..a2e87790ba 100644 --- a/dlls/vbscript/vbscript.h +++ b/dlls/vbscript/vbscript.h @@ -188,11 +188,14 @@ struct _script_ctx_t { EXCEPINFO ei; + dynamic_var_t **global_vars; + size_t global_vars_cnt; + size_t global_vars_size; + function_t **global_funcs; size_t global_funcs_cnt; size_t global_funcs_size; - dynamic_var_t *global_vars; class_desc_t *classes; class_desc_t *procs;

1 0

Hans Leidekker : bcrypt: Avoid recreating cipher handles when the initialization vector doesn't change.
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: f4d6df8947b9e0c26c92127e5e95464a64a52bde URL: https://source.winehq.org/git/wine.git/?a=commit;h=f4d6df8947b9e0c26c92127e… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Nov 6 10:16:54 2019 +0100 bcrypt: Avoid recreating cipher handles when the initialization vector doesn't change. Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/bcrypt/bcrypt_internal.h | 9 +++++- dlls/bcrypt/bcrypt_main.c | 28 +++++++++++++----- dlls/bcrypt/gnutls.c | 62 +++++++++++++++++++++++++++++--------- dlls/bcrypt/macos.c | 57 ++++++++++++++++++++++++++--------- dlls/bcrypt/tests/bcrypt.c | 69 +++++++++++++++++++++++++++++++++++++++++++ 5 files changed, 189 insertions(+), 36 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=f4d6df8947b9e0c26c92…

1 0

Jacek Caban : vbscript: Store global static arrays together with global variables.
by Alexandre Julliard Nov. 6, 2019

Nov. 6, 2019

Module: wine Branch: master Commit: 3dea7bd7fc5114ae0d6b6bcaaa503cffcdb92eca URL: https://source.winehq.org/git/wine.git/?a=commit;h=3dea7bd7fc5114ae0d6b6bca… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Nov 6 17:33:26 2019 +0100 vbscript: Store global static arrays together with global variables. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/vbscript/compile.c | 1 + dlls/vbscript/interp.c | 59 ++++++++++++++++++++++++++++++++---------------- dlls/vbscript/vbscript.h | 1 + 3 files changed, 42 insertions(+), 19 deletions(-) diff --git a/dlls/vbscript/compile.c b/dlls/vbscript/compile.c index e52b0814e0..4ccb164c4a 100644 --- a/dlls/vbscript/compile.c +++ b/dlls/vbscript/compile.c @@ -1496,6 +1496,7 @@ static HRESULT compile_func(compile_ctx_t *ctx, statement_t *stat, function_t *f V_VT(&new_var->v) = VT_EMPTY; new_var->is_const = FALSE; + new_var->array = NULL; new_var->next = ctx->global_vars; ctx->global_vars = new_var; diff --git a/dlls/vbscript/interp.c b/dlls/vbscript/interp.c index 89f9d359da..e4f982a025 100644 --- a/dlls/vbscript/interp.c +++ b/dlls/vbscript/interp.c @@ -223,6 +223,7 @@ static HRESULT add_dynamic_var(exec_ctx_t *ctx, const WCHAR *name, memcpy(str, name, size); new_var->name = str; new_var->is_const = is_const; + new_var->array = NULL; V_VT(&new_var->v) = VT_EMPTY; if(ctx->func->type == FUNC_GLOBAL) { @@ -1108,43 +1109,61 @@ static HRESULT interp_dim(exec_ctx_t *ctx) const BSTR ident = ctx->instr->arg1.bstr; const unsigned array_id = ctx->instr->arg2.uint; const array_desc_t *array_desc; - ref_t ref; + SAFEARRAY **array_ref; + VARIANT *v; HRESULT hres; TRACE("%s\n", debugstr_w(ident)); assert(array_id < ctx->func->array_cnt); - if(!ctx->arrays) { - ctx->arrays = heap_alloc_zero(ctx->func->array_cnt * sizeof(SAFEARRAY*)); - if(!ctx->arrays) - return E_OUTOFMEMORY; - } - hres = lookup_identifier(ctx, ident, VBDISP_LET, &ref); - if(FAILED(hres)) { - FIXME("lookup %s failed: %08x\n", debugstr_w(ident), hres); - return hres; - } + if(ctx->func->type == FUNC_GLOBAL) { + dynamic_var_t *var; + for(var = ctx->script->global_vars; var; var = var->next) { + if(!wcsicmp(var->name, ident)) + break; + } + assert(var != NULL); + v = &var->v; + array_ref = &var->array; + }else { + ref_t ref; - if(ref.type != REF_VAR) { - FIXME("got ref.type = %d\n", ref.type); - return E_FAIL; + if(!ctx->arrays) { + ctx->arrays = heap_alloc_zero(ctx->func->array_cnt * sizeof(SAFEARRAY*)); + if(!ctx->arrays) + return E_OUTOFMEMORY; + } + + hres = lookup_identifier(ctx, ident, VBDISP_LET, &ref); + if(FAILED(hres)) { + FIXME("lookup %s failed: %08x\n", debugstr_w(ident), hres); + return hres; + } + + if(ref.type != REF_VAR) { + FIXME("got ref.type = %d\n", ref.type); + return E_FAIL; + } + + v = ref.u.v; + array_ref = ctx->arrays + array_id; } - if(ctx->arrays[array_id]) { + if(*array_ref) { FIXME("Array already initialized\n"); return E_FAIL; } array_desc = ctx->func->array_descs + array_id; if(array_desc->dim_cnt) { - ctx->arrays[array_id] = SafeArrayCreate(VT_VARIANT, array_desc->dim_cnt, array_desc->bounds); - if(!ctx->arrays[array_id]) + *array_ref = SafeArrayCreate(VT_VARIANT, array_desc->dim_cnt, array_desc->bounds); + if(!*array_ref) return E_OUTOFMEMORY; } - V_VT(ref.u.v) = VT_ARRAY|VT_BYREF|VT_VARIANT; - V_ARRAYREF(ref.u.v) = ctx->arrays+array_id; + V_VT(v) = VT_ARRAY|VT_BYREF|VT_VARIANT; + V_ARRAYREF(v) = array_ref; return S_OK; } @@ -2201,6 +2220,8 @@ void release_dynamic_vars(dynamic_var_t *var) { while(var) { VariantClear(&var->v); + if(var->array) + SafeArrayDestroy(var->array); var = var->next; } } diff --git a/dlls/vbscript/vbscript.h b/dlls/vbscript/vbscript.h index 0bd1fce156..ac5242d94f 100644 --- a/dlls/vbscript/vbscript.h +++ b/dlls/vbscript/vbscript.h @@ -169,6 +169,7 @@ typedef struct _dynamic_var_t { VARIANT v; const WCHAR *name; BOOL is_const; + SAFEARRAY *array; } dynamic_var_t; struct _script_ctx_t {

1 0