Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-02-22 08:25:03 +00:00
Code Issues Projects Releases Packages Wiki Activity

[WINESYNC] d3dx9: Simplify effect cleanup.

Browse source 
Signed-off-by: Matteo Bruni <mbruni@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

wine commit id 009279845d9503468d5d3d39e393ad2ffa00181d by Matteo Bruni <mbruni@codeweavers.com>
This commit is contained in:
winesync 2020-09-21 22:50:18 +02:00 committed by Jérôme Gardou
parent 275bd75bd6
commit f336e915a2
2 changed files with 11 additions and 30 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

39
dll/directx/wine/d3dx9_36/effect.c
View file

@ -681,11 +681,11 @@ static void free_technique(struct d3dx_technique *technique)
technique->name = NULL;
}
static void d3dx9_base_effect_cleanup(struct d3dx_effect *effect)
static void d3dx_effect_cleanup(struct d3dx_effect *effect)
{
unsigned int i;
TRACE("Effect %p.\n", effect);
TRACE("effect %p.\n", effect);
heap_free(effect->full_name_tmp);
@ -694,7 +694,6 @@ static void d3dx9_base_effect_cleanup(struct d3dx_effect *effect)
for (i = 0; i < effect->parameter_count; ++i)
free_top_level_parameter(&effect->parameters[i]);
heap_free(effect->parameters);
effect->parameters = NULL;
}
if (effect->techniques)
@ -702,37 +701,23 @@ static void d3dx9_base_effect_cleanup(struct d3dx_effect *effect)
for (i = 0; i < effect->technique_count; ++i)
free_technique(&effect->techniques[i]);
heap_free(effect->techniques);
effect->techniques = NULL;
}
if (effect->objects)
{
for (i = 0; i < effect->object_count; ++i)
{
free_object(&effect->objects[i]);
}
heap_free(effect->objects);
effect->objects = NULL;
}
}
static void free_effect(struct d3dx_effect *effect)
{
TRACE("Free effect %p\n", effect);
d3dx9_base_effect_cleanup(effect);
if (effect->pool)
{
effect->pool->lpVtbl->Release(effect->pool);
}
if (effect->manager)
{
IUnknown_Release(effect->manager);
}
IDirect3DDevice9_Release(effect->device);
heap_free(effect);
}
static void get_vector(struct d3dx_parameter *param, D3DXVECTOR4 *vector)
@ -1826,18 +1811,15 @@ static ULONG WINAPI d3dx_effect_AddRef(ID3DXEffect *iface)
static ULONG WINAPI d3dx_effect_Release(ID3DXEffect *iface)
{
struct d3dx_effect *This = impl_from_ID3DXEffect(iface);
ULONG ref = InterlockedDecrement(&This->ref);
struct d3dx_effect *effect = impl_from_ID3DXEffect(iface);
ULONG refcount = InterlockedDecrement(&effect->ref);
TRACE("(%p)->(): Release from %u\n", This, ref + 1);
TRACE("%p decreasing refcount to %u.\n", effect, refcount);
if (!ref)
{
free_effect(This);
HeapFree(GetProcessHeap(), 0, This);
}
if (!refcount)
d3dx_effect_cleanup(effect);
return ref;
return refcount;
}
/*** ID3DXBaseEffect methods ***/
@ -6334,7 +6316,6 @@ static HRESULT d3dx9_effect_init(struct d3dx_effect *effect, struct IDirect3DDev
error_messages, pool_impl, skip_constants)))
{
FIXME("Failed to parse effect, hr %#x.\n", hr);
free_effect(effect);
return hr;
}
@ -6382,7 +6363,7 @@ HRESULT WINAPI D3DXCreateEffectEx(struct IDirect3DDevice9 *device, const void *s
if (FAILED(hr))
{
WARN("Failed to create effect object.\n");
HeapFree(GetProcessHeap(), 0, object);
d3dx_effect_cleanup(object);
return hr;
}

2
sdk/tools/winesync/d3dx9.cfg
View file

@ -15,4 +15,4 @@ files: {include/d3dx9.h: sdk/include/dxsdk/d3dx9.h, include/d3dx9anim.h: sdk/inc
include/d3dx9mesh.h: sdk/include/dxsdk/d3dx9mesh.h, include/d3dx9of.h: sdk/include/dxsdk/d3dx9of.h,
include/d3dx9shader.h: sdk/include/dxsdk/d3dx9shader.h, include/d3dx9shape.h: sdk/include/dxsdk/d3dx9shape.h,
include/d3dx9tex.h: sdk/include/dxsdk/d3dx9tex.h, include/d3dx9xof.h: sdk/include/dxsdk/d3dx9xof.h}
tags: {wine: 5fa06bbc302c0fdbf8a5e087b7f18445475add2a}
tags: {wine: 009279845d9503468d5d3d39e393ad2ffa00181d}