January 2013 - Wine-Commits

Alexandre Julliard : gdi32: Store the full flags in the face object, and add a flag for fonts added through AddFontResource.
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: f90696e813ce636fec5285424bbd152c740ea72e URL: http://source.winehq.org/git/wine.git/?a=commit;h=f90696e813ce636fec5285424… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Jan 14 15:28:42 2013 +0100 gdi32: Store the full flags in the face object, and add a flag for fonts added through AddFontResource. --- dlls/gdi32/freetype.c | 59 +++++++++++++++++++++++------------------------- 1 files changed, 28 insertions(+), 31 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=f90696e813ce636fec528…

1 0

Ken Thomases : winemac: Implement window surface rendering.
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: 7d6ebfa742eac87c9f5ac5e228c0c5f68e785f9f URL: http://source.winehq.org/git/wine.git/?a=commit;h=7d6ebfa742eac87c9f5ac5e22… Author: Ken Thomases <ken(a)codeweavers.com> Date: Mon Jan 14 20:23:55 2013 -0600 winemac: Implement window surface rendering. --- dlls/winemac.drv/Makefile.in | 1 + dlls/winemac.drv/cocoa_window.h | 3 + dlls/winemac.drv/cocoa_window.m | 82 ++++++++++ dlls/winemac.drv/macdrv.h | 4 + dlls/winemac.drv/macdrv_cocoa.h | 6 + dlls/winemac.drv/surface.c | 340 +++++++++++++++++++++++++++++++++++++++ dlls/winemac.drv/window.c | 107 ++++++++++++- 7 files changed, 542 insertions(+), 1 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=7d6ebfa742eac87c9f5ac…

1 0

Ken Thomases : winemac: Implement layered windows: SetLayeredWindowAttributes() and UpdateLayeredWindow().
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: c0d43fb51f2466660bf027b766c501c7ce1c9874 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c0d43fb51f2466660bf027b76… Author: Ken Thomases <ken(a)codeweavers.com> Date: Mon Jan 14 20:24:02 2013 -0600 winemac: Implement layered windows: SetLayeredWindowAttributes() and UpdateLayeredWindow(). --- dlls/winemac.drv/cocoa_window.h | 5 + dlls/winemac.drv/cocoa_window.m | 90 ++++++++++++++- dlls/winemac.drv/macdrv.h | 6 +- dlls/winemac.drv/macdrv_cocoa.h | 5 + dlls/winemac.drv/surface.c | 17 +++- dlls/winemac.drv/window.c | 232 ++++++++++++++++++++++++++++++++++++- dlls/winemac.drv/winemac.drv.spec | 2 + 7 files changed, 350 insertions(+), 7 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=c0d43fb51f2466660bf02…

1 0

Ken Thomases : winemac: Implement SetWindowRgn.
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: 2d4bcc47be73ae8ca78a21d7c36b04685fa78238 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2d4bcc47be73ae8ca78a21d7c… Author: Ken Thomases <ken(a)codeweavers.com> Date: Mon Jan 14 20:23:58 2013 -0600 winemac: Implement SetWindowRgn. --- dlls/winemac.drv/cocoa_window.h | 3 + dlls/winemac.drv/cocoa_window.m | 81 +++++++++++++++++ dlls/winemac.drv/macdrv.h | 7 ++ dlls/winemac.drv/macdrv_cocoa.h | 1 + dlls/winemac.drv/window.c | 172 +++++++++++++++++++++++++++++++++---- dlls/winemac.drv/winemac.drv.spec | 2 + 6 files changed, 250 insertions(+), 16 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=2d4bcc47be73ae8ca78a2…

1 0

Henri Verbeet : ddraw: Add a small tests for SetCooperativeLevel() calls with different windows.
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: ae725df54a4ea8c414043b252929bd85f7ce33f0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ae725df54a4ea8c414043b252… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Jan 15 08:46:53 2013 +0100 ddraw: Add a small tests for SetCooperativeLevel() calls with different windows. In particular, test that the windows aren't destroyed by accident. --- dlls/ddraw/tests/ddraw1.c | 31 +++++++++++++++++++++++++++++++ dlls/ddraw/tests/ddraw2.c | 31 +++++++++++++++++++++++++++++++ dlls/ddraw/tests/ddraw4.c | 31 +++++++++++++++++++++++++++++++ dlls/ddraw/tests/ddraw7.c | 31 +++++++++++++++++++++++++++++++ 4 files changed, 124 insertions(+), 0 deletions(-) diff --git a/dlls/ddraw/tests/ddraw1.c b/dlls/ddraw/tests/ddraw1.c index cf83b98..a37e0b7 100644 --- a/dlls/ddraw/tests/ddraw1.c +++ b/dlls/ddraw/tests/ddraw1.c @@ -2629,6 +2629,36 @@ static void test_coop_level_surf_create(void) IDirectDraw_Release(ddraw); } +static void test_coop_level_multi_window(void) +{ + HWND window1, window2; + IDirectDraw *ddraw; + HRESULT hr; + + window1 = CreateWindowA("static", "ddraw_test1", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + window2 = CreateWindowA("static", "ddraw_test2", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + if (!(ddraw = create_ddraw())) + { + skip("Failed to create a ddraw object, skipping test.\n"); + DestroyWindow(window2); + DestroyWindow(window1); + return; + } + + hr = IDirectDraw_SetCooperativeLevel(ddraw, window1, DDSCL_NORMAL); + ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); + hr = IDirectDraw_SetCooperativeLevel(ddraw, window2, DDSCL_NORMAL); + ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); + todo_wine ok(IsWindow(window1), "Window 1 was destroyed.\n"); + ok(IsWindow(window2), "Window 2 was destroyed.\n"); + + IDirectDraw_Release(ddraw); + DestroyWindow(window2); + DestroyWindow(window1); +} + START_TEST(ddraw1) { test_coop_level_create_device_window(); @@ -2649,4 +2679,5 @@ START_TEST(ddraw1) test_coop_level_mode_set_multi(); test_initialize(); test_coop_level_surf_create(); + test_coop_level_multi_window(); } diff --git a/dlls/ddraw/tests/ddraw2.c b/dlls/ddraw/tests/ddraw2.c index 303c634..e475f5b 100644 --- a/dlls/ddraw/tests/ddraw2.c +++ b/dlls/ddraw/tests/ddraw2.c @@ -2748,6 +2748,36 @@ static void test_coop_level_surf_create(void) IDirectDraw2_Release(ddraw); } +static void test_coop_level_multi_window(void) +{ + HWND window1, window2; + IDirectDraw2 *ddraw; + HRESULT hr; + + window1 = CreateWindowA("static", "ddraw_test1", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + window2 = CreateWindowA("static", "ddraw_test2", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + if (!(ddraw = create_ddraw())) + { + skip("Failed to create a ddraw object, skipping test.\n"); + DestroyWindow(window2); + DestroyWindow(window1); + return; + } + + hr = IDirectDraw2_SetCooperativeLevel(ddraw, window1, DDSCL_NORMAL); + ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); + hr = IDirectDraw2_SetCooperativeLevel(ddraw, window2, DDSCL_NORMAL); + ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); + todo_wine ok(IsWindow(window1), "Window 1 was destroyed.\n"); + ok(IsWindow(window2), "Window 2 was destroyed.\n"); + + IDirectDraw2_Release(ddraw); + DestroyWindow(window2); + DestroyWindow(window1); +} + START_TEST(ddraw2) { test_coop_level_create_device_window(); @@ -2770,4 +2800,5 @@ START_TEST(ddraw2) test_coop_level_mode_set_multi(); test_initialize(); test_coop_level_surf_create(); + test_coop_level_multi_window(); } diff --git a/dlls/ddraw/tests/ddraw4.c b/dlls/ddraw/tests/ddraw4.c index 2e9c845..82cd5d5 100644 --- a/dlls/ddraw/tests/ddraw4.c +++ b/dlls/ddraw/tests/ddraw4.c @@ -2935,6 +2935,36 @@ static void test_vb_discard(void) DestroyWindow(window); } +static void test_coop_level_multi_window(void) +{ + HWND window1, window2; + IDirectDraw4 *ddraw; + HRESULT hr; + + window1 = CreateWindowA("static", "ddraw_test1", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + window2 = CreateWindowA("static", "ddraw_test2", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + if (!(ddraw = create_ddraw())) + { + skip("Failed to create a ddraw object, skipping test.\n"); + DestroyWindow(window2); + DestroyWindow(window1); + return; + } + + hr = IDirectDraw4_SetCooperativeLevel(ddraw, window1, DDSCL_NORMAL); + ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); + hr = IDirectDraw4_SetCooperativeLevel(ddraw, window2, DDSCL_NORMAL); + ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); + todo_wine ok(IsWindow(window1), "Window 1 was destroyed.\n"); + ok(IsWindow(window2), "Window 2 was destroyed.\n"); + + IDirectDraw4_Release(ddraw); + DestroyWindow(window2); + DestroyWindow(window1); +} + START_TEST(ddraw4) { test_process_vertices(); @@ -2959,4 +2989,5 @@ START_TEST(ddraw4) test_initialize(); test_coop_level_surf_create(); test_vb_discard(); + test_coop_level_multi_window(); } diff --git a/dlls/ddraw/tests/ddraw7.c b/dlls/ddraw/tests/ddraw7.c index 2b7e075..cd22e0c 100644 --- a/dlls/ddraw/tests/ddraw7.c +++ b/dlls/ddraw/tests/ddraw7.c @@ -2744,6 +2744,36 @@ static void test_vb_discard(void) DestroyWindow(window); } +static void test_coop_level_multi_window(void) +{ + HWND window1, window2; + IDirectDraw7 *ddraw; + HRESULT hr; + + window1 = CreateWindowA("static", "ddraw_test1", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + window2 = CreateWindowA("static", "ddraw_test2", WS_OVERLAPPEDWINDOW, + 0, 0, 640, 480, 0, 0, 0, 0); + if (!(ddraw = create_ddraw())) + { + skip("Failed to create a ddraw object, skipping test.\n"); + DestroyWindow(window2); + DestroyWindow(window1); + return; + } + + hr = IDirectDraw7_SetCooperativeLevel(ddraw, window1, DDSCL_NORMAL); + ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); + hr = IDirectDraw7_SetCooperativeLevel(ddraw, window2, DDSCL_NORMAL); + ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); + todo_wine ok(IsWindow(window1), "Window 1 was destroyed.\n"); + ok(IsWindow(window2), "Window 2 was destroyed.\n"); + + IDirectDraw7_Release(ddraw); + DestroyWindow(window2); + DestroyWindow(window1); +} + START_TEST(ddraw7) { HMODULE module = GetModuleHandleA("ddraw.dll"); @@ -2775,4 +2805,5 @@ START_TEST(ddraw7) test_initialize(); test_coop_level_surf_create(); test_vb_discard(); + test_coop_level_multi_window(); }

1 0

Hans Leidekker : wmiutils: Implement IWbemPath::GetInfo.
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: 47e9ffa773a0c96e2ac64005f9d9b50f152442e2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=47e9ffa773a0c96e2ac64005f… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Jan 15 11:26:48 2013 +0100 wmiutils: Implement IWbemPath::GetInfo. --- dlls/wmiutils/path.c | 56 ++++++++++++++++++++++++++++--------- dlls/wmiutils/tests/path.c | 64 ++++++++++++++++++++++++++++++++++++++++++++ include/wmiutils.idl | 22 +++++++++++++++ 3 files changed, 128 insertions(+), 14 deletions(-) diff --git a/dlls/wmiutils/path.c b/dlls/wmiutils/path.c index abdf0a5..f02c9c6 100644 --- a/dlls/wmiutils/path.c +++ b/dlls/wmiutils/path.c @@ -35,16 +35,17 @@ WINE_DEFAULT_DEBUG_CHANNEL(wmiutils); struct path { IWbemPath IWbemPath_iface; - LONG refs; - WCHAR *text; - int len_text; - WCHAR *server; - int len_server; - WCHAR **namespaces; - int *len_namespaces; - int num_namespaces; - WCHAR *class; - int len_class; + LONG refs; + WCHAR *text; + int len_text; + WCHAR *server; + int len_server; + WCHAR **namespaces; + int *len_namespaces; + int num_namespaces; + WCHAR *class; + int len_class; + ULONGLONG flags; }; static void init_path( struct path *path ) @@ -58,6 +59,7 @@ static void init_path( struct path *path ) path->num_namespaces = 0; path->class = NULL; path->len_class = 0; + path->flags = 0; } static void clear_path( struct path *path ) @@ -136,6 +138,7 @@ static HRESULT parse_text( struct path *path, ULONG mode, const WCHAR *text ) memcpy( path->server, p, len * sizeof(WCHAR) ); path->server[len] = 0; path->len_server = len; + path->flags |= WBEMPATH_INFO_PATH_HAD_SERVER; } p = q; while (*q && *q != ':') @@ -180,6 +183,7 @@ static HRESULT parse_text( struct path *path, ULONG mode, const WCHAR *text ) done: if (hr != S_OK) clear_path( path ); + else path->flags |= WBEMPATH_INFO_CIM_COMPLIANT | WBEMPATH_INFO_V2_COMPLIANT; return hr; } @@ -197,6 +201,7 @@ static HRESULT WINAPI path_SetText( if (!uMode || !pszPath) return WBEM_E_INVALID_PARAMETER; clear_path( path ); + if (!pszPath[0]) return S_OK; if ((hr = parse_text( path, uMode, pszPath )) != S_OK) return hr; len = strlenW( pszPath ); @@ -404,11 +409,34 @@ static HRESULT WINAPI path_GetText( static HRESULT WINAPI path_GetInfo( IWbemPath *iface, - ULONG uRequestedInfo, - ULONGLONG *puResponse) + ULONG info, + ULONGLONG *response) { - FIXME("%p, %d, %p\n", iface, uRequestedInfo, puResponse); - return E_NOTIMPL; + struct path *path = impl_from_IWbemPath( iface ); + + TRACE("%p, %u, %p\n", iface, info, response); + + if (info || !response) return WBEM_E_INVALID_PARAMETER; + + FIXME("some flags are not implemented\n"); + + *response = path->flags; + if (!path->server || (path->len_server == 1 && path->server[0] == '.')) + *response |= WBEMPATH_INFO_ANON_LOCAL_MACHINE; + else + *response |= WBEMPATH_INFO_HAS_MACHINE_NAME; + + if (!path->class) + *response |= WBEMPATH_INFO_SERVER_NAMESPACE_ONLY; + else + { + *response |= WBEMPATH_INFO_HAS_SUBSCOPES; + if (path->text && strchrW( path->text, '=' )) /* FIXME */ + *response |= WBEMPATH_INFO_IS_INST_REF; + else + *response |= WBEMPATH_INFO_IS_CLASS_REF; + } + return S_OK; } static HRESULT WINAPI path_SetServer( diff --git a/dlls/wmiutils/tests/path.c b/dlls/wmiutils/tests/path.c index 2d435f0..3367690 100644 --- a/dlls/wmiutils/tests/path.c +++ b/dlls/wmiutils/tests/path.c @@ -370,6 +370,69 @@ static void test_IWbemPath_GetServer(void) IWbemPath_Release( path ); } +static void test_IWbemPath_GetInfo(void) +{ + IWbemPath *path; + HRESULT hr; + ULONGLONG resp; + + if (!(path = create_path())) return; + + hr = IWbemPath_GetInfo( path, 0, NULL ); + ok( hr == WBEM_E_INVALID_PARAMETER, "got %08x\n", hr ); + + hr = IWbemPath_GetInfo( path, 1, NULL ); + ok( hr == WBEM_E_INVALID_PARAMETER, "got %08x\n", hr ); + + resp = 0xdeadbeef; + hr = IWbemPath_GetInfo( path, 0, &resp ); + ok( hr == S_OK, "got %08x\n", hr ); + ok( resp == (WBEMPATH_INFO_ANON_LOCAL_MACHINE | WBEMPATH_INFO_SERVER_NAMESPACE_ONLY), + "got %lx%08lx\n", (unsigned long)(resp >> 32), (unsigned long)resp ); + + hr = IWbemPath_SetText( path, WBEMPATH_CREATE_ACCEPT_ALL, path17 ); + ok( hr == S_OK, "got %08x\n", hr ); + + hr = IWbemPath_GetInfo( path, 0, NULL ); + ok( hr == WBEM_E_INVALID_PARAMETER, "got %08x\n", hr ); + + hr = IWbemPath_GetInfo( path, 1, NULL ); + ok( hr == WBEM_E_INVALID_PARAMETER, "got %08x\n", hr ); + + resp = 0xdeadbeef; + hr = IWbemPath_GetInfo( path, 0, &resp ); + ok( hr == S_OK, "got %08x\n", hr ); + ok( resp == (WBEMPATH_INFO_ANON_LOCAL_MACHINE | WBEMPATH_INFO_IS_INST_REF | + WBEMPATH_INFO_HAS_SUBSCOPES | WBEMPATH_INFO_V2_COMPLIANT | + WBEMPATH_INFO_CIM_COMPLIANT | WBEMPATH_INFO_PATH_HAD_SERVER), + "got %lx%08lx\n", (unsigned long)(resp >> 32), (unsigned long)resp ); + + IWbemPath_Release( path ); + if (!(path = create_path())) return; + + hr = IWbemPath_SetText( path, WBEMPATH_CREATE_ACCEPT_ALL, path12 ); + ok( hr == S_OK, "got %08x\n", hr ); + + resp = 0xdeadbeef; + hr = IWbemPath_GetInfo( path, 0, &resp ); + ok( hr == S_OK, "got %08x\n", hr ); + ok( resp == (WBEMPATH_INFO_ANON_LOCAL_MACHINE | WBEMPATH_INFO_IS_CLASS_REF | + WBEMPATH_INFO_HAS_SUBSCOPES | WBEMPATH_INFO_V2_COMPLIANT | + WBEMPATH_INFO_CIM_COMPLIANT), + "got %lx%08lx\n", (unsigned long)(resp >> 32), (unsigned long)resp ); + + hr = IWbemPath_SetText( path, WBEMPATH_CREATE_ACCEPT_ALL, path1 ); + ok( hr == S_OK, "got %08x\n", hr ); + + resp = 0xdeadbeef; + hr = IWbemPath_GetInfo( path, 0, &resp ); + ok( hr == S_OK, "got %08x\n", hr ); + ok( resp == (WBEMPATH_INFO_ANON_LOCAL_MACHINE | WBEMPATH_INFO_SERVER_NAMESPACE_ONLY), + "got %lx%08lx\n", (unsigned long)(resp >> 32), (unsigned long)resp ); + + IWbemPath_Release( path ); +} + START_TEST (path) { CoInitialize( NULL ); @@ -378,6 +441,7 @@ START_TEST (path) test_IWbemPath_GetText(); test_IWbemPath_GetClassName(); test_IWbemPath_GetServer(); + test_IWbemPath_GetInfo(); CoUninitialize(); } diff --git a/include/wmiutils.idl b/include/wmiutils.idl index 82f64d1..fa790c8 100644 --- a/include/wmiutils.idl +++ b/include/wmiutils.idl @@ -21,6 +21,28 @@ import "oaidl.idl"; interface IWbemPath; interface IWbemPathKeyList; +typedef [v1_enum] enum tag_WBEM_PATH_STATUS_FLAG +{ + WBEMPATH_INFO_ANON_LOCAL_MACHINE = 0x1, + WBEMPATH_INFO_HAS_MACHINE_NAME = 0x2, + WBEMPATH_INFO_IS_CLASS_REF = 0x4, + WBEMPATH_INFO_IS_INST_REF = 0x8, + WBEMPATH_INFO_HAS_SUBSCOPES = 0x10, + WBEMPATH_INFO_IS_COMPOUND = 0x20, + WBEMPATH_INFO_HAS_V2_REF_PATHS = 0x40, + WBEMPATH_INFO_HAS_IMPLIED_KEY = 0x80, + WBEMPATH_INFO_CONTAINS_SINGLETON = 0x100, + WBEMPATH_INFO_V1_COMPLIANT = 0x200, + WBEMPATH_INFO_V2_COMPLIANT = 0x400, + WBEMPATH_INFO_CIM_COMPLIANT = 0x800, + WBEMPATH_INFO_IS_SINGLETON = 0x1000, + WBEMPATH_INFO_IS_PARENT = 0x2000, + WBEMPATH_INFO_SERVER_NAMESPACE_ONLY = 0x4000, + WBEMPATH_INFO_NATIVE_PATH = 0x8000, + WBEMPATH_INFO_WMI_PATH = 0x10000, + WBEMPATH_INFO_PATH_HAD_SERVER = 0x20000 +} tag_WBEM_PATH_STATUS_FLAG; + typedef [v1_enum] enum tag_WBEM_PATH_CREATE_FLAG { WBEMPATH_CREATE_ACCEPT_RELATIVE = 0x1,

1 0

Andrew Talbot : dbghelp: Avoid signed-unsigned integer comparisons.
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: ddf922d7a180066f8c4b65656055a1c5fac5d12a URL: http://source.winehq.org/git/wine.git/?a=commit;h=ddf922d7a180066f8c4b65656… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Mon Jan 14 21:44:19 2013 +0000 dbghelp: Avoid signed-unsigned integer comparisons. --- dlls/dbghelp/elf_module.c | 2 +- dlls/dbghelp/msc.c | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/dbghelp/elf_module.c b/dlls/dbghelp/elf_module.c index 874835d..f8015c1 100644 --- a/dlls/dbghelp/elf_module.c +++ b/dlls/dbghelp/elf_module.c @@ -273,7 +273,7 @@ static BOOL elf_map_file(struct elf_map_file_data* emfd, struct image_file_map* { static const BYTE elf_signature[4] = { ELFMAG0, ELFMAG1, ELFMAG2, ELFMAG3 }; struct stat statbuf; - int i; + unsigned int i; Elf_Phdr phdr; size_t tmp, page_mask = sysconf( _SC_PAGESIZE ) - 1; char* filename; diff --git a/dlls/dbghelp/msc.c b/dlls/dbghelp/msc.c index 7a68e1d..ddabd07 100644 --- a/dlls/dbghelp/msc.c +++ b/dlls/dbghelp/msc.c @@ -1370,7 +1370,8 @@ static void codeview_snarf_linetab(const struct msc_debug_info* msc_dbg, const B { const BYTE* ptr = linetab; int nfile, nseg; - int i, j, k; + int i, j; + unsigned int k; const unsigned int* filetab; const unsigned int* lt_ptr; const unsigned short* linenos;

1 0

Henri Verbeet : ddraw: Avoid LPD3DPROCESSVERTICES.
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: b43f897675f3c6c6f8c4bc92db0e87bfef97da2d URL: http://source.winehq.org/git/wine.git/?a=commit;h=b43f897675f3c6c6f8c4bc92d… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Jan 15 08:46:52 2013 +0100 ddraw: Avoid LPD3DPROCESSVERTICES. --- dlls/ddraw/executebuffer.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/dlls/ddraw/executebuffer.c b/dlls/ddraw/executebuffer.c index 80e44ef..1673cac 100644 --- a/dlls/ddraw/executebuffer.c +++ b/dlls/ddraw/executebuffer.c @@ -301,8 +301,9 @@ HRESULT d3d_execute_buffer_execute(struct d3d_execute_buffer *buffer, wined3d_device_get_transform(device->wined3d_device, WINED3D_TS_WORLD_MATRIX(0), (struct wined3d_matrix *)&world_mat); - for (i = 0; i < count; i++) { - LPD3DPROCESSVERTICES ci = (LPD3DPROCESSVERTICES) instr; + for (i = 0; i < count; ++i) + { + D3DPROCESSVERTICES *ci = (D3DPROCESSVERTICES *)instr; TRACE(" Start : %d Dest : %d Count : %d\n", ci->wStart, ci->wDest, ci->dwCount);

1 0

Dmitry Timoshkov : gdiplus: Avoid not necessary memory allocation for palette entries.
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: 44e554b744edd4e241118caada0972d7d2f163f8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=44e554b744edd4e241118caad… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Tue Jan 15 12:48:34 2013 +0800 gdiplus: Avoid not necessary memory allocation for palette entries. --- dlls/gdiplus/image.c | 31 +++++++++---------------------- 1 files changed, 9 insertions(+), 22 deletions(-) diff --git a/dlls/gdiplus/image.c b/dlls/gdiplus/image.c index 558e300..377cab1 100644 --- a/dlls/gdiplus/image.c +++ b/dlls/gdiplus/image.c @@ -4724,28 +4724,17 @@ GpStatus WINGDIPAPI GdipCreateBitmapFromHBITMAP(HBITMAP hbm, HPALETTE hpal, GpBi if (retval == Ok && hpal) { - WORD num_palette_entries; - PALETTEENTRY *palette_entries=NULL; + PALETTEENTRY entry[256]; ColorPalette *palette=NULL; - int i; + int i, num_palette_entries; - if (!GetObjectW(hpal, sizeof(num_palette_entries), &num_palette_entries)) + num_palette_entries = GetPaletteEntries(hpal, 0, 256, entry); + if (!num_palette_entries) retval = GenericError; - if (retval == Ok) - { - palette_entries = GdipAlloc(sizeof(PALETTEENTRY) * num_palette_entries); - palette = GdipAlloc(sizeof(ColorPalette) + sizeof(ARGB) * (num_palette_entries-1)); - - if (!palette_entries || !palette) - retval = OutOfMemory; - } - - if (retval == Ok) - { - if (!GetPaletteEntries(hpal, 0, num_palette_entries, palette_entries)) - retval = GenericError; - } + palette = GdipAlloc(sizeof(ColorPalette) + sizeof(ARGB) * (num_palette_entries-1)); + if (!palette) + retval = OutOfMemory; if (retval == Ok) { @@ -4754,15 +4743,13 @@ GpStatus WINGDIPAPI GdipCreateBitmapFromHBITMAP(HBITMAP hbm, HPALETTE hpal, GpBi for (i=0; i<num_palette_entries; i++) { - PALETTEENTRY * entry = &palette_entries[i]; - palette->Entries[i] = 0xff000000 | entry->peRed << 16 | - entry->peGreen << 8 | entry->peBlue; + palette->Entries[i] = 0xff000000 | entry[i].peRed << 16 | + entry[i].peGreen << 8 | entry[i].peBlue; } retval = GdipSetImagePalette((GpImage*)*bitmap, palette); } - GdipFree(palette_entries); GdipFree(palette); }

1 0

Henri Verbeet : ddraw: Only modify the ddraw destination window at the end of ddraw7_SetCooperativeLevel ().
by Alexandre Julliard Jan. 15, 2013

Jan. 15, 2013

Module: wine Branch: master Commit: 292522c4ad121375bd284eaeae54a0c5d0fe23f3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=292522c4ad121375bd284eaea… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Jan 15 08:46:54 2013 +0100 ddraw: Only modify the ddraw destination window at the end of ddraw7_SetCooperativeLevel(). We want this to happen after ddraw_destroy_swapchain() in particular, because dest_window being different from d3d_window will cause it to destroy d3d_window, but this seems like a good idea in general. --- dlls/ddraw/ddraw.c | 10 ++-------- dlls/ddraw/tests/ddraw1.c | 2 +- dlls/ddraw/tests/ddraw2.c | 2 +- dlls/ddraw/tests/ddraw4.c | 2 +- dlls/ddraw/tests/ddraw7.c | 2 +- 5 files changed, 6 insertions(+), 12 deletions(-) diff --git a/dlls/ddraw/ddraw.c b/dlls/ddraw/ddraw.c index ca6f8d5..21acc28 100644 --- a/dlls/ddraw/ddraw.c +++ b/dlls/ddraw/ddraw.c @@ -551,9 +551,6 @@ static HRESULT ddraw_set_focus_window(struct ddraw *ddraw, HWND window) ddraw->focuswindow = window; - /* Use the focus window for drawing too. */ - ddraw->dest_window = ddraw->focuswindow; - return DD_OK; } @@ -852,10 +849,6 @@ static HRESULT WINAPI ddraw7_SetCooperativeLevel(IDirectDraw7 *iface, HWND hwnd, } } - /* Don't override focus windows or private device windows */ - if (hwnd && !This->focuswindow && !This->devicewindow && (hwnd != window)) - This->dest_window = hwnd; - if (cooplevel & DDSCL_MULTITHREADED && !(This->cooperative_level & DDSCL_MULTITHREADED)) wined3d_device_set_multithreaded(This->wined3d_device); @@ -886,7 +879,7 @@ static HRESULT WINAPI ddraw7_SetCooperativeLevel(IDirectDraw7 *iface, HWND hwnd, ddraw_destroy_swapchain(This); } - if (FAILED(hr = ddraw_create_swapchain(This, This->dest_window, !(cooplevel & DDSCL_FULLSCREEN)))) + if (FAILED(hr = ddraw_create_swapchain(This, hwnd, !(cooplevel & DDSCL_FULLSCREEN)))) ERR("Failed to create swapchain, hr %#x.\n", hr); if (restore_state) @@ -922,6 +915,7 @@ static HRESULT WINAPI ddraw7_SetCooperativeLevel(IDirectDraw7 *iface, HWND hwnd, /* Store the cooperative_level */ This->cooperative_level = cooplevel; + This->dest_window = hwnd; TRACE("SetCooperativeLevel retuning DD_OK\n"); wined3d_mutex_unlock(); diff --git a/dlls/ddraw/tests/ddraw1.c b/dlls/ddraw/tests/ddraw1.c index a37e0b7..2cbcf44 100644 --- a/dlls/ddraw/tests/ddraw1.c +++ b/dlls/ddraw/tests/ddraw1.c @@ -2651,7 +2651,7 @@ static void test_coop_level_multi_window(void) ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); hr = IDirectDraw_SetCooperativeLevel(ddraw, window2, DDSCL_NORMAL); ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); - todo_wine ok(IsWindow(window1), "Window 1 was destroyed.\n"); + ok(IsWindow(window1), "Window 1 was destroyed.\n"); ok(IsWindow(window2), "Window 2 was destroyed.\n"); IDirectDraw_Release(ddraw); diff --git a/dlls/ddraw/tests/ddraw2.c b/dlls/ddraw/tests/ddraw2.c index e475f5b..79b0e9b 100644 --- a/dlls/ddraw/tests/ddraw2.c +++ b/dlls/ddraw/tests/ddraw2.c @@ -2770,7 +2770,7 @@ static void test_coop_level_multi_window(void) ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); hr = IDirectDraw2_SetCooperativeLevel(ddraw, window2, DDSCL_NORMAL); ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); - todo_wine ok(IsWindow(window1), "Window 1 was destroyed.\n"); + ok(IsWindow(window1), "Window 1 was destroyed.\n"); ok(IsWindow(window2), "Window 2 was destroyed.\n"); IDirectDraw2_Release(ddraw); diff --git a/dlls/ddraw/tests/ddraw4.c b/dlls/ddraw/tests/ddraw4.c index 82cd5d5..3d81d1d 100644 --- a/dlls/ddraw/tests/ddraw4.c +++ b/dlls/ddraw/tests/ddraw4.c @@ -2957,7 +2957,7 @@ static void test_coop_level_multi_window(void) ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); hr = IDirectDraw4_SetCooperativeLevel(ddraw, window2, DDSCL_NORMAL); ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); - todo_wine ok(IsWindow(window1), "Window 1 was destroyed.\n"); + ok(IsWindow(window1), "Window 1 was destroyed.\n"); ok(IsWindow(window2), "Window 2 was destroyed.\n"); IDirectDraw4_Release(ddraw); diff --git a/dlls/ddraw/tests/ddraw7.c b/dlls/ddraw/tests/ddraw7.c index cd22e0c..5668dc7 100644 --- a/dlls/ddraw/tests/ddraw7.c +++ b/dlls/ddraw/tests/ddraw7.c @@ -2766,7 +2766,7 @@ static void test_coop_level_multi_window(void) ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); hr = IDirectDraw7_SetCooperativeLevel(ddraw, window2, DDSCL_NORMAL); ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); - todo_wine ok(IsWindow(window1), "Window 1 was destroyed.\n"); + ok(IsWindow(window1), "Window 1 was destroyed.\n"); ok(IsWindow(window2), "Window 2 was destroyed.\n"); IDirectDraw7_Release(ddraw);

1 0