Re: [PATCH v3 0/4] MR2258: winegstreamer: Support WMV DMO decoder.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/wmv_decoder.c:
> HRESULT wmv_decoder_create(IUnknown *outer, IUnknown **out)
>  {
>      struct wmv_decoder *impl;
> -
>
> HRESULT hr;
>
> TRACE("outer %p, out %p.\n", outer, out);
>
> if (!(impl = calloc(1, sizeof(*impl))))

You should try to create a dummy WMV decoder transform here, like it's done for H264 and other decoders, so that we don't instantiate the class if decoding isn't actually supported.

For instance I think that, VA-API plugins incorrectly advertise support for, and register their decoder with the WMV caps, but it later most often fails to link the transform.