[FFmpeg-devel] [PATCH 07/19 v2] avformat/hls: parse ID3 timestamps for elementary audio streams

Anssi Hannula anssi.hannula at iki.fi
Sun Apr 6 16:58:13 CEST 2014


05.04.2014 15:20, Anssi Hannula kirjoitti:
> HLS provides MPEG TS timestamps via ID3 tags in the beginning of each
> segment of elementary audio streams.
> 
> v2: fix issues with streams that have multiple ID3 tags
> 
> Signed-off-by: Anssi Hannula <anssi.hannula at iki.fi>
> ---
> 
> Hi all,
> 
> and sorry for the delay in getting this patchset in.
> 
> I plan to apply the patchset (minus subtitle parts) later today or
> tomorrow, unless any issues are found (by me or anyone here).

Done, now let's see how much broke.

> The only major change is that I've dropped the separate hls_id3_audio
> subdemuxer again due to the various timestamp issues and the ID3
> timestamp mangling is now done in the hls demuxer itself. So this is
> similar to what the first version of the patchset had, except that the
> issues with it have been fixed.
> 
> The patch in this message is the major updated part of this patchset
> version. intercept_id3() contains most of the new code, other parts
> were mostly present in one of the previous versions of the patchset.
> 
> For other parts I've applied the changes previously mentioned and/or
> discussed.
> 
> The full patchset is at:
> https://github.com/anssih/FFmpeg/commits/hls-patchset-wip-3
> and with extra commits (mostly subtitles) that will remain in TODO:
> https://github.com/anssih/FFmpeg/commits/hls-patchset-wip-3-extra
> 
> What is missing from the subtitle parts is to see if they could be
> handled in a cleaner way (i.e. without the demuxer reopening hacks),
> and some additional testing. I don't likely have time to work on that
> for at least the next few weeks, though, but patches welcome of course.


-- 
Anssi Hannula


More information about the ffmpeg-devel mailing list