[FFmpeg-devel] VA API updates

[Diego Biurrun]

On Wed, Jan 07, 2009 at 10:24:31AM +0100, Gwenole Beauchesne wrote:
> 
> On Tue, 6 Jan 2009, Diego Biurrun wrote:
> 
> >> +/**
> >> + * \brief This structure is used as a callback between the FFmpeg
> >> + * decoder (vd_) and the MPlayer presentation (vo_) module.
> >> + * This is defined by the client application prior to calling the
> >> + * FFmpeg decode functions.
> >
> > I don't see how these MPlayer-related comments belong here..
> 
> Then, update vdpau.h as well.

On it..

> >> @@ -7148,6 +7157,7 @@
> >>                 ((const char*[]){"Gray","420","422","444"})[sps->chroma_format_idc]
> >>                 );
> >>      }
> >> +
> >>      av_free(h->sps_buffers[sps_id]);
> >>      h->sps_buffers[sps_id]= sps;
> >>      return 0;
> 
> Cosmetics are different in decode_picture_parameter_set() and 
> decode_seq_parameter_set() then. This hunk makes it consistent, visually.

Then send a separate patch.  You are expected not to mix cosmetic
and functional changes.

Diego