Do FIXME statements need to print out the passed-in arguments?�� Only outputting the method name, you can still tell what needs to be implemented.�� Also, to output each argument, there are typedefs to consider, and you have to choose whether to dereference any pointer arguments, which makes it rather difficult to automate the process.

I could submit just the IDL changes, without the stubs added to wmvcore_main.c.�� Sorry Alistair, they seem to include your additions as well.

On Thu, Mar 30, 2017 at 8:20 AM, Andrew Eikum <aeikum@codeweavers.com> wrote:

On Thu, Mar 30, 2017 at 08:25:03AM +0000, Alistair Leslie-Hughes wrote:
> I have a basic patch for adding tests in the works, but I was waiting
> for my IDL changes to be accepted first.
>
> So, If someone wants to review the IDL changes, Since Sebastian has been
> busy with other stuff.
>
> https://source.winehq.org/patches/data/131403
>

Okay :) I just noticed one thing:

> +�� HRESULT GetAttributeCountEx(
> +�� [in] WORD stream_num,
> +�� [out] WORD attributes );

If attributes is an out-param, it should be a pointer, right?

Andrew