Re: [PATCH 0/10] MR7407: windowscodecs: Implement query strings enumerator.

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/windowscodecs/metadataquery.c:
> static HRESULT WINAPI string_enumerator_Skip(IEnumString *iface, ULONG count)
>  {
> - FIXME("iface %p, count %lu stub.\n", iface, count);
> - struct string_enumerator *enumerator = impl_from_IEnumString(iface);
> - unsigned int index;
> - HRESULT hr = S_OK;
> - IUnknown *object;
> - ULONG fetched;
> - GUID guid;
> -
> - TRACE("iface %p, count %lu.\n", iface, count);
> -
> - if (enumerator->object_enumerator)
> - {
> -
>    while (count-- && IEnumUnknown_Next(enumerator->object_enumerator, 1, &object, &fetched) == S_OK)

I think you want to assign `hr` here so we can pass `S_FALSE` or an error to the caller.