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

[Carl Eugen Hoyos]

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.

Carl Eugen