[FFmpeg-devel] [PATCH] avformat/spdifenc: handle long TrueHD input_timing gaps

2020-04-12 Thread Anssi Hannula
Some TrueHD streams contain frames that have very long gaps in input_timing fields, while output_timing remains constant-rate. These are likely due to encoding discontinuities of some sort as the TrueHD substream terminator marker is observed before the gap. Such frames trigger a sanity check in t

[FFmpeg-devel] [PATCH] avformat/mux: allow non-monotonic ts with AVFMT_NOTIMESTAMPS

2020-02-20 Thread Anssi Hannula
Allow non-monotonic input timestamps for muxers with no timestamps at all (AVFMT_NOTIMESTAMPS). This is frequently hit when muxing TrueHD with spdifenc as many MKV files use 1ms timestamps while TrueHD frames are shorter than that (1/1200 sec for 48kHz-based and 1/1102.5 sec for 44.1kHz-based rate

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: remove myself as hls demuxer maintainer

2018-10-15 Thread Anssi Hannula
Anssi Hannula kirjoitti 2018-10-15 00:07: --- Unfortunately I haven't really had the time lately to maintain the HLS demuxer properly and I don't see that changing in the near future. So I intend to apply this removal tomorrow. If anyone is interested in taking over, please d

[FFmpeg-devel] [PATCH] MAINTAINERS: remove myself as hls demuxer maintainer

2018-10-14 Thread Anssi Hannula
affic spdif (de)muxers. - Anssi Hannula MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 3dd26e374f..bc2ae13320 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -413,7 +413,6 @@ Muxers/Demuxers: flvenc.c Michael Niederma

Re: [FFmpeg-devel] [PATCH] spdifenc: support ac3 core+eac3 dependent streams

2018-04-03 Thread Anssi Hannula
0xc0) /* fscod */ repeat = eac3_repeat[(pkt->data[4] & 0x30) >> 4]; /* numblkscod */ ctx->hd_buf = av_fast_realloc(ctx->hd_buf, &ctx->hd_buf_size, ctx->hd_buf_filled + pkt->size); Looks good to me. Thanks, -- Anssi Hannula ___

Re: [FFmpeg-devel] [PATCH v2 3/5] avformat/hls: add http_persistent option

2017-12-17 Thread Anssi Hannula
Aman Gupta kirjoitti 2017-12-17 22:41: On Sun, Dec 17, 2017 at 12:34 PM Anssi Hannula wrote: Hi! Aman Gupta kirjoitti 2017-12-13 02:35: > From: Aman Gupta > > This teaches the HLS demuxer to use the HTTP protocols > multiple_requests=1 option, to take advantage of "Connect

Re: [FFmpeg-devel] [PATCH v2 5/5] avformat/hls: add http_multiple option

2017-12-17 Thread Anssi Hannula
v->input = v->input_next; +v->input_next = tmp; Use FFSWAP(). [...] -- Anssi Hannula ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2 3/5] avformat/hls: add http_persistent option

2017-12-17 Thread Anssi Hannula
+ff_format_io_close(c->ctx, &c->playlist_pb); +return ret; +} else if (ret < 0) { +av_log(c->ctx, AV_LOG_WARNING, +"keepalive request failed for '%s', retrying with

Re: [FFmpeg-devel] [PATCH v4 2/2] lavf/hls: add option to defer parsing of variants

2017-12-09 Thread Anssi Hannula
der(). Then we could also drop all the changes you made in add_renditions_to_variant() as well as it would not be called more than once per variant. Or maybe I am missing something? Trying to keep variants, renditions, playlists, streams and programs straight in one's head does tend to cause headaches, after all... + init_playlist(c, pls); + } [...] -- Anssi Hannula ___ 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

2017-12-09 Thread Anssi Hannula
(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 Hochecker kirjoitti 2017-11-26 12:46: fixed

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

2017-12-02 Thread 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 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 Thread Anssi Hannula
n_playlists; i++) { struct playlist *pls = c->playlists[i]; -AVInputFormat *in_fmt = NULL; - [...] -- Anssi Hannula ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] avformat/hls: Add subtitle support

2016-12-12 Thread Anssi Hannula
omething), so some refactoring should be done. -- Anssi Hannula ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/3 v2] avformat/hls: Add missing error check for avcodec_parameters_copy()

2016-11-07 Thread Anssi Hannula
07.11.2016, 01:08, Andreas Cadhalpun kirjoitti: > On 06.11.2016 23:52, Anssi Hannula wrote: >> Signed-off-by: Anssi Hannula >> --- >> >> 07.11.2016, 00:35, Andreas Cadhalpun kirjoitti: >>> On 06.11.2016 22:44, Anssi Hannula wrote: >>>> Signed-o

[FFmpeg-devel] [PATCH 3/3 v2] avformat/hls: Add missing error check for avcodec_parameters_copy()

2016-11-06 Thread Anssi Hannula
Signed-off-by: Anssi Hannula --- 07.11.2016, 00:35, Andreas Cadhalpun kirjoitti: > On 06.11.2016 22:44, Anssi Hannula wrote: >> Signed-off-by: Anssi Hannula >> --- >> libavformat/hls.c | 18 ++ >> 1 file changed, 14 insertions(+), 4 deletions(-) >&

Re: [FFmpeg-devel] [PATCH 2/2] avformat/hls: Fix probing mpegts audio streams that use probing

2016-11-06 Thread Anssi Hannula
05.11.2016, 19:51, Andreas Cadhalpun kirjoitti: > On 05.11.2016 18:47, Andreas Cadhalpun wrote: >> On 05.11.2016 17:39, Anssi Hannula wrote: >>> @@ -1981,6 +1986,11 @@ static int hls_read_packet(AVFormatContext *s, >>> AVPacket *pkt) >>>

[FFmpeg-devel] [PATCH 1/3] avformat/hls: Factor copying stream info to a separate function

2016-11-06 Thread Anssi Hannula
Signed-off-by: Anssi Hannula --- libavformat/hls.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index 3c09dd8..6fb652c 100644 --- a/libavformat/hls.c +++ b/libavformat/hls.c @@ -1528,6 +1528,16 @@ static void

[FFmpeg-devel] [PATCH 2/3 v2] avformat/hls: Fix probing mpegts audio streams that use probing

2016-11-06 Thread Anssi Hannula
robed, are not passed on to the main stream. Fix that by updating the codec parameters if the codec id changes. Signed-off-by: Anssi Hannula --- v2: Added need_context_update = 1 and shortened the av_rescale_q() call to use the new ist pointer. libavformat/hls.c | 16 ++-- 1 file c

[FFmpeg-devel] [PATCH 3/3] avformat/hls: Add missing error check for avcodec_parameters_copy()

2016-11-06 Thread Anssi Hannula
Signed-off-by: Anssi Hannula --- libavformat/hls.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index ce52bf5..a744908 100644 --- a/libavformat/hls.c +++ b/libavformat/hls.c @@ -1528,9 +1528,13 @@ static void

Re: [FFmpeg-devel] [PATCH 2/2] avformat/hls: Fix probing mpegts audio streams that use probing

2016-11-06 Thread Anssi Hannula
05.11.2016, 19:27, Hendrik Leppkes kirjoitti: > On Sat, Nov 5, 2016 at 5:39 PM, Anssi Hannula wrote: >> Commit 04964ac311abe670f ("avformat/hls: Fix missing streams in some >> cases with MPEG TS") caused a regression where subdemuxer streams that >> use probing (

[FFmpeg-devel] [PATCH 2/2] avformat/hls: Fix probing mpegts audio streams that use probing

2016-11-05 Thread Anssi Hannula
robed, are not passed on to the main stream. Fix that by updating the codec parameters if the codec id changes. Signed-off-by: Anssi Hannula --- libavformat/hls.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index 6fb652

Re: [FFmpeg-devel] [PATCH] hls: call avformat_find_stream_info for mpegts subdemuxer

2016-11-05 Thread Anssi Hannula
03.11.2016, 20:34, Andreas Cadhalpun kirjoitti: > On 03.11.2016 11:18, Anssi Hannula wrote: >> Andreas Cadhalpun kirjoitti 2016-11-03 02:12: >>> This fixes probing dts/eac3/mp2 in hls. >>> >>> The problem was introduced in commit >>> 04964ac311abe670

[FFmpeg-devel] [PATCH 1/2] avformat/hls: Factor copying stream info to a separate function

2016-11-05 Thread Anssi Hannula
Signed-off-by: Anssi Hannula --- libavformat/hls.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index 3c09dd8..6fb652c 100644 --- a/libavformat/hls.c +++ b/libavformat/hls.c @@ -1528,6 +1528,16 @@ static void

Re: [FFmpeg-devel] [PATCH] hls: call avformat_find_stream_info for mpegts subdemuxer

2016-11-03 Thread Anssi Hannula
0, 97200, 115200, 3600, 4268, 0xec1235b5, F=0x0, S=1,1, 0x00e000e0 0, 100800, 108000, 3600, 1119, 0x65f51fb7, F=0x0, S=1,1, 0x00e000e0 0, 104400, 104400, 3600, 766, 0x213b78d3, F=0x0, S=1,1, 0x00e000e0 -- Anssi Hannula _

Re: [FFmpeg-devel] [PATCH 3/4] V13 - SCTE-35 support in hlsenc

2016-10-15 Thread Anssi Hannula
ate */ > +int prev_event_state; > +//TODO use AV_BASE64_SIZE to dynamically allocate the array > +char pkt_base64[1024]; > +/* keep context of its parent for log */ > +void *parent; > +/* general purpose str */ > +AVBPrint avbstr; > + > + void (

Re: [FFmpeg-devel] [PATCH] avformat/utils: Fix find_stream_info not considering the extradata it found

2016-07-27 Thread Anssi Hannula
27.07.2016, 02:04, Michael Niedermayer kirjoitti: > On Tue, Jul 26, 2016 at 08:39:03PM +0300, Anssi Hannula wrote: >> 26.07.2016, 17:59, Michael Niedermayer kirjoitti: >>> On Tue, Jul 26, 2016 at 03:56:13PM +0300, Anssi Hannula wrote: >>>> Commit 9200514ad8717c6 (&quo

Re: [FFmpeg-devel] [PATCH] avformat/utils: Fix find_stream_info not considering the extradata it found

2016-07-26 Thread Anssi Hannula
26.07.2016, 17:59, Michael Niedermayer kirjoitti: > On Tue, Jul 26, 2016 at 03:56:13PM +0300, Anssi Hannula wrote: >> Commit 9200514ad8717c6 ("lavf: replace AVStream.codec with >> AVStream.codecpar") merged in commit 6f69f7a8bf6a0d01 changed >> avformat_find_stream

Re: [FFmpeg-devel] avio starting offset and hls regression in 3.1

2016-07-26 Thread Anssi Hannula
25.07.2016, 22:13, Anssi Hannula kirjoitti: > Hi all, > > Commit d0fc5de3a643fe7f974ed14e410c2ac2f4147d7e [1] merged in > commit 81306fd4bdeb5c17d4db771e4fec684773b5790f "hls: eliminate ffurl_* > usage" from libav, which changed the hls demuxer to use AVIOContext > i

[FFmpeg-devel] [PATCH] avformat/utils: Fix find_stream_info not considering the extradata it found

2016-07-26 Thread Anssi Hannula
Commit 9200514ad8717c6 ("lavf: replace AVStream.codec with AVStream.codecpar") merged in commit 6f69f7a8bf6a0d01 changed avformat_find_stream_info() to put the extradata it got from st->parser->parser->split() to st->internal->avctx instead of st->codec (from where it will be later copied to st->co

[FFmpeg-devel] [PATCH] avformat/utils: Fix find_stream_info not considering the extradata it found

2016-07-26 Thread Anssi Hannula
Commit 9200514ad8717c6 ("lavf: replace AVStream.codec with AVStream.codecpar") merged in commit 6f69f7a8bf6a0d01 changed avformat_find_stream_info() to put the extradata it got from st->parser->parser->split() to avctx instead of st->codec. However, in the same function, the "is stream ready?" che

[FFmpeg-devel] avio starting offset and hls regression in 3.1

2016-07-25 Thread Anssi Hannula
d the ranged request. Any other ideas / opinions for the course of action? [1] https://github.com/FFmpeg/FFmpeg/commit/d0fc5de3a643fe7f974ed14 -- Anssi Hannula ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] avformat/hlsenc: pull request for single file mode

2014-07-09 Thread Anssi Hannula
10.07.2014 02:40, Nicolas Martyanoff kirjoitti: > On 2014-07-10 01:32, Anssi Hannula wrote: >> avformat/hlsenc: cleaning >> - looks good to me, and there is no maintainer so I guess this is OK >> - well, actually, maybe you could use more than one word in the subject :) >

Re: [FFmpeg-devel] understanding usage of av_compare_ts() in avformat/hlsenc

2014-07-09 Thread Anssi Hannula
esentation timestamp makes any kind of sense. > > Does anyone know what I am missing here ? I guess you may have gotten this already, but anyway: The end_pts is actually a time length as well, not an actual PTS, since it is just (number_of_segments * length_of_segment). > Thank you

Re: [FFmpeg-devel] avformat/hlsenc: pull request for single file mode

2014-07-09 Thread Anssi Hannula
utput media file" (requires "single_file") -hls_media_url - "output media file url used in the playlist" (requires "single_file" and !hls_base_url) Or would it just be unclear to have both conflicting hls_media_url and hls_base_url? -- Anssi Hannula ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel