[try 2] [PATCH 03/12] comctl32: Implement Add, ReplaceIcon, GetImageCount, SetImageCount in IImageList
Henri Verbeet
hverbeet at gmail.com
Thu Nov 12 03:40:31 CST 2009
2009/11/11 Owen Rudge <orudge at codeweavers.com>:
> @@ -2985,15 +2985,37 @@ static ULONG WINAPI ImageListImpl_Release(IImageList *iface)
> static HRESULT WINAPI ImageListImpl_Add(IImageList *iface, HBITMAP hbmImage,
> HBITMAP hbmMask, int *pi)
> {
> - FIXME("STUB: %p %p %p %p\n", iface, hbmImage, hbmMask, pi);
> - return E_NOTIMPL;
> + HIMAGELIST This = (HIMAGELIST) iface;
> + int ret;
> +
> + if (!This || !hbmImage || !pi)
> + return E_FAIL;
I don't know if you have tests for the other parameters, but "iface"
should never be NULL for COM methods.
More information about the wine-devel
mailing list