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

Qian Hong fracting at gmail.com
Fri May 8 10:31:10 CDT 2015


Hi Steve,

On Fri, May 8, 2015 at 3:03 PM, Steve Lhomme <robux4 at videolabs.io> wrote:
> Anyone has anything to say about this ?

Thanks for your contribution, glad to see VLC people!

I'm not the right person to ack your patch, but I can help to point
out a few tiny formatting issues:

- The patch title could be rename to "include: Add D3D11 video decoder
interfaces"
Here is an example of this stile of patch title: `git show
e35f82665cfdb40231c2e0a8a41c81642273160d`

- There are a couple of trailing spaces in the code which should be
removed, for example:
 94 +    UINT NumEncryptedBytesAtBeginning;<spaces>
176 +typedef struct D3D11_VIDEO_PROCESSOR_CUSTOM_RATE<space>
and about 90+ more lines with trailing spaces.

Could you fix that and resubmit?

Thanks!

-- 
Regards,
Qian Hong

-
http://www.winehq.org



More information about the wine-devel mailing list