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. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7407#note_96331