[PATCH 1/4] d3drm/tests: Add test for IDirect3DRM*::CreateDeviceFromClipper (try 2).
jem456.vasishta at gmail.com
Thu Jun 11 11:40:03 CDT 2015
On Thu, Jun 11, 2015 at 9:34 PM, Stefan Dösinger <stefandoesinger at gmail.com>
> > + hr = DirectDrawCreateClipper(0, &clipper, NULL);
> > + ref = get_refcount((IUnknown *)clipper);
> You're not reading ref anywhere as far as I can see. Did you forget to
> re-check the refcount after creating the device?
> 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.
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.
> > + memcpy(&driver, &IID_IDirect3DRGBDevice, sizeof(GUID));
> I think an assignment should do the job.
That was just to maintain consistency with the existing tests, I could
still use an assignment if you want to.
> 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.
By chaning the mode you mean the dimentions/bpp or both?
I will implement everything else you specified.
Thanks for the review! :)
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the wine-devel