July 2014 - Wine-Commits - list.winehq.org

Jacek Caban : mshtml: Added IDispatch support for IHTMLTable[23] interfaces.
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: cf50d5f70054c9759f4da89343c0ced62e76643f URL: http://source.winehq.org/git/wine.git/?a=commit;h=cf50d5f70054c9759f4da8934… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Mon Jul 14 11:57:36 2014 +0200 mshtml: Added IDispatch support for IHTMLTable[23] interfaces. --- dlls/mshtml/htmltable.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dlls/mshtml/htmltable.c b/dlls/mshtml/htmltable.c index fd7c265..de12537 100644 --- a/dlls/mshtml/htmltable.c +++ b/dlls/mshtml/htmltable.c @@ -946,6 +946,8 @@ static const NodeImplVtbl HTMLTableImplVtbl = { static const tid_t HTMLTable_iface_tids[] = { HTMLELEMENT_TIDS, IHTMLTable_tid, + IHTMLTable2_tid, + IHTMLTable3_tid, 0 };

1 0

Henri Verbeet : wined3d: Make wined3d_arbfp_blit_rb_functions static.
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: 8b8af4bee0ad1f65cfbdd4d3c951b71186eee68a URL: http://source.winehq.org/git/wine.git/?a=commit;h=8b8af4bee0ad1f65cfbdd4d3c… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jul 14 10:04:25 2014 +0200 wined3d: Make wined3d_arbfp_blit_rb_functions static. --- dlls/wined3d/arb_program_shader.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index 819a04f..d312ecf 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -6791,7 +6791,7 @@ static void arbfp_free_blit_shader(struct wine_rb_entry *entry, void *context) HeapFree(GetProcessHeap(), 0, entry_arb); } -const struct wine_rb_functions wined3d_arbfp_blit_rb_functions = +static const struct wine_rb_functions wined3d_arbfp_blit_rb_functions = { wined3d_rb_alloc, wined3d_rb_realloc,

1 0

Henri Verbeet : wined3d: Always mark vertex buffers as used in context_apply_draw_state().
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: 68339456a8f7543ae46ac504c2ce7f4f0221a789 URL: http://source.winehq.org/git/wine.git/?a=commit;h=68339456a8f7543ae46ac504c… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Jul 14 10:04:21 2014 +0200 wined3d: Always mark vertex buffers as used in context_apply_draw_state(). The issue this solves is that buffer_internal_preload() is only called when STATE_VDECL or STATE_STREAMSRC is invalidated, but that doesn't happen for regular buffer maps. We'd like to be able to just call buffer_internal_preload() here, but buffer_internal_preload() has dependencies on context_stream_info_from_declaration() and vice versa. --- dlls/wined3d/buffer.c | 7 ++++++- dlls/wined3d/context.c | 11 +++++++++++ dlls/wined3d/wined3d_private.h | 1 + 3 files changed, 18 insertions(+), 1 deletion(-) diff --git a/dlls/wined3d/buffer.c b/dlls/wined3d/buffer.c index fbbe398..901382c 100644 --- a/dlls/wined3d/buffer.c +++ b/dlls/wined3d/buffer.c @@ -721,6 +721,11 @@ static void buffer_direct_upload(struct wined3d_buffer *This, const struct wined checkGLcall("glUnmapBufferARB"); } +void buffer_mark_used(struct wined3d_buffer *buffer) +{ + buffer->flags &= ~(WINED3D_BUFFER_NOSYNC | WINED3D_BUFFER_DISCARD); +} + /* Context activation is done by the caller. */ void buffer_internal_preload(struct wined3d_buffer *buffer, struct wined3d_context *context, const struct wined3d_state *state) @@ -741,7 +746,7 @@ void buffer_internal_preload(struct wined3d_buffer *buffer, struct wined3d_conte return; } - buffer->flags &= ~(WINED3D_BUFFER_NOSYNC | WINED3D_BUFFER_DISCARD); + buffer_mark_used(buffer); if (!buffer->buffer_object) { diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 32cc191..e8423f4 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -2958,6 +2958,7 @@ BOOL context_apply_draw_state(struct wined3d_context *context, struct wined3d_de const struct StateEntry *state_table = context->state_table; const struct wined3d_fb_state *fb = state->fb; unsigned int i; + WORD map; if (!context_validate_rt_config(context->gl_info->limits.buffers, fb->render_targets, fb->depth_stencil)) @@ -2974,7 +2975,17 @@ BOOL context_apply_draw_state(struct wined3d_context *context, struct wined3d_de context_update_tex_unit_map(context, state); context_preload_textures(context, state); if (isStateDirty(context, STATE_VDECL) || isStateDirty(context, STATE_STREAMSRC)) + { context_update_stream_info(context, state); + } + else + { + for (i = 0, map = context->stream_info.use_map; map; map >>= 1, ++i) + { + if (map & 1) + buffer_mark_used(state->streams[context->stream_info.elements[i].stream_idx].buffer); + } + } if (state->index_buffer) { if (context->stream_info.all_vbo) diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 8f411cd..0c11528 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2590,6 +2590,7 @@ void buffer_get_memory(struct wined3d_buffer *buffer, struct wined3d_context *co BYTE *buffer_get_sysmem(struct wined3d_buffer *This, struct wined3d_context *context) DECLSPEC_HIDDEN; void buffer_internal_preload(struct wined3d_buffer *buffer, struct wined3d_context *context, const struct wined3d_state *state) DECLSPEC_HIDDEN; +void buffer_mark_used(struct wined3d_buffer *buffer) DECLSPEC_HIDDEN; struct wined3d_rendertarget_view {

1 0

Austin English : setupapi: Add a stub for SetupDiSetClassInstallParamsW.
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: 3013b6c6da9f02fa7afc3b49a036d063d4c4837a URL: http://source.winehq.org/git/wine.git/?a=commit;h=3013b6c6da9f02fa7afc3b49a… Author: Austin English <austinenglish(a)gmail.com> Date: Sun Jul 13 16:45:25 2014 -0700 setupapi: Add a stub for SetupDiSetClassInstallParamsW. --- dlls/setupapi/devinst.c | 14 ++++++++++++++ dlls/setupapi/setupapi.spec | 2 +- 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index ef56d2b..966af75 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -3648,6 +3648,20 @@ BOOL WINAPI SetupDiSetClassInstallParamsA( } /*********************************************************************** + * SetupDiSetClassInstallParamsW (SETUPAPI.@) + */ +BOOL WINAPI SetupDiSetClassInstallParamsW( + HDEVINFO DeviceInfoSet, + PSP_DEVINFO_DATA DeviceInfoData, + PSP_CLASSINSTALL_HEADER ClassInstallParams, + DWORD ClassInstallParamsSize) +{ + FIXME("%p %p %x %u\n",DeviceInfoSet, DeviceInfoData, + ClassInstallParams->InstallFunction, ClassInstallParamsSize); + return FALSE; +} + +/*********************************************************************** * SetupDiCallClassInstaller (SETUPAPI.@) */ BOOL WINAPI SetupDiCallClassInstaller( diff --git a/dlls/setupapi/setupapi.spec b/dlls/setupapi/setupapi.spec index a8ce41a..48ab1e4 100644 --- a/dlls/setupapi/setupapi.spec +++ b/dlls/setupapi/setupapi.spec @@ -384,7 +384,7 @@ @ stub SetupDiSelectDevice @ stub SetupDiSelectOEMDrv @ stdcall SetupDiSetClassInstallParamsA(ptr ptr ptr long) -@ stub SetupDiSetClassInstallParamsW +@ stdcall SetupDiSetClassInstallParamsW(ptr ptr ptr long) @ stdcall SetupDiSetDeviceInstallParamsA(ptr ptr ptr) @ stub SetupDiSetDeviceInstallParamsW @ stdcall SetupDiSetDeviceRegistryPropertyA(ptr ptr long ptr ptr)

1 0

Bruno Jesus : wineconsole: Free registry key name (valgrind).
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: 421d3cd9e802ec63fe5928b4f331ff85064f83d6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=421d3cd9e802ec63fe5928b4f… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Sat Jul 12 16:02:13 2014 -0300 wineconsole: Free registry key name (valgrind). --- programs/wineconsole/wineconsole.c | 1 + 1 file changed, 1 insertion(+) diff --git a/programs/wineconsole/wineconsole.c b/programs/wineconsole/wineconsole.c index 7698521..3c2faf5 100644 --- a/programs/wineconsole/wineconsole.c +++ b/programs/wineconsole/wineconsole.c @@ -519,6 +519,7 @@ static void WINECON_Delete(struct inner_data* data) if (data->hConIn) CloseHandle(data->hConIn); if (data->hConOut) CloseHandle(data->hConOut); if (data->hSynchro) CloseHandle(data->hSynchro); + HeapFree(GetProcessHeap(), 0, data->curcfg.registry); HeapFree(GetProcessHeap(), 0, data->cells); HeapFree(GetProcessHeap(), 0, data); }

1 0

Bruno Jesus : psapi/tests: Free buffer on error (valgrind).
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: 78eac8156e93d65cc15c7e7241ec90639bea3d24 URL: http://source.winehq.org/git/wine.git/?a=commit;h=78eac8156e93d65cc15c7e724… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Sat Jul 12 15:24:36 2014 -0300 psapi/tests: Free buffer on error (valgrind). --- dlls/psapi/tests/psapi_main.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/dlls/psapi/tests/psapi_main.c b/dlls/psapi/tests/psapi_main.c index ee7e8bd..62d4f5c 100644 --- a/dlls/psapi/tests/psapi_main.c +++ b/dlls/psapi/tests/psapi_main.c @@ -228,7 +228,11 @@ static BOOL nt_get_mapped_file_name(HANDLE process, LPVOID addr, LPWSTR name, DW todo_wine ok(!status, "NtQueryVirtualMemory error %x\n", status); /* FIXME: remove once Wine is fixed */ - if (status) return FALSE; + if (status) + { + HeapFree(GetProcessHeap(), 0, buf); + return FALSE; + } section_name = (MEMORY_SECTION_NAME *)buf; ok(ret_len == section_name->SectionFileName.MaximumLength + sizeof(*section_name), "got %lu, %u\n",

1 0

Bruno Jesus : dinput/tests: Release the instance after tests (valgrind).
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: 1242417d131ae9fafd66ee79e96352a1e0215fc6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1242417d131ae9fafd66ee79e… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Sat Jul 12 13:40:04 2014 -0300 dinput/tests: Release the instance after tests (valgrind). --- dlls/dinput/tests/dinput.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/dinput/tests/dinput.c b/dlls/dinput/tests/dinput.c index cc28985..0490642 100644 --- a/dlls/dinput/tests/dinput.c +++ b/dlls/dinput/tests/dinput.c @@ -603,6 +603,7 @@ static void test_DirectInputJoyConfig8(void) "IDirectInput_CreateDevice failed with guid from GetConfig hr = 0x%08x\n", hr); } + IDirectInput_Release(pDI); } START_TEST(dinput)

1 0

Piotr Caban : usp10: Limit trace length in dump_types.
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: 6d7ce98fec4bbdff86bd901e91e5636bd6680bcf URL: http://source.winehq.org/git/wine.git/?a=commit;h=6d7ce98fec4bbdff86bd901e9… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Sat Jul 12 11:56:56 2014 +0200 usp10: Limit trace length in dump_types. --- dlls/usp10/bidi.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/dlls/usp10/bidi.c b/dlls/usp10/bidi.c index 63ffa6a..2926454 100644 --- a/dlls/usp10/bidi.c +++ b/dlls/usp10/bidi.c @@ -146,10 +146,15 @@ static const char debug_type[][4] = static inline void dump_types(const char* header, WORD *types, int start, int end) { - int i; + int i, len = 0; TRACE("%s:",header); - for (i = start; i< end; i++) + for (i = start; i < end && len < 200; i++) + { TRACE(" %s",debug_type[types[i]]); + len += strlen(debug_type[types[i]])+1; + } + if (i != end) + TRACE("..."); TRACE("\n"); }

1 0

Thomas Faber : dxdiag: Do not omit mandatory argument to WriteFile.
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: f325bf745872cfeec383f33ea2b19dcaa67361c5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f325bf745872cfeec383f33ea… Author: Thomas Faber <thomas.faber(a)reactos.org> Date: Sat Jul 12 10:40:12 2014 +0200 dxdiag: Do not omit mandatory argument to WriteFile. --- programs/dxdiag/output.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/programs/dxdiag/output.c b/programs/dxdiag/output.c index 7fb9a4c..c18c75d 100644 --- a/programs/dxdiag/output.c +++ b/programs/dxdiag/output.c @@ -72,6 +72,7 @@ static BOOL output_text_header(HANDLE hFile, const char *caption) DWORD len = strlen(caption); DWORD total_len = 3 * (len + sizeof(crlf)); char *ptr = output_buffer; + DWORD bytes_written; assert(total_len <= sizeof(output_buffer)); @@ -92,7 +93,7 @@ static BOOL output_text_header(HANDLE hFile, const char *caption) memcpy(ptr, crlf, sizeof(crlf)); - return WriteFile(hFile, output_buffer, total_len, NULL, NULL); + return WriteFile(hFile, output_buffer, total_len, &bytes_written, NULL); } static BOOL output_text_field(HANDLE hFile, const char *field_name, DWORD field_width, const WCHAR *value) @@ -102,6 +103,7 @@ static BOOL output_text_field(HANDLE hFile, const char *field_name, DWORD field_ DWORD total_len = field_width + sizeof(": ") - 1 + value_lenA + sizeof(crlf); char sprintf_fmt[1 + 10 + 3 + 1]; char *ptr = output_buffer; + DWORD bytes_written; assert(total_len <= sizeof(output_buffer)); @@ -111,12 +113,13 @@ static BOOL output_text_field(HANDLE hFile, const char *field_name, DWORD field_ ptr += WideCharToMultiByte(CP_ACP, 0, value, value_lenW, ptr, value_lenA, NULL, NULL); memcpy(ptr, crlf, sizeof(crlf)); - return WriteFile(hFile, output_buffer, total_len, NULL, NULL); + return WriteFile(hFile, output_buffer, total_len, &bytes_written, NULL); } static BOOL output_crlf(HANDLE hFile) { - return WriteFile(hFile, crlf, sizeof(crlf), NULL, NULL); + DWORD bytes_written; + return WriteFile(hFile, crlf, sizeof(crlf), &bytes_written, NULL); } static inline void fill_system_text_output_table(struct dxdiag_information *dxdiag_info, struct text_information_field *fields)

1 0

Piotr Caban : usp10: Limit trace length in iso_dump_types.
by Alexandre Julliard July 14, 2014

July 14, 2014

Module: wine Branch: master Commit: 43871921204d4b598f55b6be1bfab076572ae69c URL: http://source.winehq.org/git/wine.git/?a=commit;h=43871921204d4b598f55b6be1… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Sat Jul 12 11:57:12 2014 +0200 usp10: Limit trace length in iso_dump_types. --- dlls/usp10/bidi.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/dlls/usp10/bidi.c b/dlls/usp10/bidi.c index 2926454..5b227d8 100644 --- a/dlls/usp10/bidi.c +++ b/dlls/usp10/bidi.c @@ -513,11 +513,16 @@ static inline int iso_previousChar(IsolatedRun *iso_run, int index) static inline void iso_dump_types(const char* header, IsolatedRun *iso_run) { - int i; + int i, len = 0; TRACE("%s:",header); TRACE("[ "); - for (i = 0; i < iso_run->length; i++) + for (i = 0; i < iso_run->length && len < 200; i++) + { TRACE(" %s",debug_type[*iso_run->item[i].pcls]); + len += strlen(debug_type[*iso_run->item[i].pcls])+1; + } + if (i != iso_run->length) + TRACE("..."); TRACE(" ]\n"); }

1 0