Re: [FFmpeg-devel] [PATCH v2 0/7] Support variable frame sizes in adpcm_argo

2020-09-18 Thread Zane van Iperen
On Fri, 18 Sep 2020 12:04:41 + "Zane van Iperen" wrote: > > v2: > - handle variable sizes in muxer (argo_asf) > - fix incorrect pts tracking (argo_brp) > - add test for decoding argo_asf via argo_brp Will apply the first three parts of this soon. The rest has been superceded.

Re: [FFmpeg-devel] [PATCH] web: link GH Actions based Windows-Builds by BtbN

2020-09-18 Thread Lou Logan
Wed Sep 16 14:15:34 EEST 2020, Timo Rothenpieler wrote: > --- > src/download | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/download b/src/download > index 706d402..0f090b8 100644 > --- a/src/download > +++ b/src/download > @@ -79,6 +79,9 @@ > Windows EXE Files >

Re: [FFmpeg-devel] [PATCH] avformat/3dostr: Check remaining buffer in probe before 8 byte step

2020-09-18 Thread Michael Niedermayer
On Sat, Sep 19, 2020 at 12:26:36AM +0200, Michael Niedermayer wrote: > Fixes: segfault > Fixes: signal_sigabrt_76ae7cc9_7213_0d6457b9d6897fa7c78507fa5de53510.ts > Regression since: 3ac45bf66561a667260cac37223c0393f7333fca > > Signed-off-by: Michael Niedermayer > --- > libavformat/3dostr.c |

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/diracdsp: Fix integer anomaly in dequant_subband_*

2020-09-18 Thread Michael Niedermayer
On Sat, Jul 18, 2020 at 10:47:26PM +0200, Michael Niedermayer wrote: > Fixes: negation of -2147483648 cannot be represented in type 'int32_t' (aka > 'int'); cast to an unsigned type to negate this value to itself > Fixes: >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/tiff: Fix default white level

2020-09-18 Thread Michael Niedermayer
On Fri, Jul 17, 2020 at 12:43:57AM +0200, Michael Niedermayer wrote: > According to the spec bits per sample should be used > > Fix invalid shift with bpp=32 > Fixes: shift exponent 32 is too large for 32-bit type 'unsigned int' > Fixes: >

[FFmpeg-devel] [PATCH] avformat/3dostr: Check remaining buffer in probe before 8 byte step

2020-09-18 Thread Michael Niedermayer
Fixes: segfault Fixes: signal_sigabrt_76ae7cc9_7213_0d6457b9d6897fa7c78507fa5de53510.ts Regression since: 3ac45bf66561a667260cac37223c0393f7333fca Signed-off-by: Michael Niedermayer --- libavformat/3dostr.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] avutil/pixfmt: improve definition of AVColorRange

2020-09-18 Thread James Almer
On 9/18/2020 6:05 PM, Jan Ekström wrote: > As it was brought up that the current documentation leaves things > as specific to YCbCr only, ICtCp and RGB are now mentioned. > Additionally, the specifications on which these definitions of > narrow and full range are defined are mentioned. > > This

[FFmpeg-devel] [PATCH] avutil/pixfmt: improve definition of AVColorRange

2020-09-18 Thread Jan Ekström
As it was brought up that the current documentation leaves things as specific to YCbCr only, ICtCp and RGB are now mentioned. Additionally, the specifications on which these definitions of narrow and full range are defined are mentioned. This way, the documentation of AVColorRange should now

Re: [FFmpeg-devel] [PATCH v3 2/2] dnn_backend_native_layer_conv2d.c: refine code.

2020-09-18 Thread Xu Jun
- Original Message - > From: "Yejun Guo" > To: "FFmpeg development discussions and patches" > Cc: "xujunzz" > Sent: Thursday, September 17, 2020 9:30:08 PM > Subject: RE: [FFmpeg-devel] [PATCH v3 2/2] dnn_backend_native_layer_conv2d.c: > refine code. >> -Original Message- >>

Re: [FFmpeg-devel] Patch: added V210 codec support to MXF encoder

2020-09-18 Thread Carl Eugen Hoyos
> Am 17.09.2020 um 23:34 schrieb Swami Kevala : > > Verified the container_ui and element_ui with a sample file produced by > Sony Server 1.3.0.0.1 (Sony MXF Development Kit (Win32) 4.9.1.118.1) Imo - and this is not meant to block your patch - it would make more sense to test the file

Re: [FFmpeg-devel] SCTE-35 implementation already (bounty)

2020-09-18 Thread Dennis Mungai
On Wed, 2 Sep 2020 at 07:29, MediaStream wrote: > Looking for SCTE-35 pass through implementation: > > 1. Extract SCTE-35 from MPEG-TS. > 2. Translate timing of the original SCTE-35 events to match timing in the > output file appropriately or keep timing as is. > 3. Signal encoder to force key

[FFmpeg-devel] [PATCH 2/3] avformat/mov: fix typo in comments

2020-09-18 Thread Zhao Zhili
From: Zhao Zhili --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 8c1243b48d..9fc0db24d5 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1473,7 +1473,7 @@ static int mov_read_mvhd(MOVContext *c,

[FFmpeg-devel] [PATCH 1/3] avformat/mov: replace multiple avio_r8() by avio_skip()

2020-09-18 Thread Zhao Zhili
From: Zhao Zhili --- libavformat/mov.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9462af743a..8c1243b48d 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -103,9 +103,7 @@ static int

[FFmpeg-devel] [PATCH 3/3] avformat/mov: remove an always true condition

2020-09-18 Thread Zhao Zhili
From: Zhao Zhili --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9fc0db24d5..f99605c2cd 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1475,7 +1475,7 @@ static int mov_read_mvhd(MOVContext *c,

Re: [FFmpeg-devel] [PATCH] avcodec/av1dec: check avctx->hwaccel when hwaccel pix_fmt selected

2020-09-18 Thread James Almer
On 9/18/2020 2:40 AM, Wang, Fei W wrote: > > >> -Original Message- >> From: ffmpeg-devel On Behalf Of >> Hendrik Leppkes >> Sent: Thursday, September 17, 2020 5:21 PM >> To: FFmpeg development discussions and patches >> Subject: Re: [FFmpeg-devel] [PATCH] avcodec/av1dec: check

Re: [FFmpeg-devel] [PATCH] avcodec/av1dec: check avctx->hwaccel when hwaccel pix_fmt selected

2020-09-18 Thread Michael Niedermayer
On Wed, Sep 16, 2020 at 04:57:15PM -0400, Fei Wang wrote: > Pix fmt with hwaccel flag may not be chosen in format probing, in > this case avctx->hwaccel will not be inited. > > Signed-off-by: Fei Wang > --- > libavcodec/av1dec.c | 12 > 1 file changed, 8 insertions(+), 4

Re: [FFmpeg-devel] [PATCH v2] avfilter/vf_scale: translate and verify swscale internal range flag

2020-09-18 Thread Michael Niedermayer
On Thu, Sep 17, 2020 at 11:44:39PM +0300, Jan Ekström wrote: > On Thu, Sep 17, 2020 at 11:31 PM Michael Niedermayer > wrote: > > > > On Wed, Sep 16, 2020 at 11:18:48PM +0300, Jan Ekström wrote: > > > This value - while it looks like the actual range of the content - > > > is nothing but the

[FFmpeg-devel] [PATCH v2 7/7] fate: add argo_adpcm via argo_brp demux test

2020-09-18 Thread Zane van Iperen
Signed-off-by: Zane van Iperen --- tests/fate/adpcm.mak | 3 +++ tests/ref/fate/adpcm-argo-brp | 1 + 2 files changed, 4 insertions(+) create mode 100644 tests/ref/fate/adpcm-argo-brp diff --git a/tests/fate/adpcm.mak b/tests/fate/adpcm.mak index c75a756bc3..0d6a671d34 100644 ---

[FFmpeg-devel] [PATCH v2 6/7] avformat/argo_brp: remove usage of floats

2020-09-18 Thread Zane van Iperen
Signed-off-by: Zane van Iperen --- libavformat/argo_brp.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/libavformat/argo_brp.c b/libavformat/argo_brp.c index a3e9846c74..3a5a0c0497 100644 --- a/libavformat/argo_brp.c +++ b/libavformat/argo_brp.c @@ -78,8

[FFmpeg-devel] [PATCH v2 5/7] avformat/argo_brp: support reading multiple ASF blocks at once

2020-09-18 Thread Zane van Iperen
Signed-off-by: Zane van Iperen --- libavformat/argo_brp.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/libavformat/argo_brp.c b/libavformat/argo_brp.c index 122f616ecc..a3e9846c74 100644 --- a/libavformat/argo_brp.c +++ b/libavformat/argo_brp.c @@

[FFmpeg-devel] [PATCH v2 3/7] avformat/argo_asf: read/write multiple blocks at once

2020-09-18 Thread Zane van Iperen
Signed-off-by: Zane van Iperen --- libavformat/argo_asf.c | 38 -- 1 file changed, 28 insertions(+), 10 deletions(-) diff --git a/libavformat/argo_asf.c b/libavformat/argo_asf.c index b1632f3ba5..048e5441d6 100644 --- a/libavformat/argo_asf.c +++

[FFmpeg-devel] [PATCH v2 4/7] avformat/argo_asf: add ff_argo_asf_read()

2020-09-18 Thread Zane van Iperen
For the argo_brp demuxer. Signed-off-by: Zane van Iperen --- libavformat/argo_asf.c | 45 -- libavformat/argo_asf.h | 2 ++ 2 files changed, 28 insertions(+), 19 deletions(-) diff --git a/libavformat/argo_asf.c b/libavformat/argo_asf.c index

[FFmpeg-devel] [PATCH v2 2/7] avcodec/adpcm_{psx, argo}: add missing indent

2020-09-18 Thread Zane van Iperen
Signed-off-by: Zane van Iperen --- libavcodec/adpcm.c | 86 +++--- 1 file changed, 43 insertions(+), 43 deletions(-) diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c index 14be1f4f88..4755308824 100644 --- a/libavcodec/adpcm.c +++ b/libavcodec/adpcm.c

[FFmpeg-devel] [PATCH v2 1/7] avcodec/adpcm_argo: support decoding multiple frames

2020-09-18 Thread Zane van Iperen
Increases decode speed significantly. Signed-off-by: Zane van Iperen --- libavcodec/adpcm.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c index e409a3aa6a..14be1f4f88 100644 --- a/libavcodec/adpcm.c +++

[FFmpeg-devel] [PATCH v2 0/7] Support variable frame sizes in adpcm_argo

2020-09-18 Thread Zane van Iperen
v2: - handle variable sizes in muxer (argo_asf) - fix incorrect pts tracking (argo_brp) - add test for decoding argo_asf via argo_brp Test sample is https://0x0.st/iIhK.BRP, needs to go in fate-suite/argo-brp/CROCLOGO_CUT.BRP. Original email to samples-requ...@ffmpeg.org was bounced. Zane

Re: [FFmpeg-devel] [PATCH] avformat/adxdec: demux multiple blocks at once

2020-09-18 Thread Paul B Mahol
On Fri, Sep 18, 2020 at 12:29:41PM +0200, Andreas Rheinhardt wrote: > Paul B Mahol: > > On Fri, Sep 18, 2020 at 12:16:18PM +0200, Andreas Rheinhardt wrote: > >> Paul B Mahol: > >>> Improves decoding speed by 24x > >>> > >>> Signed-off-by: Paul B Mahol > >>> --- > >>> libavformat/adxdec.c | 23

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/vlc: Add macro for ff_init_vlc_sparse()"

2020-09-18 Thread Andreas Rheinhardt
Hendrik Leppkes: > On Fri, Sep 18, 2020 at 8:25 AM Andreas Rheinhardt > wrote: >> >> This reverts commit 61669b7c40b8dc3a0841768fb39c7567513b7cfc. >> >> This commit broke building with MSVC due to its spec-incompliant handling >> of ',' in __VA_ARGS__: These are not treated as argument separators

Re: [FFmpeg-devel] [PATCH] avformat/adxdec: demux multiple blocks at once

2020-09-18 Thread Andreas Rheinhardt
Paul B Mahol: > On Fri, Sep 18, 2020 at 12:16:18PM +0200, Andreas Rheinhardt wrote: >> Paul B Mahol: >>> Improves decoding speed by 24x >>> >>> Signed-off-by: Paul B Mahol >>> --- >>> libavformat/adxdec.c | 23 +++ >>> 1 file changed, 15 insertions(+), 8 deletions(-) >>> >>>

Re: [FFmpeg-devel] [PATCH] avformat/adxdec: demux multiple blocks at once

2020-09-18 Thread Paul B Mahol
On Fri, Sep 18, 2020 at 12:16:18PM +0200, Andreas Rheinhardt wrote: > Paul B Mahol: > > Improves decoding speed by 24x > > > > Signed-off-by: Paul B Mahol > > --- > > libavformat/adxdec.c | 23 +++ > > 1 file changed, 15 insertions(+), 8 deletions(-) > > > > diff --git

[FFmpeg-devel] [PATCH] Added V210 codec support to MXF encoder

2020-09-18 Thread Swami Kevala
Verified the container_ui and element_ui with a sample file produced by Sony Server 1.3.0.0.1 (Sony MXF Development Kit (Win32) 4.9.1.118.1) Tested converting an ffv1/mkv to v210/mxf successfully Signed-off-by: Swami Kevala --- libavformat/mxfenc.c | 7 +++ 1 file changed, 7

[FFmpeg-devel] [PATCH] avformat/adxdec: demux multiple blocks at once

2020-09-18 Thread Paul B Mahol
Improves decoding speed by 24x Signed-off-by: Paul B Mahol --- libavformat/adxdec.c | 23 +++ 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/libavformat/adxdec.c b/libavformat/adxdec.c index ccd5049acd..0e4c251cfc 100644 --- a/libavformat/adxdec.c +++

Re: [FFmpeg-devel] [PATCH] avformat/adxdec: demux multiple blocks at once

2020-09-18 Thread Andreas Rheinhardt
Paul B Mahol: > Improves decoding speed by 24x > > Signed-off-by: Paul B Mahol > --- > libavformat/adxdec.c | 23 +++ > 1 file changed, 15 insertions(+), 8 deletions(-) > > diff --git a/libavformat/adxdec.c b/libavformat/adxdec.c > index ccd5049acd..0e4c251cfc 100644 > ---

Re: [FFmpeg-devel] [PATCH] Revert "avcodec/vlc: Add macro for ff_init_vlc_sparse()"

2020-09-18 Thread Hendrik Leppkes
On Fri, Sep 18, 2020 at 8:25 AM Andreas Rheinhardt wrote: > > This reverts commit 61669b7c40b8dc3a0841768fb39c7567513b7cfc. > > This commit broke building with MSVC due to its spec-incompliant handling > of ',' in __VA_ARGS__: These are not treated as argument separators for > further macros, so

Re: [FFmpeg-devel] [PATCH] opusdec: do not fail when LBRR frames are present

2020-09-18 Thread Paul B Mahol
On Fri, Sep 11, 2020 at 07:37:14PM +0200, Anton Khirnov wrote: > Decode and discard them. > > Fixes ticket 4641. > --- > libavcodec/opus_silk.c | 28 > libavcodec/opustab.c | 3 +++ > libavcodec/opustab.h | 3 +++ > 3 files changed, 26 insertions(+), 8

Re: [FFmpeg-devel] [PATCH 3/3] avdevice/lavfi: av_malloc -> av_malloc_array

2020-09-18 Thread Paul B Mahol
On Fri, Sep 18, 2020 at 09:04:26AM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavdevice/lavfi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) lgtm > > diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c > index

Re: [FFmpeg-devel] [PATCH 4/4] dnn: add a new interface DNNModel.get_output

2020-09-18 Thread Guo, Yejun
> -Original Message- > From: Guo, Yejun > Sent: 2020年9月14日 14:29 > To: ffmpeg-devel@ffmpeg.org > Cc: Guo, Yejun > Subject: [PATCH 4/4] dnn: add a new interface DNNModel.get_output > > for some cases (for example, super resolution), the DNN model changes the > frame size which impacts

[FFmpeg-devel] [PATCH] Revert "avcodec/vlc: Add macro for ff_init_vlc_sparse()"

2020-09-18 Thread Andreas Rheinhardt
This reverts commit 61669b7c40b8dc3a0841768fb39c7567513b7cfc. This commit broke building with MSVC due to its spec-incompliant handling of ',' in __VA_ARGS__: These are not treated as argument separators for further macros, so that in our case the init_vlc2() macro is treated as having only one