[FFmpeg-devel] frames not parsed correctly for VOB file (mpeg2)

[Michael Niedermayer]

On Wed, Feb 03, 2010 at 07:40:48PM +0100, Christian Cier-Zniewski wrote:
> Michael Niedermayer schrieb:
>> On Wed, Feb 03, 2010 at 10:52:23AM +0100, Christian Cier-Zniewski wrote:
>>   
>>> Christian Cier-Zniewski schrieb:
>>>
>>>     
>>>> I have analyzed the packet data a bit further and realized that it 
>>>> mostly contains at least two frames, instead of a single frame which 
>>>> would I expect to be correct.
>>>>       
>>
>> they are called field pictures
>>   
> Just before reading your reply I realized that the material is interlaced 
> and that it is correct to return two fields in one packet.
> Thanks for clarifying that. That is what I want. My patch is indeed wrong.
>> i suggest you look in svnlog we did had an alternative implementation
>> of things that did not merge field pictures but i changed it to what we
>> have now because it at that time appeared simpler. I had not considered
>> apparently that one could store timestamps on only the wrong fields.
>>   
> Could you please explain what you mean by "on only the wrong fields"?
> Since it is ok to have the two fields per packet I have no need to change 
> the current implementation.

a packet has its timestamps associated with the start of presentation/decode
the second field is later thus timestamps associated with the second field
can never be returned as they are. (i assume this is the problem with this
file)
Outputing 1 field per packet solves this with the old code for
compute_pkt_fields()
h264 also returns 1 field per packet for PAFF and has to because in h264
field pictures dont have to be paired that is you can have 
frame field frame field field frame field field field frame
ive never seen such a sample but i guess thats just because the encoder
developers all are sane enough still

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- 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/20100203/3de159d7/attachment.pgp>