[Mplayer-cvslog] CVS: main/libmpcodecs vd_ffmpeg.c, 1.113, 1.114 mp_image.h, 1.28, 1.29

D Richard Felker III dalias at aerifal.cx
Wed Dec 24 08:01:39 CET 2003


On Mon, Dec 22, 2003 at 06:26:21PM +0100, Michael Niedermayer CVS wrote:
> +#if LIBAVCODEC_BUILD >= 4697
> +    mpi->fields = MP_IMGFIELD_ORDERED;
> +    if(pic->interlaced_frame) mpi->fields |= MP_IMGFIELD_INTERLACED;
> +    if(pic->top_field_first ) mpi->fields |= MP_IMGFIELD_TOP_FIRST;    
> +#endif

This is incorrect. MP_IMGFIELD_ORDERED must NOT be set unless field
order is known! There are many codecs in lavc which do not know field
order, so it should only be set for the ones that do know field order
and export it via pic->top_field_first, i.e. only mpeg2...

Rich




More information about the MPlayer-cvslog mailing list