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

Jeff Downs heydowns
Sat Oct 6 21:12:53 CEST 2007


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





More information about the ffmpeg-devel mailing list