[FFmpeg-devel] [PATCH] lavc/videotoolbox: fix threaded decoding

AVHWAccel.end_frame can run on a worker thread. The assumption of the frame threading code is that the worker thread will change the AVFrame image data, not the AVFrame fields. So the AVFrame fields are not synced back to the main thread. But this breaks videotoolbox due to its special

Re: [FFmpeg-devel] [PATCH 4/4] af_silencedetect : fix missing log silence_end at EOS

On Fri, Feb 02, 2018 at 10:59:51AM +, Gaullier Nicolas wrote: > Attached patch fixes missing log of "silence_end" when the silence continue > until the end of the stream. > Nicolas Gaullier > af_silencedetect.c | 14 +++--- > 1 file changed, 11 insertions(+), 3 deletions(-) >

Re: [FFmpeg-devel] [PATCH] avcodec: do not use init_static_data on some codecs

On Sat, Feb 03, 2018 at 01:36:37AM +0700, Muhammad Faiz wrote: > They don't modify AVCodec, no needs to call it at register. They will be > wasteful if these codecs are unused. Instead, call static data initialization > at codecs' init. > > Benchmark: > old: 51281340 decicycles in

Re: [FFmpeg-devel] [PATCH 2/4] af_silencedetect : fix accuracy of silence_start

On Fri, Feb 02, 2018 at 11:01:20AM +, Gaullier Nicolas wrote: > (attachement + email object fixed : sorry about that x2) > Attached patch fixes accuracy of silence_start. > The benefit is mostly noticeable when the silence starts at the very > beginning (ie. silence_start=0 exactly). >

[FFmpeg-devel] Fix ctts_index calculation

From bb376fd2de5da5f9ecdef41621a579252b899d7d Mon Sep 17 00:00:00 2001 From: Xiaohan Wang Date: Fri, 2 Feb 2018 17:33:56 -0800 Subject: [PATCH] Fix ctts_index calculation An index should never be equal to the count. Hence we must make sure *ctts_index < ctts_count. ---

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: sort codec_descriptors

On Sat, Feb 03, 2018 at 02:03:49AM +0100, Michael Niedermayer wrote: > On Sat, Feb 03, 2018 at 01:35:34AM +0700, Muhammad Faiz wrote: > > Use bsearch on avcodec_descriptor_get(). > > > > Signed-off-by: Muhammad Faiz > > --- > > libavcodec/codec_desc.c | 1103 > >

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: sort codec_descriptors

On Sat, Feb 03, 2018 at 01:35:34AM +0700, Muhammad Faiz wrote: > Use bsearch on avcodec_descriptor_get(). > > Signed-off-by: Muhammad Faiz > --- > libavcodec/codec_desc.c | 1103 > --- > 1 file changed, 563 insertions(+), 540

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

On Sat, Feb 3, 2018 at 6:34 AM, Hendrik Leppkes wrote: > Am 02.02.2018 11:58 nachm. schrieb "Muhammad Faiz" : > > On Sat, Feb 3, 2018 at 1:55 AM, Hendrik Leppkes wrote: >> On Fri, Feb 2, 2018 at 7:49 PM, Muhammad Faiz

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

Am 02.02.2018 11:58 nachm. schrieb "Muhammad Faiz" : On Sat, Feb 3, 2018 at 1:55 AM, Hendrik Leppkes wrote: > On Fri, Feb 2, 2018 at 7:49 PM, Muhammad Faiz wrote: >> On Fri, Feb 2, 2018 at 10:23 PM, Josh de Kock wrote:

[FFmpeg-devel] [PATCH] Fix crash in join filter

Previously if ff_outlink_frame_wanted() returned 0 it could dereference a null pointer when trying to read nb_samples. --- libavfilter/af_join.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavfilter/af_join.c b/libavfilter/af_join.c index cf5131e8dc..4f86e13558 100644 ---

Re: [FFmpeg-devel] [PATCH] avcodec/ffv1: Support for RGBA64 and GBRAP16

On Thu, Feb 01, 2018 at 01:43:00PM +0100, Jerome Martinez wrote: > Add support for 16-bit/component RGB with Alpha encoding and decoding in > FFV1, both RGBA64 and GBRAP16 for encoding, GBRAP16 for decoding. > > Resulting bitstream was tested about lossless encoding/decoding by the > compression

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

On Sat, Feb 3, 2018 at 1:55 AM, Hendrik Leppkes wrote: > On Fri, Feb 2, 2018 at 7:49 PM, Muhammad Faiz wrote: >> On Fri, Feb 2, 2018 at 10:23 PM, Josh de Kock wrote: >>> On 1 Feb 2018, at 18:51, Muhammad Faiz

Re: [FFmpeg-devel] fail to seek key video packet

On Fri, Feb 02, 2018 at 12:02:29 -0300, James Almer wrote: > > I use following command to test ffmpeg-2.8.1: > > This list is for development of ffmpeg, not for user support or bug reports. > > Use the ffmpeg-user list for user support, and http://trac.ffmpeg.org/ > to report bugs. And please

Re: [FFmpeg-devel] [PATCH v4] avcodec/libopus: support disabling phase inversion.

On Thu, Feb 01, 2018 at 22:47:55 +0100, Menno de Gier wrote: > From: Menno > > Fix crash on decoder options. I don't think this was meant to be the final commit message? If you want to add remarks to your patch in the email, add them directly after "---" there: >

Re: [FFmpeg-devel] [PATCH] FFV1: make RGB48 support as non-experimental

On Thu, Feb 01, 2018 at 01:51:17PM +0100, Jerome Martinez wrote: > On 05/01/2018 11:18, Jerome Martinez wrote: > >1 year without RGB48 related patches, tested by a couple of users, tested > >with a FFV1 conformance checker, I suggest that FFV1 RGB48 support in > >FFmpeg does not mandate anymore

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mpegenc - accept PCM_DVD streams

On Thu, Feb 01, 2018 at 07:52:55PM +0530, Gyan Doshi wrote: > On 1/29/2018 6:53 PM, Gyan Doshi wrote: > > > >On 1/29/2018 3:47 PM, Carl Eugen Hoyos wrote: > > > >>How did you test this patch? > > > >By remuxing with patched ffmpeg and then checking with ffprobe, but not > >ffplay! > ... > >Sorry,

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mpegenc - accept PCM_DVD streams

On Thu, Feb 01, 2018 at 07:52:55PM +0530, Gyan Doshi wrote: > On 1/29/2018 6:53 PM, Gyan Doshi wrote: > > > >On 1/29/2018 3:47 PM, Carl Eugen Hoyos wrote: > > > >>How did you test this patch? > > > >By remuxing with patched ffmpeg and then checking with ffprobe, but not > >ffplay! > ... > >Sorry,

Re: [FFmpeg-devel] [PATCH v4] avcodec/libopus: support disabling phase inversion.

On Thu, Feb 01, 2018 at 10:47:55PM +0100, Menno de Gier wrote: > From: Menno > > Fix crash on decoder options. > > Signed-off-by: Menno This is not the correct commit message for the patch, that just explains what is different versus the last posted

[FFmpeg-devel] [PATCH] avcodec/utvideodec: Fix bytes left check in decode_frame()

Fixes: out of array read Fixes: poc-2017.avi Found-by: GwanYeong Kim Signed-off-by: Michael Niedermayer --- libavcodec/utvideodec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/utvideodec.c b/libavcodec/utvideodec.c

[FFmpeg-devel] [PATCH] libavformat/hls: Support metadata updates from subdemuxers

From: Richard Shaffer If a subdemuxer has the updated metadata event flag set, the metadata is copied to the corresponding stream. The flag is cleared on the subdemuxer and the appropriate event flag is set on the stream. --- This is iteration #2. In this version, we don't

Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

On 2/2/2018 5:20 PM, Derek Buitenhuis wrote: > On 2/2/2018 7:46 PM, James Almer wrote: >> Pushed as is. Removing the two lines after printing bogus values for >> four releases is imo not nice. For starters, it can't be backported. >> >> They will be removed alongside AVStream->codec in the future.

Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

On 2/2/2018 7:46 PM, James Almer wrote: > Pushed as is. Removing the two lines after printing bogus values for > four releases is imo not nice. For starters, it can't be backported. > > They will be removed alongside AVStream->codec in the future. fftools/ffprobe.c:2918:1: error: unknown type

Re: [FFmpeg-devel] [PATCH] ffprobe: Initialize coded_width/height

On 1/25/2018 3:26 AM, Li, Zhong wrote: >>> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On >> Behalf >>> Of Hendrik Leppkes >>> Sent: Friday, January 19, 2018 6:10 PM >>> To: FFmpeg development discussions and patches >>> >>> Subject: Re: [FFmpeg-devel]

[FFmpeg-devel] [PATCH v4 7/7] lavc/bsf: make BSF iteration the same as other iterators

--- fftools/cmdutils.c | 2 +- libavcodec/avcodec.h | 6 +- libavcodec/bitstream_filter.c | 4 ++-- libavcodec/bitstream_filters.c | 29 ++--- 4 files changed, 26 insertions(+), 15 deletions(-) diff --git a/fftools/cmdutils.c

[FFmpeg-devel] [PATCH v4 6/7] cmdutils: make use of new iteration APIs

--- fftools/cmdutils.c | 122 +++-- 1 file changed, 43 insertions(+), 79 deletions(-) diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index 6920ca0..9ecc51b 100644 --- a/fftools/cmdutils.c +++ b/fftools/cmdutils.c @@ -1250,19 +1250,11 @@ int

[FFmpeg-devel] [PATCH v4 5/7] lavd: add new API for iterating input and output devices

This also adds an avpriv function to register devices in libavformat --- Makefile | 3 +- configure| 27 +-- libavdevice/.gitignore | 2 + libavdevice/alldevices.c | 181 --- libavdevice/avdevice.c | 46

[FFmpeg-devel] [PATCH v4 4/7] lavf: move fifo test muxer into separate file

This fixes the fate-fifo-muxer test. --- libavformat/Makefile | 2 +- libavformat/allformats.c | 1 + libavformat/fifo_test.c| 150 + libavformat/tests/fifo_muxer.c | 115 +-- 4 files changed, 154

[FFmpeg-devel] [PATCH v4 3/7] lavf: add new API for iterating muxers and demuxers

--- Makefile | 3 +- configure| 6 +- doc/APIchanges | 7 +- libavformat/.gitignore | 2 + libavformat/allformats.c | 866 --- libavformat/avformat.h | 31 ++ libavformat/format.c | 68 +---

[FFmpeg-devel] [PATCH v4 1/7] lavc: add new API for iterating codecs and codec parsers

Based on an unfinished patch by atomnuker. --- Makefile |3 +- configure | 12 +- doc/APIchanges |4 + libavcodec/.gitignore |2 + libavcodec/allcodecs.c | 1473 libavcodec/avcodec.h | 31 +

[FFmpeg-devel] [PATCH v4 2/7] lavf/rtp: replace linked list with array

--- libavformat/allformats.c | 4 -- libavformat/rdt.c| 9 +--- libavformat/rdt.h| 5 -- libavformat/rtpdec.c | 138 ++- libavformat/rtpdec.h | 25 +++-- 5 files changed, 100 insertions(+), 81 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] libavformat/aac: Parse ID3 tags between ADTS frames.

On Fri, Feb 2, 2018 at 5:52 AM, James Almer wrote: > On 2/1/2018 11:37 PM, rshaf...@tunein.com wrote: >> From: Richard Shaffer >> >> While rare, ID3 tags may be inserted between ADTS frames. This change enables >> parsing them and setting the appropriate

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

On Fri, Feb 2, 2018 at 7:49 PM, Muhammad Faiz wrote: > On Fri, Feb 2, 2018 at 10:23 PM, Josh de Kock wrote: >> >>> On 1 Feb 2018, at 18:51, Muhammad Faiz wrote: >>> On Thu, Feb 1, 2018 at 3:25 AM, Josh de Kock wrote:

Re: [FFmpeg-devel] [PATCH] avcodec: do not use init_static_data on some codecs

On Sat, 3 Feb 2018 01:36:37 +0700 Muhammad Faiz wrote: > They don't modify AVCodec, no needs to call it at register. They will be > wasteful if these codecs are unused. Instead, call static data initialization > at codecs' init. > > Benchmark: > old: 51281340 decicycles in

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

On Fri, Feb 2, 2018, at 6:53 PM, James Almer wrote: > On 2/2/2018 12:23 PM, Josh de Kock wrote: > > > >> On 1 Feb 2018, at 18:51, Muhammad Faiz wrote: > >> > >>> On Thu, Feb 1, 2018 at 3:25 AM, Josh de Kock wrote: > >>> Also replace linked list with an array.

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

On 2/2/2018 12:23 PM, Josh de Kock wrote: > >> On 1 Feb 2018, at 18:51, Muhammad Faiz wrote: >> >>> On Thu, Feb 1, 2018 at 3:25 AM, Josh de Kock wrote: >>> Also replace linked list with an array. >>> --- >>> configure | 12 +- >>> doc/APIchanges

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

On Fri, Feb 2, 2018 at 10:23 PM, Josh de Kock wrote: > >> On 1 Feb 2018, at 18:51, Muhammad Faiz wrote: >> >>> On Thu, Feb 1, 2018 at 3:25 AM, Josh de Kock wrote: >>> Also replace linked list with an array. >>> --- >>> configure

Re: [FFmpeg-devel] [PATCH v2] avfilter: sort filter list

On Fri, Feb 2, 2018 at 5:36 PM, Nicolas George wrote: > Muhammad Faiz (2018-02-02): >> Move REGISTER_FILTER to FILTER_TABLE in configure. >> Replace linked list with static table, and sort it. >> Use bsearch() on avfilter_get_by_name(). >> Deprecate avfilter_register_all(),

[FFmpeg-devel] [PATCH] avcodec: do not use init_static_data on some codecs

They don't modify AVCodec, no needs to call it at register. They will be wasteful if these codecs are unused. Instead, call static data initialization at codecs' init. Benchmark: old: 51281340 decicycles in avcodec_register_all, 1 runs, 0 skips new: 6738960 decicycles in

[FFmpeg-devel] [PATCH] avcodec/codec_desc: sort codec_descriptors

Use bsearch on avcodec_descriptor_get(). Signed-off-by: Muhammad Faiz --- libavcodec/codec_desc.c | 1103 --- 1 file changed, 563 insertions(+), 540 deletions(-) diff --git a/libavcodec/codec_desc.c b/libavcodec/codec_desc.c index

Re: [FFmpeg-devel] [PATCH 1/2] swscale: fix dithers table for DITHER_COPY macro

W dniu 24.10.2017 o 10:02, Mateusz pisze: > The Bayer matrix 8x8 used in DITHER_COPY macro is table dithers[5]. > Remaining dithers[] matrixes are generated from this matrix by > downshift or upshift. > > This patch fixes dithers[6] and dithers[7] matrixes -- they were > too dark. > >

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

> On 2 Feb 2018, at 15:41, Rostislav Pehlivanov wrote: > >> On 31 January 2018 at 20:25, Josh de Kock wrote: >> >> Also replace linked list with an array. >> --- >> configure | 12 +- >> doc/APIchanges |4 + >>

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

On 31 January 2018 at 20:25, Josh de Kock wrote: > Also replace linked list with an array. > --- > configure | 12 +- > doc/APIchanges |4 + > libavcodec/.gitignore |2 + > libavcodec/allcodecs.c | 1473 -- >

Re: [FFmpeg-devel] [PATCH v3 1/6] lavc: add new API for iterating codecs and codec parsers

> On 1 Feb 2018, at 18:51, Muhammad Faiz wrote: > >> On Thu, Feb 1, 2018 at 3:25 AM, Josh de Kock wrote: >> Also replace linked list with an array. >> --- >> configure | 12 +- >> doc/APIchanges |4 + >> libavcodec/.gitignore |2

Re: [FFmpeg-devel] fail to build ffmpeg with icc 14

On 2/2/2018 12:04 PM, wm4 wrote: > On Fri, 2 Feb 2018 21:50:00 +0800 (CST) > qw wrote: > >> Hi, >> >> >> I use the following command to build ffmpeg: >> >> >> ./configure --cc=/opt/intel/bin/icc --enable-version3 --enable-asm >> --enable-avfilter --disable-static

Re: [FFmpeg-devel] fail to seek key video packet

On 2/2/2018 11:30 AM, qw wrote: > Hi, > > > I use following command to test ffmpeg-2.8.1: This list is for development of ffmpeg, not for user support or bug reports. Use the ffmpeg-user list for user support, and http://trac.ffmpeg.org/ to report bugs.

Re: [FFmpeg-devel] fail to build ffmpeg with icc 14

On Fri, 2 Feb 2018 21:50:00 +0800 (CST) qw wrote: > Hi, > > > I use the following command to build ffmpeg: > > > ./configure --cc=/opt/intel/bin/icc --enable-version3 --enable-asm > --enable-avfilter --disable-static --enable-shared --enable-gpl > --enable-nonfree

[FFmpeg-devel] fail to seek key video packet

Hi, I use following command to test ffmpeg-2.8.1: ffmpeg -debug_ts -ss 10 -t 1 -i hanma.ts -f null /dev/null ffmpeg version 2.8.1 Copyright (c) 2000-2015 the FFmpeg developers built with icc (ICC) 14.0.2 20140120 configuration: --cc=/opt/intel/bin/icc --enable-version3 --enable-asm

Re: [FFmpeg-devel] [PATCH] libavformat/aac: Parse ID3 tags between ADTS frames.

On 2/1/2018 11:37 PM, rshaf...@tunein.com wrote: > From: Richard Shaffer > > While rare, ID3 tags may be inserted between ADTS frames. This change enables > parsing them and setting the appropriate metadata updated event flag. > --- > I have encountered a streaming provider

[FFmpeg-devel] fail to build ffmpeg with icc 14

Hi, I use the following command to build ffmpeg: ./configure --cc=/opt/intel/bin/icc --enable-version3 --enable-asm --enable-avfilter --disable-static --enable-shared --enable-gpl --enable-nonfree --prefix=/usr/local/ --extra-cflags='-I/usr/local/include --extra-ldflags='-L/usr/local/lib

Re: [FFmpeg-devel] [PATCH] libavformat/hls: Support metadata updates from subdemuxers

On Thu, 1 Feb 2018 23:50:02 -0800 Richard Shaffer wrote: > On Thu, Feb 1, 2018 at 10:18 PM, wm4 wrote: > > On Thu, 1 Feb 2018 18:44:34 -0800 > > rshaf...@tunein.com wrote: > > > >> From: Richard Shaffer > >> > >> If a

Re: [FFmpeg-devel] [PATCH] libavformat/aac: Parse ID3 tags between ADTS frames.

On Fri, 2 Feb 2018 00:06:47 -0800 Richard Shaffer wrote: > On Thu, Feb 1, 2018 at 10:02 PM, wm4 wrote: > > On Thu, 1 Feb 2018 18:37:45 -0800 > > rshaf...@tunein.com wrote: > > > >> From: Richard Shaffer > >> > >> While rare,

[FFmpeg-devel] [PATCH 2/4] af_silencedetect : fix accuracy of silence_start

(attachement + email object fixed : sorry about that x2) Attached patch fixes accuracy of silence_start. The benefit is mostly noticeable when the silence starts at the very beginning (ie. silence_start=0 exactly). Nicolas Gaullier 0002-avfilter-af_silencedetect-fix-silence_start-accuracy.patch

[FFmpeg-devel] [PATCH 2/4] af_silencedetect : fix accuracy of silence_start)

(email object fixed : sorry about that) Attached patch fixes accuracy of silence_start. The benefit is mostly noticeable when the silence starts at the very beginning (ie. silence_start=0 exactly). Nicolas Gaullier ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH 4/4] af_silencedetect : fix missing log silence_end at EOS

Attached patch fixes missing log of "silence_end" when the silence continue until the end of the stream. Nicolas Gaullier 0004-avfilter-af_silencedetect-fix-missing-log-silence_en.patch Description: 0004-avfilter-af_silencedetect-fix-missing-log-silence_en.patch

[FFmpeg-devel] [PATCH 1/4] af_silencedetect : add mono mode

Hi! Attached patch adds a "mono mode", I mean detection of silence in any of the channels (instead of all of them simultaneously). The channel number is detected and logged, and side metadata names are suffixed with the channel number (NB: in non-mono mode, the logging and metadata remain

[FFmpeg-devel] [PATCH 3/4] af_silencedetect : fix when silence_start=0

Attached patch fixes silence_start=0 meaning "no silence start" Nicolas Gaullier 0003-avfilter-af_silencedetect-fix-silence_start-0-meanin.patch Description: 0003-avfilter-af_silencedetect-fix-silence_start-0-meanin.patch ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH 2/4] af_silencedetect : add mono mode

Attached patch fixes accuracy of silence_start. The benefit is mostly noticeable when the silence starts at the very beginning (ie. silence_start=0 exactly). Nicolas Gaullier 0002-avfilter-af_silencedetect-fix-silence_start-accuracy.patch Description:

Re: [FFmpeg-devel] [PATCH v2] avfilter: sort filter list

Muhammad Faiz (2018-02-02): > Move REGISTER_FILTER to FILTER_TABLE in configure. > Replace linked list with static table, and sort it. > Use bsearch() on avfilter_get_by_name(). > Deprecate avfilter_register_all(), avfilter_register(), and > avfilter_next(). > Add avfilter_iterate() as a

[FFmpeg-devel] [PATCH v2] avfilter: sort filter list

Move REGISTER_FILTER to FILTER_TABLE in configure. Replace linked list with static table, and sort it. Use bsearch() on avfilter_get_by_name(). Deprecate avfilter_register_all(), avfilter_register(), and avfilter_next(). Add avfilter_iterate() as a replacement for avfilter_next(). Signed-off-by:

Re: [FFmpeg-devel] [PATCH] libavformat/aac: Parse ID3 tags between ADTS frames.

On Thu, Feb 1, 2018 at 10:02 PM, wm4 wrote: > On Thu, 1 Feb 2018 18:37:45 -0800 > rshaf...@tunein.com wrote: > >> From: Richard Shaffer >> >> While rare, ID3 tags may be inserted between ADTS frames. This change enables >> parsing them and setting the