March 2009 - Wine-commits - list.winehq.org

Hans Leidekker : msi: Add tests for MsiEnumProducts.
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: f0d1d61d3b8f613dfa82b7169484c32a70ed16b9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f0d1d61d3b8f613dfa82b7169… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Mar 27 13:40:59 2009 +0100 msi: Add tests for MsiEnumProducts. --- dlls/msi/tests/msi.c | 71 ++++++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 71 insertions(+), 0 deletions(-) diff --git a/dlls/msi/tests/msi.c b/dlls/msi/tests/msi.c index 928e48a..13afd92 100644 --- a/dlls/msi/tests/msi.c +++ b/dlls/msi/tests/msi.c @@ -10874,6 +10874,76 @@ static void test_MsiGetPatchInfoEx(void) RegCloseKey(udprod); } +static void test_MsiEnumProducts(void) +{ + UINT r; + int found1, found2, found3; + DWORD index; + char product1[39], product2[39], product3[39], guid[39]; + char product_squashed1[33], product_squashed2[33], product_squashed3[33]; + char keypath1[MAX_PATH], keypath2[MAX_PATH], keypath3[MAX_PATH]; + char *usersid; + HKEY key1, key2, key3; + + create_test_guid(product1, product_squashed1); + create_test_guid(product2, product_squashed2); + create_test_guid(product3, product_squashed3); + get_user_sid(&usersid); + + strcpy(keypath1, "Software\\Classes\\Installer\\Products\\"); + strcat(keypath1, product_squashed1); + + r = RegCreateKeyA(HKEY_LOCAL_MACHINE, keypath1, &key1); + ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r); + + strcpy(keypath2, "Software\\Microsoft\\Windows\\CurrentVersion\\Installer\\Managed\\"); + strcat(keypath2, usersid); + strcat(keypath2, "\\Installer\\Products\\"); + strcat(keypath2, product_squashed2); + + r = RegCreateKeyA(HKEY_LOCAL_MACHINE, keypath2, &key2); + ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r); + + strcpy(keypath3, "Software\\Microsoft\\Installer\\Products\\"); + strcat(keypath3, product_squashed3); + + r = RegCreateKeyA(HKEY_CURRENT_USER, keypath3, &key3); + ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r); + + index = 0; + r = MsiEnumProductsA(index, NULL); + ok(r == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %u\n", r); + + index = 2; + r = MsiEnumProductsA(index, guid); + ok(r == ERROR_INVALID_PARAMETER, "Expected ERROR_INVALID_PARAMETER, got %u\n", r); + + index = 0; + r = MsiEnumProductsA(index, guid); + ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %u\n", r); + + found1 = found2 = found3 = 0; + while ((r = MsiEnumProductsA(index, guid)) == ERROR_SUCCESS) + { + if (!strcmp(product1, guid)) found1 = 1; + if (!strcmp(product2, guid)) found2 = 1; + if (!strcmp(product3, guid)) found3 = 1; + index++; + } + ok(r == ERROR_NO_MORE_ITEMS, "Expected ERROR_NO_MORE_ITEMS, got %u\n", r); + ok(found1, "product1 not found\n"); + ok(found2, "product2 not found\n"); + ok(found3, "product3 not found\n"); + + RegDeleteKeyA(key1, ""); + RegDeleteKeyA(key2, ""); + RegDeleteKeyA(key3, ""); + RegCloseKey(key1); + RegCloseKey(key2); + RegCloseKey(key3); + LocalFree(usersid); +} + START_TEST(msi) { init_functionpointers(); @@ -10901,6 +10971,7 @@ START_TEST(msi) test_MsiEnumPatchesEx(); test_MsiEnumPatches(); test_MsiGetPatchInfoEx(); + test_MsiEnumProducts(); } test_MsiGetFileVersion();

1 0

Hans Leidekker : msi: Allocate token information buffer dynamically.
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: 4365b34838f14c0a6908ea794429ba6e493a59d2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4365b34838f14c0a6908ea794… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Mar 27 13:41:35 2009 +0100 msi: Allocate token information buffer dynamically. --- dlls/msi/tests/msi.c | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/dlls/msi/tests/msi.c b/dlls/msi/tests/msi.c index 13afd92..c3ef72b 100644 --- a/dlls/msi/tests/msi.c +++ b/dlls/msi/tests/msi.c @@ -516,15 +516,19 @@ static void create_test_guid(LPSTR prodcode, LPSTR squashed) static void get_user_sid(LPSTR *usersid) { HANDLE token; - BYTE buf[1024]; DWORD size; PTOKEN_USER user; OpenProcessToken(GetCurrentProcess(), TOKEN_QUERY, &token); - size = sizeof(buf); - GetTokenInformation(token, TokenUser, buf, size, &size); - user = (PTOKEN_USER)buf; + + size = 0; + GetTokenInformation(token, TokenUser, NULL, size, &size); + user = HeapAlloc(GetProcessHeap(), 0, size); + + GetTokenInformation(token, TokenUser, user, size, &size); pConvertSidToStringSidA(user->User.Sid, usersid); + + HeapFree(GetProcessHeap(), 0, user); CloseHandle(token); }

1 0

Henri Verbeet : wined3d: Use separate structures for ddraw style strided data and wined3d' s internal stream info.
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: d7c7c768507a923902e9178f98343e22e4e22652 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d7c7c768507a923902e9178f9… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Mar 27 10:25:55 2009 +0100 wined3d: Use separate structures for ddraw style strided data and wined3d's internal stream info. --- dlls/ddraw/device.c | 76 +++++----- dlls/wined3d/buffer.c | 58 ++++---- dlls/wined3d/device.c | 270 +++++++++++++++++++++++++++++---- dlls/wined3d/drawprim.c | 331 ++++++++++----------------------------- dlls/wined3d/state.c | 303 ++++++++++++++++++------------------- dlls/wined3d/wined3d_private.h | 59 ++++++-- include/wine/wined3d.idl | 35 +---- 7 files changed, 595 insertions(+), 537 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d7c7c768507a923902e91…

1 0

Henri Verbeet : wined3d: Remove IWineD3DVertexDeclarationImpl_GetDeclaration().
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: d8a4cc664833901fcaa4af4e1756901e731933a6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d8a4cc664833901fcaa4af4e1… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Mar 27 10:25:55 2009 +0100 wined3d: Remove IWineD3DVertexDeclarationImpl_GetDeclaration(). It's never used. --- dlls/wined3d/vertexdeclaration.c | 17 ----------------- include/wine/wined3d.idl | 4 ---- 2 files changed, 0 insertions(+), 21 deletions(-) diff --git a/dlls/wined3d/vertexdeclaration.c b/dlls/wined3d/vertexdeclaration.c index cef81d3..0091ce4 100644 --- a/dlls/wined3d/vertexdeclaration.c +++ b/dlls/wined3d/vertexdeclaration.c @@ -103,22 +103,6 @@ static HRESULT WINAPI IWineD3DVertexDeclarationImpl_GetDevice(IWineD3DVertexDecl return WINED3D_OK; } -static HRESULT WINAPI IWineD3DVertexDeclarationImpl_GetDeclaration(IWineD3DVertexDeclaration *iface, - WINED3DVERTEXELEMENT *elements, UINT *element_count) { - IWineD3DVertexDeclarationImpl *This = (IWineD3DVertexDeclarationImpl *)iface; - HRESULT hr = WINED3D_OK; - - TRACE("(%p) : d3d version %d, elements %p, element_count %p\n", - This, ((IWineD3DImpl *)This->wineD3DDevice->wineD3D)->dxVersion, elements, element_count); - - *element_count = This->declarationWNumElements; - if (elements) { - CopyMemory(elements, This->pDeclarationWine, This->declarationWNumElements * sizeof(WINED3DVERTEXELEMENT)); - } - - return hr; -} - static BOOL declaration_element_valid_ffp(const WINED3DVERTEXELEMENT *element) { switch(element->Usage) @@ -287,6 +271,5 @@ const IWineD3DVertexDeclarationVtbl IWineD3DVertexDeclaration_Vtbl = /* IWineD3DVertexDeclaration */ IWineD3DVertexDeclarationImpl_GetParent, IWineD3DVertexDeclarationImpl_GetDevice, - IWineD3DVertexDeclarationImpl_GetDeclaration, IWineD3DVertexDeclarationImpl_SetDeclaration }; diff --git a/include/wine/wined3d.idl b/include/wine/wined3d.idl index 783c3fa..0c17d7d 100644 --- a/include/wine/wined3d.idl +++ b/include/wine/wined3d.idl @@ -2758,10 +2758,6 @@ interface IWineD3DVertexDeclaration : IWineD3DBase HRESULT GetDevice( [out] IWineD3DDevice **device ); - HRESULT GetDeclaration( - [out] WINED3DVERTEXELEMENT *elements, - [out] UINT *element_count - ); HRESULT SetDeclaration( [in] const WINED3DVERTEXELEMENT *elements, [in] UINT element_count

1 0

Henri Verbeet : wined3d: Remove IWineD3DVertexDeclaration_SetDeclaration() from the public interface.
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: 4330d20d0df55bc15058083a377d985caa2fd1bc URL: http://source.winehq.org/git/wine.git/?a=commit;h=4330d20d0df55bc15058083a3… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Mar 27 10:25:55 2009 +0100 wined3d: Remove IWineD3DVertexDeclaration_SetDeclaration() from the public interface. --- dlls/wined3d/device.c | 3 ++- dlls/wined3d/vertexdeclaration.c | 7 +++---- dlls/wined3d/wined3d_private.h | 3 +++ include/wine/wined3d.idl | 4 ---- 4 files changed, 8 insertions(+), 9 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 70e6f63..a115192 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -2172,7 +2172,8 @@ static HRESULT WINAPI IWineD3DDeviceImpl_CreateVertexDeclaration(IWineD3DDevice* *ppVertexDeclaration = (IWineD3DVertexDeclaration *)object; - hr = IWineD3DVertexDeclaration_SetDeclaration((IWineD3DVertexDeclaration *)object, elements, element_count); + hr = vertexdeclaration_init(object, elements, element_count); + if(FAILED(hr)) { IWineD3DVertexDeclaration_Release((IWineD3DVertexDeclaration *)object); *ppVertexDeclaration = NULL; diff --git a/dlls/wined3d/vertexdeclaration.c b/dlls/wined3d/vertexdeclaration.c index 0091ce4..ae3bbc1 100644 --- a/dlls/wined3d/vertexdeclaration.c +++ b/dlls/wined3d/vertexdeclaration.c @@ -191,9 +191,9 @@ static BOOL declaration_element_valid_ffp(const WINED3DVERTEXELEMENT *element) } } -static HRESULT WINAPI IWineD3DVertexDeclarationImpl_SetDeclaration(IWineD3DVertexDeclaration *iface, - const WINED3DVERTEXELEMENT *elements, UINT element_count) { - IWineD3DVertexDeclarationImpl *This = (IWineD3DVertexDeclarationImpl *)iface; +HRESULT vertexdeclaration_init(IWineD3DVertexDeclarationImpl *This, + const WINED3DVERTEXELEMENT *elements, UINT element_count) +{ HRESULT hr = WINED3D_OK; unsigned int i; char isPreLoaded[MAX_STREAMS]; @@ -271,5 +271,4 @@ const IWineD3DVertexDeclarationVtbl IWineD3DVertexDeclaration_Vtbl = /* IWineD3DVertexDeclaration */ IWineD3DVertexDeclarationImpl_GetParent, IWineD3DVertexDeclarationImpl_GetDevice, - IWineD3DVertexDeclarationImpl_SetDeclaration }; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 73c394f..fd37ef5 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1765,6 +1765,9 @@ typedef struct IWineD3DVertexDeclarationImpl { extern const IWineD3DVertexDeclarationVtbl IWineD3DVertexDeclaration_Vtbl; +HRESULT vertexdeclaration_init(IWineD3DVertexDeclarationImpl *This, + const WINED3DVERTEXELEMENT *elements, UINT element_count); + /***************************************************************************** * IWineD3DStateBlock implementation structure */ diff --git a/include/wine/wined3d.idl b/include/wine/wined3d.idl index 0c17d7d..2f92898 100644 --- a/include/wine/wined3d.idl +++ b/include/wine/wined3d.idl @@ -2758,10 +2758,6 @@ interface IWineD3DVertexDeclaration : IWineD3DBase HRESULT GetDevice( [out] IWineD3DDevice **device ); - HRESULT SetDeclaration( - [in] const WINED3DVERTEXELEMENT *elements, - [in] UINT element_count - ); } [

1 0

Henri Verbeet : wined3d: Separate the internal representation of vertex declaration elements from the struct used to create it .
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: 9f26fed28dc0c7f356dc9822e82dbae05245cafb URL: http://source.winehq.org/git/wine.git/?a=commit;h=9f26fed28dc0c7f356dc9822e… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Mar 27 10:25:56 2009 +0100 wined3d: Separate the internal representation of vertex declaration elements from the struct used to create it. Internally we want to store some extra data, like ffp_valid. One of the later patches also stores the format desc in the declaration elements, instead of the current WINED3DDECLTYPE. --- dlls/wined3d/device.c | 60 +++++++++++++++++++------------------- dlls/wined3d/vertexdeclaration.c | 53 +++++++++++++++++++-------------- dlls/wined3d/vertexshader.c | 7 ++-- dlls/wined3d/wined3d_private.h | 17 +++++++++-- 4 files changed, 79 insertions(+), 58 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=9f26fed28dc0c7f356dc9…

1 0

Henri Verbeet : wined3d: Don' t create an END element for wined3d vertex declarations.
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: 6f206c75ecd9d9849b47d8faa5b85f26184e8479 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6f206c75ecd9d9849b47d8faa… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Mar 27 10:25:56 2009 +0100 wined3d: Don't create an END element for wined3d vertex declarations. Wined3d doesn't need it since it already has the element count. --- dlls/d3d8/vertexdeclaration.c | 5 ----- dlls/d3d9/vertexdeclaration.c | 17 ++++++++++++----- dlls/wined3d/device.c | 13 +++++-------- dlls/wined3d/vertexdeclaration.c | 3 --- include/wine/wined3d.idl | 1 - 5 files changed, 17 insertions(+), 22 deletions(-) diff --git a/dlls/d3d8/vertexdeclaration.c b/dlls/d3d8/vertexdeclaration.c index 46957d0..f2da777 100644 --- a/dlls/d3d8/vertexdeclaration.c +++ b/dlls/d3d8/vertexdeclaration.c @@ -324,11 +324,6 @@ UINT convert_to_wined3d_declaration(const DWORD *d3d8_elements, DWORD *d3d8_elem token += parse_token(token); } - /* END */ - element = *wined3d_elements + element_count++; - element->Stream = 0xFF; - element->Type = WINED3DDECLTYPE_UNUSED; - *d3d8_elements_size = (++token - d3d8_elements) * sizeof(DWORD); return element_count; diff --git a/dlls/d3d9/vertexdeclaration.c b/dlls/d3d9/vertexdeclaration.c index 86403fd..74229e9 100644 --- a/dlls/d3d9/vertexdeclaration.c +++ b/dlls/d3d9/vertexdeclaration.c @@ -312,13 +312,16 @@ static UINT convert_to_wined3d_declaration(const D3DVERTEXELEMENT9* d3d9_element while (element++->Stream != 0xff && element_count++ < 128); if (element_count == 128) { - return 0; + return ~0U; } + /* Skip the END element */ + --element_count; + *wined3d_elements = HeapAlloc(GetProcessHeap(), 0, element_count * sizeof(WINED3DVERTEXELEMENT)); if (!*wined3d_elements) { FIXME("Memory allocation failed\n"); - return 0; + return ~0U; } for (i = 0; i < element_count; ++i) { @@ -335,6 +338,7 @@ HRESULT WINAPI IDirect3DDevice9Impl_CreateVertexDeclaration(LPDIRECT3DDEVICE9E IDirect3DDevice9Impl *This = (IDirect3DDevice9Impl *)iface; IDirect3DVertexDeclaration9Impl *object = NULL; WINED3DVERTEXELEMENT* wined3d_elements; + UINT wined3d_element_count; UINT element_count; HRESULT hr = D3D_OK; @@ -344,8 +348,9 @@ HRESULT WINAPI IDirect3DDevice9Impl_CreateVertexDeclaration(LPDIRECT3DDEVICE9E return D3DERR_INVALIDCALL; } - element_count = convert_to_wined3d_declaration(pVertexElements, &wined3d_elements); - if (!element_count) { + wined3d_element_count = convert_to_wined3d_declaration(pVertexElements, &wined3d_elements); + if (wined3d_element_count == ~0U) + { FIXME("(%p) : Error parsing vertex declaration\n", This); return D3DERR_INVALIDCALL; } @@ -361,6 +366,7 @@ HRESULT WINAPI IDirect3DDevice9Impl_CreateVertexDeclaration(LPDIRECT3DDEVICE9E object->lpVtbl = &Direct3DVertexDeclaration9_Vtbl; object->ref = 0; + element_count = wined3d_element_count + 1; object->elements = HeapAlloc(GetProcessHeap(), 0, element_count * sizeof(D3DVERTEXELEMENT9)); if (!object->elements) { HeapFree(GetProcessHeap(), 0, wined3d_elements); @@ -372,7 +378,8 @@ HRESULT WINAPI IDirect3DDevice9Impl_CreateVertexDeclaration(LPDIRECT3DDEVICE9E object->element_count = element_count; EnterCriticalSection(&d3d9_cs); - hr = IWineD3DDevice_CreateVertexDeclaration(This->WineD3DDevice, &object->wineD3DVertexDeclaration, (IUnknown *)object, wined3d_elements, element_count); + hr = IWineD3DDevice_CreateVertexDeclaration(This->WineD3DDevice, &object->wineD3DVertexDeclaration, + (IUnknown *)object, wined3d_elements, wined3d_element_count); LeaveCriticalSection(&d3d9_cs); HeapFree(GetProcessHeap(), 0, wined3d_elements); diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index dd493f5..5bb9771 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -2200,8 +2200,6 @@ static unsigned int ConvertFvfToDeclaration(IWineD3DDeviceImpl *This, /* For the DWORD num_textures = (fvf & WINED3DFVF_TEXCOUNT_MASK) >> WINED3DFVF_TEXCOUNT_SHIFT; DWORD texcoords = (fvf & 0xFFFF0000) >> 16; - - WINED3DVERTEXELEMENT end_element = WINED3DDECL_END(); WINED3DVERTEXELEMENT *elements = NULL; unsigned int size; @@ -2210,14 +2208,12 @@ static unsigned int ConvertFvfToDeclaration(IWineD3DDeviceImpl *This, /* For the /* Compute declaration size */ size = has_pos + (has_blend && num_blends > 0) + has_blend_idx + has_normal + - has_psize + has_diffuse + has_specular + num_textures + 1; + has_psize + has_diffuse + has_specular + num_textures; /* convert the declaration */ elements = HeapAlloc(GetProcessHeap(), 0, size * sizeof(WINED3DVERTEXELEMENT)); - if (!elements) - return 0; + if (!elements) return ~0U; - elements[size-1] = end_element; idx = 0; if (has_pos) { if (!has_blend && (fvf & WINED3DFVF_XYZRHW)) { @@ -2310,7 +2306,8 @@ static unsigned int ConvertFvfToDeclaration(IWineD3DDeviceImpl *This, /* For the } /* Now compute offsets, and initialize the rest of the fields */ - for (idx = 0, offset = 0; idx < size-1; idx++) { + for (idx = 0, offset = 0; idx < size; ++idx) + { elements[idx].Stream = 0; elements[idx].Method = WINED3DDECLMETHOD_DEFAULT; elements[idx].Offset = offset; @@ -2328,7 +2325,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_CreateVertexDeclarationFromFVF(IWineD3D DWORD hr; size = ConvertFvfToDeclaration(This, Fvf, &elements); - if (size == 0) return WINED3DERR_OUTOFVIDEOMEMORY; + if (size == ~0U) return WINED3DERR_OUTOFVIDEOMEMORY; hr = IWineD3DDevice_CreateVertexDeclaration(iface, ppVertexDeclaration, Parent, elements, size); HeapFree(GetProcessHeap(), 0, elements); diff --git a/dlls/wined3d/vertexdeclaration.c b/dlls/wined3d/vertexdeclaration.c index cb2915d..1c23c51 100644 --- a/dlls/wined3d/vertexdeclaration.c +++ b/dlls/wined3d/vertexdeclaration.c @@ -206,9 +206,6 @@ HRESULT vertexdeclaration_init(IWineD3DVertexDeclarationImpl *This, } } - /* Skip the END element. */ - --element_count; - This->element_count = element_count; This->elements = HeapAlloc(GetProcessHeap(), 0, sizeof(*This->elements) * element_count); if (!This->elements) diff --git a/include/wine/wined3d.idl b/include/wine/wined3d.idl index 2f92898..bed165b 100644 --- a/include/wine/wined3d.idl +++ b/include/wine/wined3d.idl @@ -846,7 +846,6 @@ typedef enum _WINED3DDECLTYPE WINED3DDECLTYPE_FLOAT16_4 = 16, WINED3DDECLTYPE_UNUSED = 17, } WINED3DDECLTYPE; -cpp_quote("#define WINED3DDECL_END() {0xFF, 0, WINED3DDECLTYPE_UNUSED, 0, 0, 0, -1}") typedef enum _WINED3DDECLUSAGE {

1 0

Henri Verbeet : wined3d: Unify vertex and texture formats.
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: 4434d00f84c0e254a8c5d27fa80cffaf621897c2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4434d00f84c0e254a8c5d27fa… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Fri Mar 27 10:25:56 2009 +0100 wined3d: Unify vertex and texture formats. Some fields should be redundant now, eg. gl_vtx_type and glType. I'll leave that for a different patch to fix though. --- dlls/d3d8/vertexdeclaration.c | 38 ++++++-- dlls/d3d9/vertexdeclaration.c | 85 ++++++++++------- dlls/ddraw/device.c | 44 +++++----- dlls/wined3d/buffer.c | 40 ++++---- dlls/wined3d/device.c | 126 +++++++++++++------------ dlls/wined3d/directx.c | 193 +++++++++++++++++++------------------- dlls/wined3d/drawprim.c | 73 +++++++------- dlls/wined3d/state.c | 41 ++++---- dlls/wined3d/utils.c | 177 ++++++++++++++++++++--------------- dlls/wined3d/vertexdeclaration.c | 123 ++++++++++++------------ dlls/wined3d/wined3d_gl.h | 12 --- dlls/wined3d/wined3d_private.h | 57 ++++++++---- include/wine/wined3d.idl | 46 +++------- 13 files changed, 552 insertions(+), 503 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=4434d00f84c0e254a8c5d…

1 0

Rein Klazes : user32/tests: Show that a submenu is still functional after calling DestroyMenu().
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: 5396fbf97d5eb859855366a41e004019ff5980c0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5396fbf97d5eb859855366a41… Author: Rein Klazes <wijn(a)online.nl> Date: Fri Mar 27 08:40:26 2009 +0100 user32/tests: Show that a submenu is still functional after calling DestroyMenu(). --- dlls/user32/tests/menu.c | 100 ++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 100 insertions(+), 0 deletions(-) diff --git a/dlls/user32/tests/menu.c b/dlls/user32/tests/menu.c index 06fa350..f375123 100644 --- a/dlls/user32/tests/menu.c +++ b/dlls/user32/tests/menu.c @@ -290,6 +290,105 @@ static void test_menu_locked_by_window(void) DestroyWindow(hwnd); } +/* demonstrates that subpopup's are locked + * even after a client calls DestroyMenu on it */ +static LRESULT WINAPI subpopuplocked_wnd_proc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam) +{ + HWND hwndmenu; + switch (msg) + { + case WM_ENTERIDLE: + hwndmenu = GetCapture(); + if( hwndmenu) { + PostMessage( hwndmenu, WM_KEYDOWN, VK_DOWN, 0); + PostMessage( hwndmenu, WM_KEYDOWN, VK_RIGHT, 0); + PostMessage( hwndmenu, WM_KEYDOWN, VK_RETURN, 0); + } + } + return DefWindowProc(hwnd, msg, wparam, lparam); +} + +static void test_subpopup_locked_by_menu(void) +{ + DWORD gle; + BOOL ret; + HMENU hmenu, hsubmenu; + MENUINFO mi = { sizeof( MENUINFO)}; + MENUITEMINFO mii = { sizeof( MENUITEMINFO)}; + HWND hwnd; + const int itemid = 0x1234567; + if( !pGetMenuInfo) + { + win_skip("GetMenuInfo is not available\n"); + return; + } + /* create window, popupmenu with one subpopup */ + hwnd = CreateWindowEx(0, MAKEINTATOM(atomMenuCheckClass), NULL, + WS_VISIBLE, CW_USEDEFAULT, CW_USEDEFAULT, 200, 200, + NULL, NULL, NULL, NULL); + ok(hwnd != NULL, "CreateWindowEx failed with error %d\n", GetLastError()); + SetWindowLongPtr( hwnd, GWLP_WNDPROC, (LONG_PTR) subpopuplocked_wnd_proc); + hmenu = CreatePopupMenu(); + ok(hmenu != NULL, "CreateMenu failed with error %d\n", GetLastError()); + hsubmenu = CreatePopupMenu(); + ok(hsubmenu != NULL, "CreateMenu failed with error %d\n", GetLastError()); + ret = InsertMenu(hmenu, 0, MF_BYPOSITION | MF_POPUP | MF_STRING, (UINT_PTR)hsubmenu, + TEXT("PopUpLockTest")); + ok(ret, "InsertMenu failed with error %d\n", GetLastError()); + ret = InsertMenu(hsubmenu, 0, MF_BYPOSITION | MF_STRING, itemid, TEXT("PopUpMenu")); + ok(ret, "InsertMenu failed with error %d\n", GetLastError()); + /* first some tests that all this functions properly */ + mii.fMask = MIIM_SUBMENU; + ret = GetMenuItemInfo( hmenu, 0, TRUE, &mii); + ok( ret, "GetMenuItemInfo failed error %d\n", GetLastError()); + ok( mii.hSubMenu == hsubmenu, "submenu is %p\n", mii.hSubMenu); + mi.fMask |= MIM_STYLE; + ret = pGetMenuInfo( hsubmenu, &mi); + ok( ret , "GetMenuInfo returned 0 with error %d\n", GetLastError()); + ret = IsMenu( hsubmenu); + ok( ret , "Menu handle is not valid\n"); + SetLastError( 0xdeadbeef); + ret = TrackPopupMenu( hmenu, 0x100, 100,100, 0, hwnd, NULL); + if( ret == (itemid & 0xffff)) { + win_skip("not on 16 bit menu subsystem\n"); + DestroyMenu( hsubmenu); + } else { + gle = GetLastError(); + ok( ret == itemid , "TrackPopupMenu returned %d error is %d\n", ret, gle); + ok( gle == 0 || + broken( gle == 0xdeadbeef), /* win2k0 */ + "Last error is %d\n", gle); + /* then destroy the sub-popup */ + ret = DestroyMenu( hsubmenu); + ok(ret, "DestroyMenu failed with error %d\n", GetLastError()); + /* and repeat the tests */ + mii.fMask = MIIM_SUBMENU; + ret = GetMenuItemInfo( hmenu, 0, TRUE, &mii); + ok( ret, "GetMenuItemInfo failed error %d\n", GetLastError()); + /* GetMenuInfo fails now */ + ok( mii.hSubMenu == hsubmenu, "submenu is %p\n", mii.hSubMenu); + mi.fMask |= MIM_STYLE; + ret = pGetMenuInfo( hsubmenu, &mi); + ok( !ret , "GetMenuInfo should have failed\n"); + /* IsMenu says it is not */ + ret = IsMenu( hsubmenu); + ok( !ret , "Menu handle should be invalid\n"); + /* but TrackPopupMenu still works! */ + SetLastError( 0xdeadbeef); + ret = TrackPopupMenu( hmenu, 0x100, 100,100, 0, hwnd, NULL); + gle = GetLastError(); + todo_wine { + ok( ret == itemid , "TrackPopupMenu returned %d error is %d\n", ret, gle); + } + ok( gle == 0 || + broken( gle == ERROR_INVALID_PARAMETER), /* win2k0 */ + "Last error is %d\n", gle); + } + /* clean up */ + DestroyMenu( hmenu); + DestroyWindow(hwnd); +} + static void test_menu_ownerdraw(void) { int i,j,k; @@ -2768,6 +2867,7 @@ START_TEST(menu) register_menu_check_class(); test_menu_locked_by_window(); + test_subpopup_locked_by_menu(); test_menu_ownerdraw(); test_menu_bmp_and_string(); /* test Get/SetMenuInfo if available */

1 0

Christian Costa : kernel32: Add stub for GetSystemTimes.
by Alexandre Julliard 27 Mar '09

27 Mar '09

Module: wine Branch: master Commit: de90eccd1ac3a65526fac0f24debe8127b74baa3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=de90eccd1ac3a65526fac0f24… Author: Christian Costa <titan.costa(a)wanadoo.fr> Date: Thu Mar 26 23:00:55 2009 +0100 kernel32: Add stub for GetSystemTimes. --- dlls/kernel32/kernel32.spec | 2 +- dlls/kernel32/time.c | 20 ++++++++++++++++++++ 2 files changed, 21 insertions(+), 1 deletions(-) diff --git a/dlls/kernel32/kernel32.spec b/dlls/kernel32/kernel32.spec index c615ef1..18b9737 100644 --- a/dlls/kernel32/kernel32.spec +++ b/dlls/kernel32/kernel32.spec @@ -628,7 +628,7 @@ @ stdcall GetSystemTime(ptr) @ stdcall GetSystemTimeAdjustment(ptr ptr ptr) @ stdcall GetSystemTimeAsFileTime(ptr) -# @ stub GetSystemTimes +@ stdcall GetSystemTimes(ptr ptr ptr) @ stdcall GetSystemWindowsDirectoryA(ptr long) @ stdcall GetSystemWindowsDirectoryW(ptr long) @ stdcall GetSystemWow64DirectoryA(ptr long) diff --git a/dlls/kernel32/time.c b/dlls/kernel32/time.c index 29b3dc8..b30314c 100644 --- a/dlls/kernel32/time.c +++ b/dlls/kernel32/time.c @@ -1013,3 +1013,23 @@ BOOL WINAPI FileTimeToDosDateTime( const FILETIME *ft, LPWORD fatdate, + tm->tm_mday; return TRUE; } + +/********************************************************************* + * GetSystemTimes (KERNEL32.@) + * + * Retrieves system timing information + * + * PARAMS + * lpIdleTime [O] Destination for idle time. + * lpKernelTime [O] Destination for kernel time. + * lpUserTime [O] Destination for user time. + * + * RETURNS + * TRUE if success, FALSE otherwise. + */ +BOOL WINAPI GetSystemTimes(LPFILETIME lpIdleTime, LPFILETIME lpKernelTime, LPFILETIME lpUserTime) +{ + FIXME("(%p,%p,%p): Stub!\n", lpIdleTime, lpKernelTime, lpUserTime); + + return FALSE; +}

1 0