[DDRAW] Only detach surfaces when releasing a non-texture surface attached to a complex root.

[Stefan Dösinger]

Am Sonntag 27 August 2006 17:23 schrieb Elie Morisse:
> Refcounting fix, Wine devs tell me if I'm wrong as from now on I get
> something like this w/ The Nomad Soul :
>
> fixme:d3d:IWineD3DDeviceImpl_Release (0x1248028) Device released with
> resources still bound, acceptable but unexpected fixme:d3d:dumpResources
> Leftover resource 0x217f988 with type 1,WINED3DRTYPE_SURFACE
> fixme:d3d:dumpResources Leftover resource 0x1abca8 with type
> 1,WINED3DRTYPE_SURFACE
>
> Other games I tried aren't affected though, so this one may just be buggy..
> Good part is this prevents memory faults in The Nomad Soul at exit and when
> switching resolution.
Just asking, does this game hit the This->first_attached != This case? This 
shouldn't happen, I suspect that the refcounting issue is somewhere else. 
Also the This->first_complex == This check should always return true in 
theory.

Can you send me a +ddraw,+d3d7 trace of the game, with and without your patch?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20060827/ca438254/attachment.pgp