[FFmpeg-devel] VA API updates

Gwenole Beauchesne gbeauchesne
Wed Jan 7 10:24:31 CET 2009


Hi,

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. Anyhow, changed as follows:

/**
  * \brief This structure is used as a callback between the FFmpeg
  * library the client video application.
  * This is defined by the client application prior to calling the
  * FFmpeg decode functions. The members are considered read-only from
  * an FFmpeg point of view.
  */

>> @@ -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.




More information about the ffmpeg-devel mailing list