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

[Thorsten Jordan]

Jeff Downs schrieb:
> On Sat, 6 Oct 2007, Carl Eugen Hoyos wrote:
> 
>> Hi!
>>
>> 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.
>>
>> Carl Eugen
> 
> I was thinking on this as well. I'm guessing mplayer's problem is that the 
> demux/parser only pushes out a field, so it delivers x fields / second 
> instead of x frames / second?
> 
> Another place to potentially fix it is in the 264 parser, making it output 
> frames and field pairs instead of frames and fields.  The docs for the 
> parser interface aren't clear on which behavior is more correct.
> And, you'd still have to deal with unmatched fields (ones that have no 
> pair of opposite parity). 

well here is my opinion as "user" of avcodec, just my 2?.
i'd suggest the latter, let the parser always output field pairs, because
- other decoders do it as well ;-) that is coreavc and also libmpeg2
(here for mpeg2 fields).
- there can be frames and fields in one stream in repeating mix, it
would be simpler for users to always use avcodec_decode_video and decide
by AVFrame->interlaced wether its fields or frames

and btw., congratulations to Jeff, Martin, Carl Eugen, Andreas and
Michael for making ffmpeg even better by implementing/integrating PAFF.

-- 
Greetings, Thorsten