August 2010 - Wine-commits - list.winehq.org

Henri Verbeet : ddraw: Merge direct3d.c into ddraw.c.
by Alexandre Julliard 18 Aug '10

18 Aug '10

Module: wine Branch: master Commit: 8394f0066144cbc7fe5634698fe6dafecf92aa27 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8394f0066144cbc7fe5634698… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Aug 17 19:03:22 2010 +0200 ddraw: Merge direct3d.c into ddraw.c. --- dlls/ddraw/Makefile.in | 1 - dlls/ddraw/ddraw.c | 1389 +++++++++++++++++++++++++++++++++++++++++ dlls/ddraw/direct3d.c | 1608 ------------------------------------------------ 3 files changed, 1389 insertions(+), 1609 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=8394f0066144cbc7fe563…

1 0

Henri Verbeet : ddraw: Remove useless light callbacks.
by Alexandre Julliard 18 Aug '10

18 Aug '10

Module: wine Branch: master Commit: 06a44abc19b748b3cb4111e4696d6139247105e2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=06a44abc19b748b3cb4111e46… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Aug 17 19:03:23 2010 +0200 ddraw: Remove useless light callbacks. --- dlls/ddraw/ddraw.c | 5 -- dlls/ddraw/ddraw_private.h | 10 +--- dlls/ddraw/light.c | 137 +++++++++++++++++++++---------------------- dlls/ddraw/viewport.c | 15 +++-- 4 files changed, 77 insertions(+), 90 deletions(-) diff --git a/dlls/ddraw/ddraw.c b/dlls/ddraw/ddraw.c index 4921519..ff44960 100644 --- a/dlls/ddraw/ddraw.c +++ b/dlls/ddraw/ddraw.c @@ -4397,11 +4397,6 @@ static HRESULT WINAPI d3d3_CreateLight(IDirect3D3 *iface, IDirect3DLight **light object->ref = 1; object->ddraw = ddraw_from_d3d3(iface); - /* Update functions */ - object->activate = light_update; - object->desactivate = light_activate; - object->update = light_desactivate; - TRACE("Created light %p.\n", object); *light = (IDirect3DLight *)object; diff --git a/dlls/ddraw/ddraw_private.h b/dlls/ddraw/ddraw_private.h index 9ff703d..220d1bb 100644 --- a/dlls/ddraw/ddraw_private.h +++ b/dlls/ddraw/ddraw_private.h @@ -525,20 +525,14 @@ struct IDirect3DLightImpl /* Chained list used for adding / removing from viewports */ IDirect3DLightImpl *next; - - /* Activation function */ - void (*activate)(IDirect3DLightImpl*); - void (*desactivate)(IDirect3DLightImpl*); - void (*update)(IDirect3DLightImpl*); }; /* Vtable */ extern const IDirect3DLightVtbl IDirect3DLight_Vtbl DECLSPEC_HIDDEN; /* Helper functions */ -void light_update(IDirect3DLightImpl *This) DECLSPEC_HIDDEN; -void light_activate(IDirect3DLightImpl *This) DECLSPEC_HIDDEN; -void light_desactivate(IDirect3DLightImpl *This) DECLSPEC_HIDDEN; +void light_activate(IDirect3DLightImpl *light) DECLSPEC_HIDDEN; +void light_deactivate(IDirect3DLightImpl *light) DECLSPEC_HIDDEN; /****************************************************************************** * IDirect3DMaterial implementation structure - Wraps to D3D7 diff --git a/dlls/ddraw/light.c b/dlls/ddraw/light.c index 030dcf0..dd41ffe 100644 --- a/dlls/ddraw/light.c +++ b/dlls/ddraw/light.c @@ -44,6 +44,71 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3d7); /***************************************************************************** + * light_update + * + * Updates the Direct3DDevice7 lighting parameters + * + *****************************************************************************/ +static void light_update(IDirect3DLightImpl *light) +{ + IDirect3DDeviceImpl *device; + + TRACE("light %p.\n", light); + + if (!light->active_viewport || !light->active_viewport->active_device) return; + device = light->active_viewport->active_device; + + IDirect3DDevice7_SetLight((IDirect3DDevice7 *)device, light->dwLightIndex, &light->light7); +} + +/***************************************************************************** + * light_activate + * + * Uses the Direct3DDevice7::LightEnable method to active the light + * + *****************************************************************************/ +void light_activate(IDirect3DLightImpl *light) +{ + IDirect3DDeviceImpl *device; + + TRACE("light %p.\n", light); + + if (!light->active_viewport || !light->active_viewport->active_device) return; + device = light->active_viewport->active_device; + + light_update(light); + if (!(light->light.dwFlags & D3DLIGHT_ACTIVE)) + { + IDirect3DDevice7_LightEnable((IDirect3DDevice7 *)device, light->dwLightIndex, TRUE); + light->light.dwFlags |= D3DLIGHT_ACTIVE; + } +} + +/***************************************************************************** + * + * light_deactivate + * + * Uses the Direct3DDevice7::LightEnable method to deactivate the light + * + *****************************************************************************/ +void light_deactivate(IDirect3DLightImpl *light) +{ + IDirect3DDeviceImpl *device; + + TRACE("light %p.\n", light); + + if (!light->active_viewport || !light->active_viewport->active_device) return; + device = light->active_viewport->active_device; + + /* If was not active, activate it */ + if (light->light.dwFlags & D3DLIGHT_ACTIVE) + { + IDirect3DDevice7_LightEnable((IDirect3DDevice7 *)device, light->dwLightIndex, FALSE); + light->light.dwFlags &= ~D3DLIGHT_ACTIVE; + } +} + +/***************************************************************************** * IUnknown Methods. *****************************************************************************/ @@ -203,9 +268,8 @@ IDirect3DLightImpl_SetLight(IDirect3DLight *iface, EnterCriticalSection(&ddraw_cs); memcpy(&This->light, lpLight, lpLight->dwSize); - if ((This->light.dwFlags & D3DLIGHT_ACTIVE) != 0) { - This->update(This); - } + if (This->light.dwFlags & D3DLIGHT_ACTIVE) + light_update(This); LeaveCriticalSection(&ddraw_cs); return D3D_OK; } @@ -240,73 +304,6 @@ IDirect3DLightImpl_GetLight(IDirect3DLight *iface, return DD_OK; } -/***************************************************************************** - * light_update - * - * Updates the Direct3DDevice7 lighting parameters - * - *****************************************************************************/ -void light_update(IDirect3DLightImpl* This) -{ - IDirect3DDeviceImpl* device; - - TRACE("(%p)\n", This); - - if (!This->active_viewport || !This->active_viewport->active_device) - return; - device = This->active_viewport->active_device; - - IDirect3DDevice7_SetLight((IDirect3DDevice7 *)device, This->dwLightIndex, &(This->light7)); -} - -/***************************************************************************** - * light_activate - * - * Uses the Direct3DDevice7::LightEnable method to active the light - * - *****************************************************************************/ -void light_activate(IDirect3DLightImpl* This) -{ - IDirect3DDeviceImpl* device; - - TRACE("(%p)\n", This); - - if (!This->active_viewport || !This->active_viewport->active_device) - return; - device = This->active_viewport->active_device; - - light_update(This); - /* If was not active, activate it */ - if ((This->light.dwFlags & D3DLIGHT_ACTIVE) == 0) { - IDirect3DDevice7_LightEnable((IDirect3DDevice7 *)device, This->dwLightIndex, TRUE); - This->light.dwFlags |= D3DLIGHT_ACTIVE; - } -} - -/***************************************************************************** - * - * light_desactivate - * - * Uses the Direct3DDevice7::LightEnable method to deactivate the light - * - *****************************************************************************/ -void light_desactivate(IDirect3DLightImpl* This) -{ - IDirect3DDeviceImpl* device; - - TRACE("(%p)\n", This); - - if (!This->active_viewport || !This->active_viewport->active_device) - return; - device = This->active_viewport->active_device; - - /* If was not active, activate it */ - if ((This->light.dwFlags & D3DLIGHT_ACTIVE) != 0) { - IDirect3DDevice7_LightEnable((IDirect3DDevice7 *)device, This->dwLightIndex, FALSE); - This->light.dwFlags &= ~D3DLIGHT_ACTIVE; - } -} - const IDirect3DLightVtbl IDirect3DLight_Vtbl = { /*** IUnknown Methods ***/ diff --git a/dlls/ddraw/viewport.c b/dlls/ddraw/viewport.c index 5e17bd0..01319f5 100644 --- a/dlls/ddraw/viewport.c +++ b/dlls/ddraw/viewport.c @@ -62,8 +62,9 @@ void viewport_activate(IDirect3DViewportImpl* This, BOOL ignore_lights) { /* Activate all the lights associated with this context */ light = This->lights; - while (light != NULL) { - light->activate(light); + while (light) + { + light_activate(light); light = light->next; } } @@ -762,9 +763,8 @@ IDirect3DViewportImpl_AddLight(IDirect3DViewport3 *iface, lpDirect3DLightImpl->active_viewport = This; /* If active, activate the light */ - if (This->active_device != NULL) { - lpDirect3DLightImpl->activate(lpDirect3DLightImpl); - } + if (This->active_device) + light_activate(lpDirect3DLightImpl); LeaveCriticalSection(&ddraw_cs); return D3D_OK; @@ -796,8 +796,9 @@ IDirect3DViewportImpl_DeleteLight(IDirect3DViewport3 *iface, EnterCriticalSection(&ddraw_cs); cur_light = This->lights; while (cur_light != NULL) { - if (cur_light == lpDirect3DLightImpl) { - lpDirect3DLightImpl->desactivate(lpDirect3DLightImpl); + if (cur_light == lpDirect3DLightImpl) + { + light_deactivate(lpDirect3DLightImpl); if (prev_light == NULL) This->lights = cur_light->next; else prev_light->next = cur_light->next; /* Detach the light to the viewport */

1 0

Henri Verbeet : ddraw: Remove a useless material callback.
by Alexandre Julliard 18 Aug '10

18 Aug '10

Module: wine Branch: master Commit: 14399f6130a5594d21312f950de523f3d35f7868 URL: http://source.winehq.org/git/wine.git/?a=commit;h=14399f6130a5594d21312f950… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Aug 17 19:03:24 2010 +0200 ddraw: Remove a useless material callback. --- dlls/ddraw/ddraw.c | 1 - dlls/ddraw/ddraw_private.h | 2 -- dlls/ddraw/device.c | 2 +- dlls/ddraw/executebuffer.c | 2 +- 4 files changed, 2 insertions(+), 5 deletions(-) diff --git a/dlls/ddraw/ddraw.c b/dlls/ddraw/ddraw.c index ff44960..670129d 100644 --- a/dlls/ddraw/ddraw.c +++ b/dlls/ddraw/ddraw.c @@ -4457,7 +4457,6 @@ static HRESULT WINAPI d3d3_CreateMaterial(IDirect3D3 *iface, IDirect3DMaterial3 object->IDirect3DMaterial_vtbl = &IDirect3DMaterial_Vtbl; object->ref = 1; object->ddraw = ddraw_from_d3d3(iface); - object->activate = material_activate; TRACE("Created material %p.\n", object); *material = (IDirect3DMaterial3 *)object; diff --git a/dlls/ddraw/ddraw_private.h b/dlls/ddraw/ddraw_private.h index 220d1bb..bb95767 100644 --- a/dlls/ddraw/ddraw_private.h +++ b/dlls/ddraw/ddraw_private.h @@ -550,8 +550,6 @@ struct IDirect3DMaterialImpl D3DMATERIAL mat; DWORD Handle; - - void (*activate)(IDirect3DMaterialImpl* this); }; /* VTables in various versions */ diff --git a/dlls/ddraw/device.c b/dlls/ddraw/device.c index 6cd0f80..a17627b 100644 --- a/dlls/ddraw/device.c +++ b/dlls/ddraw/device.c @@ -3044,7 +3044,7 @@ IDirect3DDeviceImpl_3_SetLightState(IDirect3DDevice3 *iface, } TRACE(" activating material %p.\n", m); - m->activate(m); + material_activate(m); This->material = Value; } diff --git a/dlls/ddraw/executebuffer.c b/dlls/ddraw/executebuffer.c index 59481b1..9ae24a1 100644 --- a/dlls/ddraw/executebuffer.c +++ b/dlls/ddraw/executebuffer.c @@ -261,7 +261,7 @@ IDirect3DExecuteBufferImpl_Execute(IDirect3DExecuteBufferImpl *This, if (!m) ERR("Invalid material handle %#x.\n", ci->u2.dwArg[0]); else - m->activate(m); + material_activate(m); } else if (ci->u1.dlstLightStateType == D3DLIGHTSTATE_COLORMODEL /* 3 */) {

1 0

Henri Verbeet : ddraw: Remove a useless viewport callback.
by Alexandre Julliard 18 Aug '10

18 Aug '10

Module: wine Branch: master Commit: fee9c1f2b5d00f6e75917f91536dfaf1e44243e4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fee9c1f2b5d00f6e75917f915… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Aug 17 19:03:25 2010 +0200 ddraw: Remove a useless viewport callback. --- dlls/ddraw/ddraw.c | 1 - dlls/ddraw/ddraw_private.h | 3 --- dlls/ddraw/device.c | 2 +- dlls/ddraw/executebuffer.c | 2 +- dlls/ddraw/viewport.c | 22 ++++++++++++---------- 5 files changed, 14 insertions(+), 16 deletions(-) diff --git a/dlls/ddraw/ddraw.c b/dlls/ddraw/ddraw.c index 670129d..6f060e7 100644 --- a/dlls/ddraw/ddraw.c +++ b/dlls/ddraw/ddraw.c @@ -4533,7 +4533,6 @@ static HRESULT WINAPI d3d3_CreateViewport(IDirect3D3 *iface, IDirect3DViewport3 object->ref = 1; object->ddraw = ddraw_from_d3d3(iface); object->use_vp2 = 0xff; - object->activate = viewport_activate; TRACE("Created viewport %p.\n", object); *viewport = (IDirect3DViewport3 *)object; diff --git a/dlls/ddraw/ddraw_private.h b/dlls/ddraw/ddraw_private.h index bb95767..990d4f7 100644 --- a/dlls/ddraw/ddraw_private.h +++ b/dlls/ddraw/ddraw_private.h @@ -585,9 +585,6 @@ struct IDirect3DViewportImpl D3DVIEWPORT2 vp2; } viewports; - /* Activation function */ - void (*activate)(IDirect3DViewportImpl*, BOOL); - /* Field used to chain viewports together */ IDirect3DViewportImpl *next; diff --git a/dlls/ddraw/device.c b/dlls/ddraw/device.c index a17627b..25ff428 100644 --- a/dlls/ddraw/device.c +++ b/dlls/ddraw/device.c @@ -1872,7 +1872,7 @@ IDirect3DDeviceImpl_3_SetCurrentViewport(IDirect3DDevice3 *iface, /* Activate this viewport */ This->current_viewport->active_device = This; - This->current_viewport->activate(This->current_viewport, FALSE); + viewport_activate(This->current_viewport, FALSE); LeaveCriticalSection(&ddraw_cs); return D3D_OK; diff --git a/dlls/ddraw/executebuffer.c b/dlls/ddraw/executebuffer.c index 9ae24a1..04f856a 100644 --- a/dlls/ddraw/executebuffer.c +++ b/dlls/ddraw/executebuffer.c @@ -103,7 +103,7 @@ IDirect3DExecuteBufferImpl_Execute(IDirect3DExecuteBufferImpl *This, /* Activate the viewport */ lpViewport->active_device = lpDevice; - lpViewport->activate(lpViewport, FALSE); + viewport_activate(lpViewport, FALSE); TRACE("ExecuteData :\n"); if (TRACE_ON(d3d7)) diff --git a/dlls/ddraw/viewport.c b/dlls/ddraw/viewport.c index 01319f5..403463b 100644 --- a/dlls/ddraw/viewport.c +++ b/dlls/ddraw/viewport.c @@ -325,8 +325,9 @@ IDirect3DViewportImpl_SetViewport(IDirect3DViewport3 *iface, if (This->active_device) { IDirect3DDevice3 *d3d_device3 = (IDirect3DDevice3 *)&This->active_device->IDirect3DDevice3_vtbl; IDirect3DDevice3_GetCurrentViewport(d3d_device3, &current_viewport); - if (current_viewport) { - if ((IDirect3DViewportImpl *)current_viewport == This) This->activate(This, FALSE); + if (current_viewport) + { + if ((IDirect3DViewportImpl *)current_viewport == This) viewport_activate(This, FALSE); IDirect3DViewport3_Release(current_viewport); } } @@ -697,7 +698,7 @@ static HRESULT WINAPI IDirect3DViewportImpl_Clear(IDirect3DViewport3 *iface, /* Need to temporarily activate viewport to clear it. Previously active one will be restored afterwards. */ - This->activate(This, TRUE); + viewport_activate(This, TRUE); hr = IDirect3DDevice7_Clear((IDirect3DDevice7 *)This->active_device, dwCount, lpRects, dwFlags & (D3DCLEAR_ZBUFFER | D3DCLEAR_TARGET), color, 1.0, 0x00000000); @@ -705,7 +706,7 @@ static HRESULT WINAPI IDirect3DViewportImpl_Clear(IDirect3DViewport3 *iface, IDirect3DDevice3_GetCurrentViewport(d3d_device3, &current_viewport); if(current_viewport) { IDirect3DViewportImpl *vp = (IDirect3DViewportImpl *)current_viewport; - vp->activate(vp, TRUE); + viewport_activate(vp, TRUE); IDirect3DViewport3_Release(current_viewport); } @@ -978,8 +979,9 @@ IDirect3DViewportImpl_SetViewport2(IDirect3DViewport3 *iface, if (This->active_device) { IDirect3DDevice3 *d3d_device3 = (IDirect3DDevice3 *)&This->active_device->IDirect3DDevice3_vtbl; IDirect3DDevice3_GetCurrentViewport(d3d_device3, &current_viewport); - if (current_viewport) { - if ((IDirect3DViewportImpl *)current_viewport == This) This->activate(This, FALSE); + if (current_viewport) + { + if ((IDirect3DViewportImpl *)current_viewport == This) viewport_activate(This, FALSE); IDirect3DViewport3_Release(current_viewport); } } @@ -1074,16 +1076,16 @@ IDirect3DViewportImpl_Clear2(IDirect3DViewport3 *iface, return D3DERR_VIEWPORTHASNODEVICE; } d3d_device3 = (IDirect3DDevice3 *)&This->active_device->IDirect3DDevice3_vtbl; - /* Need to temporarily activate viewport to clear it. Previously active one will be restored - afterwards. */ - This->activate(This, TRUE); + /* Need to temporarily activate viewport to clear it. Previously active + * one will be restored afterwards. */ + viewport_activate(This, TRUE); hr = IDirect3DDevice7_Clear((IDirect3DDevice7 *)This->active_device, dwCount, lpRects, dwFlags, dwColor, dvZ, dwStencil); IDirect3DDevice3_GetCurrentViewport(d3d_device3, &current_viewport); if(current_viewport) { IDirect3DViewportImpl *vp = (IDirect3DViewportImpl *)current_viewport; - vp->activate(vp, TRUE); + viewport_activate(vp, TRUE); IDirect3DViewport3_Release(current_viewport); } LeaveCriticalSection(&ddraw_cs);

1 0

Henri Verbeet : wined3d: Don't print format IDs in decimal.
by Alexandre Julliard 18 Aug '10

18 Aug '10

Module: wine Branch: master Commit: 125321d13c5febf06dd122188213adc3d0a5d3ed URL: http://source.winehq.org/git/wine.git/?a=commit;h=125321d13c5febf06dd122188… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Aug 17 19:03:26 2010 +0200 wined3d: Don't print format IDs in decimal. This is especially silly for FOURCC formats. --- dlls/wined3d/utils.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index 8396513..086350f 100644 --- a/dlls/wined3d/utils.c +++ b/dlls/wined3d/utils.c @@ -1542,7 +1542,7 @@ const struct wined3d_format_desc *getFormatDescEntry(WINED3DFORMAT fmt, const st int idx = getFmtIdx(fmt); if(idx == -1) { - FIXME("Can't find format %s(%d) in the format lookup table\n", debug_d3dformat(fmt), fmt); + FIXME("Can't find format %s (%#x) in the format lookup table\n", debug_d3dformat(fmt), fmt); /* Get the caller a valid pointer */ idx = getFmtIdx(WINED3DFMT_UNKNOWN); } @@ -1719,10 +1719,10 @@ const char* debug_d3dformat(WINED3DFORMAT fmt) { fourcc[2] = (char)(fmt >> 16); fourcc[3] = (char)(fmt >> 24); fourcc[4] = 0; - if( isprint(fourcc[0]) && isprint(fourcc[1]) && isprint(fourcc[2]) && isprint(fourcc[3]) ) - FIXME("Unrecognized %u (as fourcc: %s) WINED3DFORMAT!\n", fmt, fourcc); + if (isprint(fourcc[0]) && isprint(fourcc[1]) && isprint(fourcc[2]) && isprint(fourcc[3])) + FIXME("Unrecognized %#x (as fourcc: %s) WINED3DFORMAT!\n", fmt, fourcc); else - FIXME("Unrecognized %u WINED3DFORMAT!\n", fmt); + FIXME("Unrecognized %#x WINED3DFORMAT!\n", fmt); } return "unrecognized"; }

1 0

Henri Verbeet : wined3d: Release the previous render target only after replacing it in IWineD3DDeviceImpl_SetRenderTarget ().
by Alexandre Julliard 18 Aug '10

18 Aug '10

Module: wine Branch: master Commit: 35b9d3dbd82944d0a0b4b708087489d8cd003e3a URL: http://source.winehq.org/git/wine.git/?a=commit;h=35b9d3dbd82944d0a0b4b7080… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Aug 17 19:03:27 2010 +0200 wined3d: Release the previous render target only after replacing it in IWineD3DDeviceImpl_SetRenderTarget(). --- dlls/wined3d/device.c | 12 +++++++----- 1 files changed, 7 insertions(+), 5 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index d615004..9651b51 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -5704,6 +5704,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_SetRenderTarget(IWineD3DDevice *iface, DWORD render_target_idx, IWineD3DSurface *render_target, BOOL set_viewport) { IWineD3DDeviceImpl *device = (IWineD3DDeviceImpl *)iface; + IWineD3DSurfaceImpl *prev; TRACE("iface %p, render_target_idx %u, render_target %p, set_viewport %#x.\n", iface, render_target_idx, render_target, set_viewport); @@ -5714,7 +5715,8 @@ static HRESULT WINAPI IWineD3DDeviceImpl_SetRenderTarget(IWineD3DDevice *iface, return WINED3DERR_INVALIDCALL; } - if (render_target == (IWineD3DSurface *)device->render_targets[render_target_idx]) + prev = device->render_targets[render_target_idx]; + if (render_target == (IWineD3DSurface *)prev) { TRACE("Trying to do a NOP SetRenderTarget operation.\n"); return WINED3D_OK; @@ -5733,11 +5735,11 @@ static HRESULT WINAPI IWineD3DDeviceImpl_SetRenderTarget(IWineD3DDevice *iface, return WINED3DERR_INVALIDCALL; } - if (render_target) - IWineD3DSurface_AddRef(render_target); - if (device->render_targets[render_target_idx]) - IWineD3DSurface_Release((IWineD3DSurface *)device->render_targets[render_target_idx]); + if (render_target) IWineD3DSurface_AddRef(render_target); device->render_targets[render_target_idx] = (IWineD3DSurfaceImpl *)render_target; + /* Release after the assignment, to prevent device_resource_released() + * from seeing the surface as still in use. */ + if (prev) IWineD3DSurface_Release((IWineD3DSurface *)prev); /* Render target 0 is special. */ if (!render_target_idx && set_viewport)

1 0

Mariusz Pluciński : gameux/tests: Add test of IGameExplorer2 creation.
by Alexandre Julliard 17 Aug '10

17 Aug '10

Module: wine Branch: master Commit: 2464733ef3c0d55a0ef45a051bf570089361973b URL: http://source.winehq.org/git/wine.git/?a=commit;h=2464733ef3c0d55a0ef45a051… Author: Mariusz Pluciński <vshader(a)gmail.com> Date: Mon Aug 16 09:59:45 2010 +0200 gameux/tests: Add test of IGameExplorer2 creation. --- dlls/gameux/tests/gameexplorer.c | 13 ++++++++++++- 1 files changed, 12 insertions(+), 1 deletions(-) diff --git a/dlls/gameux/tests/gameexplorer.c b/dlls/gameux/tests/gameexplorer.c index b87fd02..8498fe1 100644 --- a/dlls/gameux/tests/gameexplorer.c +++ b/dlls/gameux/tests/gameexplorer.c @@ -1,5 +1,5 @@ /* - * IGameExplorer tests + * IGameExplorer and IGameExplorer2 tests * * Copyright (C) 2010 Mariusz Pluciński * @@ -36,6 +36,7 @@ static void test_create( void ) HRESULT hr; IGameExplorer* ge = NULL; + IGameExplorer2* ge2 = NULL; /* interface available up from Vista */ hr = CoCreateInstance( &CLSID_GameExplorer, NULL, CLSCTX_INPROC_SERVER, &IID_IGameExplorer, (LPVOID*)&ge); @@ -46,6 +47,16 @@ static void test_create( void ) } else win_skip("IGameExplorer cannot be created\n"); + + /* interface available up from Win7 */ + hr = CoCreateInstance( &CLSID_GameExplorer, NULL, CLSCTX_INPROC_SERVER, &IID_IGameExplorer2, (LPVOID*)&ge2); + if(ge2) + { + ok( hr == S_OK, "IGameExplorer2 creating failed (result false)\n"); + IGameExplorer2_Release(ge2); + } + else + win_skip("IGameExplorer2 cannot be created\n"); } START_TEST(gameexplorer)

1 0

Damjan Jovanovic : winex11.drv: Use a Wine list to store XDnD data.
by Alexandre Julliard 17 Aug '10

17 Aug '10

Module: wine Branch: master Commit: ac8c6f2944e2f80d040aa194b3da4d8d3558d01f URL: http://source.winehq.org/git/wine.git/?a=commit;h=ac8c6f2944e2f80d040aa194b… Author: Damjan Jovanovic <damjan.jov(a)gmail.com> Date: Mon Aug 16 21:07:20 2010 +0200 winex11.drv: Use a Wine list to store XDnD data. --- dlls/winex11.drv/xdnd.c | 29 +++++++++++++---------------- 1 files changed, 13 insertions(+), 16 deletions(-) diff --git a/dlls/winex11.drv/xdnd.c b/dlls/winex11.drv/xdnd.c index 9d209fc..ef8dd99 100644 --- a/dlls/winex11.drv/xdnd.c +++ b/dlls/winex11.drv/xdnd.c @@ -39,6 +39,7 @@ #include "wine/unicode.h" #include "wine/debug.h" +#include "wine/list.h" WINE_DEFAULT_DEBUG_CHANNEL(xdnd); @@ -52,10 +53,10 @@ typedef struct tagXDNDDATA Atom cf_xdnd; void *data; unsigned int size; - struct tagXDNDDATA *next; + struct list entry; } XDNDDATA, *LPXDNDDATA; -static LPXDNDDATA XDNDData = NULL; +static struct list xdndData = LIST_INIT(xdndData); static POINT XDNDxy = { 0, 0 }; static void X11DRV_XDND_InsertXDNDData(int property, int format, void* data, unsigned int len); @@ -316,12 +317,11 @@ static void X11DRV_XDND_InsertXDNDData(int property, int format, void* data, uns if (current) { EnterCriticalSection(&xdnd_cs); - current->next = XDNDData; current->cf_xdnd = property; current->cf_win = format; current->data = data; current->size = len; - XDNDData = current; + list_add_tail(&xdndData, &current->entry); LeaveCriticalSection(&xdnd_cs); } } @@ -484,21 +484,22 @@ static int X11DRV_XDND_DeconstructTextHTML(int property, void* data, int len) */ static void X11DRV_XDND_SendDropFiles(HWND hwnd) { - LPXDNDDATA current; + LPXDNDDATA current = NULL; + BOOL found = FALSE; EnterCriticalSection(&xdnd_cs); - current = XDNDData; - /* Find CF_HDROP type if any */ - while (current != NULL) + LIST_FOR_EACH_ENTRY(current, &xdndData, XDNDDATA, entry) { if (current->cf_win == CF_HDROP) + { + found = TRUE; break; - current = current->next; + } } - if (current != NULL) + if (found) { DROPFILES *lpDrop = current->data; @@ -530,17 +531,13 @@ static void X11DRV_XDND_FreeDragDropOp(void) EnterCriticalSection(&xdnd_cs); - current = XDNDData; - /** Free data cache */ - while (current != NULL) + LIST_FOR_EACH_ENTRY_SAFE(current, next, &xdndData, XDNDDATA, entry) { - next = current->next; + list_remove(&current->entry); HeapFree(GetProcessHeap(), 0, current); - current = next; } - XDNDData = NULL; XDNDxy.x = XDNDxy.y = 0; LeaveCriticalSection(&xdnd_cs);

1 0

Damjan Jovanovic : winex11.drv: Filter out all CF_* formats when CF_HDROP is present.
by Alexandre Julliard 17 Aug '10

17 Aug '10

Module: wine Branch: master Commit: 6871341c3c70c0054963d8fd9506e1aa71356293 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6871341c3c70c0054963d8fd9… Author: Damjan Jovanovic <damjan.jov(a)gmail.com> Date: Mon Aug 16 21:08:49 2010 +0200 winex11.drv: Filter out all CF_* formats when CF_HDROP is present. --- dlls/winex11.drv/xdnd.c | 27 +++++++++++++++++++++++++++ 1 files changed, 27 insertions(+), 0 deletions(-) diff --git a/dlls/winex11.drv/xdnd.c b/dlls/winex11.drv/xdnd.c index ef8dd99..f0704da 100644 --- a/dlls/winex11.drv/xdnd.c +++ b/dlls/winex11.drv/xdnd.c @@ -257,6 +257,8 @@ static void X11DRV_XDND_ResolveProperty(Display *display, Window xwin, Time tm, unsigned long bytesret, icount; int entries = 0; unsigned char* data = NULL; + XDNDDATA *current, *next; + BOOL haveHDROP = FALSE; TRACE("count(%ld)\n", *count); @@ -301,6 +303,31 @@ static void X11DRV_XDND_ResolveProperty(Display *display, Window xwin, Time tm, wine_tsx11_unlock(); } + /* On Windows when there is a CF_HDROP, there are no other CF_ formats. + * foobar2000 relies on this (spaces -> %20's without it). + */ + LIST_FOR_EACH_ENTRY(current, &xdndData, XDNDDATA, entry) + { + if (current->cf_win == CF_HDROP) + { + haveHDROP = TRUE; + break; + } + } + if (haveHDROP) + { + LIST_FOR_EACH_ENTRY_SAFE(current, next, &xdndData, XDNDDATA, entry) + { + if (current->cf_win != CF_HDROP && current->cf_win < CF_MAX) + { + list_remove(&current->entry); + HeapFree(GetProcessHeap(), 0, current->data); + HeapFree(GetProcessHeap(), 0, current); + --entries; + } + } + } + *count = entries; }

1 0

Damjan Jovanovic : winex11.drv: Implement a framework for dragging from X and dropping to OLE.
by Alexandre Julliard 17 Aug '10

17 Aug '10

Module: wine Branch: master Commit: 685e66e51d5ad4f00c3ccff56104840510c6d9fd URL: http://source.winehq.org/git/wine.git/?a=commit;h=685e66e51d5ad4f00c3ccff56… Author: Damjan Jovanovic <damjan.jov(a)gmail.com> Date: Mon Aug 16 21:10:05 2010 +0200 winex11.drv: Implement a framework for dragging from X and dropping to OLE. --- dlls/winex11.drv/Makefile.in | 4 +- dlls/winex11.drv/xdnd.c | 350 +++++++++++++++++++++++++++++++++++++++++- 2 files changed, 345 insertions(+), 9 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=685e66e51d5ad4f00c3cc…

1 0