[FFmpeg-devel] [PATCH] Implement PAFF in H.264

Mark Buechler mark.buechler
Sat Oct 6 22:59:09 CEST 2007


Jeff, can you list the currently known bugs with regard to PAFF and if the
changes discussed here have any effect on them?

Thanks, Mark.

On 10/6/07, Jeff Downs <heydowns at borg.com> wrote:
>
> Hi,
>
>
> On Sat, 6 Oct 2007, Rodion Kosovsky wrote:
>
> > In revision 10672 the following define was added:
> >  #define FIELD_OR_MBAFF_PICTURE (FRAME_MBAFF || FIELD_PICTURE)
> >
> > however in revision 10674 on line 4099
> > h->emu_edge_height= (FRAME_MBAFF || FIELD_PICTURE) ? 0 :
> h->emu_edge_width;
> >
> >
> > should it not be changed to:
> > h->emu_edge_height= (FIELD_OR_MBAFF_PICTURE) ? 0 : h->emu_edge_width;
> >
>
>
> Yes. It wasn't originally due to the ordering of all the patches and
> reviews.  If no one makes the change, I'll post a patch in a few days.
>
> There are also a number of preexisting s->picture_structure != PICT_FRAME
> that probably should instead be changed to FIELD_PICTURE.
>
>         -Jeff
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>




More information about the ffmpeg-devel mailing list