July 2011 - Wine-commits - list.winehq.org

Alexandre Julliard : cmd/tests: Rewrite replace_escaped_spaces to avoid overflows.
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: 766dc67043e3e65a729a76014654d3250cc9e76e URL: http://source.winehq.org/git/wine.git/?a=commit;h=766dc67043e3e65a729a76014… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Jul 6 19:51:51 2011 +0200 cmd/tests: Rewrite replace_escaped_spaces to avoid overflows. Spotted by Marcus Meissner. --- programs/cmd/tests/batch.c | 28 +++++++++------------------- 1 files changed, 9 insertions(+), 19 deletions(-) diff --git a/programs/cmd/tests/batch.c b/programs/cmd/tests/batch.c index 3c12fe7..95366da 100644 --- a/programs/cmd/tests/batch.c +++ b/programs/cmd/tests/batch.c @@ -29,29 +29,19 @@ static DWORD workdir_len; static const char* replace_escaped_spaces(const char *data, DWORD size, DWORD *new_size) { static const char escaped_space[] = {'@','s','p','a','c','e','@','\0'}; - const char *a, *b; - char *new_data; - DWORD len_space = sizeof(escaped_space) -1; + char *ptr, *new_data; - a = b = data; - *new_size = 0; + new_data = ptr = HeapAlloc(GetProcessHeap(), 0, size + 1); + memcpy( new_data, data, size ); + new_data[size] = 0; - new_data = HeapAlloc(GetProcessHeap(), 0, size*sizeof(char)); - ok(new_data != NULL, "HeapAlloc failed\n"); - if(!new_data) - return NULL; - - while( (b = strstr(a, escaped_space)) ) + while ((ptr = strstr(ptr, escaped_space))) { - strncpy(new_data + *new_size, a, b-a + 1); - *new_size += b-a + 1; - new_data[*new_size - 1] = ' '; - a = b + len_space; + char *end = ptr + sizeof(escaped_space) - 1; + *ptr++ = ' '; + memmove( ptr, end, strlen(end) + 1 ); } - - strncpy(new_data + *new_size, a, strlen(a) + 1); - *new_size += strlen(a); - + *new_size = strlen(new_data); return new_data; }

1 0

Jacek Caban : widl: Generate class forward declaration for coclasses.
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: 9afec542e0aafbc79e8338fcb274483f7c389f6e URL: http://source.winehq.org/git/wine.git/?a=commit;h=9afec542e0aafbc79e8338fcb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Jul 6 14:08:16 2011 +0200 widl: Generate class forward declaration for coclasses. --- tools/widl/header.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/tools/widl/header.c b/tools/widl/header.c index 4dee484..6e288d4 100644 --- a/tools/widl/header.c +++ b/tools/widl/header.c @@ -1152,6 +1152,9 @@ static void write_coclass(FILE *header, type_t *cocl) fprintf(header, " * %s coclass\n", cocl->name); fprintf(header, " */\n\n"); write_coclass_guid(header, cocl); + fprintf(header, "\n#ifdef __cplusplus\n"); + fprintf(header, "class %s;\n", cocl->name); + fprintf(header, "#endif\n"); fprintf(header, "\n"); }

1 0

Marcus Meissner : widl: Decrease is_object_interface after usage.
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: 28e92027b18ee9ba48725fe153a527942f02aecc URL: http://source.winehq.org/git/wine.git/?a=commit;h=28e92027b18ee9ba48725fe15… Author: Marcus Meissner <meissner(a)suse.de> Date: Wed Jul 6 16:00:31 2011 +0200 widl: Decrease is_object_interface after usage. --- tools/widl/header.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/widl/header.c b/tools/widl/header.c index 6e288d4..7569995 100644 --- a/tools/widl/header.c +++ b/tools/widl/header.c @@ -1267,7 +1267,7 @@ static void write_header_stmts(FILE *header, const statement_list_t *stmts, cons write_header_stmts(header, type_iface_get_stmts(iface), iface, FALSE); write_rpc_interface_end(header, iface); } - if (is_object(iface)) is_object_interface++; + if (is_object(iface)) is_object_interface--; } else if (type_get_type(stmt->u.type) == TYPE_COCLASS) write_coclass(header, stmt->u.type);

1 0

Nikolay Sivov : comctl32/tests: Simplify a workaround for old module versions, add basic PSM_ADDPAGE test.
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: a3ff177d7d668169ab389221aa0fdf582c09cd45 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a3ff177d7d668169ab389221a… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Jul 6 13:49:14 2011 +0400 comctl32/tests: Simplify a workaround for old module versions, add basic PSM_ADDPAGE test. --- dlls/comctl32/tests/propsheet.c | 111 ++++++++++++++++++++++++++------------ 1 files changed, 76 insertions(+), 35 deletions(-) diff --git a/dlls/comctl32/tests/propsheet.c b/dlls/comctl32/tests/propsheet.c index 42cb4c8..c6e3e9f 100644 --- a/dlls/comctl32/tests/propsheet.c +++ b/dlls/comctl32/tests/propsheet.c @@ -102,7 +102,7 @@ static void test_title(void) hpsp[0] = CreatePropertySheetPageA(&psp); memset(&psh, 0, sizeof(psh)); - psh.dwSize = sizeof(psh); + psh.dwSize = CCSIZEOF_STRUCT(PROPSHEETHEADERA, pfnCallback); psh.dwFlags = PSH_MODELESS | PSH_USECALLBACK; psh.pszCaption = "test caption"; psh.nPages = 1; @@ -111,12 +111,8 @@ static void test_title(void) psh.pfnCallback = sheet_callback; hdlg = (HWND)PropertySheetA(&psh); - if (hdlg == INVALID_HANDLE_VALUE) - { - win_skip("comctl32 4.70 needs dwSize adjustment\n"); - psh.dwSize = sizeof(psh) - sizeof(HBITMAP) - sizeof(HPALETTE) - sizeof(HBITMAP); - hdlg = (HWND)PropertySheetA(&psh); - } + ok(hdlg != INVALID_HANDLE_VALUE, "got invalid handle value %p\n", hdlg); + DestroyWindow(hdlg); } @@ -139,7 +135,7 @@ static void test_nopage(void) hpsp[0] = CreatePropertySheetPageA(&psp); memset(&psh, 0, sizeof(psh)); - psh.dwSize = sizeof(psh); + psh.dwSize = CCSIZEOF_STRUCT(PROPSHEETHEADERA, pfnCallback); psh.dwFlags = PSH_MODELESS | PSH_USECALLBACK; psh.pszCaption = "test caption"; psh.nPages = 1; @@ -148,12 +144,8 @@ static void test_nopage(void) psh.pfnCallback = sheet_callback; hdlg = (HWND)PropertySheetA(&psh); - if (hdlg == INVALID_HANDLE_VALUE) - { - win_skip("comctl32 4.70 needs dwSize adjustment\n"); - psh.dwSize = sizeof(psh) - sizeof(HBITMAP) - sizeof(HPALETTE) - sizeof(HBITMAP); - hdlg = (HWND)PropertySheetA(&psh); - } + ok(hdlg != INVALID_HANDLE_VALUE, "got invalid handle value %p\n", hdlg); + ShowWindow(hdlg,SW_NORMAL); SendMessage(hdlg, PSM_REMOVEPAGE, 0, 0); RedrawWindow(hdlg,NULL,NULL,RDW_UPDATENOW|RDW_ERASENOW); @@ -292,19 +284,14 @@ static void test_wiznavigation(void) /* set up the property sheet dialog */ memset(&psh, 0, sizeof(psh)); - psh.dwSize = sizeof(psh); + psh.dwSize = CCSIZEOF_STRUCT(PROPSHEETHEADERA, pfnCallback); psh.dwFlags = PSH_MODELESS | PSH_WIZARD; psh.pszCaption = "A Wizard"; psh.nPages = 4; psh.hwndParent = GetDesktopWindow(); U3(psh).phpage = hpsp; hdlg = (HWND)PropertySheetA(&psh); - if (hdlg == INVALID_HANDLE_VALUE) - { - win_skip("comctl32 4.70 needs dwSize adjustment\n"); - psh.dwSize = sizeof(psh) - sizeof(HBITMAP) - sizeof(HPALETTE) - sizeof(HBITMAP); - hdlg = (HWND)PropertySheetA(&psh); - } + ok(hdlg != INVALID_HANDLE_VALUE, "got invalid handle %p\n", hdlg); ok(active_page == 0, "Active page should be 0. Is: %d\n", active_page); @@ -395,7 +382,7 @@ static void test_buttons(void) hpsp[0] = CreatePropertySheetPageA(&psp); memset(&psh, 0, sizeof(psh)); - psh.dwSize = sizeof(psh); + psh.dwSize = CCSIZEOF_STRUCT(PROPSHEETHEADERA, pfnCallback); psh.dwFlags = PSH_MODELESS | PSH_USECALLBACK; psh.pszCaption = "test caption"; psh.nPages = 1; @@ -404,12 +391,7 @@ static void test_buttons(void) psh.pfnCallback = sheet_callback; hdlg = (HWND)PropertySheetA(&psh); - if (hdlg == INVALID_HANDLE_VALUE) - { - win_skip("comctl32 4.70 needs dwSize adjustment\n"); - psh.dwSize = sizeof(psh) - sizeof(HBITMAP) - sizeof(HPALETTE) - sizeof(HBITMAP); - hdlg = (HWND)PropertySheetA(&psh); - } + ok(hdlg != INVALID_HANDLE_VALUE, "got null handle\n"); /* OK button */ button = GetDlgItem(hdlg, IDOK); @@ -710,7 +692,7 @@ static void test_messages(void) hpsp[0] = CreatePropertySheetPageA(&psp); memset(&psh, 0, sizeof(psh)); - psh.dwSize = sizeof(psh); + psh.dwSize = CCSIZEOF_STRUCT(PROPSHEETHEADERA, pfnCallback); psh.dwFlags = PSH_NOAPPLYNOW | PSH_WIZARD | PSH_USECALLBACK | PSH_MODELESS | PSH_USEICONID; psh.pszCaption = "test caption"; @@ -720,12 +702,8 @@ static void test_messages(void) psh.pfnCallback = sheet_callback_messages; hdlg = (HWND)PropertySheetA(&psh); - if (hdlg == INVALID_HANDLE_VALUE) - { - win_skip("comctl32 4.70 needs dwSize adjustment\n"); - psh.dwSize = sizeof(psh) - sizeof(HBITMAP) - sizeof(HPALETTE) - sizeof(HBITMAP); - hdlg = (HWND)PropertySheetA(&psh); - } + ok(hdlg != INVALID_HANDLE_VALUE, "got invalid handle %p\n", hdlg); + ShowWindow(hdlg,SW_NORMAL); ok_sequence(sequences, PROPSHEET_SEQ_INDEX, property_sheet_seq, "property sheet with custom window proc", TRUE); @@ -733,6 +711,68 @@ static void test_messages(void) DestroyWindow(hdlg); } +static void test_PSM_ADDPAGE(void) +{ + HPROPSHEETPAGE hpsp[3]; + PROPSHEETPAGEA psp; + PROPSHEETHEADERA psh; + HWND hdlg, tab; + BOOL ret; + DWORD r; + + memset(&psp, 0, sizeof(psp)); + psp.dwSize = sizeof(psp); + psp.dwFlags = 0; + psp.hInstance = GetModuleHandleA(NULL); + U(psp).pszTemplate = MAKEINTRESOURCE(IDD_PROP_PAGE_MESSAGE_TEST); + U2(psp).pszIcon = NULL; + psp.pfnDlgProc = page_dlg_proc_messages; + psp.lParam = 0; + + /* two page with the same data */ + hpsp[0] = CreatePropertySheetPageA(&psp); + hpsp[1] = CreatePropertySheetPageA(&psp); + hpsp[2] = CreatePropertySheetPageA(&psp); + + memset(&psh, 0, sizeof(psh)); + psh.dwSize = sizeof(psh); + psh.dwFlags = PSH_MODELESS; + psh.pszCaption = "test caption"; + psh.nPages = 1; + psh.hwndParent = GetDesktopWindow(); + U3(psh).phpage = hpsp; + + hdlg = (HWND)PropertySheetA(&psh); + ok(hdlg != INVALID_HANDLE_VALUE, "got invalid handle %p\n", hdlg); + + /* add pages one by one */ + ret = SendMessageA(hdlg, PSM_ADDPAGE, 0, (LPARAM)hpsp[1]); + ok(ret == TRUE, "got %d\n", ret); + + /* try with null and invalid value */ + ret = SendMessageA(hdlg, PSM_ADDPAGE, 0, 0); + ok(ret == FALSE, "got %d\n", ret); + +if (0) +{ + /* crashes on native */ + ret = SendMessageA(hdlg, PSM_ADDPAGE, 0, (LPARAM)INVALID_HANDLE_VALUE); +} + /* check item count */ + tab = (HWND)SendMessageA(hdlg, PSM_GETTABCONTROL, 0, 0); + + r = SendMessageA(tab, TCM_GETITEMCOUNT, 0, 0); + ok(r == 2, "got %d\n", r); + + ret = SendMessageA(hdlg, PSM_ADDPAGE, 0, (LPARAM)hpsp[2]); + ok(ret == TRUE, "got %d\n", ret); + + r = SendMessageA(tab, TCM_GETITEMCOUNT, 0, 0); + ok(r == 3, "got %d\n", r); + + DestroyWindow(hdlg); +} + START_TEST(propsheet) { test_title(); @@ -742,4 +782,5 @@ START_TEST(propsheet) test_buttons(); test_custom_default_button(); test_messages(); + test_PSM_ADDPAGE(); }

1 0

Matteo Bruni : d3d9/tests: Fix alpha component in the unbound sampler tests .
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: 8d39a85e180c5cd72dedca4fe9ffe160c35fe2d6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8d39a85e180c5cd72dedca4fe… Author: Matteo Bruni <mbruni(a)codeweavers.com> Date: Thu Jun 30 15:00:35 2011 +0200 d3d9/tests: Fix alpha component in the unbound sampler tests. Spotted by Sean Sube. --- dlls/d3d9/tests/visual.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index 5328447..0496896 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -235,6 +235,7 @@ static void lighting_test(IDirect3DDevice9 *device) DWORD color; D3DMATERIAL9 material, old_material; DWORD cop, carg; + DWORD old_colorwrite; float mat[16] = { 1.0f, 0.0f, 0.0f, 0.0f, 0.0f, 1.0f, 0.0f, 0.0f, @@ -297,6 +298,8 @@ static void lighting_test(IDirect3DDevice9 *device) ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState returned %08x\n", hr); hr = IDirect3DDevice9_SetRenderState(device, D3DRS_CULLMODE, D3DCULL_NONE); ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed with %08x\n", hr); + hr = IDirect3DDevice9_GetRenderState(device, D3DRS_COLORWRITEENABLE, &old_colorwrite); + ok(hr == D3D_OK, "IDirect3DDevice9_GetRenderState failed with %08x\n", hr); hr = IDirect3DDevice9_SetRenderState(device, D3DRS_COLORWRITEENABLE, D3DCOLORWRITEENABLE_RED | D3DCOLORWRITEENABLE_GREEN | D3DCOLORWRITEENABLE_BLUE); ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed with %08x\n", hr); @@ -420,6 +423,8 @@ static void lighting_test(IDirect3DDevice9 *device) ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState returned %08x\n", hr); hr = IDirect3DDevice9_SetRenderState(device, D3DRS_LIGHTING, FALSE); ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState returned %08x\n", hr); + hr = IDirect3DDevice9_SetRenderState(device, D3DRS_COLORWRITEENABLE, old_colorwrite); + ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState failed with %08x\n", hr); hr = IDirect3DDevice9_SetTextureStageState(device, 0, D3DTSS_COLORARG1, carg); ok(hr == D3D_OK, "IDirect3DDevice9_SetTextureStageState returned %08x\n", hr); hr = IDirect3DDevice9_SetMaterial(device, &old_material); @@ -8759,7 +8764,7 @@ static void multiple_rendertargets_test(IDirect3DDevice9 *device) hr = IDirect3DDevice9_GetRenderTargetData(device, surf1, readback); ok(SUCCEEDED(hr), "GetRenderTargetData failed, hr %#x.\n", hr); color = getPixelColorFromSurface(readback, 8, 8); - ok(color_match(color, D3DCOLOR_ARGB(0xff, 0x00, 0xff, 0x00), 0), + ok(color_match(color, D3DCOLOR_ARGB(0x00, 0x00, 0xff, 0x00), 0), "Expected color 0xff00ff00, got 0x%08x.\n", color); hr = IDirect3DDevice9_GetRenderTargetData(device, surf2, readback); ok(SUCCEEDED(hr), "GetRenderTargetData failed, hr %#x.\n", hr); @@ -12112,7 +12117,7 @@ static void unbound_sampler_test(IDirect3DDevice9 *device) } color = getPixelColorFromSurface(rt, 32, 32); - todo_wine ok(color == 0x56000000, "Unbound sampler color is %#x.\n", color); + todo_wine ok(color == 0xff000000, "Unbound sampler color is %#x.\n", color); hr = IDirect3DDevice9_SetRenderTarget(device, 0, old_rt); ok(SUCCEEDED(hr), "IDirect3DDevice9_SetRenderTarget failed, hr %#x.\n", hr);

1 0

Matteo Bruni : wined3d: Make dummy textures black.
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: 9d3cc8b012ea6cb9c05a6cf5d58529a5aab00084 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9d3cc8b012ea6cb9c05a6cf5d… Author: Matteo Bruni <mbruni(a)codeweavers.com> Date: Thu Jun 30 15:01:07 2011 +0200 wined3d: Make dummy textures black. --- dlls/d3d9/tests/visual.c | 2 +- dlls/wined3d/device.c | 13 ++++++------- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index 0496896..16047bf 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -12117,7 +12117,7 @@ static void unbound_sampler_test(IDirect3DDevice9 *device) } color = getPixelColorFromSurface(rt, 32, 32); - todo_wine ok(color == 0xff000000, "Unbound sampler color is %#x.\n", color); + ok(color == 0xff000000, "Unbound sampler color is %#x.\n", color); hr = IDirect3DDevice9_SetRenderTarget(device, 0, old_rt); ok(SUCCEEDED(hr), "IDirect3DDevice9_SetRenderTarget failed, hr %#x.\n", hr); diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index a7e578a..bc9c716 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -958,11 +958,10 @@ static void create_dummy_textures(struct wined3d_device *device) { const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; unsigned int i; - /* Under DirectX you can have texture stage operations even if no texture is - bound, whereas opengl will only do texture operations when a valid texture is - bound. We emulate this by creating dummy textures and binding them to each - texture stage, but disable all stages by default. Hence if a stage is enabled - then the default texture will kick in until replaced by a SetTexture call */ + /* Under DirectX you can sample even if no texture is bound, whereas + * OpenGL will only allow that when a valid texture is bound. + * We emulate this by creating dummy textures and binding them + * to each texture stage when the currently set D3D texture is NULL. */ ENTER_GL(); if (gl_info->supported[APPLE_CLIENT_STORAGE]) @@ -974,7 +973,7 @@ static void create_dummy_textures(struct wined3d_device *device) for (i = 0; i < gl_info->limits.textures; ++i) { - GLubyte white = 255; + DWORD color = 0x000000ff; /* Make appropriate texture active */ GL_EXTCALL(glActiveTextureARB(GL_TEXTURE0_ARB + i)); @@ -990,7 +989,7 @@ static void create_dummy_textures(struct wined3d_device *device) glBindTexture(GL_TEXTURE_2D, device->dummyTextureName[i]); checkGLcall("glBindTexture"); - glTexImage2D(GL_TEXTURE_2D, 0, GL_LUMINANCE, 1, 1, 0, GL_LUMINANCE, GL_UNSIGNED_BYTE, &white); + glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA8, 1, 1, 0, GL_RGBA, GL_UNSIGNED_INT_8_8_8_8, &color); checkGLcall("glTexImage2D"); }

1 0

Gerald Pfeifer : urlmon: Silence a compiler warning (GCC 4.6).
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: 7d60dd7e16bf29762eb1f438d21410c44d81c25e URL: http://source.winehq.org/git/wine.git/?a=commit;h=7d60dd7e16bf29762eb1f438d… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Tue Jul 5 03:41:35 2011 +0200 urlmon: Silence a compiler warning (GCC 4.6). --- dlls/urlmon/uri.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/urlmon/uri.c b/dlls/urlmon/uri.c index 6f2897a..33d0eb2 100644 --- a/dlls/urlmon/uri.c +++ b/dlls/urlmon/uri.c @@ -3785,6 +3785,7 @@ static HRESULT validate_path(const UriBuilder *builder, parse_data *data, DWORD static const WCHAR nullW[] = {0}; ptr = nullW; check_len = FALSE; + expected_len = -1; } component = ptr; @@ -3798,7 +3799,7 @@ static HRESULT validate_path(const UriBuilder *builder, parse_data *data, DWORD if(!valid || (check_len && expected_len != data->path_len)) { TRACE("(%p %p %x): Invalid path component %s.\n", builder, data, flags, - debugstr_wn(component, check_len ? expected_len : -1) ); + debugstr_wn(component, expected_len) ); return INET_E_INVALID_URL; }

1 0

Vincent Povirk : server: Implement QS_HOTKEY.
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: 79556aeab740f28b651aac6c439ec713e6f26236 URL: http://source.winehq.org/git/wine.git/?a=commit;h=79556aeab740f28b651aac6c4… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu May 26 15:52:17 2011 -0500 server: Implement QS_HOTKEY. --- server/queue.c | 22 +++++++++++++++++++++- 1 files changed, 21 insertions(+), 1 deletions(-) diff --git a/server/queue.c b/server/queue.c index 5dd4bf3..03b0e92 100644 --- a/server/queue.c +++ b/server/queue.c @@ -120,6 +120,7 @@ struct msg_queue unsigned int changed_bits; /* changed wakeup bits */ unsigned int changed_mask; /* changed wakeup mask */ int paint_count; /* pending paint messages count */ + int hotkey_count; /* pending hotkey messages count */ int quit_message; /* is there a pending quit message? */ int exit_code; /* exit code of pending quit message */ int cursor_count; /* per-queue cursor show count */ @@ -279,6 +280,7 @@ static struct msg_queue *create_msg_queue( struct thread *thread, struct thread_ queue->changed_bits = 0; queue->changed_mask = 0; queue->paint_count = 0; + queue->hotkey_count = 0; queue->quit_message = 0; queue->cursor_count = 0; queue->recv_result = NULL; @@ -600,6 +602,8 @@ static void remove_queue_message( struct msg_queue *queue, struct message *msg, case POST_MESSAGE: if (list_empty( &queue->msg_list[kind] ) && !queue->quit_message) clear_queue_bits( queue, QS_POSTMESSAGE|QS_ALLPOSTMESSAGE ); + if (msg->msg == WM_HOTKEY && --queue->hotkey_count == 0) + clear_queue_bits( queue, QS_HOTKEY ); break; } free_message( msg ); @@ -1358,7 +1362,8 @@ found: msg->data_size = 0; list_add_tail( &hotkey->queue->msg_list[POST_MESSAGE], &msg->entry ); - set_queue_bits( hotkey->queue, QS_POSTMESSAGE|QS_ALLPOSTMESSAGE ); + set_queue_bits( hotkey->queue, QS_POSTMESSAGE|QS_ALLPOSTMESSAGE|QS_HOTKEY ); + hotkey->queue->hotkey_count++; return 1; } @@ -1942,6 +1947,11 @@ void post_message( user_handle_t win, unsigned int message, lparam_t wparam, lpa list_add_tail( &thread->queue->msg_list[POST_MESSAGE], &msg->entry ); set_queue_bits( thread->queue, QS_POSTMESSAGE|QS_ALLPOSTMESSAGE ); + if (message == WM_HOTKEY) + { + set_queue_bits( thread->queue, QS_HOTKEY ); + thread->queue->hotkey_count++; + } } release_object( thread ); } @@ -2150,6 +2160,11 @@ DECL_HANDLER(send_message) case MSG_POSTED: list_add_tail( &recv_queue->msg_list[POST_MESSAGE], &msg->entry ); set_queue_bits( recv_queue, QS_POSTMESSAGE|QS_ALLPOSTMESSAGE ); + if (msg->msg == WM_HOTKEY) + { + set_queue_bits( recv_queue, QS_HOTKEY ); + recv_queue->hotkey_count++; + } break; case MSG_HARDWARE: /* should use send_hardware_message instead */ case MSG_CALLBACK_RESULT: /* cannot send this one */ @@ -2245,6 +2260,11 @@ DECL_HANDLER(get_message) get_posted_message( queue, get_win, req->get_first, req->get_last, req->flags, reply )) return; + if ((filter & QS_HOTKEY) && queue->hotkey_count && + req->get_first <= WM_HOTKEY && req->get_last >= WM_HOTKEY && + get_posted_message( queue, get_win, WM_HOTKEY, WM_HOTKEY, req->flags, reply )) + return; + /* only check for quit messages if not posted messages pending. * note: the quit message isn't filtered */ if (get_quit_message( queue, req->flags, reply ))

1 0

Frédéric Delanoy : d3dx9_36: Remove dead assignment (Clang).
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: 67d10dfa537e606e68567a390e13abbef13f747b URL: http://source.winehq.org/git/wine.git/?a=commit;h=67d10dfa537e606e68567a390… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Jul 6 10:34:49 2011 +0200 d3dx9_36: Remove dead assignment (Clang). --- dlls/d3dx9_36/mesh.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/dlls/d3dx9_36/mesh.c b/dlls/d3dx9_36/mesh.c index 2807150..dfe6a35 100644 --- a/dlls/d3dx9_36/mesh.c +++ b/dlls/d3dx9_36/mesh.c @@ -3763,7 +3763,6 @@ HRESULT WINAPI D3DXCreateSphere(LPDIRECT3DDEVICE9 device, FLOAT radius, UINT sli vertex = 0; face = 0; - stack = 0; vertices[vertex].normal.x = 0.0f; vertices[vertex].normal.y = 0.0f;

1 0

Frédéric Delanoy : mpr: Remove dead assignment ( Clang).
by Alexandre Julliard 06 Jul '11

06 Jul '11

Module: wine Branch: master Commit: 09eec46daf35419d6a9f8abd634d39c152dd70c6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=09eec46daf35419d6a9f8abd6… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Wed Jul 6 10:34:50 2011 +0200 mpr: Remove dead assignment (Clang). --- dlls/mpr/pwcache.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/dlls/mpr/pwcache.c b/dlls/mpr/pwcache.c index 0ed65b6..efd9c23 100644 --- a/dlls/mpr/pwcache.c +++ b/dlls/mpr/pwcache.c @@ -260,7 +260,6 @@ UINT WINAPI WNetEnumCachedPasswords( sprintf(prefix, "X-%02X-", nType ); - i = 0; for( i=0; ; i++ ) { val_sz = sizeof val;

1 0