# HG changeset patch # User reimar # Date 1228847775 0 # Node ID f2606c3433ef4dc1950cc6ab480f15de30efebc2 # Parent f38929a49c2f1e8d4e90a0124098b0989c5eb5c3 Slightly simplify the conditional release/free code diff -r f38929a49c2f -r f2606c3433ef libvo/vo_direct3d.c --- a/libvo/vo_direct3d.c Tue Dec 09 18:31:52 2008 +0000 +++ b/libvo/vo_direct3d.c Tue Dec 09 18:36:15 2008 +0000 @@ -202,31 +202,26 @@ mp_msg(MSGT_VO, MSGL_V, "destroy_d3d_surfaces called\r\n"); /* Let's destroy the old (if any) D3D Surfaces */ - if (priv->locked_rect.pBits) { + if (priv->locked_rect.pBits) IDirect3DSurface9_UnlockRect(priv->d3d_surface); - priv->locked_rect.pBits = NULL; - } + priv->locked_rect.pBits = NULL; - if (priv->d3d_surface) { + if (priv->d3d_surface) IDirect3DSurface9_Release(priv->d3d_surface); - priv->d3d_surface = NULL; - } + priv->d3d_surface = NULL; /* kill the OSD texture and its shadow copy */ - if (priv->d3d_texture_osd) { + if (priv->d3d_texture_osd) IDirect3DTexture9_Release(priv->d3d_texture_osd); - priv->d3d_texture_osd = NULL; - } + priv->d3d_texture_osd = NULL; - if (priv->d3d_texture_system) { + if (priv->d3d_texture_system) IDirect3DTexture9_Release(priv->d3d_texture_system); - priv->d3d_texture_system = NULL; - } + priv->d3d_texture_system = NULL; - if (priv->d3d_backbuf) { + if (priv->d3d_backbuf) IDirect3DSurface9_Release(priv->d3d_backbuf); - priv->d3d_backbuf = NULL; - } + priv->d3d_backbuf = NULL; } /** @brief Create D3D Offscreen and Backbuffer surfaces. @@ -409,10 +404,9 @@ destroy_d3d_surfaces(); /* Destroy the D3D Device */ - if (priv->d3d_device) { + if (priv->d3d_device) IDirect3DDevice9_Release(priv->d3d_device); - priv->d3d_device = NULL; - } + priv->d3d_device = NULL; /* Stop the whole Direct3D */ IDirect3D9_Release(priv->d3d_handle); @@ -778,10 +772,9 @@ destroy_d3d_surfaces(); /* Destroy the D3D Device */ - if (priv->d3d_device) { + if (priv->d3d_device) IDirect3DDevice9_Release(priv->d3d_device); - priv->d3d_device = NULL; - } + priv->d3d_device = NULL; if (!configure_d3d()) return VO_ERROR;