Re: [PATCH 0/8] MR2093: Add ir50_32 winmm video codec using winegstreamer.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/video_decoder.c:
> -
>        IMFMediaType_Release(decoder->input_type);
> -
>    if (decoder->output_type)
> -
>        IMFMediaType_Release(decoder->output_type);
> -
> -
>    wg_sample_queue_destroy(decoder->wg_sample_queue);
> -
>    free(decoder);
> - }
> -
> - return refcount;
>
> }
>
> static HRESULT WINAPI transform_GetStreamLimits(IMFTransform *iface, DWORD *input_minimum,
>          DWORD *input_maximum, DWORD *output_minimum, DWORD *output_maximum)
>  {
> - FIXME("iface %p, input_minimum %p, input_maximum %p, output_minimum %p, output_maximum %p.\n",
> - TRACE("iface %p, input_minimum %p, input_maximum %p, output_minimum %p, output_maximum %p.\n",

I think you should keep all the methods you're not using in the client as stubs for now. So these and the media type enumeration methods. It'll avoid dead code, make the changes smaller, and easier to get reviewed and merged.