Re: [FFmpeg-devel] [PATCH v6 6/9] avcodec: add vvc parser

2021-02-17 Thread Nuo Mi
On Thu, Feb 18, 2021 at 8:10 AM Mark Thompson wrote: > On 17/02/2021 01:51, Nuo Mi wrote: > > --- > > configure | 1 + > > libavcodec/Makefile | 1 + > > libavcodec/parsers.c| 1 + > > libavcodec/vvc_parser.c | 310 > > 4

Re: [FFmpeg-devel] [PATCH v3] avcodec/libvpxenc: fix potential memory leak.

2021-02-17 Thread Wonkap Jang
On Wed, Feb 17, 2021 at 11:30 AM Wonkap Jang wrote: > > > On Wed, Feb 17, 2021 at 9:50 AM Nicolas George wrote: > >> Wonkap Jang (12021-02-17): >> > While parsing ref_frame_config, AVdictionary needs to be manually >> > deallocated. >> > --- >> > libavcodec/libvpxenc.c | 21

[FFmpeg-devel] [PATCH 29/29] avformat/wavdec: Share wav and w64 options

2021-02-17 Thread Andreas Rheinhardt
The options of the w64 demuxer are a proper subset of the options for the wav demuxer, making it possible to reuse a part of the options for the wav demuxer for the w64 demuxer. Signed-off-by: Andreas Rheinhardt --- Due to w64 having a configure dependency on wav the wav demuxer is always

[FFmpeg-devel] [PATCH 28/29] avformat/wav: Deduplicate codec tags lists

2021-02-17 Thread Andreas Rheinhardt
Also saves relocations. Signed-off-by: Andreas Rheinhardt --- libavformat/riff.c | 4 libavformat/riff.h | 2 ++ libavformat/wavdec.c | 4 ++-- libavformat/wavenc.c | 4 ++-- 4 files changed, 10 insertions(+), 4 deletions(-) diff --git a/libavformat/riff.c b/libavformat/riff.c index

[FFmpeg-devel] [PATCH 27/29] avformat/avienc, wtvenc: Deduplicate codec tags list

2021-02-17 Thread Andreas Rheinhardt
Also saves relocations. Signed-off-by: Andreas Rheinhardt --- libavformat/avienc.c | 4 +--- libavformat/riff.c | 7 +++ libavformat/riff.h | 2 ++ libavformat/wtvenc.c | 3 +-- 4 files changed, 11 insertions(+), 5 deletions(-) diff --git a/libavformat/avienc.c b/libavformat/avienc.c

[FFmpeg-devel] [PATCH 26/29] avformat/aiff: Deduplicate codec tags and codec tags list

2021-02-17 Thread Andreas Rheinhardt
Also saves a relocation. Signed-off-by: Andreas Rheinhardt --- libavformat/Makefile | 4 ++-- libavformat/aiff.c| 54 +++ libavformat/aiff.h| 30 ++-- libavformat/aiffdec.c | 2 +- libavformat/aiffenc.c | 2 +- 5 files

[FFmpeg-devel] [PATCH 25/29] avformat/ast: Deduplicate codec tags lists

2021-02-17 Thread Andreas Rheinhardt
Also saves a relocation. Signed-off-by: Andreas Rheinhardt --- libavformat/ast.c| 5 - libavformat/ast.h| 1 + libavformat/astdec.c | 2 +- libavformat/astenc.c | 2 +- 4 files changed, 7 insertions(+), 3 deletions(-) diff --git a/libavformat/ast.c b/libavformat/ast.c index

[FFmpeg-devel] [PATCH 24/29] avformat/movenc: Deduplicate codec tags lists

2021-02-17 Thread Andreas Rheinhardt
Also saves relocations. Signed-off-by: Andreas Rheinhardt --- libavformat/movenc.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index f97383fcfe..545b0885ae 100644 --- a/libavformat/movenc.c +++

[FFmpeg-devel] [PATCH 23/29] avformat/oma: Move stuff only used by demuxer to demuxer

2021-02-17 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavformat/oma.c| 17 + libavformat/oma.h| 3 --- libavformat/omadec.c | 18 -- 3 files changed, 17 insertions(+), 21 deletions(-) diff --git a/libavformat/oma.c b/libavformat/oma.c index e7bf8553f0..7282d464aa

[FFmpeg-devel] [PATCH 22/29] avformat/oma: Deduplicate codec tags list

2021-02-17 Thread Andreas Rheinhardt
Also saves a relocation. Signed-off-by: Andreas Rheinhardt --- libavformat/oma.c| 3 +++ libavformat/oma.h| 1 + libavformat/omadec.c | 2 +- libavformat/omaenc.c | 2 +- 4 files changed, 6 insertions(+), 2 deletions(-) diff --git a/libavformat/oma.c b/libavformat/oma.c index

[FFmpeg-devel] [PATCH 21/29] avformat/rso: Deduplicate codec tags list

2021-02-17 Thread Andreas Rheinhardt
Also saves a relocation. Signed-off-by: Andreas Rheinhardt --- libavformat/rso.c| 2 ++ libavformat/rso.h| 1 + libavformat/rsodec.c | 2 +- libavformat/rsoenc.c | 2 +- 4 files changed, 5 insertions(+), 2 deletions(-) diff --git a/libavformat/rso.c b/libavformat/rso.c index

[FFmpeg-devel] [PATCH 20/29] avformat/voc: Deduplicate codec tags list

2021-02-17 Thread Andreas Rheinhardt
Also saves a relocation. Signed-off-by: Andreas Rheinhardt --- libavformat/voc.c| 8 ++-- libavformat/voc.h| 1 + libavformat/vocdec.c | 2 +- libavformat/vocenc.c | 2 +- 4 files changed, 9 insertions(+), 4 deletions(-) diff --git a/libavformat/voc.c b/libavformat/voc.c index

[FFmpeg-devel] [PATCH 19/29] avformat/caf: Deduplicate codec tags list

2021-02-17 Thread Andreas Rheinhardt
Also saves a relocation. Signed-off-by: Andreas Rheinhardt --- libavformat/caf.c| 1 + libavformat/caf.h| 1 + libavformat/cafdec.c | 2 +- libavformat/cafenc.c | 2 +- 4 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/caf.c b/libavformat/caf.c index

[FFmpeg-devel] [PATCH 15/29] avformat/isom: Split movaudio/movvideo tags off into a separate file

2021-02-17 Thread Andreas Rheinhardt
The NUT and avi demuxers only need ff_codec_movvideo_tags and so this removes a dependency on the rest of isom.c as well as on mpeg4audio.c (which isom depends on); it is similar for the Matroska demuxer and muxers, except that the mpeg4audio.c dependency can't be avoided. Signed-off-by: Andreas

[FFmpeg-devel] [PATCH 18/29] avformat/asfenc: Deduplicate codec tags lists

2021-02-17 Thread Andreas Rheinhardt
Also saves relocations. Signed-off-by: Andreas Rheinhardt --- libavformat/asfenc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavformat/asfenc.c b/libavformat/asfenc.c index 8b24264c94..23a09efe05 100644 --- a/libavformat/asfenc.c +++

[FFmpeg-devel] [PATCH 17/29] avformat/au: Deduplicate codec_tag lists

2021-02-17 Thread Andreas Rheinhardt
Also saves a relocation. Signed-off-by: Andreas Rheinhardt --- libavformat/au.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/au.c b/libavformat/au.c index 4f2b81119f..c2c1b7a3a3 100644 --- a/libavformat/au.c +++ b/libavformat/au.c @@ -53,6 +53,8 @@

[FFmpeg-devel] [PATCH 16/29] avformat/isom: Remove outcommented function

2021-02-17 Thread Andreas Rheinhardt
There is another ff_mov_read_chan in mov_chan.c. Signed-off-by: Andreas Rheinhardt --- libavformat/isom.c | 39 --- 1 file changed, 39 deletions(-) diff --git a/libavformat/isom.c b/libavformat/isom.c index e0e50f71b1..df98779149 100644 ---

[FFmpeg-devel] [PATCH 09/29] avcodec/Makefile: Don't build mjpegenc_huffman unconditionally

2021-02-17 Thread Andreas Rheinhardt
Only the mjpeg and amv encoders as well as its testprogram actually need it. Signed-off-by: Andreas Rheinhardt --- libavcodec/Makefile | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index c427de78be..8a835a26ab 100644 ---

[FFmpeg-devel] [PATCH 08/29] avcodec/mjpegenc_common: Move stuff only used by mjpegenc.c to it

2021-02-17 Thread Andreas Rheinhardt
This allows to make ff_init_uni_ac_vlc static; ff_mjpeg_encode_picture_frame has also been made static, but it could always have been made static. Signed-off-by: Andreas Rheinhardt --- libavcodec/ljpegenc.c| 2 + libavcodec/mjpegenc.c| 191 +-

[FFmpeg-devel] [PATCH 14/29] avformat/Makefile: Remove outdated AIFF demuxer dependency on iso_media

2021-02-17 Thread Andreas Rheinhardt
This is a result of the mov channel parsing stuff being factored out of mov.c twice: Once in 91b782720fd0df5571775b6591bc41797d6ecf78 to isom.c and later in 3bab7cd12802dc5abf2c5cc6dec49e9e249ce204. Also remove the isom.h header; and while just at it, remove an unused mathematics.h inclusion.

[FFmpeg-devel] [PATCH 07/29] avcodec/jpegtables: Move ff_mjpeg_build_huffman_codes to mjpegenc_common

2021-02-17 Thread Andreas Rheinhardt
Since g2meet.c doesn't use it any more, only encoders use it and the place for their common code is mjpegenc_common.c. Signed-off-by: Andreas Rheinhardt --- libavcodec/jpegtables.c | 21 - libavcodec/jpegtables.h | 4 libavcodec/mjpegenc_common.c | 21

[FFmpeg-devel] [PATCH 12/29] avcodec/mpeg4audio: Use proper logcontext for logging

2021-02-17 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/mpeg4audio.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/mpeg4audio.c b/libavcodec/mpeg4audio.c index 77cf2fb61c..2286303b54 100644 --- a/libavcodec/mpeg4audio.c +++ b/libavcodec/mpeg4audio.c @@ -30,7 +30,7

[FFmpeg-devel] [PATCH 13/29] avcodec/Makefile: Remove outdated dependency of FLV demuxer on mpeg4audio

2021-02-17 Thread Andreas Rheinhardt
Unneeded since f96a653184e63cea91e08ea75ae60d309e431f40 and b2bb09bcc330156e9d79d7ddfa59f9c5d05ca149. Signed-off-by: Andreas Rheinhardt --- libavcodec/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 8a835a26ab..24725d8666 100644 ---

[FFmpeg-devel] [PATCH 06/29] avcodec/g2meet, mjpegdec: Factor out common VLC initialization code

2021-02-17 Thread Andreas Rheinhardt
While just at it, remove the nb_codes parameter: It is redundant (the number of codes is implicitly contained in the array containing how many entries of a specific size there are) and for this reason it might even be wrong, so it is better to check what is actually used instead. Signed-off-by:

[FFmpeg-devel] [PATCH 11/29] avcodec/scpr3: Avoid code duplication when updating models

2021-02-17 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/scpr3.c | 24 1 file changed, 4 insertions(+), 20 deletions(-) diff --git a/libavcodec/scpr3.c b/libavcodec/scpr3.c index 1ed764baa1..78c58889cb 100644 --- a/libavcodec/scpr3.c +++ b/libavcodec/scpr3.c @@ -524,32 +524,16

[FFmpeg-devel] [PATCH 05/29] avcodec/Makefile: Remove redundant mjpegdec dependency from tiff decoder

2021-02-17 Thread Andreas Rheinhardt
The MJPEG decoder is already activated by configure whenever the tiff decoder is selected; ergo it is unnecessary to add a dependency in the Makefile. Signed-off-by: Andreas Rheinhardt --- libavcodec/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[FFmpeg-devel] [PATCH 04/29] configure, libavcodec/Makefile: Fix avrn dependencies

2021-02-17 Thread Andreas Rheinhardt
The avrn decoder actually only needs one thing: The MJPEG decoder. Instead the Makefile made it compile mjpegdec and configure required some of the prerequisites of the MJPEG decoder (exif and jpegtables). Even if all the prerequisites of the MJPEG decoder were required, it would still not make

[FFmpeg-devel] [PATCH 10/29] configure: Remove mpegvideo_enc dependency from ljpeg encoder

2021-02-17 Thread Andreas Rheinhardt
It only existed because some code in mjpegenc_common.c relied on it; yet said code was actually only used by mjpegenc.c and has been moved there. Signed-off-by: Andreas Rheinhardt --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index

[FFmpeg-devel] [PATCH 03/29] avcodec/avrndec: Remove unnecessary headers

2021-02-17 Thread Andreas Rheinhardt
The avrn decoder does not use any internals of the MJPEG decoder since e0031ca29a471c4a540ba2e01b3f81af03ef757b. Signed-off-by: Andreas Rheinhardt --- libavcodec/avrndec.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/avrndec.c b/libavcodec/avrndec.c index

[FFmpeg-devel] [PATCH 02/29] avcodec/Makefile: Remove spurios dcaenc dependency

2021-02-17 Thread Andreas Rheinhardt
It does not need dca.c. Signed-off-by: Andreas Rheinhardt --- libavcodec/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index f43e717714..92a3dcf0fc 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -276,7

[FFmpeg-devel] [PATCH 01/29] configure, libavcodec/Makefile: Remove spurious CAF demuxer dependencies

2021-02-17 Thread Andreas Rheinhardt
Forgotten in 604fbb3132e88727e496c96c92cfe02748c25a1a. Signed-off-by: Andreas Rheinhardt --- configure | 2 +- libavcodec/Makefile | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/configure b/configure index a76c2ec4ae..c28a7403db 100755 --- a/configure +++

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-17 Thread Christopher Degawa
> > It does seem like the number correlates somehow with how much CPU it > uses, but it's not an upper bound. > If it isn't, then that might be an issue, but the idea is that with the logical_processor option you can limit the amount of ram a single encoder instance will use since on a system

[FFmpeg-devel] [PATCH v4] avcodec/libvpxenc: optimize parsing vpx_svc_ref_frame_config parameters

2021-02-17 Thread Wonkap Jang
Getting rid of unnecessary use of AVDictionary object in parsing vpx_svc_ref_frame_config. --- libavcodec/libvpxenc.c | 76 -- 1 file changed, 58 insertions(+), 18 deletions(-) diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index

Re: [FFmpeg-devel] [PATCH v6 6/9] avcodec: add vvc parser

2021-02-17 Thread Mark Thompson
On 17/02/2021 01:51, Nuo Mi wrote: --- configure | 1 + libavcodec/Makefile | 1 + libavcodec/parsers.c| 1 + libavcodec/vvc_parser.c | 310 4 files changed, 313 insertions(+) create mode 100644 libavcodec/vvc_parser.c

Re: [FFmpeg-devel] [PATCH v6 4/9] avcodec: add cbs for h266/vvc

2021-02-17 Thread Mark Thompson
On 17/02/2021 01:51, Nuo Mi wrote: --- configure |2 + libavcodec/Makefile |1 + libavcodec/cbs.c |6 + libavcodec/cbs_h2645.c| 541 - libavcodec/cbs_h266.h | 817 +++

[FFmpeg-devel] [PATCH v2] avisynth: fix audio on big endian

2021-02-17 Thread Stephen Hutchinson
--- libavformat/avisynth.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/libavformat/avisynth.c b/libavformat/avisynth.c index f029a0e842..4cd6d6bc90 100644 --- a/libavformat/avisynth.c +++ b/libavformat/avisynth.c @@ -42,6 +42,13 @@ #define AVISYNTH_LIB

Re: [FFmpeg-devel] [PATCH] avisynth: fix audio on big endian

2021-02-17 Thread Andreas Rheinhardt
Stephen Hutchinson: > --- > Open for bikeshedding, as I'm not too sure of > the names, but I didn't want to use anything > too close to regular AV_CODEC_ID* defines. > libavformat/avisynth.c | 21 + > 1 file changed, 17 insertions(+), 4 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH] avfilter/vf_pseudocolor: Add missing braces

2021-02-17 Thread Mark Thompson
On 06/02/2021 22:04, Paul B Mahol wrote: Trivial, thus lgtm. Rebased and applied, apologies for the delay. Thanks, - Mark ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit

[FFmpeg-devel] [PATCH] avisynth: fix audio on big endian

2021-02-17 Thread Stephen Hutchinson
--- Open for bikeshedding, as I'm not too sure of the names, but I didn't want to use anything too close to regular AV_CODEC_ID* defines. libavformat/avisynth.c | 21 + 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/libavformat/avisynth.c

Re: [FFmpeg-devel] [PATCH v6 3/9] avformat: add vvc raw demux

2021-02-17 Thread Mark Thompson
On 17/02/2021 01:51, Nuo Mi wrote: --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/vvcdec.c | 61 3 files changed, 63 insertions(+) create mode 100644 libavformat/vvcdec.c diff --git a/libavformat/Makefile

Re: [FFmpeg-devel] [PATCH] libavfilter: avoid UB nullptr-with-offset.

2021-02-17 Thread Andreas Rheinhardt
Michael Niedermayer: > On Tue, Feb 16, 2021 at 03:53:08PM +, Jeremy Leconte wrote: >> --- >> libavfilter/vf_scale.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavfilter/vf_scale.c b/libavfilter/vf_scale.c >> index 58eee96744..98cef5eb4b 100644 >> ---

Re: [FFmpeg-devel] [PATCH v6 2/9] avcodec/vvc: add shared header for vvc

2021-02-17 Thread Mark Thompson
On 17/02/2021 01:51, Nuo Mi wrote: --- libavcodec/vvc.h | 142 +++ 1 file changed, 142 insertions(+) create mode 100644 libavcodec/vvc.h diff --git a/libavcodec/vvc.h b/libavcodec/vvc.h new file mode 100644 index 00..ca15297d7a ---

Re: [FFmpeg-devel] [PATCH] libavfilter: avoid UB nullptr-with-offset.

2021-02-17 Thread Michael Niedermayer
On Tue, Feb 16, 2021 at 03:53:08PM +, Jeremy Leconte wrote: > --- > libavfilter/vf_scale.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/vf_scale.c b/libavfilter/vf_scale.c > index 58eee96744..98cef5eb4b 100644 > --- a/libavfilter/vf_scale.c > +++

Re: [FFmpeg-devel] [RFC] Event loop

2021-02-17 Thread Mark Thompson
On 17/02/2021 20:57, Nicolas George wrote: Mark Thompson (12021-02-17): I can see that this is all sensible stuff on Unix, but can you explain a bit more of what your Windows implementation is going to look like? I have no intention to write Windows code for this. Our current protocols use

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-17 Thread Mark Thompson
On 17/02/2021 18:58, Christopher Degawa wrote: On Sun, Feb 14, 2021 at 12:27 PM Christopher Degawa wrote: That said, if you have a general use-case where this is helpful and the documentation explains what it is doing (and warns about the bad cases) then maybe? The main use case we have

Re: [FFmpeg-devel] [RFC] Event loop

2021-02-17 Thread Lynne
Feb 17, 2021, 15:38 by geo...@nsup.org: > Lynne (12021-02-17): > >> I think I'd prefer an asynchronous library rather than libev. >> So libuv? >> > > I have on occasion straced node to debug things, and what I have > observed is: I don't want to touch it even with a very long stick. > > Wan you

Re: [FFmpeg-devel] [RFC] Event loop

2021-02-17 Thread Nicolas George
Mark Thompson (12021-02-17): > I can see that this is all sensible stuff on Unix, but can you explain > a bit more of what your Windows implementation is going to look like? I have no intention to write Windows code for this. Our current protocols use poll() internally, this event loop will do

Re: [FFmpeg-devel] [RFC] Event loop

2021-02-17 Thread Mark Thompson
On 01/02/2021 19:14, Nicolas George wrote: Based on this discussion, I say that the need for an event loop is confirmed. I will now start discussing actual plans for going forward. Since the coding work will be significant and not incremental, I want the outcome of this discussion to be

Re: [FFmpeg-devel] [PATCH] Moves yuv2yuvX_sse3 to yasm, unrolls main loop and other small optimizations for ~20% speedup.

2021-02-17 Thread Paul B Mahol
On Tue, Feb 16, 2021 at 6:31 PM Alan Kelly < alankelly-at-google@ffmpeg.org> wrote: > Looks like there are no comments, is this OK to be applied? Thanks > Applied, thanks for pinging. > > On Tue, Feb 9, 2021 at 6:25 PM Paul B Mahol wrote: > > > Will apply in no comments. > >

[FFmpeg-devel] GSoC 2020 results page

2021-02-17 Thread Michael Niedermayer
Hi all If you mentored a project in 2020, please fill the results on the results page https://trac.ffmpeg.org/wiki/SponsoringPrograms/GSoC/2020/Results Its empty ATM and previous years it was filled If you where a student of 2020 you can of course fill in your projects results too :) also

Re: [FFmpeg-devel] [PATCH] avcodec: add initial exr image encoder

2021-02-17 Thread Paul B Mahol
Will apply with these issues fixed soon. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH 2/2] avformat: add Simbiosis IMX demuxer

2021-02-17 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/imx.c| 160 +++ 3 files changed, 162 insertions(+) create mode 100644 libavformat/imx.c diff --git a/libavformat/Makefile

[FFmpeg-devel] [PATCH 1/2] avcodec: add Simbiosis IMX video decoder

2021-02-17 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/codec_desc.c | 7 ++ libavcodec/codec_id.h | 1 + libavcodec/imx.c| 153 5 files changed, 163 insertions(+) create mode 100644

Re: [FFmpeg-devel] [PATCH v3] avcodec/libvpxenc: fix potential memory leak.

2021-02-17 Thread Wonkap Jang
On Wed, Feb 17, 2021 at 9:50 AM Nicolas George wrote: > Wonkap Jang (12021-02-17): > > While parsing ref_frame_config, AVdictionary needs to be manually > > deallocated. > > --- > > libavcodec/libvpxenc.c | 21 + > > 1 file changed, 13 insertions(+), 8 deletions(-) > > > >

[FFmpeg-devel] [PATCH 2/3] checkasm/vf_gblur: split off the horiz_slice test into its own function

2021-02-17 Thread James Almer
Will come in handy for the following commit. Signed-off-by: James Almer --- tests/checkasm/vf_gblur.c | 28 +++- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/tests/checkasm/vf_gblur.c b/tests/checkasm/vf_gblur.c index 1d63fc22a0..8ff47a338f 100644 ---

Re: [FFmpeg-devel] Interested in contributing to gsoc

2021-02-17 Thread Michael Niedermayer
Hello Sanskar On Wed, Feb 17, 2021 at 06:54:05PM +0530, Sanskar Khandelwal wrote: > Hello, > > I am Sanskar Khandelwal, a 3rd year undergrad student ,I am interested in > contributing to ffmpeg and also participating in gsoc along with this year. > I have a few questions if someone can answer

[FFmpeg-devel] [PATCH] libavformat/dashenc.c: In some circumstances a total_duration equals to zero so that it makes possible for SIGFPE to appear and crash an application.

2021-02-17 Thread Sergei Iashin
--- libavformat/dashenc.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 2d757b3a87..fbdee126e9 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -844,7 +844,11 @@ static int

Re: [FFmpeg-devel] [PATCH] libsvtav1: Add logical_processors option

2021-02-17 Thread Christopher Degawa
On Sun, Feb 14, 2021 at 12:27 PM Christopher Degawa wrote: > That said, if you have a general use-case where this is helpful and the >> documentation explains what it is doing (and warns about the bad cases) >> then maybe? >> > > The main use case we have internally is to able to run multiple

Re: [FFmpeg-devel] GSoC 2021

2021-02-17 Thread Michael Niedermayer
On Tue, Feb 02, 2021 at 10:48:13AM +0100, Michael Niedermayer wrote: > Hi all > > Most people probably already know but just to be sure everyone knows > GSoC 2021 is 175h not 350h > https://groups.google.com/g/google-summer-of-code-discuss/c/GgvbLrFBcUQ?pli=1 > > Some project ideas may need to

Re: [FFmpeg-devel] [PATCH 1/3] x86/vf_gblur: fix postscale_slice prologue

2021-02-17 Thread James Almer
On 2/17/2021 3:34 PM, Michael Niedermayer wrote: On Wed, Feb 17, 2021 at 01:41:04PM -0300, James Almer wrote: x86_32 ABI does not pass float arguments directly on xmm regs, and the Win64 ABI uses only the first four regs for this purpose. Signed-off-by: James Almer ---

Re: [FFmpeg-devel] [PATCH 1/3] x86/vf_gblur: fix postscale_slice prologue

2021-02-17 Thread Michael Niedermayer
On Wed, Feb 17, 2021 at 01:41:04PM -0300, James Almer wrote: > x86_32 ABI does not pass float arguments directly on xmm regs, and the Win64 > ABI uses only the first four regs for this purpose. > > Signed-off-by: James Almer > --- > libavfilter/vf_gblur.c | 3 +-- >

Re: [FFmpeg-devel] [PATCH v3] avcodec/libvpxenc: fix potential memory leak.

2021-02-17 Thread Nicolas George
Wonkap Jang (12021-02-17): > While parsing ref_frame_config, AVdictionary needs to be manually > deallocated. > --- > libavcodec/libvpxenc.c | 21 + > 1 file changed, 13 insertions(+), 8 deletions(-) > > diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c > index

Re: [FFmpeg-devel] [PATCH v3] avcodec/libvpxenc: fix potential memory leak.

2021-02-17 Thread Nicolas George
Wonkap Jang (12021-02-17): > Or are you saying after getting the string with en->value, I should just > parse through the string without dictionary? Yes, exactly. You would use a dictionary if you need to keep all the values for a later use. But in this case, there is no later use, you only

[FFmpeg-devel] [PATCH v3] avcodec/libvpxenc: fix potential memory leak.

2021-02-17 Thread Wonkap Jang
While parsing ref_frame_config, AVdictionary needs to be manually deallocated. --- libavcodec/libvpxenc.c | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index 284cb9a108..56a1b5aafe 100644 ---

[FFmpeg-devel] [PATCH] avcodec/cfhd: Fix negative shift in cfhd_decode()

2021-02-17 Thread Michael Niedermayer
Fixes: left shift of negative value -1 Fixes: 30714/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_CFHD_fuzzer-4867823371419648 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/cfhd.c | 4

Re: [FFmpeg-devel] [PATCH 3/3] checkasm/vf_gblur: add a test for postscale_slice

2021-02-17 Thread James Almer
On 2/17/2021 1:47 PM, Paul B Mahol wrote: probably ok if tested Set pushed. Thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email

Re: [FFmpeg-devel] [PATCH v3] avcodec/libvpxenc: fix potential memory leak.

2021-02-17 Thread Wonkap Jang
On Wed, Feb 17, 2021 at 8:52 AM Wonkap Jang wrote: > Hi Nicolas, > > On Wed, Feb 17, 2021 at 3:00 AM Nicolas George wrote: > >> Wonkap Jang (12021-02-16): >> > While parsing ref_frame_config, AVdictionary needs to be manually >> > deallocated. >> > --- >> > libavcodec/libvpxenc.c | 19

Re: [FFmpeg-devel] [PATCH v3] avcodec/libvpxenc: fix potential memory leak.

2021-02-17 Thread Wonkap Jang
Hi Nicolas, On Wed, Feb 17, 2021 at 3:00 AM Nicolas George wrote: > Wonkap Jang (12021-02-16): > > While parsing ref_frame_config, AVdictionary needs to be manually > > deallocated. > > --- > > libavcodec/libvpxenc.c | 19 --- > > 1 file changed, 12 insertions(+), 7

Re: [FFmpeg-devel] [PATCH 3/3] checkasm/vf_gblur: add a test for postscale_slice

2021-02-17 Thread Paul B Mahol
probably ok if tested ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 1/3] x86/vf_gblur: fix postscale_slice prologue

2021-02-17 Thread Paul B Mahol
lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH 3/3] checkasm/vf_gblur: add a test for postscale_slice

2021-02-17 Thread James Almer
Signed-off-by: James Almer --- tests/checkasm/vf_gblur.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/tests/checkasm/vf_gblur.c b/tests/checkasm/vf_gblur.c index 8ff47a338f..b9fe2f9a36 100644 --- a/tests/checkasm/vf_gblur.c +++ b/tests/checkasm/vf_gblur.c @@ -16,6

[FFmpeg-devel] [PATCH 1/3] x86/vf_gblur: fix postscale_slice prologue

2021-02-17 Thread James Almer
x86_32 ABI does not pass float arguments directly on xmm regs, and the Win64 ABI uses only the first four regs for this purpose. Signed-off-by: James Almer --- libavfilter/vf_gblur.c | 3 +-- libavfilter/x86/vf_gblur.asm | 29 + 2 files changed, 14

[FFmpeg-devel] Fix test for complete frame in MxPEG decoder

2021-02-17 Thread Gabriele Sales
Hi, I've noticed that the MxPEG decoder fails to read a video stream if the bitmask used to track updated blocks has a length which is not a multiple of 8. The attached patch provides a tentative fix. Best, Gabriele 0001-Fix-test-for-complete-frame-in-MxPEG-decoder.patch Description: Binary

[FFmpeg-devel] [PATCH 2/2] avcodec/mediacodec_wrapper: use MediaCodecInfo.isSoftwareOnly() when available

2021-02-17 Thread sfan5
>From 22ebde779f61fb030633a881ef320264ea446b6b Mon Sep 17 00:00:00 2001 From: sfan5 Date: Thu, 11 Feb 2021 20:48:54 +0100 Subject: [PATCH 2/2] avcodec/mediacodec_wrapper: use MediaCodecInfo.isSoftwareOnly() when available Added in Android 10 it provides a reliable way of filtering out

[FFmpeg-devel] [PATCH 1/2] avcodec/mediacodec_wrapper: clean up ff_AMediaCodecList_getCodecNameByType a bit

2021-02-17 Thread sfan5
Hi, while looking into mediacodec for unrelated reasons I saw some room for improvement. Therefore, here's a series with two small patches. >From cadd2b2d4a5ffbb4dcc34faf2d3e139e1d4d608b Mon Sep 17 00:00:00 2001 From: sfan5 Date: Thu, 11 Feb 2021 19:23:26 +0100 Subject: [PATCH 1/2]

Re: [FFmpeg-devel] [RFC] Event loop

2021-02-17 Thread Nicolas George
Lynne (12021-02-17): > I think I'd prefer an asynchronous library rather than libev. > So libuv? I have on occasion straced node to debug things, and what I have observed is: I don't want to touch it even with a very long stick. Wan you explain more precisely why you think libuv would be a

Re: [FFmpeg-devel] [RFC] Event loop

2021-02-17 Thread Lynne
Feb 17, 2021, 12:47 by geo...@nsup.org: > James Almer (12021-02-01): > >> I support the idea of it being a build option (Perhaps simply >> --enable-libev, like any other external dep). There's a precedent of >> external libraries being used as backend for certain features, with the most >>

[FFmpeg-devel] [PATCH] avcodec: add exr format parser

2021-02-17 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 1 + libavcodec/exr_parser.c | 217 libavcodec/parsers.c| 1 + 3 files changed, 219 insertions(+) create mode 100644 libavcodec/exr_parser.c diff --git a/libavcodec/Makefile

Re: [FFmpeg-devel] [PATCH 5/6] pngdec: fix and simplify apng reference handling

2021-02-17 Thread Paul B Mahol
On Wed, Feb 17, 2021 at 12:33 PM Anton Khirnov wrote: > Quoting Paul B Mahol (2021-02-17 11:52:31) > > On Tue, Feb 16, 2021 at 9:26 PM Anton Khirnov wrote: > > > @@ -1088,23 +1084,23 @@ static int handle_p_frame_apng(AVCodecContext > > > *avctx, PNGDecContext *s, > > > if (!buffer) > > >

Re: [FFmpeg-devel] [PATCH 4/6] Revert "avcodec/pngdec: fix possible race condition with APNG decoding"

2021-02-17 Thread Paul B Mahol
On Wed, Feb 17, 2021 at 12:31 PM Anton Khirnov wrote: > Quoting Paul B Mahol (2021-02-17 11:45:02) > > On Wed, Feb 17, 2021 at 9:14 AM Anton Khirnov wrote: > > > > > Quoting Paul B Mahol (2021-02-16 21:57:47) > > > > Do you have actual proof for such claims? > > > > > > The burden of proof is

[FFmpeg-devel] Interested in contributing to gsoc

2021-02-17 Thread Sanskar Khandelwal
Hello, I am Sanskar Khandelwal, a 3rd year undergrad student ,I am interested in contributing to ffmpeg and also participating in gsoc along with this year. I have a few questions if someone can answer them and help me get started it'll be nice. 1. Is this a right platform to ask questions, I

Re: [FFmpeg-devel] [RFC] Event loop

2021-02-17 Thread Nicolas George
James Almer (12021-02-01): > I support the idea of it being a build option (Perhaps simply > --enable-libev, like any other external dep). There's a precedent of > external libraries being used as backend for certain features, with the most > prominent example being libsoxr as the resample engine

Re: [FFmpeg-devel] [PATCH 5/6] pngdec: fix and simplify apng reference handling

2021-02-17 Thread Anton Khirnov
Quoting Paul B Mahol (2021-02-17 11:52:31) > On Tue, Feb 16, 2021 at 9:26 PM Anton Khirnov wrote: > > @@ -1088,23 +1084,23 @@ static int handle_p_frame_apng(AVCodecContext > > *avctx, PNGDecContext *s, > > if (!buffer) > > return AVERROR(ENOMEM); > > > > +

Re: [FFmpeg-devel] [PATCH] lavfi/drawtext: ignore final LF of textfile.

2021-02-17 Thread Nicolas George
Nicolas George (12020-11-30): > Good idea, thanks. Pushed. Regards, -- Nicolas George signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe,

Re: [FFmpeg-devel] [PATCH 4/6] Revert "avcodec/pngdec: fix possible race condition with APNG decoding"

2021-02-17 Thread Anton Khirnov
Quoting Paul B Mahol (2021-02-17 11:45:02) > On Wed, Feb 17, 2021 at 9:14 AM Anton Khirnov wrote: > > > Quoting Paul B Mahol (2021-02-16 21:57:47) > > > Do you have actual proof for such claims? > > > > The burden of proof is on you here - you are supposed to prove that your > > commit fixes

Re: [FFmpeg-devel] [PATCH 1/1] avcodec/libopusdec: Enable FEC/PLC

2021-02-17 Thread Andreas Rheinhardt
Philip-Dylan Gleonec: >> Could you elaborate? >> I would have expected that the normal use case is not have a >> lossy input and that the new feature is always useful if data >> was lost. > > The use-case for FEC is typically RTP stream where audio is compressed > with opus. In that case,

Re: [FFmpeg-devel] [PATCH 1/1] avcodec/libopusdec: Enable FEC/PLC

2021-02-17 Thread Philip-Dylan Gleonec
> I've added him to the CC list Now done. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 1/1] avcodec/libopusdec: Enable FEC/PLC

2021-02-17 Thread Philip-Dylan Gleonec
>> Am Di., 16. Feb. 2021 um 15:02 Uhr schrieb Philip-Dylan Gleonec >> : >> >>> >>> Adds FEC/PLC support to libopus. The lost packets are detected as a >>> discontinuity in the audio stream. When a discontinuity is used, this >>> patch tries to decode the FEC data. If FEC data is present in the >>>

Re: [FFmpeg-devel] [PATCH 1/1] avcodec/libopusdec: Enable FEC/PLC

2021-02-17 Thread Philip-Dylan Gleonec
> Could you elaborate? > I would have expected that the normal use case is not have a > lossy input and that the new feature is always useful if data > was lost. The use-case for FEC is typically RTP stream where audio is compressed with opus. In that case, depending on the network conditions,

Re: [FFmpeg-devel] [PATCH 4/6] Revert "avcodec/pngdec: fix possible race condition with APNG decoding"

2021-02-17 Thread Jean-Baptiste Kempf
On Wed, 17 Feb 2021, at 11:45, Paul B Mahol wrote: > > It didn't fix anything though. Before your commit, frame 69 of the > > sample in #9017 is almost black with 1 thread and looks okay with 2 > > threads. After your commit, it is almost black in both cases. So your > > commit made it

Re: [FFmpeg-devel] [PATCH v3] avcodec/libvpxenc: fix potential memory leak.

2021-02-17 Thread Nicolas George
Wonkap Jang (12021-02-16): > While parsing ref_frame_config, AVdictionary needs to be manually > deallocated. > --- > libavcodec/libvpxenc.c | 19 --- > 1 file changed, 12 insertions(+), 7 deletions(-) NAK. This code is all wrong, it looks like Java or Python, it should not be

Re: [FFmpeg-devel] [PATCH 5/6] pngdec: fix and simplify apng reference handling

2021-02-17 Thread Paul B Mahol
On Tue, Feb 16, 2021 at 9:26 PM Anton Khirnov wrote: > Current code is very confused and confusing. It uses two different > reference frames - "previous" and "last" - when only one is really > necessary. It also confuses the two, leading to incorrect output with > APNG_DISPOSE_OP_PREVIOUS mode.

Re: [FFmpeg-devel] [PATCH 4/6] Revert "avcodec/pngdec: fix possible race condition with APNG decoding"

2021-02-17 Thread Paul B Mahol
On Wed, Feb 17, 2021 at 9:14 AM Anton Khirnov wrote: > Quoting Paul B Mahol (2021-02-16 21:57:47) > > Do you have actual proof for such claims? > > The burden of proof is on you here - you are supposed to prove that your > commit fixes something. And when I asked you about details, you couldn't

Re: [FFmpeg-devel] [PATCH] arm/aarch64: Use mach_absolute_time as timer on apple platforms

2021-02-17 Thread Martin Storsjö
On Fri, 12 Feb 2021, Martin Storsjö wrote: This is much less precise than the cycle counter register, but the cycle counter register is not available on apple platforms (and on linux, it requires a kernel module for allowing user mode access). --- libavutil/aarch64/timer.h | 8 +++-

[FFmpeg-devel] [PATCH 06/19] avformat/matroskadec: Make reading zero-length elements spec-compliant

2021-02-17 Thread Andreas Rheinhardt
For a very long time, the payload of integer and float elements had to have a length > 0. Our parser treated such invalid elements as having a value zero. But now it has been defined what an EBML element with length zero means: It is a shorthand for the default value. This has also been defined

[FFmpeg-devel] [PATCH 16/19] avformat/matroskaenc: Add support for FlagTextDescriptions

2021-02-17 Thread Andreas Rheinhardt
This is the Matroska equivalent of D_WEBVTT_DESCRIPTIONS and is therefore only enabled for subtitles. Signed-off-by: Andreas Rheinhardt --- libavformat/matroskaenc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index

[FFmpeg-devel] [PATCH 19/19] fate/matroska: Add fate-matroska target

2021-02-17 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- tests/fate/matroska.mak | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/fate/matroska.mak b/tests/fate/matroska.mak index b907da40f3..c7da2465b5 100644 --- a/tests/fate/matroska.mak +++ b/tests/fate/matroska.mak @@ -129,3 +129,5 @@

[FFmpeg-devel] [PATCH 08/19] avformat/matroskadec: Beautify setting default values

2021-02-17 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavformat/matroskadec.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 636ed23f0d..b9c6047f56 100644 --- a/libavformat/matroskadec.c +++

[FFmpeg-devel] [PATCH 12/19] avformat/matroskaenc: Add support for FlagOriginal

2021-02-17 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavformat/matroskaenc.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 746c0b347a..0dd093ae8b 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -1193,6 +1193,12

[FFmpeg-devel] [PATCH 09/19] avformat/matroska, matroskadec: Support FlagCommentary

2021-02-17 Thread Andreas Rheinhardt
Hint: Matroska actually provides a way to distinguish the cases of "track is no commentary track" and "it is unknown whether the track is a commentary track", but our disposition flags do not. Therefore we need not use a CountedElement. Signed-off-by: Andreas Rheinhardt ---

  1   2   >