ddraw: Forward AddAttachedSurface to the correct equivalent

Stefan Dösinger stefandoesinger at gmx.at
Tue May 15 09:59:50 CDT 2012


Am Montag, 14. Mai 2012, 18:45:01 schrieb paulo lesgaz:
> But I can write a small test. Would writing a test attaching an offscreen 
surface to another offscreen surface be enough to convince you of the 
correctness of the change?
I recommend to port the entire AttachmentTest from dlls/ddraw/tests/dsurface.c 
to IDirectDrawSurface4. Please place the new test in ddraw4.c and follow the 
code style there. I'd be extra greatful if you could move AttachmentTest to 
ddraw1.c, AttachmentTest7 to ddraw7.c and also create a IDirectDrawSurface3 
version in ddraw2.c.

Yes, that's a lot of work for fixing the bug. But ddraw is a complex API with 
many quirks, and we're past the point where quick fixes can be made without 
thurough testing.

As Matteo says, it probably makes more sense to forward Surface3 to Surface4 
and change the Surface4 behavior than the other way around.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20120515/bbedf06b/attachment.pgp>


More information about the wine-devel mailing list