[PATCH] dsound/tests: Add media object query tests
Andrew Eikum
aeikum at codeweavers.com
Thu Nov 7 15:22:07 CST 2019
On Thu, Nov 07, 2019 at 07:28:36PM +0000, Alistair Leslie-Hughes wrote:
>
> + rc = IUnknown_QueryInterface(dmo, &IID_IMediaObject, (void**)&inplace);
>
> > Wrong IID here.
> >
> > Also, these tests seem like implementation details. What's your
> > motivation here? Is there an app that depends on this behavior?
> No app depends on this behavior.
>
> dsound (builtin) queriesfor these interfaces and I was showing that DMO's support the interfaces
> and at the same time trying to work out how much of the
> IMediaObjec/IMediaObjectInPlace code I could share between them
>
>
> My current implementation has a complete copy of each these interfaces.
>
> I'm happy to drop tests and just prove the interfaces exists and we can
> get back to the object original, if that helps.
Yeah I think the pointer value tests aren't interesting unless an
application needs it.
Andrew
More information about the wine-devel
mailing list