[FFmpeg-devel] Google Summer of Code participation

[Thilo Borgmann]

Ronald S. Bultje schrieb:
> Hi,
>
> On Tue, Apr 7, 2009 at 10:15 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
>   
>> On Tue, Apr 07, 2009 at 02:51:08PM +0200, Thilo Borgmann wrote:
>>     
>>> Ronald S. Bultje schrieb:
>>>       
>>>> @@ -89,7 +92,9 @@ static int sp5x_decode_frame(AVCodecContext *avctx,
>>>>      recoded[j++] = 0xD9;
>>>>
>>>>      avctx->flags &= ~CODEC_FLAG_EMU_EDGE;
>>>> -    i = ff_mjpeg_decode_frame(avctx, data, data_size, recoded, j);
>>>> +    avpkt_recoded.data = recoded;
>>>> +    avpkt_recoded.size = j;
>>>> +    i = ff_mjpeg_decode_frame(avctx, data, data_size, &avpkt_recoded);
>>>>
>>>>      av_free(recoded);
>>>>
>>>> That leaves some fields uninitialized, you should use av_init_packet()
>>>> here also.
>>>>         
> [..]
>   
>>> OK, added the init function in revision 7.
>>>       
> [..]
>   
>> tabs
>>     
>
> Well yeah, applied without that. Good work, Thilo! I'll apply your
> last patch (moving of av_packet_*() functions) after lunch.
>
>
>   
Thanks to you and the other reviewers! ...And I was just about to mail 
revision 8 without that tab :)
To be correct, I attached revision 8, too, just for the case that 
revision 7 has not been applied.
By the way the tab issue is now history as I figured out to configure my 
editor...

I also attached the patch for the API changes in ffplay.c and ffmpeg.c 
to use the new functions.

TB
-------------- next part --------------
A non-text attachment was scrubbed...
Name: tb.changedAVCodec.rev8.patch
Type: text/x-patch
Size: 98632 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090407/08f4db77/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: tb.APICalls.patch
Type: text/x-patch
Size: 3852 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090407/08f4db77/attachment-0001.bin>