Re: [PATCH] add the interfaces necessary to use the D3D11 video decoder

[Matteo Bruni]

2015-05-08 9:03 GMT+02:00 Steve Lhomme :
> Anyone has anything to say about this ?
>
> Thanks
>
> On Tue, May 5, 2015 at 1:52 PM, Steve Lhomme  wrote:
> > This is needed by VLC.

Hi Steve,

some parts of the patch are clearly copypasted from the MS idl, which
is a deal breaker. While rewriting those parts try to use the wined3d
style, like putting the '*' next to the variable instead of the type.

Another potential issue (noticed by Henri) is that
D3D11_VIDEO_PROCESSOR_COLOR_SPACE uses bitfields which aren't
generally portable. I don't know if their implementation on Windows is
compatible with the one used by the OSs and architectures we support.
If that's not the case then we have a problem...