On Thu, Jul 28, 2016 at 12:21 AM, Stefan D������singer <stefandoesinger@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