[FFmpeg-devel] Google Summer of Code participation
Tue Apr 7 20:20:59 CEST 2009
On Tue, Apr 07, 2009 at 12:01:35PM -0400, Ronald S. Bultje wrote:
> 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.
you seem not to have noticed but that after lunch patch is a prerequestite
to this patch, your commit made lavc depend on lavf
also, the qualification tasks are to find out if the students can create
patches that pass review, not if there is someone who can fix it halfway
up before commit
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 189 bytes
Desc: Digital signature
More information about the ffmpeg-devel