Jacek Caban : wined3d: Removed no longer needed wine_rb_init error handling.
Alexandre Julliard
julliard at winehq.org
Wed Aug 31 11:02:42 CDT 2016
Module: wine
Branch: master
Commit: 249b494cce4e281628c43b4855c4dd2584af23dc
URL: http://source.winehq.org/git/wine.git/?a=commit;h=249b494cce4e281628c43b4855c4dd2584af23dc
Author: Jacek Caban <jacek at codeweavers.com>
Date: Tue Aug 30 21:31:44 2016 +0200
wined3d: Removed no longer needed wine_rb_init error handling.
Signed-off-by: Jacek Caban <jacek at codeweavers.com>
Signed-off-by: Henri Verbeet <hverbeet at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/wined3d/arb_program_shader.c | 24 +++---------------------
dlls/wined3d/ati_fragment_shader.c | 8 +-------
dlls/wined3d/device.c | 6 +-----
dlls/wined3d/glsl_shader.c | 22 +++-------------------
4 files changed, 8 insertions(+), 52 deletions(-)
diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c
index c206745..e45f1c4 100644
--- a/dlls/wined3d/arb_program_shader.c
+++ b/dlls/wined3d/arb_program_shader.c
@@ -4993,14 +4993,7 @@ static HRESULT shader_arb_alloc(struct wined3d_device *device, const struct wine
memset(priv->pshader_const_dirty, 1,
sizeof(*priv->pshader_const_dirty) * d3d_info->limits.ps_uniform_count);
- if (wine_rb_init(&priv->signature_tree, &sig_tree_functions) == -1)
- {
- ERR("RB tree init failed\n");
- fragment_pipe->free_private(device);
- vertex_pipe->vp_free(device);
- HeapFree(GetProcessHeap(), 0, priv);
- return E_OUTOFMEMORY;
- }
+ wine_rb_init(&priv->signature_tree, &sig_tree_functions);
priv->vertex_pipe = vertex_pipe;
priv->fragment_pipe = fragment_pipe;
@@ -5884,13 +5877,7 @@ static void *arbfp_alloc(const struct wined3d_shader_backend_ops *shader_backend
else if (!(priv = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*priv))))
return NULL;
- if (wine_rb_init(&priv->fragment_shaders, &wined3d_ffp_frag_program_rb_functions) == -1)
- {
- ERR("Failed to initialize rbtree.\n");
- if (priv != shader_priv)
- HeapFree(GetProcessHeap(), 0, priv);
- return NULL;
- }
+ wine_rb_init(&priv->fragment_shaders, &wined3d_ffp_frag_program_rb_functions);
priv->use_arbfp_fixed_func = TRUE;
return priv;
@@ -7080,12 +7067,7 @@ static HRESULT arbfp_blit_alloc(struct wined3d_device *device)
if (!(priv = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*priv))))
return E_OUTOFMEMORY;
- if (wine_rb_init(&priv->shaders, &wined3d_arbfp_blit_rb_functions) == -1)
- {
- ERR("Failed to initialize rbtree.\n");
- HeapFree(GetProcessHeap(), 0, priv);
- return E_OUTOFMEMORY;
- }
+ wine_rb_init(&priv->shaders, &wined3d_arbfp_blit_rb_functions);
device->blit_priv = priv;
diff --git a/dlls/wined3d/ati_fragment_shader.c b/dlls/wined3d/ati_fragment_shader.c
index 6a71c24..e5f65c6 100644
--- a/dlls/wined3d/ati_fragment_shader.c
+++ b/dlls/wined3d/ati_fragment_shader.c
@@ -1323,13 +1323,7 @@ static void *atifs_alloc(const struct wined3d_shader_backend_ops *shader_backend
if (!(priv = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*priv))))
return NULL;
- if (wine_rb_init(&priv->fragment_shaders, &wined3d_ffp_frag_program_rb_functions) == -1)
- {
- ERR("Failed to initialize rbtree.\n");
- HeapFree(GetProcessHeap(), 0, priv);
- return NULL;
- }
-
+ wine_rb_init(&priv->fragment_shaders, &wined3d_ffp_frag_program_rb_functions);
return priv;
}
diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c
index 3f163a6..5b80814 100644
--- a/dlls/wined3d/device.c
+++ b/dlls/wined3d/device.c
@@ -5089,11 +5089,7 @@ HRESULT device_init(struct wined3d_device *device, struct wined3d *wined3d,
fragment_pipeline = adapter->fragment_pipe;
- if (wine_rb_init(&device->samplers, &wined3d_sampler_rb_functions) == -1)
- {
- ERR("Failed to initialize sampler rbtree.\n");
- return E_OUTOFMEMORY;
- }
+ wine_rb_init(&device->samplers, &wined3d_sampler_rb_functions);
if (vertex_pipeline->vp_states && fragment_pipeline->states
&& FAILED(hr = compile_state_table(device->StateTable, device->multistate_funcs,
diff --git a/dlls/wined3d/glsl_shader.c b/dlls/wined3d/glsl_shader.c
index 5ccb027..a5ec999 100644
--- a/dlls/wined3d/glsl_shader.c
+++ b/dlls/wined3d/glsl_shader.c
@@ -8483,11 +8483,7 @@ static HRESULT shader_glsl_alloc(struct wined3d_device *device, const struct win
goto fail;
}
- if (wine_rb_init(&priv->program_lookup, &wined3d_glsl_program_rb_functions) == -1)
- {
- ERR("Failed to initialize rbtree.\n");
- goto fail;
- }
+ wine_rb_init(&priv->program_lookup, &wined3d_glsl_program_rb_functions);
priv->next_constant_version = 1;
priv->vertex_pipe = vertex_pipe;
@@ -8921,13 +8917,7 @@ static void *glsl_vertex_pipe_vp_alloc(const struct wined3d_shader_backend_ops *
if (shader_backend == &glsl_shader_backend)
{
priv = shader_priv;
-
- if (wine_rb_init(&priv->ffp_vertex_shaders, &wined3d_ffp_vertex_program_rb_functions) == -1)
- {
- ERR("Failed to initialize rbtree.\n");
- return NULL;
- }
-
+ wine_rb_init(&priv->ffp_vertex_shaders, &wined3d_ffp_vertex_program_rb_functions);
return priv;
}
@@ -9447,13 +9437,7 @@ static void *glsl_fragment_pipe_alloc(const struct wined3d_shader_backend_ops *s
if (shader_backend == &glsl_shader_backend)
{
priv = shader_priv;
-
- if (wine_rb_init(&priv->ffp_fragment_shaders, &wined3d_ffp_frag_program_rb_functions) == -1)
- {
- ERR("Failed to initialize rbtree.\n");
- return NULL;
- }
-
+ wine_rb_init(&priv->ffp_fragment_shaders, &wined3d_ffp_frag_program_rb_functions);
return priv;
}
More information about the wine-cvs
mailing list