Commit 86609976 authored by Henri Verbeet's avatar Henri Verbeet Committed by Alexandre Julliard

ddraw: Attaching an already attached surface is not a FIXME.

According to the tests it should simply fail.
parent 3fd66bc6
...@@ -988,13 +988,14 @@ static HRESULT WINAPI ddraw_surface_attach_surface(IDirectDrawSurfaceImpl *This, ...@@ -988,13 +988,14 @@ static HRESULT WINAPI ddraw_surface_attach_surface(IDirectDrawSurfaceImpl *This,
EnterCriticalSection(&ddraw_cs); EnterCriticalSection(&ddraw_cs);
/* Check if the surface is already attached somewhere */ /* Check if the surface is already attached somewhere */
if( (Surf->next_attached != NULL) || if (Surf->next_attached || Surf->first_attached != Surf)
(Surf->first_attached != Surf) )
{ {
/* TODO: Test for the structure of the manual attachment. Is it a chain or a list? /* TODO: Test for the structure of the manual attachment. Is it a
* What happens if one surface is attached to 2 different surfaces? * chain or a list? What happens if one surface is attached to 2
*/ * different surfaces? */
FIXME("(%p) The Surface %p is already attached somewhere else: next_attached = %p, first_attached = %p, can't handle by now\n", This, Surf, Surf->next_attached, Surf->first_attached); WARN("Surface %p is already attached somewhere. next_attached %p, first_attached %p.\n",
Surf, Surf->next_attached, Surf->first_attached);
LeaveCriticalSection(&ddraw_cs); LeaveCriticalSection(&ddraw_cs);
return DDERR_SURFACEALREADYATTACHED; return DDERR_SURFACEALREADYATTACHED;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment