[FFmpeg-devel] Google Summer of Code participation

[Thilo Borgmann]

Michael Niedermayer schrieb:
>> Index: ffmpeg.c
>> ===================================================================
>> --- ffmpeg.c	(Revision 18211)
>> +++ ffmpeg.c	(Arbeitskopie)
>> @@ -1246,7 +1246,7 @@
>>                      avcodec_get_frame_defaults(&picture);
>>  
>>                      ret = avcodec_decode_video(ist->st->codec,
>> -                                               &picture, &got_picture, ptr, len);
>> +                                               &picture, &got_picture, pkt);
>>     
>
> avcodec_decode_video2() otherwise ABI breaks
>   
ABI?
Ok, do you mean just to change the name (and there is no 
avcodec_decode_video() anymore) or to introduce avcodec_decode_video2() 
next to the other one but use avcodec_decode_video2() everywhere?

> [...]
>   
>> Index: libavcodec/avcodec.h
>> ===================================================================
>> --- libavcodec/avcodec.h	(Revision 18211)
>> +++ libavcodec/avcodec.h	(Arbeitskopie)
>> @@ -842,6 +842,57 @@
>>  #define FF_BUFFER_HINTS_PRESERVE 0x04 // User must not alter buffer content.
>>  #define FF_BUFFER_HINTS_REUSABLE 0x08 // Codec will reuse the buffer (update).
>>  
>> +
>> +typedef struct AVPacket {
>> +    /**
>> +     * Presentation timestamp in time_base units; the time at which the
>>     
>
> moving AVPacket should be a seperate  patch
>   
>
Ok.


Thanks to all reviewers!

TB