Re: [FFmpeg-devel] Unable to submit patches

2020-02-07 Thread Christopher Degawa
If you are talking about "[FFmpeg-devel] [PATCH] doc/ffmpeg: Fix bug #8204", it seems the list has received it On Sat, Feb 8, 2020 at 12:33 AM Gautam Ramakrishnan wrote: > Hello, > > I wish to contribute to ffmpeg and though of submitting a small patch > to fix a bug listed on the bug tracker. H

[FFmpeg-devel] Unable to submit patches

2020-02-07 Thread Gautam Ramakrishnan
Hello, I wish to contribute to ffmpeg and though of submitting a small patch to fix a bug listed on the bug tracker. However, my patch submission does not reach the mailing list. I have checked all the instructions and use git format-path with git send-email. What should I do? -- - G

[FFmpeg-devel] [PATCH] doc/ffmpeg: Fix bug #8204

2020-02-07 Thread gautamramk
From: Gautam Ramakrishnan Change the documentation of -hwaccels as mentioned in bug #8204 Signed-off-by: Gautam Ramakrishnan --- doc/ffmpeg.texi | 1 + 1 file changed, 1 insertion(+) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index 92337d..c700aa2aee 100644 --- a/doc/ffmpeg.texi +++ b

Re: [FFmpeg-devel] [PATCH v6] avcodec/libvpxenc: add a way to explicitly set temporal layer id

2020-02-07 Thread James Zern
On Fri, Feb 7, 2020 at 10:44 AM Wonkap Jang wrote: > > Hi James, > > On Fri, Feb 7, 2020 at 10:36 AM Wonkap Jang wrote: > > > In order for rate control to correctly allocate bitrate to each temporal > > layer, correct temporal layer id has to be set to each frame. This > > commit provides the abi

Re: [FFmpeg-devel] [PATCH 2/7] avcodec/h264_levels: Remove unused ff_h264_get_level()

2020-02-07 Thread Michael Niedermayer
On Fri, Feb 07, 2020 at 12:41:58PM -0300, James Almer wrote: > On 2/7/2020 10:48 AM, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/h264_levels.c | 12 > > libavcodec/h264_levels.h | 3 --- > > 2 files changed, 15 deletions(-) > > > > diff

Re: [FFmpeg-devel] [PATCH] fate/adpcm: add adpcm_ima_ssi tests

2020-02-07 Thread Michael Niedermayer
On Fri, Feb 07, 2020 at 11:38:25AM +, Zane van Iperen wrote: > Signed-off-by: Zane van Iperen > --- > tests/fate/adpcm.mak| 6 ++ > tests/ref/fate/adpcm-ima-ssi-mono | 1 + > tests/ref/fate/adpcm-ima-ssi-stereo | 1 + > 3 files changed, 8 insertions(+) > create mode 100

Re: [FFmpeg-devel] [PATCH v7 3/3] avcodec/mpeg12dec: Add CPB coded side data

2020-02-07 Thread James Almer
On 2/7/2020 7:43 PM, James Almer wrote: > On 1/14/2020 8:42 PM, Nicolas Gaullier wrote: >> This fixes mpeg2video stream copies to mpeg muxer like this: >> ffmpeg -i xdcamhd.mxf -c:v copy output.mpg >> --- >> libavcodec/mpeg12dec.c | 7 +++ >> tests/ref/fate/mxf-probe-d10 | 3 +++ >> te

Re: [FFmpeg-devel] [PATCH v7 3/3] avcodec/mpeg12dec: Add CPB coded side data

2020-02-07 Thread James Almer
On 1/14/2020 8:42 PM, Nicolas Gaullier wrote: > This fixes mpeg2video stream copies to mpeg muxer like this: > ffmpeg -i xdcamhd.mxf -c:v copy output.mpg > --- > libavcodec/mpeg12dec.c | 7 +++ > tests/ref/fate/mxf-probe-d10 | 3 +++ > tests/ref/fate/ts-demux | 2 +- > 3 files cha

Re: [FFmpeg-devel] [PATCH v7 3/3] avcodec/mpeg12dec: Add CPB coded side data

2020-02-07 Thread Michael Niedermayer
On Wed, Jan 15, 2020 at 12:42:13AM +0100, Nicolas Gaullier wrote: > This fixes mpeg2video stream copies to mpeg muxer like this: > ffmpeg -i xdcamhd.mxf -c:v copy output.mpg > --- > libavcodec/mpeg12dec.c | 7 +++ > tests/ref/fate/mxf-probe-d10 | 3 +++ > tests/ref/fate/ts-demux |

Re: [FFmpeg-devel] [PATCH v4] avfilter/vf_zoompan: fix shaking when zooming

2020-02-07 Thread Paul B Mahol
On 2/6/20, Robert Deibel wrote: > Would appreciate further review or info. Are you sure this change does not break older scripts? > > On 29.01.20 10:54, Robert Deibel wrote: >> Fix shaking of image when zoom is applied by the zoompan filter. >> Resolves ticket https://trac.ffmpeg.org/ticket/4298

[FFmpeg-devel] [PATCH] avcodec/mlpdec: use get_bits_long for huff lsbs

2020-02-07 Thread Jai Luthra
lsb bits may go beyond 25 bits, so to be safe use get_bits_long Signed-off-by: Jai Luthra --- libavcodec/mlpdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/mlpdec.c b/libavcodec/mlpdec.c index 22a6efd63d..1a2c0f29ac 100644 --- a/libavcodec/mlpdec.c +++ b/liba

Re: [FFmpeg-devel] [PATCH 3/4] avfilter/vf_paletteuse: Fix potential double-free of AVFrame

2020-02-07 Thread Paul B Mahol
LGTM On 1/27/20, Andreas Rheinhardt wrote: > apply_palette() would free an AVFrame given to it only via an AVFrame * > (and not via AVFrame **) in three of its four exists (namely in the > normal path and in two error paths). So upon error the caller has no way > to know whether the frame has alr

Re: [FFmpeg-devel] [PATCH 3/4] avfilter/vf_paletteuse: Fix potential double-free of AVFrame

2020-02-07 Thread Andreas Rheinhardt
Andreas Rheinhardt: > apply_palette() would free an AVFrame given to it only via an AVFrame * > (and not via AVFrame **) in three of its four exists (namely in the > normal path and in two error paths). So upon error the caller has no way > to know whether the frame has already been freed or not; >

Re: [FFmpeg-devel] [PATCH v6] avcodec/libvpxenc: add a way to explicitly set temporal layer id

2020-02-07 Thread Wonkap Jang
Hi James, On Fri, Feb 7, 2020 at 10:36 AM Wonkap Jang wrote: > In order for rate control to correctly allocate bitrate to each temporal > layer, correct temporal layer id has to be set to each frame. This > commit provides the ability to set correct temporal layer id for each > frame. > --- > d

[FFmpeg-devel] [PATCH v6] avcodec/libvpxenc: add a way to explicitly set temporal layer id

2020-02-07 Thread Wonkap Jang
In order for rate control to correctly allocate bitrate to each temporal layer, correct temporal layer id has to be set to each frame. This commit provides the ability to set correct temporal layer id for each frame. --- doc/encoders.texi | 8 +++- libavcodec/libvpxenc.c | 13 +++

Re: [FFmpeg-devel] [PATCHv2 2/3] libfdk-aacdec: Use the decoder's default level limiter settings

2020-02-07 Thread James Almer
On 2/7/2020 3:08 PM, Martin Storsjö wrote: > On Fri, 7 Feb 2020, Martin Storsjö wrote: > >> On Fri, 7 Feb 2020, James Almer wrote: >> >>> On 2/5/2020 10:06 AM, Martin Storsjö wrote: It was disabled by default in >> 2dbd35b00c6433e587d5f44d5dbc8972ebbaa88e as it added delay, but now we c

Re: [FFmpeg-devel] [PATCHv2 2/3] libfdk-aacdec: Use the decoder's default level limiter settings

2020-02-07 Thread Martin Storsjö
On Fri, 7 Feb 2020, Martin Storsjö wrote: On Fri, 7 Feb 2020, James Almer wrote: On 2/5/2020 10:06 AM, Martin Storsjö wrote: It was disabled by default in 2dbd35b00c6433e587d5f44d5dbc8972ebbaa88e as it added delay, but now we compensate for the delay properly by offsetting timestamps. ---

Re: [FFmpeg-devel] [PATCHv2 2/3] libfdk-aacdec: Use the decoder's default level limiter settings

2020-02-07 Thread James Almer
On 2/7/2020 3:04 PM, Martin Storsjö wrote: > On Fri, 7 Feb 2020, James Almer wrote: > >> On 2/5/2020 10:06 AM, Martin Storsjö wrote: >>> It was disabled by default in 2dbd35b00c6433e587d5f44d5dbc8972ebbaa88e >>> as it added delay, but now we compensate for the delay properly >>> by offsetting time

Re: [FFmpeg-devel] [PATCHv2 2/3] libfdk-aacdec: Use the decoder's default level limiter settings

2020-02-07 Thread Martin Storsjö
On Fri, 7 Feb 2020, James Almer wrote: On 2/5/2020 10:06 AM, Martin Storsjö wrote: It was disabled by default in 2dbd35b00c6433e587d5f44d5dbc8972ebbaa88e as it added delay, but now we compensate for the delay properly by offsetting timestamps. --- libavcodec/libfdk-aacdec.c | 3 ++- 1 file cha

Re: [FFmpeg-devel] [PATCH 2/3] libfdk-aacdec: Use the decoder's default level limiter settings

2020-02-07 Thread James Almer
On 2/5/2020 8:07 AM, Martin Storsjö wrote: > It was disabled by default in 2dbd35b00c6433e587d5f44d5dbc8972ebbaa88e > as it added delay, but now we compensate for the delay properly > by offsetting timestamps. > --- > libavcodec/libfdk-aacdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(

Re: [FFmpeg-devel] [PATCH] checkasm: sbrdsp: Fix a spurious test failure by calculating a better epsilon for sum_square

2020-02-07 Thread Martin Storsjö
On Wed, 5 Feb 2020, Martin Storsjö wrote: --- tests/checkasm/sbrdsp.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/checkasm/sbrdsp.c b/tests/checkasm/sbrdsp.c index 558f452c9b..516b9f0ec6 100644 --- a/tests/checkasm/sbrdsp.c +++ b/tests/checkasm/sbrdsp.c @@ -17,6 +

Re: [FFmpeg-devel] [PATCHv2 2/3] libfdk-aacdec: Use the decoder's default level limiter settings

2020-02-07 Thread James Almer
On 2/5/2020 10:06 AM, Martin Storsjö wrote: > It was disabled by default in 2dbd35b00c6433e587d5f44d5dbc8972ebbaa88e > as it added delay, but now we compensate for the delay properly > by offsetting timestamps. > --- > libavcodec/libfdk-aacdec.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deleti

[FFmpeg-devel] [PATCH] Update HDR10+ metadata structure.

2020-02-07 Thread Mohammad Izadi
From: Mohammad Izadi Trying to read HDR10+ metadata from HEVC/SEI and pass it to packet side data in the follow-up CLs. --- libavutil/hdr_dynamic_metadata.c | 165 +++ libavutil/hdr_dynamic_metadata.h | 12 ++- libavutil/version.h | 2 +- 3 files chan

Re: [FFmpeg-devel] [PATCH 2/7] avcodec/h264_levels: Remove unused ff_h264_get_level()

2020-02-07 Thread James Almer
On 2/7/2020 10:48 AM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/h264_levels.c | 12 > libavcodec/h264_levels.h | 3 --- > 2 files changed, 15 deletions(-) > > diff --git a/libavcodec/h264_levels.c b/libavcodec/h264_levels.c > index dd517f100

Re: [FFmpeg-devel] [PATCH] ac3enc: drop a global variable

2020-02-07 Thread James Almer
On 2/7/2020 1:31 PM, Paul B Mahol wrote: > On 1/30/20, Michael Niedermayer wrote: >> On Wed, Jan 29, 2020 at 04:34:48PM +0100, Anton Khirnov wrote: >>> Log the warning message once per encoder instance instead. >>> --- >>> libavcodec/ac3enc.c | 5 ++--- >>> libavcodec/ac3enc.h | 2 ++ >>> 2 files

Re: [FFmpeg-devel] [PATCH] ac3enc: drop a global variable

2020-02-07 Thread Paul B Mahol
On 1/30/20, Michael Niedermayer wrote: > On Wed, Jan 29, 2020 at 04:34:48PM +0100, Anton Khirnov wrote: >> Log the warning message once per encoder instance instead. >> --- >> libavcodec/ac3enc.c | 5 ++--- >> libavcodec/ac3enc.h | 2 ++ >> 2 files changed, 4 insertions(+), 3 deletions(-) > > LGT

[FFmpeg-devel] [PATCH 1/7] tools/target_dec_fuzzer: Fuzz skip_frame

2020-02-07 Thread Michael Niedermayer
Should allow coverage of related code Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c index 2487b6ca94..67a2a41874 100644 --- a/tools/target_dec_fuzzer.c +++ b/tools/targe

[FFmpeg-devel] [PATCH 4/7] tools/target_dec_fuzzer: Set extradata for the parser

2020-02-07 Thread Michael Niedermayer
This should improve coverage Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c index 2e718c2e95..f1768d4dce 100644 --- a/tools/target_dec_fuzzer.c +++ b/tools/target_dec_fu

[FFmpeg-devel] [PATCH 5/7] tools/target_dec_fuzzer: Fuzz AV_CODEC_FLAG2_EXPORT_MVS

2020-02-07 Thread Michael Niedermayer
This should increase coverage Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c index f1768d4dce..7e73c83686 100644 --- a/tools/target_dec_fuzzer.c +++ b/tools/target_dec_fu

[FFmpeg-devel] [PATCH 6/7] tools/target_dec_fuzzer: Fuzz FF_DEBUG_*

2020-02-07 Thread Michael Niedermayer
This should increase coverage Signed-off-by: Michael Niedermayer --- tools/target_dec_fuzzer.c | 9 + 1 file changed, 9 insertions(+) diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c index 7e73c83686..dcf9e06121 100644 --- a/tools/target_dec_fuzzer.c +++ b/tools/target

[FFmpeg-devel] [PATCH 2/7] avcodec/h264_levels: Remove unused ff_h264_get_level()

2020-02-07 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/h264_levels.c | 12 libavcodec/h264_levels.h | 3 --- 2 files changed, 15 deletions(-) diff --git a/libavcodec/h264_levels.c b/libavcodec/h264_levels.c index dd517f1005..801b27fdd9 100644 --- a/libavcodec/h264_levels.c +++ b/libavco

[FFmpeg-devel] [PATCH 7/7] avcodec/bitpacked: Add codec_tags list

2020-02-07 Thread Michael Niedermayer
This should improve coverage Signed-off-by: Michael Niedermayer --- libavcodec/bitpacked.c | 4 1 file changed, 4 insertions(+) diff --git a/libavcodec/bitpacked.c b/libavcodec/bitpacked.c index f0b417d595..be7d1e3629 100644 --- a/libavcodec/bitpacked.c +++ b/libavcodec/bitpacked.c @@ -146

[FFmpeg-devel] [PATCH 3/7] tools/target_dec_fuzzer: enable mjpeg for tiff or tdsc

2020-02-07 Thread Michael Niedermayer
This is needed for fuzzing tiff/tdsc and should increase coverage Signed-off-by: Michael Niedermayer --- libavcodec/allcodecs.c| 1 + tools/target_dec_fuzzer.c | 6 ++ 2 files changed, 7 insertions(+) diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c index 0ad3338f9a..5735864

Re: [FFmpeg-devel] [PATCH 2/3] avformat/tty: Fix division by 0 in probe

2020-02-07 Thread Michael Niedermayer
On Fri, Feb 07, 2020 at 12:19:32AM +0100, Hendrik Leppkes wrote: > On Fri, Feb 7, 2020 at 12:16 AM Michael Niedermayer wrote: > > > > On Thu, Feb 06, 2020 at 03:27:11PM +0100, Paul B Mahol wrote: > > > Nonsense, why would this code be called with 0 buffer? > > > > I guess because code calls av_pro

[FFmpeg-devel] [PATCH] fate/adpcm: add adpcm_ima_ssi tests

2020-02-07 Thread Zane van Iperen
Signed-off-by: Zane van Iperen --- tests/fate/adpcm.mak| 6 ++ tests/ref/fate/adpcm-ima-ssi-mono | 1 + tests/ref/fate/adpcm-ima-ssi-stereo | 1 + 3 files changed, 8 insertions(+) create mode 100644 tests/ref/fate/adpcm-ima-ssi-mono create mode 100644 tests/ref/fate/adpcm-

Re: [FFmpeg-devel] [PATCH] lavd/avfoundation: Add an option to drop late frames.

2020-02-07 Thread Thilo Borgmann
Am 03.02.20 um 21:47 schrieb Thilo Borgmann: > Am 19.01.20 um 17:41 schrieb Thilo Borgmann: >> $subject >> >> If you record live sources like during screen capture, you want this enabled >> (current behavior). >> For the case you're capturing from etc you want the most possible frames >> captured

Re: [FFmpeg-devel] GSoC 2020

2020-02-07 Thread Thilo Borgmann
Am 06.02.20 um 23:59 schrieb Michael Niedermayer: > Hi all > > please help fill the 2020 GSoC Ideas page > https://trac.ffmpeg.org/wiki/SponsoringPrograms/GSoC/2020 > > (This page is key to being acccepted to GSoC) please come up with ideas that you find useful and possible to do, even if you d

[FFmpeg-devel] [PATCH] configure: fix order of linking for libglslang

2020-02-07 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 58419cd88c..51cdc7b2cb 100755 --- a/configure +++ b/configure @@ -6267,7 +6267,7 @@ enabled fontconfig&& enable libfontconfig enabled libfontconfig

Re: [FFmpeg-devel] [PATCH 2/2] avformat/tty: add seeking support

2020-02-07 Thread Paul B Mahol
On 2/2/20, Andreas Rheinhardt wrote: > Paul B Mahol: >> Signed-off-by: Paul B Mahol >> --- >> libavformat/tty.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/libavformat/tty.c b/libavformat/tty.c >> index 61e9f3e95c..2065fc0a53 100644 >> --- a/libavformat/tty.c >> +++ b/libavfor

Re: [FFmpeg-devel] [PATCH] avfilter: add afirsrc filter

2020-02-07 Thread Paul B Mahol
On 2/1/20, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 38 + > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/asrc_afirsrc.c | 330 + > 4 files changed, 370 insertions(+)