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

[Jeff Downs]

On Sat, 6 Oct 2007, Mark Buechler wrote:

> 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.

Please don't top post.


> 
> 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


Known problems were posted recently. See
http://article.gmane.org/gmane.comp.video.ffmpeg.devel/57399
and
http://article.gmane.org/gmane.comp.video.ffmpeg.devel/57274

The changes discussed here are purely cosmetic. So, no, they won't 
fix/change anything.

As should be obvious by this thread, current SVN does not yet contain 
complete PAFF support.

	-Jeff