[FFmpeg-devel] [PATCH] avcodec/h264_parser: set missing pts for top/bottom field frames

Paul B Mahol onemda at gmail.com
Wed Sep 14 20:35:29 EEST 2016


On 9/14/16, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
> 2016-09-14 18:11 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
>> On Wednesday, September 14, 2016, Carl Eugen Hoyos wrote:
>>
>>> 2016-09-14 16:42 GMT+02:00 Paul B Mahol <onemda at gmail.com>:
>>> > Adopted from 4eb49fdde8f84d54a763cfb5d355527b525ee2bf revert.
>>>
>>> I am of course not against this patch but wasn't the reason for the
>>> old code in utils.c that our h264 parser has insufficiencies?
>>
>> Like what? Only h264 have those fields. Doing it outside of
>> parser seems strange.
>
> I thought - and please correct me if this is wrong - that this code was
> always (only?) needed because our h264 parser does not do what
> the specification requires to correctly determine h264 timestamps.

This code appears to be needed for field based h264 streams.
The code that was reverted was added in 2009. and than there was no
avctx->pkt_timebase.
I see nothing wrong doing this in parser instead of in generic code.


More information about the ffmpeg-devel mailing list