[FFmpeg-cvslog] r18351 - in trunk/libavcodec: 4xm.c 8bps.c 8svx.c aac.c aasc.c ac3dec.c adpcm.c adxdec.c alac.c apedec.c asv1.c atrac3.c avcodec.h avs.c bethsoftvideo.c bfi.c bmp.c c93.c cavsdec.c cinepak.c cljr.c...

Thilo Borgmann thilo.borgmann
Thu Jul 8 20:39:23 CEST 2010


Am 08.07.10 20:23, schrieb Diego Biurrun:
> On Tue, Apr 07, 2009 at 05:59:51PM +0200, rbultje wrote:
>>
>> Log:
>> Implement avcodec_decode_video2(), _audio3() and _subtitle2() which takes an
>> AVPacket argument rather than a const uint8_t *buf + int buf_size. This allows
>> passing of packet-specific flags from demuxer to decoder, such as the keyframe
>> flag, which appears necessary to playback corePNG P-frames.
>>
>> --- trunk/libavcodec/rl2.c	Tue Apr  7 17:37:26 2009	(r18350)
>> +++ trunk/libavcodec/rl2.c	Tue Apr  7 17:59:50 2009	(r18351)
>> @@ -181,8 +181,10 @@ static av_cold int rl2_decode_init(AVCod
>>   */
>>  static int rl2_decode_frame(AVCodecContext *avctx,
>>                                void *data, int *data_size,
>> -                              const uint8_t *buf, int buf_size)
>> +                              AVPacket *avpkt)
>>  {
>> +    const uint8_t *buf = avpkt->data;
>> +    int buf_size = avpkt->size;
>>      Rl2Context *s = avctx->priv_data;
> 
> Here (and probably in many other places) the function signature is
> changed without updating the doxygen documentation.  Please update
> the doxygen comments to match.

IIRC this was one of the patches committed during my GSoC qualification
task (which should also be part of the commit message btw.).

I will have a look at all the files again and change the doxy's
accordingly at the weekend.

-Thilo



More information about the ffmpeg-cvslog mailing list