Re: [PATCH 0/4] MR6330: H264enc

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/mf/tests/transform.c:
> +    for (desc = &expect_codec_api_attributes[0]; desc->key; ++desc)
> +    {
> +        PROPVARIANT propvar;
> +        char buffer[1024];
> +        VARIANT var;
> +        GUID guid;
> +
> +        hr = ICodecAPI_GetValue(codec_api, desc->key, &var);
> +        todo_wine_if(desc->todo)
> +        ok(hr == S_OK, "%s is missing.\n", debugstr_a(desc->name));
> +        if (hr != S_OK)
> +            continue;
> +        hr = VariantToPropVariant(&var, &propvar);
> +        ok(hr == S_OK, "VariantToPropVariant returned %#lx.\n", hr);
> +
> +        if (SUCCEEDED(PropVariantToGUID(&propvar, &guid)))
> Do you really need the GUID case?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6330#note_79385