[PATCH 2/6] d3drm: Implement IDirect3DRM*::CreateViewport. (v2 resend)

[Aaryaman Vasishta]

On Thu, Jul 28, 2016 at 12:21 AM, Stefan Dösinger <stefandoesinger at gmail.com
> wrote:

> > +    if (!device || !camera)
> > +        return D3DRMERR_BADOBJECT;
> > +    if (!viewport)
> > +        return D3DRMERR_BADVALUE;
> These checks are redundant, Init() does that as well.
>
Agreed for version 3 of CreateViewport, but this this apply for version 1/2
as well? Since I'm dereferencing these interfaces there I thought the
checks were justified.

>
> > +    ok(ref4 == frame_ref, "Expected ref4 == frame_ref, got frame_ref =
> %u, ref4 = %u.\n", frame_ref, ref4);
> > +
> > +    device_ref = get_refcount((IUnknown *)device1);
> > +    frame_ref = get_refcount((IUnknown *)frame);
> The get_refcount here seems redundant because you've just shown that it is
> back to where it used to be.
>
Right, these are definitely redundant.

Thanks for the review!

Cheers,
Aaryaman
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20160728/75d948be/attachment.html>