[FFmpeg-devel] [PATCH] H264 DXVA2 implementation

Laurent Aimar fenrir
Sun Jan 10 22:05:05 CET 2010


On Sun, Jan 10, 2010, Reimar D?ffinger wrote:
> On Sun, Jan 10, 2010 at 08:48:46PM +0100, Laurent Aimar wrote:
> > Hi,
> > 
> > On Sun, Jan 10, 2010, Reimar D?ffinger wrote:
> > > Which of course means the requirements for FFmpeg change from "requires a C99 compatible compiler" to
> > > "requires a compiler compatible to some bastardized mix of C99 and whatever Microsoft calls C", not to
> > > mention that you'd probably also need -mms-bitfields.
> > > And with all that compiler checks to make sure we don't mess up too much if any of these are not available.
> > 
> >  I can also send a patch that only use the field bPicEntry and then remove the
> > nameless struct/union in the header (it won't be complete anymore but still
> > usable).
> 
> _I_ think that would be a good solution, adding the appropriate compiler flags
> would be possible, too, but it certainly will be more than one line in configure
> (at least: Makefile so it is only used for that file, configure: only check
> if the flag is available, don't add it to cflags, fail or disable dxva if it
> is not supported).

Attached is a diff against dxva2_h264.c to avoid using bitfields in dxva2api.h.
(If oked, I will send a complete patch once I have validated the changes).

-- 
fenrir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-dxva2-nobitfields.patch
Type: text/x-diff
Size: 6121 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100110/f6f3a92e/attachment.patch>



More information about the ffmpeg-devel mailing list