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

Ričardas Barkauskas : ddraw: Use helper function to delete attached surfaces.
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: 104e68da459490a6e348620651fc181c2177fc9b URL: http://source.winehq.org/git/wine.git/?a=commit;h=104e68da459490a6e34862065… Author: Ričardas Barkauskas <rbarkauskas(a)codeweavers.com> Date: Mon Jun 20 23:46:26 2011 +0300 ddraw: Use helper function to delete attached surfaces. --- dlls/ddraw/surface.c | 69 +++++++++++++++++++++++++++++++++++++++----------- 1 files changed, 54 insertions(+), 15 deletions(-) diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index 36c4084..fdb199e 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -1452,14 +1452,12 @@ static HRESULT WINAPI ddraw_surface1_AddAttachedSurface(IDirectDrawSurface *ifac * DDERR_SURFACENOTATTACHED if the surface isn't attached to * *****************************************************************************/ -static HRESULT WINAPI ddraw_surface7_DeleteAttachedSurface(IDirectDrawSurface7 *iface, - DWORD Flags, IDirectDrawSurface7 *Attach) +static HRESULT ddraw_surface_delete_attached_surface(IDirectDrawSurfaceImpl *This, + IDirectDrawSurfaceImpl *Surf) { - IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface7(iface); - IDirectDrawSurfaceImpl *Surf = unsafe_impl_from_IDirectDrawSurface7(Attach); IDirectDrawSurfaceImpl *Prev = This; - TRACE("iface %p, flags %#x, attachment %p.\n", iface, Flags, Attach); + TRACE("surface %p, attachment %p.\n", This, Surf); EnterCriticalSection(&ddraw_cs); if (!Surf || (Surf->first_attached != This) || (Surf == This) ) @@ -1497,21 +1495,44 @@ static HRESULT WINAPI ddraw_surface7_DeleteAttachedSurface(IDirectDrawSurface7 * { IDirect3DDeviceImpl_UpdateDepthStencil(This->ddraw->d3ddevice); } - - ddraw_surface7_Release(Attach); LeaveCriticalSection(&ddraw_cs); return DD_OK; } +static HRESULT WINAPI ddraw_surface7_DeleteAttachedSurface(IDirectDrawSurface7 *iface, + DWORD flags, IDirectDrawSurface7 *attachment) +{ + IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface7(iface); + IDirectDrawSurfaceImpl *attachment_impl = unsafe_impl_from_IDirectDrawSurface7(attachment); + HRESULT hr; + + TRACE("iface %p, flags %#x, attachment %p.\n", iface, flags, attachment); + + hr = ddraw_surface_delete_attached_surface(This, attachment_impl); + if (FAILED(hr)) + { + return hr; + } + ddraw_surface7_Release(attachment); + return hr; +} + static HRESULT WINAPI ddraw_surface4_DeleteAttachedSurface(IDirectDrawSurface4 *iface, DWORD flags, IDirectDrawSurface4 *attachment) { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface4(iface); IDirectDrawSurfaceImpl *attachment_impl = unsafe_impl_from_IDirectDrawSurface4(attachment); + HRESULT hr; + TRACE("iface %p, flags %#x, attachment %p.\n", iface, flags, attachment); - return ddraw_surface7_DeleteAttachedSurface(&This->IDirectDrawSurface7_iface, flags, - attachment_impl ? &attachment_impl->IDirectDrawSurface7_iface : NULL); + hr = ddraw_surface_delete_attached_surface(This, attachment_impl); + if (FAILED(hr)) + { + return hr; + } + ddraw_surface4_Release(attachment); + return hr; } static HRESULT WINAPI ddraw_surface3_DeleteAttachedSurface(IDirectDrawSurface3 *iface, @@ -1519,10 +1540,16 @@ static HRESULT WINAPI ddraw_surface3_DeleteAttachedSurface(IDirectDrawSurface3 * { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface3(iface); IDirectDrawSurfaceImpl *attachment_impl = unsafe_impl_from_IDirectDrawSurface3(attachment); + HRESULT hr; TRACE("iface %p, flags %#x, attachment %p.\n", iface, flags, attachment); - return ddraw_surface7_DeleteAttachedSurface(&This->IDirectDrawSurface7_iface, flags, - attachment_impl ? &attachment_impl->IDirectDrawSurface7_iface : NULL); + hr = ddraw_surface_delete_attached_surface(This, attachment_impl); + if (FAILED(hr)) + { + return hr; + } + ddraw_surface3_Release(attachment); + return hr; } static HRESULT WINAPI ddraw_surface2_DeleteAttachedSurface(IDirectDrawSurface2 *iface, @@ -1530,10 +1557,16 @@ static HRESULT WINAPI ddraw_surface2_DeleteAttachedSurface(IDirectDrawSurface2 * { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface2(iface); IDirectDrawSurfaceImpl *attachment_impl = unsafe_impl_from_IDirectDrawSurface2(attachment); + HRESULT hr; TRACE("iface %p, flags %#x, attachment %p.\n", iface, flags, attachment); - return ddraw_surface7_DeleteAttachedSurface(&This->IDirectDrawSurface7_iface, flags, - attachment_impl ? &attachment_impl->IDirectDrawSurface7_iface : NULL); + hr = ddraw_surface_delete_attached_surface(This, attachment_impl); + if (FAILED(hr)) + { + return hr; + } + ddraw_surface2_Release(attachment); + return hr; } static HRESULT WINAPI ddraw_surface1_DeleteAttachedSurface(IDirectDrawSurface *iface, @@ -1541,10 +1574,16 @@ static HRESULT WINAPI ddraw_surface1_DeleteAttachedSurface(IDirectDrawSurface *i { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface(iface); IDirectDrawSurfaceImpl *attachment_impl = unsafe_impl_from_IDirectDrawSurface(attachment); + HRESULT hr; TRACE("iface %p, flags %#x, attachment %p.\n", iface, flags, attachment); - return ddraw_surface7_DeleteAttachedSurface(&This->IDirectDrawSurface7_iface, flags, - attachment_impl ? &attachment_impl->IDirectDrawSurface7_iface : NULL); + hr = ddraw_surface_delete_attached_surface(This, attachment_impl); + if (FAILED(hr)) + { + return hr; + } + ddraw_surface1_Release(attachment); + return hr; } /*****************************************************************************

1 0

Ričardas Barkauskas : ddraw: Test ddraw surface reference counts a bit more.
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: a9f72a3067b049fea2e9d8fc554137b5d1b5977f URL: http://source.winehq.org/git/wine.git/?a=commit;h=a9f72a3067b049fea2e9d8fc5… Author: Ričardas Barkauskas <rbarkauskas(a)codeweavers.com> Date: Tue Jun 21 00:40:54 2011 +0300 ddraw: Test ddraw surface reference counts a bit more. --- dlls/ddraw/tests/dsurface.c | 21 +++++++++++++++++++++ 1 files changed, 21 insertions(+), 0 deletions(-) diff --git a/dlls/ddraw/tests/dsurface.c b/dlls/ddraw/tests/dsurface.c index 7416c93..8bc400d 100644 --- a/dlls/ddraw/tests/dsurface.c +++ b/dlls/ddraw/tests/dsurface.c @@ -1053,6 +1053,7 @@ static void IFaceRefCount(void) HRESULT ret; IDirectDrawSurface2 *surf2; IDirectDrawSurface2 *surf2a; + IDirectDrawSurface3 *surf3; IDirectDrawSurface4 *surf4; IDirectDrawSurface7 *surf7a; IDirectDrawSurface7 *surf7b; @@ -1089,6 +1090,10 @@ static void IFaceRefCount(void) ref = getRefcount((IUnknown *) surf); todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); + IDirectDrawSurface_QueryInterface(surf, &IID_IDirectDrawSurface3, (void **) &surf3); + ref = getRefcount((IUnknown *) surf3); + todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); + IDirectDrawSurface_QueryInterface(surf, &IID_IDirectDrawSurface4, (void **) &surf4); ref = getRefcount((IUnknown *) surf4); todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); @@ -1142,6 +1147,9 @@ static void IFaceRefCount(void) ref = IDirectDrawSurface2_Release(surf2a); /* Release the other */ todo_wine ok(ref == 0, "Refcount is %u, expected 0\n", ref); + ref = IDirectDrawSurface3_Release(surf3); + todo_wine ok(ref == 0, "Refcount is %u, expected 0\n", ref); + ref = IDirectDrawSurface4_Release(surf4); todo_wine ok(ref == 0, "Refcount is %u, expected 0\n", ref); @@ -3080,6 +3088,7 @@ static void GetDCTest(void) IDirectDraw4 *dd4; IDirectDraw7 *dd7; HDC dc; + ULONG ref; memset(&ddsd, 0, sizeof(ddsd)); ddsd.dwSize = sizeof(ddsd); @@ -3125,6 +3134,11 @@ static void GetDCTest(void) hr = IDirectDrawSurface4_QueryInterface(surf4, &IID_IDirectDrawSurface, (void **)&surf); ok(SUCCEEDED(hr), "QueryInterface failed, hr %#x.\n", hr); + ref = getRefcount((IUnknown *) surf); + todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); + ref = getRefcount((IUnknown *) surf4); + todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); + hr = IDirectDrawSurface4_GetDC(surf4, &dc); ok(SUCCEEDED(hr), "GetDC failed, hr %#x.\n", hr); @@ -3135,6 +3149,13 @@ static void GetDCTest(void) ok(SUCCEEDED(hr), "GetSurfaceFromDC failed, hr %#x.\n", hr); ok(tmp == surf, "Expected surface %p, got %p.\n\n", surf, tmp); + ref = getRefcount((IUnknown *) surf); + todo_wine ok(ref == 2, "Refcount is %u, expected 2\n", ref); + ref = getRefcount((IUnknown *) tmp); + todo_wine ok(ref == 2, "Refcount is %u, expected 2\n", ref); + ref = getRefcount((IUnknown *) surf4); + todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); + hr = IDirectDrawSurface4_ReleaseDC(surf4, dc); ok(SUCCEEDED(hr), "ReleaseDC failed, hr %#x.\n", hr);

1 0

Ričardas Barkauskas : ddraw: Separate IDirectDrawSurface2 reference count.
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: 167ffc7b4b58b8fe39a9bdeb0f04e0568dc79161 URL: http://source.winehq.org/git/wine.git/?a=commit;h=167ffc7b4b58b8fe39a9bdeb0… Author: Ričardas Barkauskas <rbarkauskas(a)codeweavers.com> Date: Sun Jun 19 17:24:32 2011 +0300 ddraw: Separate IDirectDrawSurface2 reference count. --- dlls/ddraw/ddraw_private.h | 2 +- dlls/ddraw/main.c | 9 ++++++--- dlls/ddraw/surface.c | 28 ++++++++++++++++++++++------ dlls/ddraw/tests/dsurface.c | 12 ++++++------ 4 files changed, 35 insertions(+), 16 deletions(-) diff --git a/dlls/ddraw/ddraw_private.h b/dlls/ddraw/ddraw_private.h index 7b00deb..5d74715 100644 --- a/dlls/ddraw/ddraw_private.h +++ b/dlls/ddraw/ddraw_private.h @@ -161,7 +161,7 @@ struct IDirectDrawSurfaceImpl const IDirect3DTexture2Vtbl *IDirect3DTexture2_vtbl; const IDirect3DTextureVtbl *IDirect3DTexture_vtbl; - LONG ref, iface_count; + LONG ref, ref2, iface_count; IUnknown *ifaceToRelease; int version; diff --git a/dlls/ddraw/main.c b/dlls/ddraw/main.c index 7a693ee..863c5b0 100644 --- a/dlls/ddraw/main.c +++ b/dlls/ddraw/main.c @@ -752,10 +752,12 @@ DestroyCallback(IDirectDrawSurface7 *surf, void *context) { IDirectDrawSurfaceImpl *Impl = impl_from_IDirectDrawSurface7(surf); - ULONG ref; + ULONG ref, ref2; ref = IDirectDrawSurface7_Release(surf); /* For the EnumSurfaces */ - WARN("Surface %p has an reference count of %d\n", Impl, ref); + IDirectDrawSurface2_AddRef(&Impl->IDirectDrawSurface2_iface); + ref2 = IDirectDrawSurface2_Release(&Impl->IDirectDrawSurface2_iface); + WARN("Surface %p has an reference counts of %d 2: %d\n", Impl, ref, ref2); /* Skip surfaces which are attached somewhere or which are * part of a complex compound. They will get released when destroying @@ -765,7 +767,8 @@ DestroyCallback(IDirectDrawSurface7 *surf, return DDENUMRET_OK; /* Destroy the surface */ - while(ref) ref = IDirectDrawSurface7_Release(surf); + while (ref) ref = IDirectDrawSurface7_Release(surf); + while (ref2) ref2 = IDirectDrawSurface2_Release(&Impl->IDirectDrawSurface2_iface); return DDENUMRET_OK; } diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index fdb199e..6204703 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -97,7 +97,7 @@ static HRESULT WINAPI ddraw_surface7_QueryInterface(IDirectDrawSurface7 *iface, } else if (IsEqualGUID(riid, &IID_IDirectDrawSurface2)) { - IUnknown_AddRef(iface); + IDirectDrawSurface2_AddRef(&This->IDirectDrawSurface2_iface); *obj = &This->IDirectDrawSurface2_iface; TRACE("(%p) returning IDirectDrawSurface2 interface at %p\n", This, *obj); return S_OK; @@ -274,9 +274,16 @@ static ULONG WINAPI ddraw_surface3_AddRef(IDirectDrawSurface3 *iface) static ULONG WINAPI ddraw_surface2_AddRef(IDirectDrawSurface2 *iface) { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface2(iface); - TRACE("iface %p.\n", iface); + ULONG refcount = InterlockedIncrement(&This->ref2); - return ddraw_surface7_AddRef(&This->IDirectDrawSurface7_iface); + TRACE("iface %p increasing refcount to %u.\n", iface, refcount); + + if (refcount == 1) + { + ddraw_surface_add_iface(This); + } + + return refcount; } static ULONG WINAPI ddraw_surface1_AddRef(IDirectDrawSurface *iface) @@ -335,7 +342,7 @@ void ddraw_surface_destroy(IDirectDrawSurfaceImpl *This) * because the 2nd surface was addref()ed when the app * called GetAttachedSurface */ - WARN("(%p): Destroying surface with refount %d\n", This, This->ref); + WARN("(%p): Destroying surface with refcounts %d 2: %d\n", This, This->ref, This->ref2); } if (This->wined3d_surface) @@ -522,9 +529,16 @@ static ULONG WINAPI ddraw_surface3_Release(IDirectDrawSurface3 *iface) static ULONG WINAPI ddraw_surface2_Release(IDirectDrawSurface2 *iface) { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface2(iface); - TRACE("iface %p.\n", iface); + ULONG refcount = InterlockedDecrement(&This->ref2); - return ddraw_surface7_Release(&This->IDirectDrawSurface7_iface); + TRACE("iface %p decreasing refcount to %u.\n", iface, refcount); + + if (refcount == 0) + { + ddraw_surface_release_iface(This); + } + + return refcount; } static ULONG WINAPI ddraw_surface1_Release(IDirectDrawSurface *iface) @@ -755,6 +769,8 @@ static HRESULT WINAPI ddraw_surface2_GetAttachedSurface(IDirectDrawSurface2 *ifa } attachment_impl = impl_from_IDirectDrawSurface7(attachment7); *attachment = &attachment_impl->IDirectDrawSurface2_iface; + ddraw_surface2_AddRef(*attachment); + ddraw_surface7_Release(attachment7); return hr; } diff --git a/dlls/ddraw/tests/dsurface.c b/dlls/ddraw/tests/dsurface.c index 8bc400d..99058ab 100644 --- a/dlls/ddraw/tests/dsurface.c +++ b/dlls/ddraw/tests/dsurface.c @@ -1080,15 +1080,15 @@ static void IFaceRefCount(void) IDirectDrawSurface_QueryInterface(surf, &IID_IDirectDrawSurface2, (void **) &surf2); ref = getRefcount((IUnknown *) surf); - todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); /* Check the ref count is one */ + ok(ref == 1, "Refcount is %u, expected 1\n", ref); /* Check the ref count is one */ ref = getRefcount((IUnknown *) surf2); - todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); /* This should also be one */ + ok(ref == 1, "Refcount is %u, expected 1\n", ref); /* This should also be one */ IDirectDrawSurface_QueryInterface(surf, &IID_IDirectDrawSurface2, (void **) &surf2a); ref = getRefcount((IUnknown *) surf2); - todo_wine ok(ref == 2, "Refcount is %u, expected 2\n", ref); /* Surf2's refcount should be 2 now, but surf should be 1 */ + ok(ref == 2, "Refcount is %u, expected 2\n", ref); /* Surf2's refcount should be 2 now, but surf should be 1 */ ref = getRefcount((IUnknown *) surf); - todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); + ok(ref == 1, "Refcount is %u, expected 1\n", ref); IDirectDrawSurface_QueryInterface(surf, &IID_IDirectDrawSurface3, (void **) &surf3); ref = getRefcount((IUnknown *) surf3); @@ -1142,10 +1142,10 @@ static void IFaceRefCount(void) } ref = IDirectDrawSurface2_Release(surf2); /* Release one of the 2 surf2 interfaces */ - todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); + ok(ref == 1, "Refcount is %u, expected 1\n", ref); ref = IDirectDrawSurface2_Release(surf2a); /* Release the other */ - todo_wine ok(ref == 0, "Refcount is %u, expected 0\n", ref); + ok(ref == 0, "Refcount is %u, expected 0\n", ref); ref = IDirectDrawSurface3_Release(surf3); todo_wine ok(ref == 0, "Refcount is %u, expected 0\n", ref);

1 0

Ričardas Barkauskas : ddraw: Separate IDirectDrawSurface3 reference count.
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: f0634f7a4bf125f1c7221fe675765a7b157d0b92 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f0634f7a4bf125f1c7221fe67… Author: Ričardas Barkauskas <rbarkauskas(a)codeweavers.com> Date: Mon Jun 20 23:51:01 2011 +0300 ddraw: Separate IDirectDrawSurface3 reference count. --- dlls/ddraw/ddraw_private.h | 2 +- dlls/ddraw/main.c | 7 +++++-- dlls/ddraw/surface.c | 29 +++++++++++++++++++++++------ dlls/ddraw/tests/dsurface.c | 4 ++-- 4 files changed, 31 insertions(+), 11 deletions(-) diff --git a/dlls/ddraw/ddraw_private.h b/dlls/ddraw/ddraw_private.h index 5d74715..e15219d 100644 --- a/dlls/ddraw/ddraw_private.h +++ b/dlls/ddraw/ddraw_private.h @@ -161,7 +161,7 @@ struct IDirectDrawSurfaceImpl const IDirect3DTexture2Vtbl *IDirect3DTexture2_vtbl; const IDirect3DTextureVtbl *IDirect3DTexture_vtbl; - LONG ref, ref2, iface_count; + LONG ref, ref3, ref2, iface_count; IUnknown *ifaceToRelease; int version; diff --git a/dlls/ddraw/main.c b/dlls/ddraw/main.c index 863c5b0..36c0ef7 100644 --- a/dlls/ddraw/main.c +++ b/dlls/ddraw/main.c @@ -752,12 +752,14 @@ DestroyCallback(IDirectDrawSurface7 *surf, void *context) { IDirectDrawSurfaceImpl *Impl = impl_from_IDirectDrawSurface7(surf); - ULONG ref, ref2; + ULONG ref, ref3, ref2; ref = IDirectDrawSurface7_Release(surf); /* For the EnumSurfaces */ + IDirectDrawSurface3_AddRef(&Impl->IDirectDrawSurface3_iface); + ref3 = IDirectDrawSurface3_Release(&Impl->IDirectDrawSurface3_iface); IDirectDrawSurface2_AddRef(&Impl->IDirectDrawSurface2_iface); ref2 = IDirectDrawSurface2_Release(&Impl->IDirectDrawSurface2_iface); - WARN("Surface %p has an reference counts of %d 2: %d\n", Impl, ref, ref2); + WARN("Surface %p has an reference counts of %d 3: %d 2: %d\n", Impl, ref, ref3, ref2); /* Skip surfaces which are attached somewhere or which are * part of a complex compound. They will get released when destroying @@ -768,6 +770,7 @@ DestroyCallback(IDirectDrawSurface7 *surf, /* Destroy the surface */ while (ref) ref = IDirectDrawSurface7_Release(surf); + while (ref3) ref3 = IDirectDrawSurface3_Release(&Impl->IDirectDrawSurface3_iface); while (ref2) ref2 = IDirectDrawSurface2_Release(&Impl->IDirectDrawSurface2_iface); return DDENUMRET_OK; diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index 6204703..412bc29 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -90,7 +90,7 @@ static HRESULT WINAPI ddraw_surface7_QueryInterface(IDirectDrawSurface7 *iface, } else if (IsEqualGUID(riid, &IID_IDirectDrawSurface3)) { - IUnknown_AddRef(iface); + IDirectDrawSurface3_AddRef(&This->IDirectDrawSurface3_iface); *obj = &This->IDirectDrawSurface3_iface; TRACE("(%p) returning IDirectDrawSurface3 interface at %p\n", This, *obj); return S_OK; @@ -266,9 +266,16 @@ static ULONG WINAPI ddraw_surface4_AddRef(IDirectDrawSurface4 *iface) static ULONG WINAPI ddraw_surface3_AddRef(IDirectDrawSurface3 *iface) { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface3(iface); - TRACE("iface %p.\n", iface); + ULONG refcount = InterlockedIncrement(&This->ref3); - return ddraw_surface7_AddRef(&This->IDirectDrawSurface7_iface); + TRACE("iface %p increasing refcount to %u.\n", iface, refcount); + + if (refcount == 1) + { + ddraw_surface_add_iface(This); + } + + return refcount; } static ULONG WINAPI ddraw_surface2_AddRef(IDirectDrawSurface2 *iface) @@ -342,7 +349,8 @@ void ddraw_surface_destroy(IDirectDrawSurfaceImpl *This) * because the 2nd surface was addref()ed when the app * called GetAttachedSurface */ - WARN("(%p): Destroying surface with refcounts %d 2: %d\n", This, This->ref, This->ref2); + WARN("(%p): Destroying surface with refcounts %d 3: %d 2: %d\n", This, This->ref, + This->ref3, This->ref2); } if (This->wined3d_surface) @@ -521,9 +529,16 @@ static ULONG WINAPI ddraw_surface4_Release(IDirectDrawSurface4 *iface) static ULONG WINAPI ddraw_surface3_Release(IDirectDrawSurface3 *iface) { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface3(iface); - TRACE("iface %p.\n", iface); + ULONG refcount = InterlockedDecrement(&This->ref3); - return ddraw_surface7_Release(&This->IDirectDrawSurface7_iface); + TRACE("iface %p decreasing refcount to %u.\n", iface, refcount); + + if (refcount == 0) + { + ddraw_surface_release_iface(This); + } + + return refcount; } static ULONG WINAPI ddraw_surface2_Release(IDirectDrawSurface2 *iface) @@ -740,6 +755,8 @@ static HRESULT WINAPI ddraw_surface3_GetAttachedSurface(IDirectDrawSurface3 *ifa } attachment_impl = impl_from_IDirectDrawSurface7(attachment7); *attachment = &attachment_impl->IDirectDrawSurface3_iface; + ddraw_surface3_AddRef(*attachment); + ddraw_surface7_Release(attachment7); return hr; } diff --git a/dlls/ddraw/tests/dsurface.c b/dlls/ddraw/tests/dsurface.c index 99058ab..4cfe469 100644 --- a/dlls/ddraw/tests/dsurface.c +++ b/dlls/ddraw/tests/dsurface.c @@ -1092,7 +1092,7 @@ static void IFaceRefCount(void) IDirectDrawSurface_QueryInterface(surf, &IID_IDirectDrawSurface3, (void **) &surf3); ref = getRefcount((IUnknown *) surf3); - todo_wine ok(ref == 1, "Refcount is %u, expected 1\n", ref); + ok(ref == 1, "Refcount is %u, expected 1\n", ref); IDirectDrawSurface_QueryInterface(surf, &IID_IDirectDrawSurface4, (void **) &surf4); ref = getRefcount((IUnknown *) surf4); @@ -1148,7 +1148,7 @@ static void IFaceRefCount(void) ok(ref == 0, "Refcount is %u, expected 0\n", ref); ref = IDirectDrawSurface3_Release(surf3); - todo_wine ok(ref == 0, "Refcount is %u, expected 0\n", ref); + ok(ref == 0, "Refcount is %u, expected 0\n", ref); ref = IDirectDrawSurface4_Release(surf4); todo_wine ok(ref == 0, "Refcount is %u, expected 0\n", ref);

1 0

Michael Stefaniuc : ole32: COM cleanup for the static object ComCatMgrImpl.
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: 7b3d1bbc4ea8d0067b01d5268b0faedfc2cac946 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7b3d1bbc4ea8d0067b01d5268… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Jun 21 10:38:53 2011 +0200 ole32: COM cleanup for the static object ComCatMgrImpl. --- dlls/ole32/comcat.c | 47 +++++++++++++++++++---------------------------- 1 files changed, 19 insertions(+), 28 deletions(-) diff --git a/dlls/ole32/comcat.c b/dlls/ole32/comcat.c index fe0f7e5..20151ae 100644 --- a/dlls/ole32/comcat.c +++ b/dlls/ole32/comcat.c @@ -36,12 +36,22 @@ WINE_DEFAULT_DEBUG_CHANNEL(ole); +static const ICatRegisterVtbl COMCAT_ICatRegister_Vtbl; +static const ICatInformationVtbl COMCAT_ICatInformation_Vtbl; + typedef struct { - const ICatRegisterVtbl *lpVtbl; - const ICatInformationVtbl *infVtbl; + ICatRegister ICatRegister_iface; + ICatInformation ICatInformation_iface; } ComCatMgrImpl; +/* static ComCatMgr instance */ +static ComCatMgrImpl COMCAT_ComCatMgr = +{ + { &COMCAT_ICatRegister_Vtbl }, + { &COMCAT_ICatInformation_Vtbl } +}; + struct class_categories { LPCWSTR impl_strings; LPCWSTR req_strings; @@ -63,12 +73,6 @@ static const WCHAR req_keyname[] = { static const WCHAR clsid_keyname[] = { 'C','L','S','I','D',0 }; -static inline ComCatMgrImpl *impl_from_ICatInformation( ICatInformation *iface ) -{ - return (ComCatMgrImpl *)((char*)iface - FIELD_OFFSET(ComCatMgrImpl, infVtbl)); -} - - /********************************************************************** * COMCAT_RegisterClassCategories */ @@ -279,20 +283,19 @@ static HRESULT WINAPI COMCAT_ICatRegister_QueryInterface( REFIID riid, LPVOID *ppvObj) { - ComCatMgrImpl *This = (ComCatMgrImpl *)iface; TRACE("%s\n",debugstr_guid(riid)); if (ppvObj == NULL) return E_POINTER; if (IsEqualGUID(riid, &IID_IUnknown) || IsEqualGUID(riid, &IID_ICatRegister)) { *ppvObj = iface; - IUnknown_AddRef(iface); + ICatRegister_AddRef(iface); return S_OK; } if (IsEqualGUID(riid, &IID_ICatInformation)) { - *ppvObj = &This->infVtbl; - IUnknown_AddRef(iface); + *ppvObj = &COMCAT_ComCatMgr.ICatInformation_iface; + ICatRegister_AddRef(iface); return S_OK; } @@ -462,8 +465,7 @@ static HRESULT WINAPI COMCAT_ICatInformation_QueryInterface( REFIID riid, LPVOID *ppvObj) { - ComCatMgrImpl *This = impl_from_ICatInformation( iface ); - return IUnknown_QueryInterface((LPUNKNOWN)This, riid, ppvObj); + return ICatRegister_QueryInterface(&COMCAT_ComCatMgr.ICatRegister_iface, riid, ppvObj); } /********************************************************************** @@ -471,8 +473,7 @@ static HRESULT WINAPI COMCAT_ICatInformation_QueryInterface( */ static ULONG WINAPI COMCAT_ICatInformation_AddRef(LPCATINFORMATION iface) { - ComCatMgrImpl *This = impl_from_ICatInformation( iface ); - return IUnknown_AddRef((LPUNKNOWN)This); + return ICatRegister_AddRef(&COMCAT_ComCatMgr.ICatRegister_iface); } /********************************************************************** @@ -480,8 +481,7 @@ static ULONG WINAPI COMCAT_ICatInformation_AddRef(LPCATINFORMATION iface) */ static ULONG WINAPI COMCAT_ICatInformation_Release(LPCATINFORMATION iface) { - ComCatMgrImpl *This = impl_from_ICatInformation( iface ); - return IUnknown_Release((LPUNKNOWN)This); + return ICatRegister_Release(&COMCAT_ComCatMgr.ICatRegister_iface); } /********************************************************************** @@ -705,15 +705,6 @@ static const ICatInformationVtbl COMCAT_ICatInformation_Vtbl = }; /********************************************************************** - * static ComCatMgr instance - */ -static ComCatMgrImpl COMCAT_ComCatMgr = -{ - &COMCAT_ICatRegister_Vtbl, - &COMCAT_ICatInformation_Vtbl -}; - -/********************************************************************** * COMCAT_IClassFactory_QueryInterface (also IUnknown) */ static HRESULT WINAPI COMCAT_IClassFactory_QueryInterface( @@ -769,7 +760,7 @@ static HRESULT WINAPI COMCAT_IClassFactory_CreateInstance( /* Don't support aggregation (Windows doesn't) */ if (pUnkOuter != NULL) return CLASS_E_NOAGGREGATION; - res = IUnknown_QueryInterface((LPUNKNOWN)&COMCAT_ComCatMgr, riid, ppvObj); + res = ICatRegister_QueryInterface(&COMCAT_ComCatMgr.ICatRegister_iface, riid, ppvObj); if (SUCCEEDED(res)) { return res; }

1 0

Jacek Caban : mshtml.idl: Added head element interfaces.
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: 0f8300c407ac9a83336b16b57db70a93cbf6da8a URL: http://source.winehq.org/git/wine.git/?a=commit;h=0f8300c407ac9a83336b16b57… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Jun 21 13:46:53 2011 +0200 mshtml.idl: Added head element interfaces. --- include/mshtmdid.h | 6 +++++ include/mshtml.idl | 62 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 68 insertions(+), 0 deletions(-) diff --git a/include/mshtmdid.h b/include/mshtmdid.h index 2dad6fb..ad09bdb 100644 --- a/include/mshtmdid.h +++ b/include/mshtmdid.h @@ -2282,6 +2282,12 @@ /* IHTMLSelectElement4 */ #define DISPID_IHTMLSELECTELEMENT4_NAMEDITEM DISPID_COLLECTION+6 +/* IHTMLHeadElement */ +#define DISPID_IHTMLHEADELEMENT_PROFILE DISPID_HEDELEMS+1 + +/* IHTMLHeadElement2 */ +#define DISPID_IHTMLHEADELEMENT2_IE8_PROFILE DISPID_IE8_HEAD + /* IHTMLRect */ #define DISPID_IHTMLRECT_LEFT DISPID_OMRECT+1 #define DISPID_IHTMLRECT_TOP DISPID_OMRECT+2 diff --git a/include/mshtml.idl b/include/mshtml.idl index c2e0874..575f15f 100644 --- a/include/mshtml.idl +++ b/include/mshtml.idl @@ -9187,6 +9187,68 @@ methods: } /***************************************************************************** + * IHTMLHeadElement interface + */ +[ + odl, + oleautomation, + dual, + uuid(3050f81d-98b5-11cf-bb82-00aa00bdce0b) +] +interface IHTMLHeadElement : IDispatch +{ + [propput, id(DISPID_IHTMLHEADELEMENT_PROFILE), displaybind, bindable] + HRESULT profile([in] BSTR v); + + [propget, id(DISPID_IHTMLHEADELEMENT_PROFILE), displaybind, bindable] + HRESULT profile([retval, out] BSTR *p); +} + +/***************************************************************************** + * IHTMLHeadElement2 interface + */ +[ + odl, + oleautomation, + dual, + uuid(3051042f-98b5-11cf-bb82-00aa00bdce0b) +] +interface IHTMLHeadElement2 : IDispatch +{ + [propput, id(DISPID_IHTMLHEADELEMENT2_IE8_PROFILE), displaybind, bindable] + HRESULT profile([in] BSTR v); + + [propget, id(DISPID_IHTMLHEADELEMENT2_IE8_PROFILE), displaybind, bindable] + HRESULT profile([retval, out] BSTR *p); +} + +/***************************************************************************** + * DispHTMLHeadElement interface + */ +[ + hidden, + uuid(3050f561-98b5-11cf-bb82-00aa00bdce0b) +] +dispinterface DispHTMLHeadElement +{ +properties: +methods: + WINE_HTMLELEMENT_DISPINTERFACE_DECL; + + [propput, id(DISPID_IHTMLHEADELEMENT_PROFILE), displaybind, bindable] + void profile(BSTR v); + + [propget, id(DISPID_IHTMLHEADELEMENT_PROFILE), displaybind, bindable] + BSTR profile(); + + [propput, id(DISPID_IHTMLHEADELEMENT2_IE8_PROFILE), displaybind, bindable] + void ie8_profile(BSTR v); + + [propget, id(DISPID_IHTMLHEADELEMENT2_IE8_PROFILE), displaybind, bindable] + BSTR ie8_profile(); +} + +/***************************************************************************** * DispHTMLUnknownElement interface */ [

1 0

Jacek Caban : mshtml: Added IHTMLHeadElement stub implementation.
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: 74e95a13b302a2c63d73609a546078a0d7a020f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=74e95a13b302a2c63d73609a5… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Jun 21 13:47:09 2011 +0200 mshtml: Added IHTMLHeadElement stub implementation. --- dlls/mshtml/Makefile.in | 1 + dlls/mshtml/htmlelem.c | 2 + dlls/mshtml/htmlhead.c | 177 ++++++++++++++++++++++++++++++++++++++++++ dlls/mshtml/mshtml_private.h | 1 + dlls/mshtml/tests/dom.c | 9 ++- 5 files changed, 189 insertions(+), 1 deletions(-) diff --git a/dlls/mshtml/Makefile.in b/dlls/mshtml/Makefile.in index ea716fb..1732951 100644 --- a/dlls/mshtml/Makefile.in +++ b/dlls/mshtml/Makefile.in @@ -27,6 +27,7 @@ C_SRCS = \ htmlframe.c \ htmlframebase.c \ htmlgeneric.c \ + htmlhead.c \ htmliframe.c \ htmlimg.c \ htmlinput.c \ diff --git a/dlls/mshtml/htmlelem.c b/dlls/mshtml/htmlelem.c index 603ae9b..24d7b08 100644 --- a/dlls/mshtml/htmlelem.c +++ b/dlls/mshtml/htmlelem.c @@ -42,6 +42,7 @@ static const WCHAR bodyW[] = {'B','O','D','Y',0}; static const WCHAR embedW[] = {'E','M','B','E','D',0}; static const WCHAR formW[] = {'F','O','R','M',0}; static const WCHAR frameW[] = {'F','R','A','M','E',0}; +static const WCHAR headW[] = {'H','E','A','D',0}; static const WCHAR iframeW[] = {'I','F','R','A','M','E',0}; static const WCHAR imgW[] = {'I','M','G',0}; static const WCHAR inputW[] = {'I','N','P','U','T',0}; @@ -65,6 +66,7 @@ static const tag_desc_t tag_descs[] = { {embedW, HTMLEmbedElement_Create}, {formW, HTMLFormElement_Create}, {frameW, HTMLFrameElement_Create}, + {headW, HTMLHeadElement_Create}, {iframeW, HTMLIFrame_Create}, {imgW, HTMLImgElement_Create}, {inputW, HTMLInputElement_Create}, diff --git a/dlls/mshtml/htmlhead.c b/dlls/mshtml/htmlhead.c new file mode 100644 index 0000000..aee2157 --- /dev/null +++ b/dlls/mshtml/htmlhead.c @@ -0,0 +1,177 @@ + /* + * Copyright 2011 Jacek Caban for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <stdarg.h> + +#define COBJMACROS + +#include "windef.h" +#include "winbase.h" +#include "winuser.h" +#include "ole2.h" + +#include "wine/debug.h" + +#include "mshtml_private.h" + +WINE_DEFAULT_DEBUG_CHANNEL(mshtml); + +typedef struct { + HTMLElement element; + + IHTMLHeadElement IHTMLHeadElement_iface; +} HTMLHeadElement; + +static inline HTMLHeadElement *impl_from_IHTMLHeadElement(IHTMLHeadElement *iface) +{ + return CONTAINING_RECORD(iface, HTMLHeadElement, IHTMLHeadElement_iface); +} + +static HRESULT WINAPI HTMLHeadElement_QueryInterface(IHTMLHeadElement *iface, + REFIID riid, void **ppv) +{ + HTMLHeadElement *This = impl_from_IHTMLHeadElement(iface); + + return IHTMLDOMNode_QueryInterface(&This->element.node.IHTMLDOMNode_iface, riid, ppv); +} + +static ULONG WINAPI HTMLHeadElement_AddRef(IHTMLHeadElement *iface) +{ + HTMLHeadElement *This = impl_from_IHTMLHeadElement(iface); + + return IHTMLDOMNode_AddRef(&This->element.node.IHTMLDOMNode_iface); +} + +static ULONG WINAPI HTMLHeadElement_Release(IHTMLHeadElement *iface) +{ + HTMLHeadElement *This = impl_from_IHTMLHeadElement(iface); + + return IHTMLDOMNode_Release(&This->element.node.IHTMLDOMNode_iface); +} + +static HRESULT WINAPI HTMLHeadElement_GetTypeInfoCount(IHTMLHeadElement *iface, UINT *pctinfo) +{ + HTMLHeadElement *This = impl_from_IHTMLHeadElement(iface); + + return IDispatchEx_GetTypeInfoCount(&This->element.node.dispex.IDispatchEx_iface, pctinfo); +} + +static HRESULT WINAPI HTMLHeadElement_GetTypeInfo(IHTMLHeadElement *iface, UINT iTInfo, + LCID lcid, ITypeInfo **ppTInfo) +{ + HTMLHeadElement *This = impl_from_IHTMLHeadElement(iface); + + return IDispatchEx_GetTypeInfo(&This->element.node.dispex.IDispatchEx_iface, iTInfo, lcid, + ppTInfo); +} + +static HRESULT WINAPI HTMLHeadElement_GetIDsOfNames(IHTMLHeadElement *iface, REFIID riid, + LPOLESTR *rgszNames, UINT cNames, + LCID lcid, DISPID *rgDispId) +{ + HTMLHeadElement *This = impl_from_IHTMLHeadElement(iface); + + return IDispatchEx_GetIDsOfNames(&This->element.node.dispex.IDispatchEx_iface, riid, rgszNames, + cNames, lcid, rgDispId); +} + +static HRESULT WINAPI HTMLHeadElement_Invoke(IHTMLHeadElement *iface, DISPID dispIdMember, + REFIID riid, LCID lcid, WORD wFlags, DISPPARAMS *pDispParams, + VARIANT *pVarResult, EXCEPINFO *pExcepInfo, UINT *puArgErr) +{ + HTMLHeadElement *This = impl_from_IHTMLHeadElement(iface); + + return IDispatchEx_Invoke(&This->element.node.dispex.IDispatchEx_iface, dispIdMember, riid, + lcid, wFlags, pDispParams, pVarResult, pExcepInfo, puArgErr); +} + +static HRESULT WINAPI HTMLHeadElement_put_profile(IHTMLHeadElement *iface, BSTR v) +{ + HTMLHeadElement *This = impl_from_IHTMLHeadElement(iface); + FIXME("(%p)->(%s)\n", This, debugstr_w(v)); + return E_NOTIMPL; +} + +static HRESULT WINAPI HTMLHeadElement_get_profile(IHTMLHeadElement *iface, BSTR *p) +{ + HTMLHeadElement *This = impl_from_IHTMLHeadElement(iface); + FIXME("(%p)->(%p)\n", This, p); + return E_NOTIMPL; +} + +static const IHTMLHeadElementVtbl HTMLHeadElementVtbl = { + HTMLHeadElement_QueryInterface, + HTMLHeadElement_AddRef, + HTMLHeadElement_Release, + HTMLHeadElement_GetTypeInfoCount, + HTMLHeadElement_GetTypeInfo, + HTMLHeadElement_GetIDsOfNames, + HTMLHeadElement_Invoke, + HTMLHeadElement_put_profile, + HTMLHeadElement_get_profile +}; + +static inline HTMLHeadElement *impl_from_HTMLDOMNode(HTMLDOMNode *iface) +{ + return CONTAINING_RECORD(iface, HTMLHeadElement, element.node); +} + +static HRESULT HTMLHeadElement_QI(HTMLDOMNode *iface, REFIID riid, void **ppv) +{ + HTMLHeadElement *This = impl_from_HTMLDOMNode(iface); + + if(IsEqualGUID(&IID_IHTMLHeadElement, riid)) { + TRACE("(%p)->(IID_IHTMLHeadElement %p)\n", This, ppv); + *ppv = &This->IHTMLHeadElement_iface; + }else { + HTMLElement_QI(&This->element.node, riid, ppv); + } + + IUnknown_AddRef((IUnknown*)*ppv); + return S_OK; +} + +static void HTMLHeadElement_destructor(HTMLDOMNode *iface) +{ + HTMLHeadElement *This = impl_from_HTMLDOMNode(iface); + + HTMLElement_destructor(&This->element.node); +} + +static const NodeImplVtbl HTMLHeadElementImplVtbl = { + HTMLHeadElement_QI, + HTMLHeadElement_destructor, + HTMLElement_clone +}; + +HRESULT HTMLHeadElement_Create(HTMLDocumentNode *doc, nsIDOMHTMLElement *nselem, HTMLElement **elem) +{ + HTMLHeadElement *ret; + + ret = heap_alloc_zero(sizeof(*ret)); + if(!ret) + return E_OUTOFMEMORY; + + ret->IHTMLHeadElement_iface.lpVtbl = &HTMLHeadElementVtbl; + ret->element.node.vtbl = &HTMLHeadElementImplVtbl; + + HTMLElement_Init(&ret->element, doc, nselem, NULL); + + *elem = &ret->element; + return S_OK; +} diff --git a/dlls/mshtml/mshtml_private.h b/dlls/mshtml/mshtml_private.h index cc17a64..ea62a46 100644 --- a/dlls/mshtml/mshtml_private.h +++ b/dlls/mshtml/mshtml_private.h @@ -779,6 +779,7 @@ HRESULT HTMLBodyElement_Create(HTMLDocumentNode*,nsIDOMHTMLElement*,HTMLElement* HRESULT HTMLEmbedElement_Create(HTMLDocumentNode*,nsIDOMHTMLElement*,HTMLElement**) DECLSPEC_HIDDEN; HRESULT HTMLFormElement_Create(HTMLDocumentNode*,nsIDOMHTMLElement*,HTMLElement**) DECLSPEC_HIDDEN; HRESULT HTMLFrameElement_Create(HTMLDocumentNode*,nsIDOMHTMLElement*,HTMLElement**) DECLSPEC_HIDDEN; +HRESULT HTMLHeadElement_Create(HTMLDocumentNode*,nsIDOMHTMLElement*,HTMLElement**) DECLSPEC_HIDDEN; HRESULT HTMLIFrame_Create(HTMLDocumentNode*,nsIDOMHTMLElement*,HTMLElement**) DECLSPEC_HIDDEN; HRESULT HTMLStyleElement_Create(HTMLDocumentNode*,nsIDOMHTMLElement*,HTMLElement**) DECLSPEC_HIDDEN; HRESULT HTMLImgElement_Create(HTMLDocumentNode*,nsIDOMHTMLElement*,HTMLElement**) DECLSPEC_HIDDEN; diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index 414da44..9672943 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -281,6 +281,13 @@ static const IID * const frame_iids[] = { NULL }; +static const IID * const head_iids[] = { + ELEM_IFACES, + &IID_IHTMLHeadElement, + &IID_IConnectionPointContainer, + NULL +}; + static const IID * const object_iids[] = { ELEM_IFACES, &IID_IHTMLObjectElement, @@ -361,7 +368,7 @@ typedef struct { static const elem_type_info_t elem_type_infos[] = { {"", none_iids, NULL}, {"HTML", elem_iids, NULL}, - {"HEAD", elem_iids, NULL}, + {"HEAD", head_iids, NULL}, {"TITLE", elem_iids, NULL}, {"BODY", body_iids, &DIID_DispHTMLBody}, {"A", anchor_iids, &DIID_DispHTMLAnchorElement},

1 0

Piotr Pawlow : wined3d: Use a separate mutex for wndproc_table access.
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: b5e04640cc358dda13f70961c31295b522611577 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b5e04640cc358dda13f70961c… Author: Piotr Pawlow <pp(a)siedziba.pl> Date: Fri Jun 17 21:49:38 2011 +0200 wined3d: Use a separate mutex for wndproc_table access. If CreateDevice is called with hFocusWindow belonging to another thread, then any code that does SendMessage (or equivalent) to the window while holding the main wined3d mutex causes a deadlock, because wined3d_wndproc tries to acquire the same mutex for wndproc_table access. --- dlls/wined3d/wined3d_main.c | 44 +++++++++++++++++++++++++++++++----------- 1 files changed, 32 insertions(+), 12 deletions(-) diff --git a/dlls/wined3d/wined3d_main.c b/dlls/wined3d/wined3d_main.c index 4f7b1cb..ad3643d 100644 --- a/dlls/wined3d/wined3d_main.c +++ b/dlls/wined3d/wined3d_main.c @@ -60,6 +60,16 @@ static CRITICAL_SECTION_DEBUG wined3d_cs_debug = }; static CRITICAL_SECTION wined3d_cs = {&wined3d_cs_debug, -1, 0, 0, 0, 0}; +static CRITICAL_SECTION wined3d_wndproc_cs; +static CRITICAL_SECTION_DEBUG wined3d_wndproc_cs_debug = +{ + 0, 0, &wined3d_wndproc_cs, + {&wined3d_wndproc_cs_debug.ProcessLocksList, + &wined3d_wndproc_cs_debug.ProcessLocksList}, + 0, 0, {(DWORD_PTR)(__FILE__ ": wined3d_wndproc_cs")} +}; +static CRITICAL_SECTION wined3d_wndproc_cs = {&wined3d_wndproc_cs_debug, -1, 0, 0, 0, 0}; + /* When updating default value here, make sure to update winecfg as well, * where appropriate. */ struct wined3d_settings wined3d_settings = @@ -381,6 +391,16 @@ void WINAPI wined3d_mutex_unlock(void) LeaveCriticalSection(&wined3d_cs); } +static void wined3d_wndproc_mutex_lock(void) +{ + EnterCriticalSection(&wined3d_wndproc_cs); +} + +static void wined3d_wndproc_mutex_unlock(void) +{ + LeaveCriticalSection(&wined3d_wndproc_cs); +} + static struct wined3d_wndproc *wined3d_find_wndproc(HWND window) { unsigned int i; @@ -403,12 +423,12 @@ static LRESULT CALLBACK wined3d_wndproc(HWND window, UINT message, WPARAM wparam BOOL unicode; WNDPROC proc; - wined3d_mutex_lock(); + wined3d_wndproc_mutex_lock(); entry = wined3d_find_wndproc(window); if (!entry) { - wined3d_mutex_unlock(); + wined3d_wndproc_mutex_unlock(); ERR("Window %p is not registered with wined3d.\n", window); return DefWindowProcW(window, message, wparam, lparam); } @@ -416,7 +436,7 @@ static LRESULT CALLBACK wined3d_wndproc(HWND window, UINT message, WPARAM wparam device = entry->device; unicode = entry->unicode; proc = entry->proc; - wined3d_mutex_unlock(); + wined3d_wndproc_mutex_unlock(); return device_process_message(device, window, unicode, message, wparam, lparam, proc); } @@ -425,11 +445,11 @@ BOOL wined3d_register_window(HWND window, struct wined3d_device *device) { struct wined3d_wndproc *entry; - wined3d_mutex_lock(); + wined3d_wndproc_mutex_lock(); if (wined3d_find_wndproc(window)) { - wined3d_mutex_unlock(); + wined3d_wndproc_mutex_unlock(); WARN("Window %p is already registered with wined3d.\n", window); return TRUE; } @@ -444,7 +464,7 @@ BOOL wined3d_register_window(HWND window, struct wined3d_device *device) if (!new_entries) { - wined3d_mutex_unlock(); + wined3d_wndproc_mutex_unlock(); ERR("Failed to grow table.\n"); return FALSE; } @@ -465,7 +485,7 @@ BOOL wined3d_register_window(HWND window, struct wined3d_device *device) entry->proc = (WNDPROC)SetWindowLongPtrA(window, GWLP_WNDPROC, (LONG_PTR)wined3d_wndproc); entry->device = device; - wined3d_mutex_unlock(); + wined3d_wndproc_mutex_unlock(); return TRUE; } @@ -475,11 +495,11 @@ void wined3d_unregister_window(HWND window) struct wined3d_wndproc *entry, *last; LONG_PTR proc; - wined3d_mutex_lock(); + wined3d_wndproc_mutex_lock(); if (!(entry = wined3d_find_wndproc(window))) { - wined3d_mutex_unlock(); + wined3d_wndproc_mutex_unlock(); ERR("Window %p is not registered with wined3d.\n", window); return; } @@ -489,7 +509,7 @@ void wined3d_unregister_window(HWND window) proc = GetWindowLongPtrW(window, GWLP_WNDPROC); if (proc != (LONG_PTR)wined3d_wndproc) { - wined3d_mutex_unlock(); + wined3d_wndproc_mutex_unlock(); WARN("Not unregistering window %p, window proc %#lx doesn't match wined3d window proc %p.\n", window, proc, wined3d_wndproc); return; @@ -502,7 +522,7 @@ void wined3d_unregister_window(HWND window) proc = GetWindowLongPtrA(window, GWLP_WNDPROC); if (proc != (LONG_PTR)wined3d_wndproc) { - wined3d_mutex_unlock(); + wined3d_wndproc_mutex_unlock(); WARN("Not unregistering window %p, window proc %#lx doesn't match wined3d window proc %p.\n", window, proc, wined3d_wndproc); return; @@ -514,7 +534,7 @@ void wined3d_unregister_window(HWND window) last = &wndproc_table.entries[--wndproc_table.count]; if (entry != last) *entry = *last; - wined3d_mutex_unlock(); + wined3d_wndproc_mutex_unlock(); } /* At process attach */

1 0

Ričardas Barkauskas : ddraw: Remove ddraw_set_surface_version and set surface version when initialising it .
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: 8959a1395acffb02e98da598352fb0d536ef195c URL: http://source.winehq.org/git/wine.git/?a=commit;h=8959a1395acffb02e98da5983… Author: Ričardas Barkauskas <rbarkauskas(a)codeweavers.com> Date: Mon Jun 20 09:30:39 2011 +0300 ddraw: Remove ddraw_set_surface_version and set surface version when initialising it. --- dlls/ddraw/ddraw.c | 70 ++++++++++++++++++------------------------- dlls/ddraw/ddraw_private.h | 2 +- dlls/ddraw/surface.c | 4 +- 3 files changed, 32 insertions(+), 44 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=8959a1395acffb02e98da…

1 0

Vincent Povirk : server: Post WM_HOTKEY when a hotkey is pressed.
by Alexandre Julliard 21 Jun '11

21 Jun '11

Module: wine Branch: master Commit: cc0ea6986772f90b83358080178da9b3bb974378 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cc0ea6986772f90b833580801… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Thu May 26 15:52:17 2011 -0500 server: Post WM_HOTKEY when a hotkey is pressed. --- dlls/user32/tests/msg.c | 8 ++++---- server/queue.c | 37 +++++++++++++++++++++++++++++++++++++ 2 files changed, 41 insertions(+), 4 deletions(-) diff --git a/dlls/user32/tests/msg.c b/dlls/user32/tests/msg.c index 7585c2c..ee1637d 100644 --- a/dlls/user32/tests/msg.c +++ b/dlls/user32/tests/msg.c @@ -13247,7 +13247,7 @@ static void test_hotkey(void) } DispatchMessage(&msg); } - ok_sequence(WmHotkeyPress, "window hotkey press", TRUE); + ok_sequence(WmHotkeyPress, "window hotkey press", FALSE); key_state = GetAsyncKeyState(hotkey_letter); ok((key_state & 0x8000) == 0x8000, "unexpected key state %x\n", key_state); @@ -13278,7 +13278,7 @@ static void test_hotkey(void) } DispatchMessage(&msg); } - ok_sequence(WmHotkeyCombined, "window hotkey combined", TRUE); + ok_sequence(WmHotkeyCombined, "window hotkey combined", FALSE); /* Register same hwnd/id with different key combination */ ret = RegisterHotKey(test_window, 5, 0, hotkey_letter); @@ -13307,7 +13307,7 @@ static void test_hotkey(void) } DispatchMessage(&msg); } - ok_sequence(WmHotkeyNew, "window hotkey new", TRUE); + ok_sequence(WmHotkeyNew, "window hotkey new", FALSE); /* Unregister hotkey properly */ ret = UnregisterHotKey(test_window, 5); @@ -13351,7 +13351,7 @@ static void test_hotkey(void) ok(msg.hwnd != NULL, "unexpected thread message %x\n", msg.message); DispatchMessage(&msg); } - ok_sequence(WmHotkeyPress, "thread hotkey press", TRUE); + ok_sequence(WmHotkeyPress, "thread hotkey press", FALSE); keybd_event(hotkey_letter, 0, KEYEVENTF_KEYUP, 0); while (PeekMessage(&msg, NULL, 0, 0, PM_REMOVE)) diff --git a/server/queue.c b/server/queue.c index 2fceacd..ce773a1 100644 --- a/server/queue.c +++ b/server/queue.c @@ -1316,6 +1316,42 @@ static void release_hardware_message( struct msg_queue *queue, unsigned int hw_i } } +static int queue_hotkey_message( struct desktop *desktop, struct message *msg ) +{ + struct hotkey *hotkey; + unsigned int modifiers = 0; + + if (msg->msg != WM_KEYDOWN) return 0; + + if (desktop->keystate[VK_MENU] & 0x80) modifiers |= MOD_ALT; + if (desktop->keystate[VK_CONTROL] & 0x80) modifiers |= MOD_CONTROL; + if (desktop->keystate[VK_SHIFT] & 0x80) modifiers |= MOD_SHIFT; + if ((desktop->keystate[VK_LWIN] & 0x80) || (desktop->keystate[VK_RWIN] & 0x80)) modifiers |= MOD_WIN; + + LIST_FOR_EACH_ENTRY( hotkey, &desktop->hotkeys, struct hotkey, entry ) + { + if (hotkey->vkey != msg->wparam) continue; + if ((hotkey->flags & (MOD_ALT|MOD_CONTROL|MOD_SHIFT|MOD_WIN)) == modifiers) goto found; + } + + return 0; + +found: + msg->type = MSG_POSTED; + msg->win = hotkey->win; + msg->msg = WM_HOTKEY; + msg->wparam = hotkey->id; + msg->lparam = ((hotkey->vkey & 0xffff) << 16) | modifiers; + + free( msg->data ); + msg->data = NULL; + msg->data_size = 0; + + list_add_tail( &hotkey->queue->msg_list[POST_MESSAGE], &msg->entry ); + set_queue_bits( hotkey->queue, QS_POSTMESSAGE|QS_ALLPOSTMESSAGE ); + return 1; +} + /* find the window that should receive a given hardware message */ static user_handle_t find_hardware_message_window( struct desktop *desktop, struct thread_input *input, struct message *msg, unsigned int *msg_code ) @@ -1358,6 +1394,7 @@ static void queue_hardware_message( struct desktop *desktop, struct message *msg if (is_keyboard_msg( msg )) { + if (queue_hotkey_message( desktop, msg )) return; if (desktop->keystate[VK_MENU] & 0x80) msg->lparam |= KF_ALTDOWN << 16; if (msg->wparam == VK_SHIFT || msg->wparam == VK_LSHIFT || msg->wparam == VK_RSHIFT) msg->lparam &= ~(KF_EXTENDED << 16);

1 0