2009/8/10 Owen Rudge <oru...@codeweavers.com>:
> +@ stdcall HIMAGELIST_QueryInterface(long long long)
...
> +@ stdcall ImageList_CoCreateInstance(long long long long)
These should all be pointers.

> +    FIXME("STUB: %s %p %s %p\n",debugstr_guid(rclsid), punkOuter, 
> debugstr_guid(riid), ppv);
> +    return E_NOINTERFACE;
You have some trailing white space here. Also, the white space in the
FIXME is somewhat inconsistent.

Personally, I'd say the wording in the comments and the parameter
naming is closer to MSDN than I'd be comfortable with.


Reply via email to