RetroSearch Browse

Fri Jan 26 12:38:49 CET 2007 · https://ffmpeg.org/pipermail/ffmpeg-devel/2007-January/028057.html

Hi Michael,

On Thu, 2007-01-25 at 20:52 +0100, Michael Niedermayer wrote:
[...]
> > So, I tried to add support for coded_frame... I came up with the
> > attached patch, but I do not know if it is correct (in particular, is it
> > ok to set coded_frame = &t->po?).
> 
> i guess so / patch looks ok
Good, thanks. I'll extend it and resubmit in a hopefully short time.

> [...]
> > Index: ffmpeg/libavcodec/h264enc.c
> > ===================================================================
> > --- ffmpeg.orig/libavcodec/h264enc.c	2007-01-25 16:32:12.840459312 +0100
> > +++ ffmpeg/libavcodec/h264enc.c	2007-01-25 16:32:27.045299848 +0100
> > @@ -2044,6 +2044,10 @@
> >      t->frame_num++;
> >      if (isIDR)
> >          t->IDR_frame_num++;
> > +
> > +    avctx->coded_frame = (AVFrame *)&t->po;
> 
> why that cast?
Opsss...  I introduced it when doing some experiments with the code, and
forgot to remove it.

			Thanks,
				Luca
-- 
_____________________________________________________________________________
Copy this in your signature, if you think it is important:
                               N O    W A R ! ! !

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Showing content from https://ffmpeg.org/pipermail/ffmpeg-devel/2007-January/028057.html below:

[Ffmpeg-devel] ffh264 & coded_frame