[FFmpeg-devel] [PATCH 17/18] avformat/hls: do not care about stream start timestamps

Anssi Hannula anssi.hannula at iki.fi
Fri Jan 3 08:40:01 CET 2014

01.01.2014 04:21, Michael Niedermayer kirjoitti:
> On Mon, Dec 30, 2013 at 01:14:31PM +0200, Anssi Hannula wrote:
>> While selecting a packet to return to caller in read_packet(), the code
>> corrects the timestamps for starting timestamps.
>> However, this is wrong, since for live streams the initial timestamps
>> might differ just because of the time delay between the retrieval of the
>> various Media Playlists.
>> Fortunately, spec 6.2.4 mandates that all variant streams must have
>> matching timestamps, so we do not need to correct for initial
>> timestamps.
>> Drop the correction code.
>> Note that ID3 and WebVTT MPEGTS timestamps were previously ignored, so
>> this code was previously actually needed.
> i assume that also older hls "protocol" versions didnt need this
> if so patch LGTM

Correct, synced timestamps are mandatory in all versions.

Anssi Hannula

More information about the ffmpeg-devel mailing list