[PATCH 2/6] wined3d: Get rid of wined3d_surface_update_overlay_z_order().
Henri Verbeet
hverbeet at codeweavers.com
Mon Feb 8 11:32:02 CST 2016
Signed-off-by: Henri Verbeet <hverbeet at codeweavers.com>
---
dlls/ddraw/ddraw_private.h | 1 -
dlls/ddraw/surface.c | 22 +++++++++-------------
dlls/wined3d/surface.c | 14 --------------
dlls/wined3d/wined3d.spec | 1 -
include/wine/wined3d.h | 2 --
5 files changed, 9 insertions(+), 31 deletions(-)
diff --git a/dlls/ddraw/ddraw_private.h b/dlls/ddraw/ddraw_private.h
index f2399e4..d73d87b 100644
--- a/dlls/ddraw/ddraw_private.h
+++ b/dlls/ddraw/ddraw_private.h
@@ -162,7 +162,6 @@ struct ddraw_surface
/* Connections to other Objects */
struct ddraw *ddraw;
- struct wined3d_surface *wined3d_surface;
struct wined3d_texture *wined3d_texture;
unsigned int sub_resource_idx;
struct wined3d_rendertarget_view *wined3d_rtv;
diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c
index 9b7d76e..d5ebe31 100644
--- a/dlls/ddraw/surface.c
+++ b/dlls/ddraw/surface.c
@@ -1216,7 +1216,6 @@ static HRESULT WINAPI DECLSPEC_HOTPATCH ddraw_surface7_Flip(IDirectDrawSurface7
DDSCAPS2 caps = {DDSCAPS_FLIP, 0, 0, {0}};
struct wined3d_texture *texture;
IDirectDrawSurface7 *current;
- struct wined3d_surface *tmp;
HRESULT hr;
TRACE("iface %p, src %p, flags %#x.\n", iface, src, flags);
@@ -1237,7 +1236,6 @@ static HRESULT WINAPI DECLSPEC_HOTPATCH ddraw_surface7_Flip(IDirectDrawSurface7
}
tmp_rtv = ddraw_surface_get_rendertarget_view(dst_impl);
- tmp = dst_impl->wined3d_surface;
if (dst_impl->sub_resource_idx)
ERR("Invalid sub-resource index %u on surface %p.\n", dst_impl->sub_resource_idx, dst_impl);
texture = dst_impl->wined3d_texture;
@@ -1269,7 +1267,6 @@ static HRESULT WINAPI DECLSPEC_HOTPATCH ddraw_surface7_Flip(IDirectDrawSurface7
wined3d_rendertarget_view_set_parent(src_rtv, dst_impl);
dst_impl->wined3d_rtv = src_rtv;
wined3d_resource_set_parent(wined3d_texture_get_sub_resource(src_impl->wined3d_texture, 0), dst_impl);
- dst_impl->wined3d_surface = src_impl->wined3d_surface;
prev_ddraw_texture = wined3d_texture_get_parent(src_impl->wined3d_texture);
wined3d_resource_set_parent(wined3d_texture_get_resource(src_impl->wined3d_texture), ddraw_texture);
if (src_impl->sub_resource_idx)
@@ -1301,7 +1298,6 @@ static HRESULT WINAPI DECLSPEC_HOTPATCH ddraw_surface7_Flip(IDirectDrawSurface7
wined3d_rendertarget_view_set_parent(src_rtv, dst_impl);
dst_impl->wined3d_rtv = src_rtv;
wined3d_resource_set_parent(wined3d_texture_get_sub_resource(src_impl->wined3d_texture, 0), dst_impl);
- dst_impl->wined3d_surface = src_impl->wined3d_surface;
prev_ddraw_texture = wined3d_texture_get_parent(src_impl->wined3d_texture);
wined3d_resource_set_parent(wined3d_texture_get_resource(src_impl->wined3d_texture), ddraw_texture);
ddraw_texture = prev_ddraw_texture;
@@ -1319,7 +1315,6 @@ static HRESULT WINAPI DECLSPEC_HOTPATCH ddraw_surface7_Flip(IDirectDrawSurface7
wined3d_rendertarget_view_set_parent(tmp_rtv, src_impl);
src_impl->wined3d_rtv = tmp_rtv;
wined3d_resource_set_parent(wined3d_texture_get_sub_resource(texture, 0), src_impl);
- src_impl->wined3d_surface = tmp;
wined3d_resource_set_parent(wined3d_texture_get_resource(texture), ddraw_texture);
src_impl->wined3d_texture = texture;
@@ -3843,20 +3838,22 @@ static HRESULT WINAPI ddraw_surface1_UpdateOverlayDisplay(IDirectDrawSurface *if
*
*****************************************************************************/
static HRESULT WINAPI ddraw_surface7_UpdateOverlayZOrder(IDirectDrawSurface7 *iface,
- DWORD Flags, IDirectDrawSurface7 *DDSRef)
+ DWORD flags, IDirectDrawSurface7 *reference)
{
struct ddraw_surface *surface = impl_from_IDirectDrawSurface7(iface);
- struct ddraw_surface *reference_impl = unsafe_impl_from_IDirectDrawSurface7(DDSRef);
- HRESULT hr;
- TRACE("iface %p, flags %#x, reference %p.\n", iface, Flags, DDSRef);
+ FIXME("iface %p, flags %#x, reference %p stub!\n", iface, flags, reference);
wined3d_mutex_lock();
- hr = wined3d_surface_update_overlay_z_order(surface->wined3d_surface,
- Flags, reference_impl ? reference_impl->wined3d_surface : NULL);
+ if (!(surface->surface_desc.ddsCaps.dwCaps & DDSCAPS_OVERLAY))
+ {
+ WARN("Not an overlay surface.\n");
+ wined3d_mutex_unlock();
+ return DDERR_NOTAOVERLAYSURFACE;
+ }
wined3d_mutex_unlock();
- return hr;
+ return DD_OK;
}
static HRESULT WINAPI ddraw_surface4_UpdateOverlayZOrder(IDirectDrawSurface4 *iface,
@@ -6302,7 +6299,6 @@ void ddraw_surface_init(struct ddraw_surface *surface, struct ddraw *ddraw,
}
desc->lpSurface = NULL;
- surface->wined3d_surface = wined3d_surface;
wined3d_texture_incref(surface->wined3d_texture = wined3d_texture);
surface->sub_resource_idx = sub_resource_idx;
*parent_ops = &ddraw_surface_wined3d_parent_ops;
diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c
index 53f4d09..11db96e 100644
--- a/dlls/wined3d/surface.c
+++ b/dlls/wined3d/surface.c
@@ -1909,20 +1909,6 @@ DWORD CDECL wined3d_surface_get_pitch(const struct wined3d_surface *surface)
return pitch;
}
-HRESULT CDECL wined3d_surface_update_overlay_z_order(struct wined3d_surface *surface,
- DWORD flags, struct wined3d_surface *ref)
-{
- FIXME("surface %p, flags %#x, ref %p stub!\n", surface, flags, ref);
-
- if (!(surface->resource.usage & WINED3DUSAGE_OVERLAY))
- {
- TRACE("Not an overlay surface.\n");
- return WINEDDERR_NOTAOVERLAYSURFACE;
- }
-
- return WINED3D_OK;
-}
-
HRESULT wined3d_surface_update_desc(struct wined3d_surface *surface,
const struct wined3d_gl_info *gl_info, void *mem, unsigned int pitch)
{
diff --git a/dlls/wined3d/wined3d.spec b/dlls/wined3d/wined3d.spec
index 53eb97a..d31cf9a 100644
--- a/dlls/wined3d/wined3d.spec
+++ b/dlls/wined3d/wined3d.spec
@@ -224,7 +224,6 @@
@ cdecl wined3d_surface_get_parent(ptr)
@ cdecl wined3d_surface_get_pitch(ptr)
-@ cdecl wined3d_surface_update_overlay_z_order(ptr long ptr)
@ cdecl wined3d_swapchain_create(ptr ptr ptr ptr ptr)
@ cdecl wined3d_swapchain_decref(ptr)
diff --git a/include/wine/wined3d.h b/include/wine/wined3d.h
index 40f3d49..0b8bf7b 100644
--- a/include/wine/wined3d.h
+++ b/include/wine/wined3d.h
@@ -2477,8 +2477,6 @@ ULONG __cdecl wined3d_stateblock_incref(struct wined3d_stateblock *stateblock);
void * __cdecl wined3d_surface_get_parent(const struct wined3d_surface *surface);
DWORD __cdecl wined3d_surface_get_pitch(const struct wined3d_surface *surface);
-HRESULT __cdecl wined3d_surface_update_overlay_z_order(struct wined3d_surface *surface,
- DWORD flags, struct wined3d_surface *ref);
HRESULT __cdecl wined3d_swapchain_create(struct wined3d_device *device, struct wined3d_swapchain_desc *desc,
void *parent, const struct wined3d_parent_ops *parent_ops, struct wined3d_swapchain **swapchain);
--
2.1.4
More information about the wine-patches
mailing list