Re: [PATCH 0/4] MR6330: H264enc

[Rémi Bernon (＠rbernon)]

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?