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

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/windowscodecs/tests/metadata.c:
> - ok(!!ifd_stream && app1_stream != ifd_stream, "Unexpected stream.\n");
> -
> - /* Stream is positioned right after header, and point to Ifd data. It's not
> -
>   possible to make this offset an origin, as following data contains relative offsets. */
> - move.QuadPart = 0;
> - hr = IStream_Seek(ifd_stream, move, STREAM_SEEK_CUR, &pos);
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> - todo_wine
> - ok(pos.QuadPart == 8, "Unexpected position %s.\n", wine_dbgstr_longlong(pos.QuadPart));
> -
> - hr = CoCreateInstance(&CLSID_WICIfdMetadataReader, NULL, CLSCTX_INPROC_SERVER,
> -
>        &IID_IWICMetadataReader, (void **)&reader2);
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> -
> - hr = IWICMetadataReader_QueryInterface(reader2, &IID_IWICPersistStream, (void **)&persist_stream);
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);

persist_stream is never freed.