[FFmpeg-devel] Google Summer of Code participation

Ronald S. Bultje rsbultje
Tue Apr 7 18:01:35 CEST 2009


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.

Ronald



More information about the ffmpeg-devel mailing list