<div dir="ltr"><div>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.<br><br></div><div>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.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 30, 2017 at 8:20 AM, Andrew Eikum <span dir="ltr"><<a href="mailto:aeikum@codeweavers.com" target="_blank">aeikum@codeweavers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Mar 30, 2017 at 08:25:03AM +0000, Alistair Leslie-Hughes wrote:<br>
> I have a basic patch for adding tests in the works, but I was waiting<br>
> for my IDL changes to be accepted first.<br>
><br>
> So, If someone wants to review the IDL changes, Since Sebastian has been<br>
> busy with other stuff.<br>
><br>
> <a href="https://source.winehq.org/patches/data/131403" rel="noreferrer" target="_blank">https://source.winehq.org/<wbr>patches/data/131403</a><br>
><br>
<br>
</span>Okay :) I just noticed one thing:<br>
<br>
> +    HRESULT GetAttributeCountEx(<br>
> +        [in] WORD stream_num,<br>
> +        [out] WORD attributes );<br>
<br>
If attributes is an out-param, it should be a pointer, right?<br>
<span class="HOEnZb"><font color="#888888"><br>
Andrew<br>
</font></span></blockquote></div><br></div>