Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

2017-12-09 20:17 GMT+01:00 Anssi Hannula : >> Selecting the first available program, if none was selected >> seems the most natural way. Selecting all or none make >> less sense. > > Hmm.. I wonder if selecting the highest-bandwidth-one might > make more sense This is what

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

(Re-added ffmpeg-devel@) Rainer Hochecker kirjoitti 2017-12-03 09:16: 2017-12-02 16:09 GMT+01:00 Anssi Hannula : Hi, Sorry about the delay. Rainer Hochecker kirjoitti 2017-11-28 00:23: 2017-11-27 22:53 GMT+01:00 Anssi Hannula : Hi, Rainer

[FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

v4: fixed the memleak properly ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

fixed mem leak poined out by Steven --- doc/demuxers.texi | 5 + libavformat/hls.c | 304 -- 2 files changed, 209 insertions(+), 100 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 73dc0feec1..634b122e10 100644 ---

[FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

I tried to implement what you requested ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

Hi, Sorry about the delay. Rainer Hochecker kirjoitti 2017-11-28 00:23: 2017-11-27 22:53 GMT+01:00 Anssi Hannula : Hi, Rainer Hochecker kirjoitti 2017-11-26 12:46: fixed mem leak poined out by Steven --- doc/demuxers.texi | 5 + libavformat/hls.c | 304

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

2017-11-27 22:53 GMT+01:00 Anssi Hannula : > Hi, > > Rainer Hochecker kirjoitti 2017-11-26 12:46: >> >> fixed mem leak poined out by Steven >> >> --- >> doc/demuxers.texi | 5 + >> libavformat/hls.c | 304 >> -- >> 2

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

Hi, Rainer Hochecker kirjoitti 2017-11-26 12:46: fixed mem leak poined out by Steven --- doc/demuxers.texi | 5 + libavformat/hls.c | 304 -- 2 files changed, 209 insertions(+), 100 deletions(-) [...] + +@item load_all_variants +If 0,

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

Variants are presented as programs and can be loaded later by setting discard flags on the program. Currently Kodi chooses the program that best matches the desired bit rate. Rainer ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

2017-11-26 12:04 GMT+01:00 Steven Liu : > 2017-11-26 18:46 GMT+08:00 Rainer Hochecker : >> fixed mem leak poined out by Steven > Hi Rainer, > > I'm not sure that is memleak, but looks like memleak when reading > the code, i see the code always in

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

>> +/* >> + * If this is a live stream and this playlist looks like it is one >> segment >> + * behind, try to sync it up so that every substream starts at the same >> + * time position (so e.g. avformat_find_stream_info() will see packets >> from >> + * all active streams

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

2017-11-26 18:46 GMT+08:00 Rainer Hochecker : > fixed mem leak poined out by Steven Hi Rainer, I'm not sure that is memleak, but looks like memleak when reading the code, i see the code always in hls.c before this patch, but no people report it memleak. If that is

[FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

fixed mem leak poined out by Steven --- doc/demuxers.texi | 5 + libavformat/hls.c | 304 -- 2 files changed, 209 insertions(+), 100 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 73dc0feec1..634b122e10 100644 ---

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

2017-11-24 7:52 GMT+01:00 Rainer Hochecker : >> Shouldn't this be: >> 0 (or -1) for all streams (default), n>0 (or > -1) for the nth stream. >> I guess it is at least possible that the user knows in advance which >> streams the source will offer. >> No? > It is

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

2017-11-25 17:31 GMT+08:00 Rainer Hochecker : > fate runs now without error, sorry for that > > --- > doc/demuxers.texi | 5 + > libavformat/hls.c | 302 > -- > 2 files changed, 207 insertions(+), 100 deletions(-) > >

[FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

fate runs now without error, sorry for that --- doc/demuxers.texi | 5 + libavformat/hls.c | 302 -- 2 files changed, 207 insertions(+), 100 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 73dc0feec1..634b122e10 100644

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

On Thu, Nov 23, 2017 at 11:20:10PM +0100, Rainer Hochecker wrote: > --- > doc/demuxers.texi | 5 + > libavformat/hls.c | 299 > -- > 2 files changed, 204 insertions(+), 100 deletions(-) breaks fate-segment-mp4-to-ts ... Output #0, framecrc,

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

Shouldn't this be: 0 (or -1) for all streams (default), n>0 (or > -1) for the nth stream. I guess it is at least possible that the user knows in advance which streams the source will offer. No? Carl Eugen It is very unlikely that this is known in advance. Also there is no reason why you want

Re: [FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

2017-11-23 23:20 GMT+01:00 Rainer Hochecker : > +@item load_all_variants > +If 0, only the first variant/playlist is loaded on open. All other variants > +get disabled and can be enabled by setting discard option in program. > +Default value is 1. Shouldn't this be: 0 (or

[FFmpeg-devel] [PATCH] hls demuxer: add option to defer parsing of variants

--- doc/demuxers.texi | 5 + libavformat/hls.c | 299 -- 2 files changed, 204 insertions(+), 100 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 73dc0feec1..634b122e10 100644 --- a/doc/demuxers.texi +++