Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Steven Liu
2016-09-09 9:59 GMT+08:00 Aman Gupta : > Yea this doesn't work at all. I wish someone had bothered to check before > the patch was proposed or merged. > > With "-ss 20 -i file -segment_time 2 -segment_start_number 10 > -initial_offset 20": > > #EXTM3U > #EXT-X-VERSION:3 > #EXT-X-MEDIA-SEQUENCE:0 >

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Aman Gupta
Yea this doesn't work at all. I wish someone had bothered to check before the patch was proposed or merged. With "-ss 20 -i file -segment_time 2 -segment_start_number 10 -initial_offset 20": #EXTM3U #EXT-X-VERSION:3 #EXT-X-MEDIA-SEQUENCE:0 #EXT-X-ALLOW-CACHE:YES #EXT-X-TARGETDURATION:3 #EXTINF:2.

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Aman Gupta
I tried to switch from -initial_offset to -output_ts_offset, but it does not work as expected. In Safari, the generated HLS stream stalls out and stops playing after the first segment. Aman On Thu, Sep 8, 2016 at 5:54 PM, Michael Niedermayer wrote: > On Fri, Sep 09, 2016 at 02:59:52AM +0800, St

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Michael Niedermayer
On Fri, Sep 09, 2016 at 02:59:52AM +0800, Steven Liu wrote: > 2016-09-08 22:52 GMT+08:00 Carl Eugen Hoyos : > > > 2016-09-08 16:01 GMT+02:00 Steven Liu : > > > > > +if (seg->initial_offset > 0) { > > > +av_log(s, AV_LOG_WARNING, "NOTE: the option initial_offset will > > be > > > deprec

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Steven Liu
2016-09-08 22:52 GMT+08:00 Carl Eugen Hoyos : > 2016-09-08 16:01 GMT+02:00 Steven Liu : > > > +if (seg->initial_offset > 0) { > > +av_log(s, AV_LOG_WARNING, "NOTE: the option initial_offset will > be > > deprecated soon," > > "is deprecated" even if you decide not to ignore the value s

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 16:01 GMT+02:00 Steven Liu : > +if (seg->initial_offset > 0) { > +av_log(s, AV_LOG_WARNING, "NOTE: the option initial_offset will be > deprecated soon," "is deprecated" even if you decide not to ignore the value set. Carl Eugen _

[FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Steven Liu
ffmpeg have a generic solution working with all muxer named output_ts_offset, output_ts_offset will instead of initial_offset Signed-off-by: LiuQi --- libavformat/segment.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/segment.c b/libavformat/segment.c index bf29ef8..23e1a