Re: [PATCH 5/5] d3drm/tests: Modify all QI tests to compare vtables. (resend)

[Henri Verbeet]

On 19 June 2016 at 00:29, Aaryaman Vasishta  wrote:
> -
>    { &IID_IUnknown,                   &IID_IUnknown, NULL,                   S_OK                      },

Is the NULL "vtable_iid" here intentional?

[PATCH 5/5] d3drm/tests: Modify all QI tests to compare vtables. (resend)

[Aaryaman Vasishta]

On Sunday, June 19, 2016, Henri Verbeet  wrote:
> On 19 June 2016 at 00:29, Aaryaman Vasishta <[e-mail]
> javascript:;> wrote:
> > -
> >    { &IID_IUnknown,                   &IID_IUnknown, NULL,
>        S_OK                      },
>
> Is the NULL "vtable_iid" here intentional?
>
> Yes, it means that it the particular interface hasn't been implemented in

QI yet. In this case, IUnknown QI's to frame1in wine, but in Windows it
QI's to something else, possibly IUnknown's own vtable.

Cheers,
Aaryaman

[PATCH 5/5] d3drm/tests: Modify all QI tests to compare vtables. (resend)

[Aaryaman Vasishta]

Sorry for the typos, I'm replying from my phone.

On Sunday, June 19, 2016, Aaryaman Vasishta 
wrote:
> On Sunday, June 19, 2016, Henri Verbeet <[e-mail]
> > wrote:
> > On 19 June 2016 at 00:29, Aaryaman Vasishta 
> > wrote:
> > > -
> > >    { &IID_IUnknown,                   &IID_IUnknown, NULL,
> >        S_OK                      },
> >
> > Is the NULL "vtable_iid" here intentional?
> >
> > Yes, it means that it the particular interface hasn't been implemented in
>
> QI yet. In this case, IUnknown QI's to frame1in wine, but in Windows it
> QI's to something else, possibly IUnknown's own vtable.
>
> Cheers,
> Aaryaman