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 <jem456.vasishta(a)gmail.com> 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 <hverbeet(a)gmail.com> wrote:
> On 19 June 2016 at 00:29, Aaryaman Vasishta <jem456.vasishta(a)gmail.com
> <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 <jem456.vasishta(a)gmail.com>
wrote:
> On Sunday, June 19, 2016, Henri Verbeet <hverbeet(a)gmail.com
> <javascript:_e(%7B%7D,'cvml','hverbeet(a)gmail.com');>> wrote:
> > On 19 June 2016 at 00:29, Aaryaman Vasishta <jem456.vasishta(a)gmail.com>
> > 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