<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 11, 2015 at 9:34 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<span class=""><br>
> +    hr = DirectDrawCreateClipper(0, &clipper, NULL);<br>
> +    ref = get_refcount((IUnknown *)clipper);<br>
</span>You're not reading ref anywhere as far as I can see. Did you forget to re-check the refcount after creating the device?<br>
<br>
If the refcount is incremented by 2 or more: See if the clipper is assigned to the render target and / or depth stencil. This wouldn't really do anything, but it's Microsoft we're talking about here.<br></blockquote><div>That was a mistake from me. The refcount tests are in patch 4. I'll also take a look at where the clipper is assigned. <br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> +    memcpy(&driver, &IID_IDirect3DRGBDevice, sizeof(GUID));<br>
I think an assignment should do the job.<br></blockquote><div>That was just to maintain consistency with the existing tests, I could still use an assignment if you want to. <span class=""></span><span class=""></span><br></div><div> <span class=""></span><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
</span>There's no need to try to set the display mode to 8. Also testing if changing the mode works is outside the scope of this test. What's interesting is that the pixel format of the primary surface of a new device after changing the mode.<br></blockquote><div>By chaning the mode you mean the dimentions/bpp or both? <br><br><br></div><div>I will implement everything else you specified.<br><br></div><div>Thanks for the review! :)<br></div></div><br></div></div>