Re: [PATCH 0/5] MR5732: winegstreamer: Get rid of the wg_transform internal output_format.

[Elizabeth Figura (＠zfigura)]

Elizabeth Figura (@zfigura) commented about dlls/winegstreamer/video_decoder.c:
>      IMFMediaType_Release(decoder->output_type);
>  IMFMediaType_AddRef((decoder->output_type = type));
>
> - /* WMV decoder outputs RGB formats with default stride forced to negative, likely a result of internal conversion to DMO media type */
> - if (!decoder->IMediaObject_iface.lpVtbl || FAILED(video_decoder_set_default_stride(decoder->output_type, &output_type)))

Isn't failure from video_decoder_set_default_stride() only exceptional? Falling back to the previous type seems odd here in that case.