[Ffmpeg-devel] [Ffmpeg-devel-old] Questionaboutinterlaced_frame and top_field_first

[Michael Niedermayer]

Hi

On Mon, Jan 30, 2006 at 12:10:18PM +0100, Luca Abeni wrote:
> Hi all,
> 
> On Mon, 2006-01-30 at 10:17 +0100, Luca Abeni wrote:
> [...]
> > So, I think raw_decode() should be fixed to copy the fields form
> > coded_frame to the decoded frame... Or am I misunderstanding how things
> > work?
> > 
> > BTW, I see that raw_decode() does this:
> > static int raw_decode(AVCodecContext *avctx,
> >                             void *data, int *data_size,
> >                             uint8_t *buf, int buf_size)
> > {
> >     RawVideoContext *context = avctx->priv_data;
> >     int bytesNeeded;
> > 
> >     AVPicture * picture = (AVPicture *) data;
> > [...]
> > 
> > I think this "AVPicture * picture" should be changed in "AVFrame *",
> > right?
> > I'll try to patch raw.c in such way, to see if this idea works.
> Ok, I did this, and it works. I attach the patch for libavcodec/raw.c
> 
> If the patch is accepted and applied, I think libavformat/yuv4mpeg.c
> (and maybe other) can be patched to take advantage of it and correctly
> set interlaced_frame and top_field_first.

patch looks ok, can be applied

[...]

-- 
Michael