Re: [FFmpeg-devel] [PATCH 2/4] avcodec/encode: restructure the core encoding code

2020-02-29 Thread Andriy Gelman
On Thu, 27. Feb 15:02, James Almer wrote: > This commit follows the same logic as 061a0c14bb, but for the encode API: The > new public encoding API will no longer be a wrapper around the old deprecated > one, and the internal API used by the encoders now consists of a single > receive_packet() call

[FFmpeg-devel] [PATCH] avcodec/cuviddec: use AVCodec.bsfs to filter packets

2020-02-29 Thread James Almer
Simplifies code considerably. Signed-off-by: James Almer --- libavcodec/cuviddec.c | 78 +-- 1 file changed, 16 insertions(+), 62 deletions(-) diff --git a/libavcodec/cuviddec.c b/libavcodec/cuviddec.c index acee78cf2c..69c64055fe 100644 --- a/libavcodec/

[FFmpeg-devel] [PATCH] avcodec/siren: Check several indexes

2020-02-29 Thread Michael Niedermayer
Fixes: Multiple out of array accesses Fixes: 20817/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SIREN_fuzzer-5754041227542528.fuzz Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/siren.c

[FFmpeg-devel] [PATCH 2/2] fftools/ffmpeg_opt: remove bogus warning of multiple -af and -vf usage

2020-02-29 Thread Marton Balint
This is redundant after the last patch and also fixes ticket #7712. Signed-off-by: Marton Balint --- fftools/ffmpeg_opt.c | 4 1 file changed, 4 deletions(-) diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c index 3d6fafe073..df6928ee10 100644 --- a/fftools/ffmpeg_opt.c +++ b/fftool

[FFmpeg-devel] [PATCH 1/2] fftools/ffmpeg_opt: warn about overwritten parsed options

2020-02-29 Thread Marton Balint
Signed-off-by: Marton Balint --- fftools/ffmpeg_opt.c | 70 +--- 1 file changed, 67 insertions(+), 3 deletions(-) diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c index 12d44886ee..3d6fafe073 100644 --- a/fftools/ffmpeg_opt.c +++ b/fftools/

Re: [FFmpeg-devel] [PATCH 2/2] avfilter/vf_cas: Remove superfluous ;

2020-02-29 Thread Michael Niedermayer
On Fri, Feb 28, 2020 at 11:30:43PM +0100, Paul B Mahol wrote: > lgtm will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many things microsoft did are stupid, but not doing something just because microsoft did it is even more stupid. If everything ms

Re: [FFmpeg-devel] [PATCH 1/2] fftools/ffmpeg_opt: Fix leak of options when parsing options fails

2020-02-29 Thread Michael Niedermayer
On Fri, Feb 28, 2020 at 11:30:28PM +0100, Paul B Mahol wrote: > lgtm will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Dictatorship naturally arises out of democracy, and the most aggravated form of tyranny and slavery out of the most extreme liber

Re: [FFmpeg-devel] [PATCH 1/1] Patch for adding Documentation of ff_http_match_no_proxy

2020-02-29 Thread Michael Niedermayer
On Sat, Feb 29, 2020 at 05:07:30PM +0530, Sourabh Sharma wrote: > Patch for adding Documentation of ff_http_match_no_proxy > > Function ff_http_match_no_proxy check for host of proxy address > matches with hostname or not. > --- > libavformat/network.h | 8 +++- > 1 file changed, 7 insertions

Re: [FFmpeg-devel] [avfomat/rtp: source ips lost when specified as URL options] Patch for ffmpeg using rtp protocol where sources option is not retained

2020-02-29 Thread Ross Nicholson
Ping... > On 26 Feb 2020, at 08:57, Ross Nicholson wrote: > >  > Thanks, just thought you might have another idea, as it works just was not > sure if I was accomplishing it in the right way. > > Ya, I submitted the patch formally. Let’s see what’s a review comes back like. > > Thanks > >>>

[FFmpeg-devel] [PATCH v2] avformat/mxfenc: use a zero based continuity counter

2020-02-29 Thread Marton Balint
The standard does not seem to require the counter to be zero based, but some checker tools (MyriadBits MXFInspect, Interra Baton) have validations against 0 start... Fixes ticket #6781. Signed-off-by: Marton Balint --- libavformat/mxfenc.c | 2 +- tests/ref/fate/mxf-user-comments |

[FFmpeg-devel] [PATCH 4/4 v2] avcodec/librav1e: adapt to the new internal encode API

2020-02-29 Thread James Almer
Signed-off-by: James Almer --- Fixed retrying to feed librav1e a frame after it returned RA_ENCODER_STATUS_ENOUGH_DATA the previous time (Can't be reproduced right now as librav1e hasn't yet introduced a queue limit). Also, no reindenting stuff this time to show the needed changes are trivial. Mu

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-02-29 Thread Carl Eugen Hoyos
> Am 29.02.2020 um 17:19 schrieb Marton Balint : > > > >> On Sat, 29 Feb 2020, Carl Eugen Hoyos wrote: >> >> Am Fr., 28. Feb. 2020 um 21:57 Uhr schrieb Andriy Gelman >> : >> >>> @@ -1789,6 +1790,7 @@ EXTERNAL_LIBRARY_LIST=" >>> libopenmpt >>> libopus >>> libpulse >>> +librabb

[FFmpeg-devel] [PATCH] avformat/mxfenc: use a zero based continuity counter

2020-02-29 Thread Marton Balint
The standard does not seem to require the counter to be zero based, but some checker tools (MyriadBits MXFInspect, Interra Baton) have validations against 0 start... Fixes ticket #6781. Signed-off-by: Marton Balint --- libavformat/mxfenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [FFmpeg-devel] GSoC: Regarding Parsing and FLIF16 Frame Encoding

2020-02-29 Thread Jai Luthra
Hi Anamitra, On Sat, Feb 29, 2020 at 04:50:23AM +, Anamitra Ghorui wrote: Hello, I have been reading through the parsing API and other things and here's what I've managed to gather (I will be ignoring overruns in these functions for now). Please tell me if I am right or wrong: 1. As long as

Re: [FFmpeg-devel] GSoC: Regarding Parsing and FLIF16 Frame Encoding

2020-02-29 Thread Kartik K. Khullar
It is just to remind that I am already working on Transformations involved in FLIF and the functions which these transformations use like Symbol Encoding @Anamitra. It would be helpful if someone could help me clear what does RAC refer to in FLIF spec. It is mentioned under Symbol Encoding and is b

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-02-29 Thread Andriy Gelman
On Sat, 29. Feb 11:21, Paul B Mahol wrote: > I think this was already rejected? Thilo had a question on broker performance, which I believe addressed: http://ffmpeg.org/pipermail/ffmpeg-devel/2020-February/257483.html j-b had concerns about how it fits into libavformat: http://ffmpeg.org/pipermai

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-02-29 Thread Paul B Mahol
On 2/29/20, Andriy Gelman wrote: > On Sat, 29. Feb 11:21, Paul B Mahol wrote: >> I think this was already rejected? > > Thilo had a question on broker performance, which I believe addressed: > http://ffmpeg.org/pipermail/ffmpeg-devel/2020-February/257483.html > > j-b had concerns about how it fits

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-02-29 Thread Marton Balint
On Sat, 29 Feb 2020, Carl Eugen Hoyos wrote: Am Fr., 28. Feb. 2020 um 21:57 Uhr schrieb Andriy Gelman : @@ -1789,6 +1790,7 @@ EXTERNAL_LIBRARY_LIST=" libopenmpt libopus libpulse +librabbitmq librav1e librsvg librtmp If the patch is acceptable, I believe th

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-02-29 Thread Marton Balint
On Sat, 29 Feb 2020, Paul B Mahol wrote: I think this was already rejected? jb questioned if this belongs to libavformat, and timo asked how well the message brokers handle high bitrates/big message sizes, no hard rejects were made as far as I remember. Andriy provided numbers for scalin

[FFmpeg-devel] [PATCH 2/4] avutil/frame: Rename QP related fields so unintentional use is impossible

2020-02-29 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavutil/frame.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavutil/frame.h b/libavutil/frame.h index b5afb58634..b966f37fe0 100644 --- a/libavutil/frame.h +++ b/libavutil/frame.h @@ -614,18 +614,18 @@ typedef struct AVFra

[FFmpeg-devel] [PATCH 4/4] libavutil: AVEncodeInfo data structures

2020-02-29 Thread Michael Niedermayer
From: Juan De León AVEncodeInfoFrame data structure to store as AVFrameSideData of type AV_FRAME_DATA_ENCODE_INFO. The structure stores quantization index for each plane, DC/AC deltas for luma and chroma planes, and an array of AVEncodeInfoBlock type denoting position, size, and delta quantizer f

[FFmpeg-devel] [PATCH 3/4] avutil: frame Update the existing QP API

2020-02-29 Thread Michael Niedermayer
This should extend the API to allow exporting internal tables for codecs with block sizes different from 16x16 and different values per plane or multiple values per block. This is unfinished and only to demonstrate how such API (which maintains API/ABI compatibility) would look This API may allow

[FFmpeg-devel] [RFC] 2 Alternative approaches for AVFrame QP Tables

2020-02-29 Thread Michael Niedermayer
[PATCH 1/4] Remove all uses of AVFrame QP tables and related fields [PATCH 2/4] avutil/frame: Rename QP related fields so unintentional These 2 are common in both options, they break API but not ABI so should not be applied before the next release [PATCH 3/4] avutil: frame Update the existing Q

[FFmpeg-devel] [PATCH 1/4] Remove all uses of AVFrame QP tables and related fields in AVFrame

2020-02-29 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavutil/frame.c | 62 +-- 1 file changed, 12 insertions(+), 50 deletions(-) diff --git a/libavutil/frame.c b/libavutil/frame.c index e4038096c2..89089c6cc0 100644 --- a/libavutil/frame.c +++ b/libavutil/frame.c

Re: [FFmpeg-devel] GSoC: Regarding Parsing and FLIF16 Frame Encoding

2020-02-29 Thread Anamitra Ghorui
Oh, sorry about that. ___ 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 1/1] Patch for adding Documentation of ff_http_match_no_proxy

2020-02-29 Thread Sourabh Sharma
Patch for adding Documentation of ff_http_match_no_proxy Function ff_http_match_no_proxy check for host of proxy address matches with hostname or not. --- libavformat/network.h | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavformat/network.h b/libavformat/network.h

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-02-29 Thread Carl Eugen Hoyos
Am Fr., 28. Feb. 2020 um 21:57 Uhr schrieb Andriy Gelman : > @@ -1789,6 +1790,7 @@ EXTERNAL_LIBRARY_LIST=" > libopenmpt > libopus > libpulse > +librabbitmq > librav1e > librsvg > librtmp If the patch is acceptable, I believe this belongs in EXTERNAL_LIBRARY_GPL_L

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat: Add AMQP version 0-9-1 protocol support

2020-02-29 Thread Paul B Mahol
I think this was already rejected? On 2/28/20, Andriy Gelman wrote: > From: Andriy Gelman > > Supports connecting to a RabbitMQ broker via AMQP version 0-9-1. > > Signed-off-by: Andriy Gelman > --- > > Changes in v2: > - Addressed comments from Marton > - Updated documentation > > Compi

Re: [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: fix the infinite loop while framerate lower than input

2020-02-29 Thread Fu, Linjie
> -Original Message- > From: Zhong Li > Sent: Saturday, February 29, 2020 13:14 > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Cc: Fu, Linjie > Subject: Re: [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: fix the infinite loop > while framerate lower than input > > Linjie