Re: [PATCH 0/4] MR6733: Mfplat HTTP byte stream implementation

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mf/tests/mf.c:
> -
>    ok(hr == S_OK, "Got hr %#lx\n", hr);
> -
>    ok(IsEqualGUID(&key, &MF_BYTESTREAM_EFFECTIVE_URL), "Got key %s\n", debugstr_guid(&key));
> -
>    ok(var.vt == VT_LPWSTR, "Got type %d\n", var.vt);
> -
>    todo_wine
> -
>    ok(!lstrcmpW(var.pwszVal, EFFECTIVE_URL), "Got value %s\n", var.pszVal);
> -
>    memset(&key, 0, sizeof(key));
> -
>    PropVariantClear(&var);
> -
> -
>    hr = IMFAttributes_GetItemByIndex(attr, 1, &key, &var);
> -
>    ok(hr == S_OK, "Got hr %#lx\n", hr);
> -
>    ok(IsEqualGUID(&key, &MF_BYTESTREAM_CONTENT_TYPE), "Got key %s\n", debugstr_guid(&key));
> -
>    ok(var.vt == VT_LPWSTR, "Got type %d\n", var.vt);
> -
>    todo_wine
> -
>    ok(!lstrcmpW(var.pwszVal, CONTENT_TYPE), "Got value %s\n", var.pszVal);
> -
>    memset(&key, 0, sizeof(key));
> -
>    PropVariantClear(&var);

Unless you find a use case where indexing matters, I would remove this. Use GetItem() instead.