October 2012 - Wine-Commits

Henri Verbeet : winex11: Don' t send SC_RESTORE when a maximized window switches to fullscreen.
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: 32125fe65ec9f6fe9386ac657c03e73422338dc9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=32125fe65ec9f6fe9386ac657… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Oct 4 13:53:29 2012 +0200 winex11: Don't send SC_RESTORE when a maximized window switches to fullscreen. --- dlls/winex11.drv/event.c | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/dlls/winex11.drv/event.c b/dlls/winex11.drv/event.c index 9eedbc5..f908413 100644 --- a/dlls/winex11.drv/event.c +++ b/dlls/winex11.drv/event.c @@ -1065,19 +1065,19 @@ void X11DRV_ConfigureNotify( HWND hwnd, XEvent *xev ) data->window_rect.bottom - data->window_rect.top, cx, cy ); style = GetWindowLongW( data->hwnd, GWL_STYLE ); - if ((style & WS_CAPTION) == WS_CAPTION && is_net_wm_state_maximized( event->display, data )) + if ((style & WS_CAPTION) == WS_CAPTION) { - if (!(style & WS_MAXIMIZE)) + if (is_net_wm_state_maximized( event->display, data )) { - TRACE( "win %p/%lx is maximized\n", data->hwnd, data->whole_window ); - release_win_data( data ); - SendMessageW( data->hwnd, WM_SYSCOMMAND, SC_MAXIMIZE, 0 ); - return; + if (!(style & WS_MAXIMIZE)) + { + TRACE( "win %p/%lx is maximized\n", data->hwnd, data->whole_window ); + release_win_data( data ); + SendMessageW( data->hwnd, WM_SYSCOMMAND, SC_MAXIMIZE, 0 ); + return; + } } - } - else - { - if (style & WS_MAXIMIZE) + else if (style & WS_MAXIMIZE) { TRACE( "window %p/%lx is no longer maximized\n", data->hwnd, data->whole_window ); release_win_data( data );

1 0

Hans Leidekker : wbemprox: Remove dynamic tables from the global list when they're freed.
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: 804e7e91ccc4f56b7c9183900137fb6ecee7f724 URL: http://source.winehq.org/git/wine.git/?a=commit;h=804e7e91ccc4f56b7c9183900… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Oct 4 15:36:33 2012 +0200 wbemprox: Remove dynamic tables from the global list when they're freed. --- dlls/wbemprox/table.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/dlls/wbemprox/table.c b/dlls/wbemprox/table.c index 4305f1f..c414aa9 100644 --- a/dlls/wbemprox/table.c +++ b/dlls/wbemprox/table.c @@ -308,8 +308,10 @@ void free_table( struct table *table ) clear_table( table ); if (table->flags & TABLE_FLAG_DYNAMIC) { + TRACE("destroying %p\n", table); heap_free( (WCHAR *)table->name ); free_columns( (struct column *)table->columns, table->num_cols ); + list_remove( &table->entry ); heap_free( table ); } } @@ -342,6 +344,7 @@ struct table *create_table( const WCHAR *name, UINT num_cols, const struct colum table->data = data; table->fill = fill; table->flags = TABLE_FLAG_DYNAMIC; + list_init( &table->entry ); return table; }

1 0

Jacek Caban : jscript: Added support for pstrFormalParams argument in ParseProcedureText.
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: bf650032713ad0bc736376ccb700c4e864f6901f URL: http://source.winehq.org/git/wine.git/?a=commit;h=bf650032713ad0bc736376ccb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Oct 4 14:58:05 2012 +0200 jscript: Added support for pstrFormalParams argument in ParseProcedureText. --- dlls/jscript/compile.c | 106 +++++++++++++++++++++++++++++++++++++++++-- dlls/jscript/engine.h | 2 +- dlls/jscript/function.c | 2 +- dlls/jscript/global.c | 2 +- dlls/jscript/jscript.c | 4 +- dlls/jscript/tests/run.c | 11 +++++ dlls/mshtml/tests/activex.c | 2 - 7 files changed, 117 insertions(+), 12 deletions(-) diff --git a/dlls/jscript/compile.c b/dlls/jscript/compile.c index 034a55d..7099a6a 100644 --- a/dlls/jscript/compile.c +++ b/dlls/jscript/compile.c @@ -131,24 +131,32 @@ static WCHAR *compiler_alloc_string(bytecode_t *code, const WCHAR *str) return ret; } -static BSTR compiler_alloc_bstr(compiler_ctx_t *ctx, const WCHAR *str) +static BOOL ensure_bstr_slot(compiler_ctx_t *ctx) { if(!ctx->code->bstr_pool_size) { ctx->code->bstr_pool = heap_alloc(8 * sizeof(BSTR)); if(!ctx->code->bstr_pool) - return NULL; + return FALSE; ctx->code->bstr_pool_size = 8; }else if(ctx->code->bstr_pool_size == ctx->code->bstr_cnt) { BSTR *new_pool; new_pool = heap_realloc(ctx->code->bstr_pool, ctx->code->bstr_pool_size*2*sizeof(BSTR)); if(!new_pool) - return NULL; + return FALSE; ctx->code->bstr_pool = new_pool; ctx->code->bstr_pool_size *= 2; } + return TRUE; +} + +static BSTR compiler_alloc_bstr(compiler_ctx_t *ctx, const WCHAR *str) +{ + if(!ensure_bstr_slot(ctx)) + return NULL; + ctx->code->bstr_pool[ctx->code->bstr_cnt] = SysAllocString(str); if(!ctx->code->bstr_pool[ctx->code->bstr_cnt]) return NULL; @@ -156,6 +164,18 @@ static BSTR compiler_alloc_bstr(compiler_ctx_t *ctx, const WCHAR *str) return ctx->code->bstr_pool[ctx->code->bstr_cnt++]; } +static BSTR compiler_alloc_bstr_len(compiler_ctx_t *ctx, const WCHAR *str, size_t len) +{ + if(!ensure_bstr_slot(ctx)) + return NULL; + + ctx->code->bstr_pool[ctx->code->bstr_cnt] = SysAllocStringLen(str, len); + if(!ctx->code->bstr_pool[ctx->code->bstr_cnt]) + return NULL; + + return ctx->code->bstr_pool[ctx->code->bstr_cnt++]; +} + static unsigned push_instr(compiler_ctx_t *ctx, jsop_t op) { assert(ctx->code_size >= ctx->code_off); @@ -1885,8 +1905,78 @@ static HRESULT compile_function(compiler_ctx_t *ctx, source_elements_t *source, return S_OK; } -HRESULT compile_script(script_ctx_t *ctx, const WCHAR *code, const WCHAR *delimiter, BOOL from_eval, BOOL use_decode, - bytecode_t **ret) +static HRESULT parse_arguments(compiler_ctx_t *ctx, const WCHAR *args, BSTR *arg_array, unsigned *args_size) +{ + const WCHAR *ptr = args, *ptr2; + unsigned arg_cnt = 0; + + while(isspaceW(*ptr)) + ptr++; + if(!*ptr) { + if(args_size) + *args_size = 0; + return S_OK; + } + + while(1) { + if(!isalphaW(*ptr) && *ptr != '_') { + FIXME("expected alpha or '_': %s\n", debugstr_w(ptr)); + return E_FAIL; + } + + ptr2 = ptr; + while(isalnumW(*ptr) || *ptr == '_') + ptr++; + + if(*ptr && *ptr != ',' && !isspaceW(*ptr)) { + FIXME("unexpected har %s\n", debugstr_w(ptr)); + return E_FAIL; + } + + if(arg_array) { + arg_array[arg_cnt] = compiler_alloc_bstr_len(ctx, ptr2, ptr-ptr2); + if(!arg_array[arg_cnt]) + return E_OUTOFMEMORY; + } + arg_cnt++; + + while(isspaceW(*ptr)) + ptr++; + if(!*ptr) + break; + if(*ptr != ',') { + FIXME("expected ',': %s\n", debugstr_w(ptr)); + return E_FAIL; + } + + ptr++; + while(isspaceW(*ptr)) + ptr++; + } + + if(args_size) + *args_size = arg_cnt; + return S_OK; +} + +static HRESULT compile_arguments(compiler_ctx_t *ctx, const WCHAR *args) +{ + HRESULT hres; + + hres = parse_arguments(ctx, args, NULL, &ctx->code->global_code.param_cnt); + if(FAILED(hres)) + return hres; + + ctx->code->global_code.params = compiler_alloc(ctx->code, + ctx->code->global_code.param_cnt * sizeof(*ctx->code->global_code.params)); + if(!ctx->code->global_code.params) + return E_OUTOFMEMORY; + + return parse_arguments(ctx, args, ctx->code->global_code.params, NULL); +} + +HRESULT compile_script(script_ctx_t *ctx, const WCHAR *code, const WCHAR *args, const WCHAR *delimiter, + BOOL from_eval, BOOL use_decode, bytecode_t **ret) { compiler_ctx_t compiler = {0}; HRESULT hres; @@ -1895,6 +1985,12 @@ HRESULT compile_script(script_ctx_t *ctx, const WCHAR *code, const WCHAR *delimi if(FAILED(hres)) return hres; + if(args) { + hres = compile_arguments(&compiler, args); + if(FAILED(hres)) + return hres; + } + if(use_decode) { hres = decode_source(compiler.code->source); if(FAILED(hres)) { diff --git a/dlls/jscript/engine.h b/dlls/jscript/engine.h index f7e03f7..49232ee 100644 --- a/dlls/jscript/engine.h +++ b/dlls/jscript/engine.h @@ -178,7 +178,7 @@ typedef struct _bytecode_t { struct _bytecode_t *next; } bytecode_t; -HRESULT compile_script(script_ctx_t*,const WCHAR*,const WCHAR*,BOOL,BOOL,bytecode_t**) DECLSPEC_HIDDEN; +HRESULT compile_script(script_ctx_t*,const WCHAR*,const WCHAR*,const WCHAR*,BOOL,BOOL,bytecode_t**) DECLSPEC_HIDDEN; void release_bytecode(bytecode_t*) DECLSPEC_HIDDEN; static inline void bytecode_addref(bytecode_t *code) diff --git a/dlls/jscript/function.c b/dlls/jscript/function.c index f1ed559..7a32de7 100644 --- a/dlls/jscript/function.c +++ b/dlls/jscript/function.c @@ -750,7 +750,7 @@ static HRESULT construct_function(script_ctx_t *ctx, unsigned argc, jsval_t *arg if(FAILED(hres)) return hres; - hres = compile_script(ctx, str, NULL, FALSE, FALSE, &code); + hres = compile_script(ctx, str, NULL, NULL, FALSE, FALSE, &code); heap_free(str); if(FAILED(hres)) return hres; diff --git a/dlls/jscript/global.c b/dlls/jscript/global.c index 60aae51..1c0f72c 100644 --- a/dlls/jscript/global.c +++ b/dlls/jscript/global.c @@ -370,7 +370,7 @@ static HRESULT JSGlobal_eval(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, uns } TRACE("parsing %s\n", debugstr_jsval(argv[0])); - hres = compile_script(ctx, get_string(argv[0]), NULL, TRUE, FALSE, &code); + hres = compile_script(ctx, get_string(argv[0]), NULL, NULL, TRUE, FALSE, &code); if(FAILED(hres)) { WARN("parse (%s) failed: %08x\n", debugstr_jsval(argv[0]), hres); return throw_syntax_error(ctx, hres, NULL); diff --git a/dlls/jscript/jscript.c b/dlls/jscript/jscript.c index e38987c..8dfa0c8 100644 --- a/dlls/jscript/jscript.c +++ b/dlls/jscript/jscript.c @@ -763,7 +763,7 @@ static HRESULT WINAPI JScriptParse_ParseScriptText(IActiveScriptParse *iface, if(This->thread_id != GetCurrentThreadId() || This->ctx->state == SCRIPTSTATE_CLOSED) return E_UNEXPECTED; - hres = compile_script(This->ctx, pstrCode, pstrDelimiter, FALSE, This->is_encode, &code); + hres = compile_script(This->ctx, pstrCode, NULL, pstrDelimiter, FALSE, This->is_encode, &code); if(FAILED(hres)) return hres; @@ -830,7 +830,7 @@ static HRESULT WINAPI JScriptParseProcedure_ParseProcedureText(IActiveScriptPars if(This->thread_id != GetCurrentThreadId() || This->ctx->state == SCRIPTSTATE_CLOSED) return E_UNEXPECTED; - hres = compile_script(This->ctx, pstrCode, pstrDelimiter, FALSE, This->is_encode, &code); + hres = compile_script(This->ctx, pstrCode, pstrFormalParams, pstrDelimiter, FALSE, This->is_encode, &code); if(FAILED(hres)) { WARN("Parse failed %08x\n", hres); return hres; diff --git a/dlls/jscript/tests/run.c b/dlls/jscript/tests/run.c index e360515..f0aff60 100644 --- a/dlls/jscript/tests/run.c +++ b/dlls/jscript/tests/run.c @@ -1927,6 +1927,17 @@ static void test_parse_proc(void) dp.cArgs = 1; V_VT(args) = VT_EMPTY; invoke_procedure(NULL, "return arguments.length == 1;", &dp); + + V_VT(args) = VT_BOOL; + V_BOOL(args) = VARIANT_TRUE; + invoke_procedure(" x ", "return x;", &dp); + + dp.cArgs = 2; + V_VT(args) = VT_I4; + V_I4(args) = 2; + V_VT(args+1) = VT_I4; + V_I4(args+1) = 1; + invoke_procedure(" _x1 , y_2", "return _x1 === 1 && y_2 === 2;", &dp); } static void run_encoded_tests(void) diff --git a/dlls/mshtml/tests/activex.c b/dlls/mshtml/tests/activex.c index 8e4ee58..e861d37 100644 --- a/dlls/mshtml/tests/activex.c +++ b/dlls/mshtml/tests/activex.c @@ -2355,10 +2355,8 @@ static void test_event_call(void) dp.cArgs = 2; V_VT(&res) = VT_EMPTY; hres = IDispatch_Invoke(sink_disp, 2, &IID_NULL, 0, DISPATCH_METHOD, &dp, &res, &ei, NULL); - todo_wine { /* Needs jscript fixes */ ok(hres == S_OK, "Invoke failed: %08x\n", hres); ok(V_VT(&res) == VT_I4 && V_I4(&res) == 7, "unexpected result: %d\n", V_I4(&res)); - } V_VT(&res) = VT_ERROR; hres = IDispatch_Invoke(sink_disp, 10, &IID_NULL, 0, DISPATCH_METHOD, &dp, &res, &ei, NULL);

1 0

Henri Verbeet : wined3d: Just return the surface from wined3d_swapchain_get_back_buffer().
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: 5fcc1f8f70100a3b1b29c36d53fa69d11804c2a4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5fcc1f8f70100a3b1b29c36d5… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Oct 4 13:53:27 2012 +0200 wined3d: Just return the surface from wined3d_swapchain_get_back_buffer(). --- dlls/d3d8/device.c | 1 - dlls/d3d8/swapchain.c | 12 +++++++----- dlls/d3d9/device.c | 1 - dlls/d3d9/swapchain.c | 12 +++++++----- dlls/dxgi/swapchain.c | 8 +++----- dlls/wined3d/device.c | 4 +++- dlls/wined3d/stateblock.c | 5 +---- dlls/wined3d/swapchain.c | 19 +++++++------------ dlls/wined3d/wined3d.spec | 2 +- include/wine/wined3d.h | 4 ++-- 10 files changed, 31 insertions(+), 37 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=5fcc1f8f70100a3b1b29c…

1 0

Henri Verbeet : d3d10core: Implement d3d10_device_OMGetBlendState().
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: 272f63263e4fc8647052067daa021671816caab6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=272f63263e4fc8647052067da… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Oct 4 13:53:26 2012 +0200 d3d10core: Implement d3d10_device_OMGetBlendState(). --- dlls/d3d10core/device.c | 9 ++++++++- 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/dlls/d3d10core/device.c b/dlls/d3d10core/device.c index 95b337a..43cafa5 100644 --- a/dlls/d3d10core/device.c +++ b/dlls/d3d10core/device.c @@ -571,8 +571,15 @@ static void STDMETHODCALLTYPE d3d10_device_OMGetRenderTargets(ID3D10Device *ifac static void STDMETHODCALLTYPE d3d10_device_OMGetBlendState(ID3D10Device *iface, ID3D10BlendState **blend_state, FLOAT blend_factor[4], UINT *sample_mask) { - FIXME("iface %p, blend_state %p, blend_factor %p, sample_mask %p stub!\n", + struct d3d10_device *device = impl_from_ID3D10Device(iface); + + TRACE("iface %p, blend_state %p, blend_factor %p, sample_mask %p.\n", iface, blend_state, blend_factor, sample_mask); + + if ((*blend_state = device->blend_state ? &device->blend_state->ID3D10BlendState_iface : NULL)) + ID3D10BlendState_AddRef(*blend_state); + memcpy(blend_factor, device->blend_factor, 4 * sizeof(*blend_factor)); + *sample_mask = device->sample_mask; } static void STDMETHODCALLTYPE d3d10_device_OMGetDepthStencilState(ID3D10Device *iface,

1 0

Piotr Caban : msvcp60: Copy source files from msvcp90.
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: 9d7f3c7651940c7f3164ccf09b9576bb563586f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9d7f3c7651940c7f3164ccf09… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Oct 4 13:15:41 2012 +0200 msvcp60: Copy source files from msvcp90. --- dlls/msvcp60/Makefile.in | 4 +- dlls/msvcp60/cxx.h | 2 - dlls/msvcp60/exception.c | 36 +- dlls/msvcp60/ios.c |11564 +++++++++++++++++++++++++++++++++++++++++++++ dlls/msvcp60/locale.c | 8198 ++++++++++++++++++++++++++++++++ dlls/msvcp60/main.c | 9 +- dlls/msvcp60/memory.c | 264 +- dlls/msvcp60/misc.c | 42 +- dlls/msvcp60/msvcp.h | 396 ++- dlls/msvcp60/msvcp60.spec | 3208 +++++++------- dlls/msvcp60/string.c | 67 +- 11 files changed, 21851 insertions(+), 1939 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=9d7f3c7651940c7f3164c…

1 0

Piotr Caban : msvcp60: Fixed vtables in IO classes.
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: b84d5d991ce0a6e8da7bc3af94795d88fdab516a URL: http://source.winehq.org/git/wine.git/?a=commit;h=b84d5d991ce0a6e8da7bc3af9… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Oct 4 13:16:34 2012 +0200 msvcp60: Fixed vtables in IO classes. --- dlls/msvcp60/ios.c | 56 ++++++++++++++++++--------------------------------- 1 files changed, 20 insertions(+), 36 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=b84d5d991ce0a6e8da7bc…

1 0

Francois Gouget : shell32/tests: Write proper tests for CommandLineToArgvW( ).
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: 9ec7ab3fa03ca5ba467cfb7a856d50014bae5211 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9ec7ab3fa03ca5ba467cfb7a8… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Thu Oct 4 09:54:16 2012 +0200 shell32/tests: Write proper tests for CommandLineToArgvW(). --- dlls/kernel32/tests/process.c | 12 -- dlls/shell32/tests/shlexec.c | 326 +++++++++++++++++++++++++++++------------ 2 files changed, 232 insertions(+), 106 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=9ec7ab3fa03ca5ba467cf…

1 0

Piotr Caban : msvcp90: Added _Winit class implementation.
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: ea5b845b32913048931864750fcb78630d501e4b URL: http://source.winehq.org/git/wine.git/?a=commit;h=ea5b845b32913048931864750… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Oct 4 13:14:15 2012 +0200 msvcp90: Added _Winit class implementation. --- dlls/msvcp100/ios.c | 42 ++++++++++++++++++++++++++++++++++++++++++ dlls/msvcp100/msvcp100.spec | 14 +++++++------- dlls/msvcp71/ios.c | 42 ++++++++++++++++++++++++++++++++++++++++++ dlls/msvcp71/msvcp71.spec | 14 +++++++------- dlls/msvcp90/ios.c | 42 ++++++++++++++++++++++++++++++++++++++++++ dlls/msvcp90/msvcp90.spec | 14 +++++++------- 6 files changed, 147 insertions(+), 21 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=ea5b845b3291304893186…

1 0

Piotr Caban : msvcp90: Added ios_base::Init class implementation.
by Alexandre Julliard Oct. 4, 2012

Oct. 4, 2012

Module: wine Branch: master Commit: a221f87709c959a10a69cc819687cdd11d04971d URL: http://source.winehq.org/git/wine.git/?a=commit;h=a221f87709c959a10a69cc819… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Oct 4 13:13:56 2012 +0200 msvcp90: Added ios_base::Init class implementation. --- dlls/msvcp100/ios.c | 70 ++++++++++++++++++++++++++++++++++++++++-- dlls/msvcp100/msvcp100.spec | 30 +++++++++--------- dlls/msvcp71/ios.c | 63 ++++++++++++++++++++++++++++++++++++-- dlls/msvcp71/msvcp71.spec | 18 +++++----- dlls/msvcp90/ios.c | 70 ++++++++++++++++++++++++++++++++++++++++-- dlls/msvcp90/msvcp90.spec | 30 +++++++++--------- 6 files changed, 230 insertions(+), 51 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a221f87709c959a10a69c…

1 0