[FFmpeg-devel] [PATCH] Tell calling application about PAFF fields

Michael Niedermayer michaelni
Wed Oct 10 01:13:38 CEST 2007


On Wed, Oct 10, 2007 at 12:22:26AM +0200, Carl Eugen Hoyos wrote:
> Michael Niedermayer wrote:
> >> I tested the PAFF patch with mplayer, and it works well, but A/V sync
> >> breaks because frame rate has to be doubled for PAFF fields.
> >>
> >> Attached patch is a possibility to solve this problem, please comment.
> > 
> > interlaced_frame should be 1 for all interlaced frames not just PAFF fields
> > 
> > it for example could be used by the application to decide how to display
> > the 2 fields (at the same time or not ...) or how to convert 4:2:0 yuv to rgb
> I had similar ideas since I sent the first patch.
> Second try attached.
> Please comment, Carl Eugen

i think the patch is ok, feel free to commit
its at least a lot better than never setting interlaced_frame
though i do not know if it is completely correct, i think the picture
timing SEI would have to be considered too to decide bewteen progressive
and interlaced ... 

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071010/a9e73b50/attachment.pgp>

More information about the ffmpeg-devel mailing list