Re: [PATCH v3 0/2] MR6130: Added stubs for the EnumerableObjectCollection COM class.

[Alfred Agrell (＠Alcaro)]

Alfred Agrell (@Alcaro) commented about dlls/shell32/enumobjects.c:
> - return IEnumObjects_AddRef(&This->IEnumObjects_iface);
>
> +}
> -
>
> +static ULONG WINAPI object_collection_Release(IObjectCollection *iface)
> +{
> - struct enum_objects *This = impl_from_IObjectCollection(iface);
> - return IEnumObjects_Release(&This->IEnumObjects_iface);
>
> +}
> -
>
> +static HRESULT WINAPI object_collection_GetCount(IObjectCollection *iface, UINT *count)
> +{
> - struct enum_objects *This = impl_from_IObjectCollection(iface);
> -
> - FIXME("(%p/%p)->(%n): stub!\n", This, iface, count);
> -
> - if (count)

I don't think that argument is optional. Better delete this check and just segfault, unless winetest says otherwise.