<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 24, 2015 at 2:17 AM, Stefan Dösinger <span dir="ltr"><<a href="mailto:stefandoesinger@gmail.com" target="_blank">stefandoesinger@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
Hi,<br>
<br>
One I don't like here is the hardcoded DDSCAPS_SYSTEMEMORY for the Z<br>
buffer. You need it on Windows because of the RGB device and Wine<br>
doesn't check, so it kinda works. But I recommend to do something like<br>
this:<br>
<br>
</blockquote><div>I had actually tested for DDSCAPS_SYSTEMMEMORY without writing a test for it but you're right. It should be there.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Another thing to test: After destroying the d3drm device is the Z<br>
buffer removed from the surface when (a) d3drm created it and (b) you<br>
created it?<br></blockquote><div>There are probably two ways to do this. One is use EnumAttachedSurfaces and increment a counter if a surface is attached (similar to ddraw tests) then check if the counter == 1 (meaning z surface is attached) just after releasing the device for both cases (whether we attached the z surface or let d3drm attach one for us).<br></div><div>The other way could be simply calling GetAttachedSurface with DDSCAPS_ZBUFFER as a caps parameter.<br></div></div><br></div></div>