December 2010 - Wine-commits

Henri Verbeet : wined3d: Remove the unused IWineD3DBaseTexture::GetDirty() method.
by Alexandre Julliard 09 Dec '10

09 Dec '10

Module: wine Branch: master Commit: 4dd3066ea3f96ab25ff86ebf3c54c489a88c551e URL: http://source.winehq.org/git/wine.git/?a=commit;h=4dd3066ea3f96ab25ff86ebf3… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Dec 8 18:28:08 2010 +0100 wined3d: Remove the unused IWineD3DBaseTexture::GetDirty() method. --- dlls/wined3d/basetexture.c | 6 ------ dlls/wined3d/cubetexture.c | 6 ------ dlls/wined3d/texture.c | 5 ----- dlls/wined3d/volumetexture.c | 5 ----- dlls/wined3d/wined3d_private.h | 1 - include/wine/wined3d.idl | 2 -- 6 files changed, 0 insertions(+), 25 deletions(-) diff --git a/dlls/wined3d/basetexture.c b/dlls/wined3d/basetexture.c index 83f3797..166584d 100644 --- a/dlls/wined3d/basetexture.c +++ b/dlls/wined3d/basetexture.c @@ -247,12 +247,6 @@ BOOL basetexture_set_dirty(IWineD3DBaseTexture *iface, BOOL dirty) return old; } -BOOL basetexture_get_dirty(IWineD3DBaseTexture *iface) -{ - IWineD3DBaseTextureImpl *This = (IWineD3DBaseTextureImpl *)iface; - return This->baseTexture.texture_rgb.dirty || This->baseTexture.texture_srgb.dirty; -} - /* Context activation is done by the caller. */ HRESULT basetexture_bind(IWineD3DBaseTexture *iface, BOOL srgb, BOOL *set_surface_desc) { diff --git a/dlls/wined3d/cubetexture.c b/dlls/wined3d/cubetexture.c index d53f6cf..d2f2f19 100644 --- a/dlls/wined3d/cubetexture.c +++ b/dlls/wined3d/cubetexture.c @@ -275,11 +275,6 @@ static BOOL WINAPI IWineD3DCubeTextureImpl_SetDirty(IWineD3DCubeTexture *iface, return basetexture_set_dirty((IWineD3DBaseTexture *)iface, dirty); } -/* Internal function, No d3d mapping */ -static BOOL WINAPI IWineD3DCubeTextureImpl_GetDirty(IWineD3DCubeTexture *iface) { - return basetexture_get_dirty((IWineD3DBaseTexture *)iface); -} - /* Context activation is done by the caller. */ static HRESULT WINAPI IWineD3DCubeTextureImpl_BindTexture(IWineD3DCubeTexture *iface, BOOL srgb) { IWineD3DCubeTextureImpl *This = (IWineD3DCubeTextureImpl *)iface; @@ -440,7 +435,6 @@ static const IWineD3DCubeTextureVtbl IWineD3DCubeTexture_Vtbl = IWineD3DCubeTextureImpl_GetAutoGenFilterType, IWineD3DCubeTextureImpl_GenerateMipSubLevels, IWineD3DCubeTextureImpl_SetDirty, - IWineD3DCubeTextureImpl_GetDirty, IWineD3DCubeTextureImpl_BindTexture, IWineD3DCubeTextureImpl_IsCondNP2, /* IWineD3DCubeTexture */ diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index 915a55f..9fe7280 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -266,10 +266,6 @@ static BOOL WINAPI IWineD3DTextureImpl_SetDirty(IWineD3DTexture *iface, BOOL dir return basetexture_set_dirty((IWineD3DBaseTexture *)iface, dirty); } -static BOOL WINAPI IWineD3DTextureImpl_GetDirty(IWineD3DTexture *iface) { - return basetexture_get_dirty((IWineD3DBaseTexture *)iface); -} - /* Context activation is done by the caller. */ static HRESULT WINAPI IWineD3DTextureImpl_BindTexture(IWineD3DTexture *iface, BOOL srgb) { IWineD3DTextureImpl *This = (IWineD3DTextureImpl *)iface; @@ -454,7 +450,6 @@ static const IWineD3DTextureVtbl IWineD3DTexture_Vtbl = IWineD3DTextureImpl_GetAutoGenFilterType, IWineD3DTextureImpl_GenerateMipSubLevels, IWineD3DTextureImpl_SetDirty, - IWineD3DTextureImpl_GetDirty, IWineD3DTextureImpl_BindTexture, IWineD3DTextureImpl_IsCondNP2, /* IWineD3DTexture */ diff --git a/dlls/wined3d/volumetexture.c b/dlls/wined3d/volumetexture.c index 99fa75f..3d1a22b 100644 --- a/dlls/wined3d/volumetexture.c +++ b/dlls/wined3d/volumetexture.c @@ -230,10 +230,6 @@ static BOOL WINAPI IWineD3DVolumeTextureImpl_SetDirty(IWineD3DVolumeTexture *ifa return basetexture_set_dirty((IWineD3DBaseTexture *)iface, dirty); } -static BOOL WINAPI IWineD3DVolumeTextureImpl_GetDirty(IWineD3DVolumeTexture *iface) { - return basetexture_get_dirty((IWineD3DBaseTexture *)iface); -} - /* Context activation is done by the caller. */ static HRESULT WINAPI IWineD3DVolumeTextureImpl_BindTexture(IWineD3DVolumeTexture *iface, BOOL srgb) { @@ -370,7 +366,6 @@ static const IWineD3DVolumeTextureVtbl IWineD3DVolumeTexture_Vtbl = IWineD3DVolumeTextureImpl_GetAutoGenFilterType, IWineD3DVolumeTextureImpl_GenerateMipSubLevels, IWineD3DVolumeTextureImpl_SetDirty, - IWineD3DVolumeTextureImpl_GetDirty, /* not in d3d */ IWineD3DVolumeTextureImpl_BindTexture, IWineD3DVolumeTextureImpl_IsCondNP2, diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index b21d617..4b6b109 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1915,7 +1915,6 @@ HRESULT basetexture_bind(IWineD3DBaseTexture *iface, BOOL srgb, BOOL *set_surfac void basetexture_cleanup(IWineD3DBaseTexture *iface) DECLSPEC_HIDDEN; void basetexture_generate_mipmaps(IWineD3DBaseTexture *iface) DECLSPEC_HIDDEN; WINED3DTEXTUREFILTERTYPE basetexture_get_autogen_filter_type(IWineD3DBaseTexture *iface) DECLSPEC_HIDDEN; -BOOL basetexture_get_dirty(IWineD3DBaseTexture *iface) DECLSPEC_HIDDEN; DWORD basetexture_get_level_count(IWineD3DBaseTexture *iface) DECLSPEC_HIDDEN; DWORD basetexture_get_lod(IWineD3DBaseTexture *iface) DECLSPEC_HIDDEN; IWineD3DResourceImpl *basetexture_get_sub_resource(IWineD3DBaseTextureImpl *texture, diff --git a/include/wine/wined3d.idl b/include/wine/wined3d.idl index 8876c3b..3b4fda4 100644 --- a/include/wine/wined3d.idl +++ b/include/wine/wined3d.idl @@ -2528,8 +2528,6 @@ interface IWineD3DBaseTexture : IWineD3DResource BOOL SetDirty( BOOL dirty ); - BOOL GetDirty( - ); HRESULT BindTexture( [in] BOOL srgb );

1 0

Henri Verbeet : wined3d: Remove IWineD3DBaseTexture::SetDirty() from the public interface.
by Alexandre Julliard 09 Dec '10

09 Dec '10

Module: wine Branch: master Commit: a3b2407824e364406d28403b1f64c61bdd3c037c URL: http://source.winehq.org/git/wine.git/?a=commit;h=a3b2407824e364406d28403b1… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Dec 8 18:28:09 2010 +0100 wined3d: Remove IWineD3DBaseTexture::SetDirty() from the public interface. --- dlls/wined3d/basetexture.c | 13 +++++++------ dlls/wined3d/cubetexture.c | 6 ------ dlls/wined3d/surface.c | 6 +++--- dlls/wined3d/texture.c | 6 ------ dlls/wined3d/volumetexture.c | 6 ------ dlls/wined3d/wined3d_private.h | 2 +- include/wine/wined3d.idl | 3 --- 7 files changed, 11 insertions(+), 31 deletions(-) diff --git a/dlls/wined3d/basetexture.c b/dlls/wined3d/basetexture.c index 166584d..5047081 100644 --- a/dlls/wined3d/basetexture.c +++ b/dlls/wined3d/basetexture.c @@ -237,13 +237,14 @@ void basetexture_generate_mipmaps(IWineD3DBaseTexture *iface) FIXME("iface %p stub!\n", iface); } -BOOL basetexture_set_dirty(IWineD3DBaseTexture *iface, BOOL dirty) +BOOL basetexture_set_dirty(IWineD3DBaseTextureImpl *texture, BOOL dirty) { BOOL old; - IWineD3DBaseTextureImpl *This = (IWineD3DBaseTextureImpl *)iface; - old = This->baseTexture.texture_rgb.dirty || This->baseTexture.texture_srgb.dirty; - This->baseTexture.texture_rgb.dirty = dirty; - This->baseTexture.texture_srgb.dirty = dirty; + + old = texture->baseTexture.texture_rgb.dirty || texture->baseTexture.texture_srgb.dirty; + texture->baseTexture.texture_rgb.dirty = dirty; + texture->baseTexture.texture_srgb.dirty = dirty; + return old; } @@ -293,7 +294,7 @@ HRESULT basetexture_bind(IWineD3DBaseTexture *iface, BOOL srgb, BOOL *set_surfac gl_tex->states[WINED3DTEXSTA_MAXANISOTROPY] = 1; gl_tex->states[WINED3DTEXSTA_SRGBTEXTURE] = 0; gl_tex->states[WINED3DTEXSTA_SHADOW] = FALSE; - IWineD3DBaseTexture_SetDirty(iface, TRUE); + basetexture_set_dirty(This, TRUE); isNewTexture = TRUE; if(This->resource.usage & WINED3DUSAGE_AUTOGENMIPMAP) { diff --git a/dlls/wined3d/cubetexture.c b/dlls/wined3d/cubetexture.c index d2f2f19..5287e37 100644 --- a/dlls/wined3d/cubetexture.c +++ b/dlls/wined3d/cubetexture.c @@ -270,11 +270,6 @@ static void WINAPI IWineD3DCubeTextureImpl_GenerateMipSubLevels(IWineD3DCubeText basetexture_generate_mipmaps((IWineD3DBaseTexture *)iface); } -/* Internal function, No d3d mapping */ -static BOOL WINAPI IWineD3DCubeTextureImpl_SetDirty(IWineD3DCubeTexture *iface, BOOL dirty) { - return basetexture_set_dirty((IWineD3DBaseTexture *)iface, dirty); -} - /* Context activation is done by the caller. */ static HRESULT WINAPI IWineD3DCubeTextureImpl_BindTexture(IWineD3DCubeTexture *iface, BOOL srgb) { IWineD3DCubeTextureImpl *This = (IWineD3DCubeTextureImpl *)iface; @@ -434,7 +429,6 @@ static const IWineD3DCubeTextureVtbl IWineD3DCubeTexture_Vtbl = IWineD3DCubeTextureImpl_SetAutoGenFilterType, IWineD3DCubeTextureImpl_GetAutoGenFilterType, IWineD3DCubeTextureImpl_GenerateMipSubLevels, - IWineD3DCubeTextureImpl_SetDirty, IWineD3DCubeTextureImpl_BindTexture, IWineD3DCubeTextureImpl_IsCondNP2, /* IWineD3DCubeTexture */ diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 91cc880..e10ed0c 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -980,7 +980,7 @@ void surface_add_dirty_rect(IWineD3DSurfaceImpl *surface, const RECT *dirty_rect if (surface->container.type == WINED3D_CONTAINER_TEXTURE) { TRACE("Passing to container.\n"); - IWineD3DBaseTexture_SetDirty((IWineD3DBaseTexture *)surface->container.u.texture, TRUE); + basetexture_set_dirty(surface->container.u.texture, TRUE); } } @@ -4299,7 +4299,7 @@ void surface_modify_location(IWineD3DSurfaceImpl *surface, DWORD flag, BOOL pers if (surface->container.type == WINED3D_CONTAINER_TEXTURE) { TRACE("Passing to container.\n"); - IWineD3DBaseTexture_SetDirty((IWineD3DBaseTexture *)surface->container.u.texture, TRUE); + basetexture_set_dirty(surface->container.u.texture, TRUE); } } surface->flags &= ~SFLAG_LOCATIONS; @@ -4321,7 +4321,7 @@ void surface_modify_location(IWineD3DSurfaceImpl *surface, DWORD flag, BOOL pers if (surface->container.type == WINED3D_CONTAINER_TEXTURE) { TRACE("Passing to container\n"); - IWineD3DBaseTexture_SetDirty((IWineD3DBaseTexture *)surface->container.u.texture, TRUE); + basetexture_set_dirty(surface->container.u.texture, TRUE); } } surface->flags &= ~flag; diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index 9fe7280..e610846 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -261,11 +261,6 @@ static void WINAPI IWineD3DTextureImpl_GenerateMipSubLevels(IWineD3DTexture *ifa basetexture_generate_mipmaps((IWineD3DBaseTexture *)iface); } -/* Internal function, No d3d mapping */ -static BOOL WINAPI IWineD3DTextureImpl_SetDirty(IWineD3DTexture *iface, BOOL dirty) { - return basetexture_set_dirty((IWineD3DBaseTexture *)iface, dirty); -} - /* Context activation is done by the caller. */ static HRESULT WINAPI IWineD3DTextureImpl_BindTexture(IWineD3DTexture *iface, BOOL srgb) { IWineD3DTextureImpl *This = (IWineD3DTextureImpl *)iface; @@ -449,7 +444,6 @@ static const IWineD3DTextureVtbl IWineD3DTexture_Vtbl = IWineD3DTextureImpl_SetAutoGenFilterType, IWineD3DTextureImpl_GetAutoGenFilterType, IWineD3DTextureImpl_GenerateMipSubLevels, - IWineD3DTextureImpl_SetDirty, IWineD3DTextureImpl_BindTexture, IWineD3DTextureImpl_IsCondNP2, /* IWineD3DTexture */ diff --git a/dlls/wined3d/volumetexture.c b/dlls/wined3d/volumetexture.c index 3d1a22b..0c61111 100644 --- a/dlls/wined3d/volumetexture.c +++ b/dlls/wined3d/volumetexture.c @@ -225,11 +225,6 @@ static void WINAPI IWineD3DVolumeTextureImpl_GenerateMipSubLevels(IWineD3DVolume basetexture_generate_mipmaps((IWineD3DBaseTexture *)iface); } -/* Internal function, No d3d mapping */ -static BOOL WINAPI IWineD3DVolumeTextureImpl_SetDirty(IWineD3DVolumeTexture *iface, BOOL dirty) { - return basetexture_set_dirty((IWineD3DBaseTexture *)iface, dirty); -} - /* Context activation is done by the caller. */ static HRESULT WINAPI IWineD3DVolumeTextureImpl_BindTexture(IWineD3DVolumeTexture *iface, BOOL srgb) { @@ -365,7 +360,6 @@ static const IWineD3DVolumeTextureVtbl IWineD3DVolumeTexture_Vtbl = IWineD3DVolumeTextureImpl_SetAutoGenFilterType, IWineD3DVolumeTextureImpl_GetAutoGenFilterType, IWineD3DVolumeTextureImpl_GenerateMipSubLevels, - IWineD3DVolumeTextureImpl_SetDirty, /* not in d3d */ IWineD3DVolumeTextureImpl_BindTexture, IWineD3DVolumeTextureImpl_IsCondNP2, diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 4b6b109..ebcb4ea 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1925,7 +1925,7 @@ HRESULT basetexture_init(IWineD3DBaseTextureImpl *texture, UINT layer_count, UIN const struct wined3d_parent_ops *parent_ops) DECLSPEC_HIDDEN; HRESULT basetexture_set_autogen_filter_type(IWineD3DBaseTexture *iface, WINED3DTEXTUREFILTERTYPE filter_type) DECLSPEC_HIDDEN; -BOOL basetexture_set_dirty(IWineD3DBaseTexture *iface, BOOL dirty) DECLSPEC_HIDDEN; +BOOL basetexture_set_dirty(IWineD3DBaseTextureImpl *texture, BOOL dirty) DECLSPEC_HIDDEN; DWORD basetexture_set_lod(IWineD3DBaseTexture *iface, DWORD new_lod) DECLSPEC_HIDDEN; void basetexture_unload(IWineD3DBaseTexture *iface) DECLSPEC_HIDDEN; diff --git a/include/wine/wined3d.idl b/include/wine/wined3d.idl index 3b4fda4..45e4143 100644 --- a/include/wine/wined3d.idl +++ b/include/wine/wined3d.idl @@ -2525,9 +2525,6 @@ interface IWineD3DBaseTexture : IWineD3DResource ); void GenerateMipSubLevels( ); - BOOL SetDirty( - BOOL dirty - ); HRESULT BindTexture( [in] BOOL srgb );

1 0

Henri Verbeet : wined3d: Pass an IWineD3DDeviceImpl pointer to alloc_private().
by Alexandre Julliard 09 Dec '10

09 Dec '10

Module: wine Branch: master Commit: 4ba3b64c576302e7391902e85dac0dcbd3618d7b URL: http://source.winehq.org/git/wine.git/?a=commit;h=4ba3b64c576302e7391902e85… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Dec 8 18:28:10 2010 +0100 wined3d: Pass an IWineD3DDeviceImpl pointer to alloc_private(). --- dlls/wined3d/arb_program_shader.c | 4 ++-- dlls/wined3d/device.c | 4 ++-- dlls/wined3d/surface.c | 4 ++-- dlls/wined3d/wined3d_private.h | 2 +- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index e3b6e5d..eb0056f 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -6503,8 +6503,8 @@ struct arbfp_blit_priv { GLuint palette_texture; }; -static HRESULT arbfp_blit_alloc(IWineD3DDevice *iface) { - IWineD3DDeviceImpl *device = (IWineD3DDeviceImpl *) iface; +static HRESULT arbfp_blit_alloc(IWineD3DDeviceImpl *device) +{ device->blit_priv = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(struct arbfp_blit_priv)); if(!device->blit_priv) { ERR("Out of memory\n"); diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index ec50693..53b0c36 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -1973,7 +1973,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_Init3D(IWineD3DDevice *iface, TRACE("Fragment pipeline private data couldn't be allocated\n"); goto err_out; } - hr = This->blitter->alloc_private(iface); + hr = This->blitter->alloc_private(This); if(FAILED(hr)) { TRACE("Blitter private data couldn't be allocated\n"); goto err_out; @@ -6259,7 +6259,7 @@ static HRESULT create_primary_opengl_context(IWineD3DDevice *iface, IWineD3DSwap goto err; } - hr = This->blitter->alloc_private(iface); + hr = This->blitter->alloc_private(This); if (FAILED(hr)) { ERR("Failed to allocate blitter private data, hr %#x.\n", hr); diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index e10ed0c..05ce89b 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -4732,7 +4732,7 @@ const IWineD3DSurfaceVtbl IWineD3DSurface_Vtbl = IWineD3DSurfaceImpl_DrawOverlay }; -static HRESULT ffp_blit_alloc(IWineD3DDevice *iface) { return WINED3D_OK; } +static HRESULT ffp_blit_alloc(IWineD3DDeviceImpl *device) { return WINED3D_OK; } /* Context activation is done by the caller. */ static void ffp_blit_free(IWineD3DDevice *iface) { } @@ -4865,7 +4865,7 @@ const struct blit_shader ffp_blit = { ffp_blit_color_fill }; -static HRESULT cpu_blit_alloc(IWineD3DDevice *iface) +static HRESULT cpu_blit_alloc(IWineD3DDeviceImpl *device) { return WINED3D_OK; } diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index ebcb4ea..a20a0ad 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1175,7 +1175,7 @@ enum blit_operation * already under the GL lock. */ struct blit_shader { - HRESULT (*alloc_private)(IWineD3DDevice *iface); + HRESULT (*alloc_private)(IWineD3DDeviceImpl *device); void (*free_private)(IWineD3DDevice *iface); HRESULT (*set_shader)(IWineD3DDevice *iface, IWineD3DSurfaceImpl *surface); void (*unset_shader)(IWineD3DDevice *iface);

1 0

Henri Verbeet : wined3d: Pass an IWineD3DDeviceImpl pointer to free_private ().
by Alexandre Julliard 09 Dec '10

09 Dec '10

Module: wine Branch: master Commit: 7f09eb6a7a7a8a285d0f90a8998bcc1fda32cdc9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7f09eb6a7a7a8a285d0f90a89… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Dec 8 18:28:11 2010 +0100 wined3d: Pass an IWineD3DDeviceImpl pointer to free_private(). --- dlls/wined3d/arb_program_shader.c | 4 ++-- dlls/wined3d/device.c | 6 +++--- dlls/wined3d/surface.c | 4 ++-- dlls/wined3d/wined3d_private.h | 2 +- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index eb0056f..7dc3c81 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -6514,8 +6514,8 @@ static HRESULT arbfp_blit_alloc(IWineD3DDeviceImpl *device) } /* Context activation is done by the caller. */ -static void arbfp_blit_free(IWineD3DDevice *iface) { - IWineD3DDeviceImpl *device = (IWineD3DDeviceImpl *) iface; +static void arbfp_blit_free(IWineD3DDeviceImpl *device) +{ const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; struct arbfp_blit_priv *priv = device->blit_priv; diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 53b0c36..15b9020 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -2050,7 +2050,7 @@ err_out: This->stateBlock = NULL; } if (This->blit_priv) { - This->blitter->free_private(iface); + This->blitter->free_private(This); } if (This->fragment_priv) { This->frag_pipe->free_private(This); @@ -2185,7 +2185,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_Uninit3D(IWineD3DDevice *iface, } /* Destroy the shader backend. Note that this has to happen after all shaders are destroyed. */ - This->blitter->free_private(iface); + This->blitter->free_private(This); This->frag_pipe->free_private(This); This->shader_backend->shader_free_private(This); @@ -6199,7 +6199,7 @@ static void delete_opengl_contexts(IWineD3DDevice *iface, IWineD3DSwapChainImpl } LEAVE_GL(); - This->blitter->free_private(iface); + This->blitter->free_private(This); This->frag_pipe->free_private(This); This->shader_backend->shader_free_private(This); destroy_dummy_textures(This, gl_info); diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 05ce89b..994f3bc 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -4734,7 +4734,7 @@ const IWineD3DSurfaceVtbl IWineD3DSurface_Vtbl = static HRESULT ffp_blit_alloc(IWineD3DDeviceImpl *device) { return WINED3D_OK; } /* Context activation is done by the caller. */ -static void ffp_blit_free(IWineD3DDevice *iface) { } +static void ffp_blit_free(IWineD3DDeviceImpl *device) { } /* This function is used in case of 8bit paletted textures using GL_EXT_paletted_texture */ /* Context activation is done by the caller. */ @@ -4871,7 +4871,7 @@ static HRESULT cpu_blit_alloc(IWineD3DDeviceImpl *device) } /* Context activation is done by the caller. */ -static void cpu_blit_free(IWineD3DDevice *iface) +static void cpu_blit_free(IWineD3DDeviceImpl *device) { } diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index a20a0ad..bb7b1f5 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1176,7 +1176,7 @@ enum blit_operation struct blit_shader { HRESULT (*alloc_private)(IWineD3DDeviceImpl *device); - void (*free_private)(IWineD3DDevice *iface); + void (*free_private)(IWineD3DDeviceImpl *device); HRESULT (*set_shader)(IWineD3DDevice *iface, IWineD3DSurfaceImpl *surface); void (*unset_shader)(IWineD3DDevice *iface); BOOL (*blit_supported)(const struct wined3d_gl_info *gl_info, enum blit_operation blit_op,

1 0

Andrew Bogott : winemenubuilder: Release property variants. Plugs a one-string memory leak.
by Alexandre Julliard 09 Dec '10

09 Dec '10

Module: wine Branch: master Commit: 3b00ef9ebd7e7b0caf33271f24bc95c5b0a7b966 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3b00ef9ebd7e7b0caf33271f2… Author: Andrew Bogott <Andrew(a)CodeWeavers.com> Date: Wed Dec 8 12:20:36 2010 -0600 winemenubuilder: Release property variants. Plugs a one-string memory leak. --- programs/winemenubuilder/winemenubuilder.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/programs/winemenubuilder/winemenubuilder.c b/programs/winemenubuilder/winemenubuilder.c index 78d6bd6..af1aa56 100644 --- a/programs/winemenubuilder/winemenubuilder.c +++ b/programs/winemenubuilder/winemenubuilder.c @@ -2692,6 +2692,8 @@ static BOOL InvokeShellLinkerForURL( IUniformResourceLocatorW *url, LPCWSTR link icon_name = extract_icon( pv[0].pwszVal, pv[1].iVal, NULL, bWait ); WINE_TRACE("URL icon path: %s icon index: %d icon name: %s\n", wine_dbgstr_w(pv[0].pwszVal), pv[1].iVal, icon_name); + PropVariantClear(&pv[0]); + PropVariantClear(&pv[1]); } IPropertyStorage_Release(pPropStg); }

1 0

Alexandre Julliard : widl: Signal an error when trying to output an undefined interface into a typelib .
by Alexandre Julliard 09 Dec '10

09 Dec '10

Module: wine Branch: master Commit: 8c78994866430c1e56212bba1e1c3ff3ce62e77d URL: http://source.winehq.org/git/wine.git/?a=commit;h=8c78994866430c1e56212bba1… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 8 20:32:08 2010 +0100 widl: Signal an error when trying to output an undefined interface into a typelib. --- tools/widl/write_msft.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/tools/widl/write_msft.c b/tools/widl/write_msft.c index 309145f..d762445 100644 --- a/tools/widl/write_msft.c +++ b/tools/widl/write_msft.c @@ -2014,6 +2014,12 @@ static void add_interface_typeinfo(msft_typelib_t *typelib, type_t *interface) if (-1 < interface->typelib_idx) return; + if (!interface->details.iface) + { + error( "interface %s is referenced but not defined\n", interface->name ); + return; + } + if (is_attr(interface->attrs, ATTR_DISPINTERFACE)) return add_dispinterface_typeinfo(typelib, interface);

1 0

Alexandre Julliard : widl: Support encoding the DECIMAL type in typelibs.
by Alexandre Julliard 09 Dec '10

09 Dec '10

Module: wine Branch: master Commit: cb246ca7a7bb34c815321a49e3013bc7a35cdc3a URL: http://source.winehq.org/git/wine.git/?a=commit;h=cb246ca7a7bb34c815321a49e… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 9 00:05:28 2010 +0100 widl: Support encoding the DECIMAL type in typelibs. --- tools/widl/write_msft.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/tools/widl/write_msft.c b/tools/widl/write_msft.c index d762445..f11de5c 100644 --- a/tools/widl/write_msft.c +++ b/tools/widl/write_msft.c @@ -849,6 +849,12 @@ static int encode_type( *alignment = 8; break; + case VT_DECIMAL: + *encoded_type = default_type; + *width = 16; + *alignment = 8; + break; + case VT_VOID: *encoded_type = 0x80000000 | (VT_EMPTY << 16) | vt; *width = 0;

1 0

Alexandre Julliard : include: Define more wuapi interfaces to avoid undefined forward declarations.
by Alexandre Julliard 09 Dec '10

09 Dec '10

Module: wine Branch: master Commit: 500c47ccab3b51ead2031be68a75eebb5a44806f URL: http://source.winehq.org/git/wine.git/?a=commit;h=500c47ccab3b51ead2031be68… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Dec 8 20:27:24 2010 +0100 include: Define more wuapi interfaces to avoid undefined forward declarations. --- include/wuapi.idl | 952 ++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 941 insertions(+), 11 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=500c47ccab3b51ead2031…

1 0

Hwang YunSong : appwiz.cpl: Update Korean resource.
by Alexandre Julliard 08 Dec '10

08 Dec '10

Module: wine Branch: master Commit: 792051baaf1e8df173e2e0cfea0bb440645c67a7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=792051baaf1e8df173e2e0cfe… Author: Hwang YunSong <hys545(a)dreamwiz.com> Date: Wed Dec 8 17:27:01 2010 +0900 appwiz.cpl: Update Korean resource. --- dlls/appwiz.cpl/Ko.rc | 83 +++++++++++++++++++++++++++++++----------------- 1 files changed, 53 insertions(+), 30 deletions(-) diff --git a/dlls/appwiz.cpl/Ko.rc b/dlls/appwiz.cpl/Ko.rc index d76bb97..711d7a3 100644 --- a/dlls/appwiz.cpl/Ko.rc +++ b/dlls/appwiz.cpl/Ko.rc @@ -2,7 +2,7 @@ * Add/Remove Programs Korean resources * * Copyright 2001-2002, 2008 Owen Rudge -* Copyright 2005,2006,2007,2008 YunSong Hwang +* Copyright 2005,2006,2007,2008,2010 YunSong Hwang * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -22,57 +22,65 @@ #include "res.h" +#pragma code_page(65001) + LANGUAGE LANG_KOREAN, SUBLANG_DEFAULT STRINGTABLE { - IDS_CPL_TITLE, "��α׷� �߰�/��" - IDS_CPL_DESC, "�� Ʈ�� ��ġ�ϰų�,�̹� ��ġ�� Ʈ�� �� ǻ�Ϳ�� ֽ��ϴ�." - IDS_TAB1_TITLE, "��α׷�" + IDS_CPL_TITLE, "프로그램 추가/삭제" + IDS_CPL_DESC, "새 소프트웨어를 설치하거나,이미 설치된 소프트웨어를 당신의 컴퓨터에서 삭제 할 수 있습니다." + IDS_TAB1_TITLE, "프로그램" + + IDS_UNINSTALL_FAILED, "언인스톨러를 실행할 수 없습니다, '%s'. 당신은 이 프로그램의 언인스톨 정보를 레지스트리에서 지우겠습니까?" + IDS_NOT_SPECIFIED, "지정되지 않음" + IDS_COLUMN_NAME, "이름" + IDS_COLUMN_PUBLISHER, "제작사" + IDS_COLUMN_VERSION, "버젼" + IDS_INSTALL_FILTER, "설치 프로그램0*instal*.exe;*setup*.exe;*.msi\0프로그램 (*.exe)\0*.exe\0모든 파일들 (*.*)\0*.*\0\0" - IDS_UNINSTALL_FAILED, "��ν��緯�� ϴ�, '%s'. �� α׷�� ν�� Ʈ�� ڽ��ϱ�?" - IDS_NOT_SPECIFIED, "�� " - IDS_COLUMN_NAME, "�̸�" - IDS_COLUMN_PUBLISHER, "��ۻ�" - IDS_COLUMN_VERSION, "��" - IDS_INSTALL_FILTER, "��ġ ��α׷�0*instal*.exe;*setup*.exe;*.msi\0��α׷� (*.exe)\0*.exe\0�� ϵ� (*.*)\0*.*\0\0" + IDS_REMOVE, "제거(&R)..." + IDS_MODIFY_REMOVE, "고치기(&M)/제거..." +} - IDS_REMOVE, "��(&R)..." - IDS_MODIFY_REMOVE, "��ġ��(&M)/��..." +STRINGTABLE +{ + IDS_DOWNLOADING "다운로드중..." + IDS_INSTALLING "설치하는 중..." } IDD_MAIN DIALOG 0, 0, 320, 220 STYLE DS_MODALFRAME | DS_3DLOOK | WS_POPUP | WS_VISIBLE | WS_CAPTION | WS_SYSMENU -CAPTION "��ġ/��" +CAPTION "설치/삭제" FONT 9, "MS Shell Dlg" { - CONTROL "�÷�� ũ�� õ��̳� ,�ϵ��ũ�� α׷�� ġ�Ϸ�� [��ġ]�� Ŭ��Ͻʽÿ�.", 1000, "STATIC", SS_LEFT | WS_CHILD | WS_VISIBLE | WS_GROUP, 40, 7, 270, 20 - CONTROL "��ġ(&I)...", IDC_INSTALL, "BUTTON", BS_PUSHBUTTON | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 246, 26, 64, 14 + CONTROL "플로피 디스크나 시디롬이나 ,하드디스크에서 새 프로그램을 설치하려면 [설치]를 클릭하십시오.", 1000, "STATIC", SS_LEFT | WS_CHILD | WS_VISIBLE | WS_GROUP, 40, 7, 270, 20 + CONTROL "설치(&I)...", IDC_INSTALL, "BUTTON", BS_PUSHBUTTON | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 246, 26, 64, 14 CONTROL "", -1, "STATIC", SS_LEFT | SS_SUNKEN | WS_CHILD | WS_VISIBLE, 7, 46, 303, 1 CONTROL 2, 1001, "STATIC", SS_ICON | WS_CHILD | WS_VISIBLE, 7, 7, 21, 20 - CONTROL "�� Ʈ�� ڵ�� Ǿ�� ֽ��ϴ�.��ġ�� Ҹ� ��ġ�ų� ��α׷�� , �߰�/�� Ͽ�� Ͻʽÿ�..", 1002, "STATIC", SS_LEFT | WS_CHILD | WS_VISIBLE | WS_GROUP, 40, 57, 270, 36 + CONTROL "이 소프트웨어는 자동으로 삭제되었을 수도 있습니다.설치된 구성요소를 고치거나 프로그램을 지우려면 , 추가/삭제 목록에서 선택하십시오..", 1002, "STATIC", SS_LEFT | WS_CHILD | WS_VISIBLE | WS_GROUP, 40, 57, 270, 36 CONTROL "", IDL_PROGRAMS, "SysListView32", LVS_REPORT | LVS_SINGLESEL | LVS_SORTASCENDING | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 7, 90, 303, 100 - CONTROL "�� (&S)...", IDC_SUPPORT_INFO, "button", BS_PUSHBUTTON | BS_CENTER | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 104, 198, 64, 14 - CONTROL "��ġ��(&M)...", IDC_MODIFY, "BUTTON", BS_PUSHBUTTON | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 172, 198, 68, 14 - CONTROL "��(&R)...", IDC_ADDREMOVE, "BUTTON", BS_PUSHBUTTON | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 244, 198, 68, 14 + CONTROL "지원 정보(&S)...", IDC_SUPPORT_INFO, "button", BS_PUSHBUTTON | BS_CENTER | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 104, 198, 64, 14 + CONTROL "고치기(&M)...", IDC_MODIFY, "BUTTON", BS_PUSHBUTTON | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 172, 198, 68, 14 + CONTROL "지우기(&R)...", IDC_ADDREMOVE, "BUTTON", BS_PUSHBUTTON | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 244, 198, 68, 14 CONTROL 3, 1003, "STATIC", SS_ICON | WS_CHILD | WS_VISIBLE, 7, 57, 21, 20 } IDD_INFO DIALOG 0, 0, 256, 138 STYLE DS_MODALFRAME | DS_3DLOOK | WS_POPUP | WS_VISIBLE | WS_CAPTION | WS_SYSMENU -CAPTION "�� " +CAPTION "지원 정보" FONT 9, "MS Shell Dlg" { - CONTROL "Ȯ��", IDOK, "BUTTON", BS_DEFPUSHBUTTON | BS_CENTER | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 103, 116, 50, 14 - CONTROL "�� %s: �� ޱ� �� ˴ϴ�", IDC_INFO_LABEL, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 12, 9, 228, 19 - CONTROL "��ۻ�:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 30, 60, 8 - CONTROL "��:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 40, 60, 8 - CONTROL "��ó:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 50, 60, 8 - CONTROL "�� :", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 60, 64, 8 - CONTROL "�� ȭ:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 70, 68, 8 - CONTROL "��ǻ��:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 80, 60, 8 - CONTROL "��ǰ ��Ʈ:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 90, 60, 8 - CONTROL "��:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 100, 60, 8 + CONTROL "확인", IDOK, "BUTTON", BS_DEFPUSHBUTTON | BS_CENTER | WS_CHILD | WS_VISIBLE | WS_TABSTOP, 103, 116, 50, 14 + CONTROL "이 정보는 %s: 에서 기술 지원을 받기 위해 사용됩니다", IDC_INFO_LABEL, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 12, 9, 228, 19 + CONTROL "제작사:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 30, 60, 8 + CONTROL "버젼:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 40, 60, 8 + CONTROL "연락처:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 50, 60, 8 + CONTROL "지원 정보:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 60, 64, 8 + CONTROL "지원 전화:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 70, 68, 8 + CONTROL "주의사항:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 80, 60, 8 + CONTROL "제품 업데이트:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 90, 60, 8 + CONTROL "설명:", -1, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 16, 100, 60, 8 CONTROL "", IDC_INFO_PUBLISHER, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 104, 30, 136, 8 CONTROL "", IDC_INFO_VERSION, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 104, 40, 136, 8 CONTROL "", IDC_INFO_CONTACT, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 104, 50, 136, 8 @@ -82,3 +90,18 @@ FONT 9, "MS Shell Dlg" CONTROL "", IDC_INFO_UPDATES, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 104, 90, 136, 8 CONTROL "", IDC_INFO_COMMENTS, "static", SS_LEFT | WS_CHILD | WS_VISIBLE, 104, 100, 136, 8 } + +ID_DWL_DIALOG DIALOG 0, 0, 260, 95 +STYLE DS_MODALFRAME | DS_CENTER | WS_POPUP | WS_VISIBLE | WS_CAPTION | WS_SYSMENU +CAPTION "Wine Gecko 설치관리자" +FONT 8, "MS Shell Dlg" +{ + LTEXT "Wine은 HTML 임베딩 프로그램이 정상적으로 작동하기 위해서 " \ + "필요로 하는 Gecko 패키지를 찾을 수 없습니다. WIne은 당신을 위해 자동으로 다운로드하고 설치할 것입니다.\n\n\n" \ + "주위:당신의 배포판의 패키지를 대신 사용하는 것을 추천합니다. " \ + "자세한 것은 http://wiki.winehq.org/Gecko를 보십시오.", + ID_DWL_STATUS, 10, 10, 240, 60, SS_LEFT + CONTROL "진행상황", ID_DWL_PROGRESS, PROGRESS_CLASSA, WS_BORDER|PBS_SMOOTH, 10, 40, 240, 12 + DEFPUSHBUTTON "설치(&I)", ID_DWL_INSTALL, 200, 70, 50, 15, WS_GROUP | WS_TABSTOP + PUSHBUTTON "취소(&C)", IDCANCEL, 140, 70, 50, 15, WS_GROUP | WS_TABSTOP +}

1 0

Michael Stefaniuc : ole32: Use an iface instead of a vtbl pointer in PropertyStorage_impl.
by Alexandre Julliard 08 Dec '10

08 Dec '10

Module: wine Branch: master Commit: e13f9ff300ed23fafb8bb505d9b6ba0360c7dfe0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e13f9ff300ed23fafb8bb505d… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Dec 8 13:15:12 2010 +0100 ole32: Use an iface instead of a vtbl pointer in PropertyStorage_impl. --- dlls/ole32/stg_prop.c | 41 +++++++++++++++++++++++------------------ 1 files changed, 23 insertions(+), 18 deletions(-) diff --git a/dlls/ole32/stg_prop.c b/dlls/ole32/stg_prop.c index 65b4b2e..cc29c0e 100644 --- a/dlls/ole32/stg_prop.c +++ b/dlls/ole32/stg_prop.c @@ -152,7 +152,7 @@ static HRESULT create_EnumSTATPROPSTG(PropertyStorage_impl *, IEnumSTATPROPSTG** */ struct tagPropertyStorage_impl { - const IPropertyStorageVtbl *vtbl; + IPropertyStorage IPropertyStorage_iface; LONG ref; CRITICAL_SECTION cs; IStream *stm; @@ -171,6 +171,11 @@ struct tagPropertyStorage_impl struct dictionary *propid_to_prop; }; +static inline PropertyStorage_impl *impl_from_IPropertyStorage(IPropertyStorage *iface) +{ + return CONTAINING_RECORD(iface, PropertyStorage_impl, IPropertyStorage_iface); +} + /************************************************************************ * IPropertyStorage_fnQueryInterface (IPropertyStorage) */ @@ -179,7 +184,7 @@ static HRESULT WINAPI IPropertyStorage_fnQueryInterface( REFIID riid, void** ppvObject) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); TRACE("(%p, %s, %p)\n", This, debugstr_guid(riid), ppvObject); @@ -205,7 +210,7 @@ static HRESULT WINAPI IPropertyStorage_fnQueryInterface( static ULONG WINAPI IPropertyStorage_fnAddRef( IPropertyStorage *iface) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); return InterlockedIncrement(&This->ref); } @@ -215,7 +220,7 @@ static ULONG WINAPI IPropertyStorage_fnAddRef( static ULONG WINAPI IPropertyStorage_fnRelease( IPropertyStorage *iface) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); ULONG ref; ref = InterlockedDecrement(&This->ref); @@ -293,7 +298,7 @@ static HRESULT WINAPI IPropertyStorage_fnReadMultiple( const PROPSPEC rgpspec[], PROPVARIANT rgpropvar[]) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); HRESULT hr = S_OK; ULONG i; @@ -551,7 +556,7 @@ static HRESULT WINAPI IPropertyStorage_fnWriteMultiple( const PROPVARIANT rgpropvar[], PROPID propidNameFirst) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); HRESULT hr = S_OK; ULONG i; @@ -650,7 +655,7 @@ static HRESULT WINAPI IPropertyStorage_fnDeleteMultiple( ULONG cpspec, const PROPSPEC rgpspec[]) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); ULONG i; HRESULT hr; @@ -696,7 +701,7 @@ static HRESULT WINAPI IPropertyStorage_fnReadPropertyNames( const PROPID rgpropid[], LPOLESTR rglpwstrName[]) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); ULONG i; HRESULT hr = S_FALSE; @@ -736,7 +741,7 @@ static HRESULT WINAPI IPropertyStorage_fnWritePropertyNames( const PROPID rgpropid[], const LPOLESTR rglpwstrName[]) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); ULONG i; HRESULT hr; @@ -769,7 +774,7 @@ static HRESULT WINAPI IPropertyStorage_fnDeletePropertyNames( ULONG cpropid, const PROPID rgpropid[]) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); ULONG i; HRESULT hr; @@ -805,7 +810,7 @@ static HRESULT WINAPI IPropertyStorage_fnCommit( IPropertyStorage* iface, DWORD grfCommitFlags) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); HRESULT hr; TRACE("(%p, 0x%08x)\n", iface, grfCommitFlags); @@ -828,7 +833,7 @@ static HRESULT WINAPI IPropertyStorage_fnRevert( IPropertyStorage* iface) { HRESULT hr; - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); TRACE("%p\n", iface); @@ -853,7 +858,7 @@ static HRESULT WINAPI IPropertyStorage_fnEnum( IPropertyStorage* iface, IEnumSTATPROPSTG** ppenum) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); return create_EnumSTATPROPSTG(This, ppenum); } @@ -877,7 +882,7 @@ static HRESULT WINAPI IPropertyStorage_fnSetClass( IPropertyStorage* iface, REFCLSID clsid) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); TRACE("%p, %s\n", iface, debugstr_guid(clsid)); @@ -899,7 +904,7 @@ static HRESULT WINAPI IPropertyStorage_fnStat( IPropertyStorage* iface, STATPROPSETSTG* statpsstg) { - PropertyStorage_impl *This = (PropertyStorage_impl *)iface; + PropertyStorage_impl *This = impl_from_IPropertyStorage(iface); STATSTG stat; HRESULT hr; @@ -1979,7 +1984,7 @@ static HRESULT PropertyStorage_BaseConstruct(IStream *stm, if (!*pps) return E_OUTOFMEMORY; - (*pps)->vtbl = &IPropertyStorage_Vtbl; + (*pps)->IPropertyStorage_iface.lpVtbl = &IPropertyStorage_Vtbl; (*pps)->ref = 1; InitializeCriticalSection(&(*pps)->cs); (*pps)->cs.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": PropertyStorage_impl.cs"); @@ -2013,7 +2018,7 @@ static HRESULT PropertyStorage_ConstructFromStream(IStream *stm, hr = PropertyStorage_ReadFromStream(ps); if (SUCCEEDED(hr)) { - *pps = (IPropertyStorage *)ps; + *pps = &ps->IPropertyStorage_iface; TRACE("PropertyStorage %p constructed\n", ps); hr = S_OK; } @@ -2047,7 +2052,7 @@ static HRESULT PropertyStorage_ConstructEmpty(IStream *stm, ps->codePage = CP_UNICODE; ps->locale = LOCALE_SYSTEM_DEFAULT; TRACE("Code page is %d, locale is %d\n", ps->codePage, ps->locale); - *pps = (IPropertyStorage *)ps; + *pps = &ps->IPropertyStorage_iface; TRACE("PropertyStorage %p constructed\n", ps); hr = S_OK; }

1 0