[FFmpeg-devel] [PATCH] VAAPI/mpeg2: record s->mb_x

Reimar Döffinger Reimar.Doeffinger
Tue Sep 15 11:11:57 CEST 2009


On Tue, Sep 15, 2009 at 10:48:31AM +0200, Michael Niedermayer wrote:
> On Tue, Sep 15, 2009 at 09:41:38AM +0200, Gwenole Beauchesne wrote:
> > This patch fills in the new VAAPI slice_horizontal_position field. It's 
> > needed for the Intel G45 driver.
> >
> > Upstream libVA GIT (0.31) is needed, or one of the 0.31 versions I provide 
> > here: <http://www.splitted-desktop.com/~gbeauchesne/libva/>
> >
> 
> > Should I add a compile-time version check? e.g.
> > #if VA_CHECK_VERSION(0,31,0)
> > // fill in the new field
> > #endif
> 
> if you are willing to maintain such code ...

I don't see a point in supporting old versions of what I think is beta
code which nobody really uses.
But of course the configure check (we do have one, right?) should be
adjusted to check for this.



More information about the ffmpeg-devel mailing list