Re: [PATCH 0/5] MR3884: winegstreamer/aac decoder: Correctly handle various number of channels in transform_GetOutputAvailableType().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/mf/tests/transform.c:
> -
>    input_desc[num_channels_index].value.ulVal = num_channels;
> -
> -
>    hr = MFCreateMediaType(&type);
> -
>    ok(hr == S_OK, "got %#lx.\n", hr);
> -
>    init_media_type(type, input_desc, -1);
> -
>    hr = IMFTransform_SetInputType(transform, 0, type, 0);
> -
>    if (num_channels <= 6)
> -
>        ok(hr == S_OK, "got %#lx.\n", hr);
> -
>    else
> -
>        todo_wine ok(hr == MF_E_INVALIDMEDIATYPE, "got %#lx.\n", hr);
> -
>    IMFMediaType_Release(type);
> -
>    if (FAILED(hr) || num_channels > 6)
> -
>    {
> -
>        winetest_pop_context();
> -
>        continue;
> -
>    }

```suggestion:-10+0
        hr = IMFTransform_SetInputType(transform, 0, type, 0);
        IMFMediaType_Release(type);
        if (num_channels <= 6)
            ok(hr == S_OK, "got %#lx.\n", hr);
        else
        {
            todo_wine ok(hr == MF_E_INVALIDMEDIATYPE, "got %#lx.\n", hr);
            winetest_pop_context();
            continue;
        }
```