Re: [FFmpeg-devel] [PATCH v14 3/4] avcodec/libjxl: add Jpeg XL encoding via libjxl

2022-04-14 Thread Leo Izen
On 4/14/22 13:49, Anton Khirnov wrote: Quoting Leo Izen (2022-04-12 07:53:32) + +while (1) { +jret = JxlEncoderProcessOutput(ctx->encoder, _out, ); +if (jret == JXL_ENC_ERROR) { +av_log(avctx, AV_LOG_ERROR, "Unspecified libjxl error occurred\n"); +

[FFmpeg-devel] [PATCH] avfilter: add warp video filter

2022-04-14 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_warp.c| 628 +++ 3 files changed, 630 insertions(+) create mode 100644 libavfilter/vf_warp.c diff --git a/libavfilter/Makefile

Re: [FFmpeg-devel] [PATCH 6/7] avcodec: Make avcodec_decoder_subtitles2 accept a const AVPacket*

2022-04-14 Thread Hendrik Leppkes
On Thu, Apr 14, 2022 at 9:33 PM Andreas Rheinhardt wrote: > > Anton Khirnov: > > Quoting Andreas Rheinhardt (2022-03-31 00:49:57) > >> From: Andreas Rheinhardt > >> > >> Signed-off-by: Andreas Rheinhardt > >> --- > >> doc/APIchanges| 3 +++ > >> fftools/ffmpeg.c | 4 ++-- >

Re: [FFmpeg-devel] [RFC] Switching ffmpeg.c to a threaded architecture

2022-04-14 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of Paul > B Mahol > Sent: Thursday, April 14, 2022 12:02 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [RFC] Switching ffmpeg.c to a threaded > architecture > > On Wed, Apr 13,

[FFmpeg-devel] [PATCH] avutil/cpu: #define _GNU_SOURCE before including any standard headers

2022-04-14 Thread Marton Balint
Otherwise its effect might not work causing CPU_COUNT to not get defined. Fixes cpu count detection to actually use sched_getaffinity if available. Signed-off-by: Marton Balint --- libavutil/cpu.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git

[FFmpeg-devel] [PATCH] avformat/rtmpproto: send proper status for response to play command

2022-04-14 Thread Marton Balint
This fixes referencing the uninitialized filename variable. Fixes ticket #9711. Signed-off-by: Marton Balint --- libavformat/rtmpproto.c | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index

Re: [FFmpeg-devel] [PATCH] swscale/aarch64: add hscale specializations

2022-04-14 Thread Clément Bœsch
Hi, On Thu, Apr 14, 2022 at 04:25:19PM +, Swinney, Jonathan wrote: > Clement (and others), do you have any comments on this patch? > I'm sorry I won't have time to review the asm patches. Regards, -- Clément B. ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 6/7] avcodec: Make avcodec_decoder_subtitles2 accept a const AVPacket*

2022-04-14 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2022-03-31 00:49:57) >> From: Andreas Rheinhardt >> >> Signed-off-by: Andreas Rheinhardt >> --- >> doc/APIchanges| 3 +++ >> fftools/ffmpeg.c | 4 ++-- >> fftools/ffprobe.c | 2 +- >> libavcodec/avcodec.h | 3 +-- >>

Re: [FFmpeg-devel] [PATCH v14 3/4] avcodec/libjxl: add Jpeg XL encoding via libjxl

2022-04-14 Thread Anton Khirnov
Quoting Leo Izen (2022-04-12 07:53:32) > + > +while (1) { > +jret = JxlEncoderProcessOutput(ctx->encoder, _out, ); > +if (jret == JXL_ENC_ERROR) { > +av_log(avctx, AV_LOG_ERROR, "Unspecified libjxl error > occurred\n"); > +return AVERROR_EXTERNAL; > +

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/mjpegdec: Don't create unnecessary AVFrame reference

2022-04-14 Thread Andreas Rheinhardt
James Almer: > > > On 4/14/2022 1:43 PM, Andreas Rheinhardt wrote: >> James Almer: >>> On 4/14/2022 12:57 PM, Andreas Rheinhardt wrote: MJPEG is an intra-codec, so it makes no sense to keep the reference. It is basically unused lateron anyway. >>> >>> Wouldn't this mean you can remove

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/mjpegdec: Don't create unnecessary AVFrame reference

2022-04-14 Thread James Almer
On 4/14/2022 1:43 PM, Andreas Rheinhardt wrote: James Almer: On 4/14/2022 12:57 PM, Andreas Rheinhardt wrote: MJPEG is an intra-codec, so it makes no sense to keep the reference. It is basically unused lateron anyway. Wouldn't this mean you can remove the AV_GET_BUFFER_FLAG_REF flag from

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/mjpegdec: Don't create unnecessary AVFrame reference

2022-04-14 Thread Andreas Rheinhardt
James Almer: > On 4/14/2022 12:57 PM, Andreas Rheinhardt wrote: >> MJPEG is an intra-codec, so it makes no sense to keep the reference. >> It is basically unused lateron anyway. > > Wouldn't this mean you can remove the AV_GET_BUFFER_FLAG_REF flag from > the ff_get_buffer() call? > Not in case

Re: [FFmpeg-devel] [PATCH 2/5] avcodec/mjpegdec: Don't create unnecessary AVFrame reference

2022-04-14 Thread James Almer
On 4/14/2022 12:57 PM, Andreas Rheinhardt wrote: MJPEG is an intra-codec, so it makes no sense to keep the reference. It is basically unused lateron anyway. Wouldn't this mean you can remove the AV_GET_BUFFER_FLAG_REF flag from the ff_get_buffer() call? Signed-off-by: Andreas Rheinhardt

[FFmpeg-devel] [PATCH 5/5] avcodec/mjpegbdec: Don't use GetBit-API for byte-aligned reads

2022-04-14 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/mjpegbdec.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/libavcodec/mjpegbdec.c b/libavcodec/mjpegbdec.c index 46b9eb377e..50fff2562b 100644 --- a/libavcodec/mjpegbdec.c +++ b/libavcodec/mjpegbdec.c

[FFmpeg-devel] [PATCH 3/5] avcodec/mjpegdec: Avoid copying data when flipping image

2022-04-14 Thread Andreas Rheinhardt
Basically reverts af15c17daa5d8d2940c0263ba4d3ecec761c11ee. Flipping a picture by modifying the pointers is so common that even users of direct rendering should take it into account. Signed-off-by: Andreas Rheinhardt --- libavcodec/mjpegdec.c | 18 -- 1 file changed, 4

[FFmpeg-devel] [PATCH 4/5] avcodec/mjpegbdec: Don't create unnecessary AVFrame reference

2022-04-14 Thread Andreas Rheinhardt
MJPEG-B is an intra-codec, so it makes no sense to keep the reference. It is unused lateron anyway. Signed-off-by: Andreas Rheinhardt --- libavcodec/mjpegbdec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/mjpegbdec.c b/libavcodec/mjpegbdec.c index

[FFmpeg-devel] [PATCH 2/5] avcodec/mjpegdec: Don't create unnecessary AVFrame reference

2022-04-14 Thread Andreas Rheinhardt
MJPEG is an intra-codec, so it makes no sense to keep the reference. It is basically unused lateron anyway. Signed-off-by: Andreas Rheinhardt --- libavcodec/mjpegdec.c | 45 +-- 1 file changed, 22 insertions(+), 23 deletions(-) diff --git

[FFmpeg-devel] [PATCH 1/5] avcodec/mjpegdec: Always reset got_picture at the beginnig of decoding

2022-04-14 Thread Andreas Rheinhardt
Said field is set when parsing a SOF; yet a picture is only allocated if skip_frame is != AVDISCARD_ALL. This leads to a crash in the following case: If a jpeg is split into two parts, the first containing everything before the scans including the SOF and the second part containing the rest, and

Re: [FFmpeg-devel] [PATCH v1] avformat/ipfsgateway: define PATH_MAX

2022-04-14 Thread Mark Gaiser
On Thu, Apr 14, 2022 at 3:41 PM Martin Storsjö wrote: > On Thu, 14 Apr 2022, Mark Gaiser wrote: > > > On Thu, Apr 14, 2022 at 10:25 AM Martin Storsjö > wrote: > > > >> On Wed, 13 Apr 2022, Mark Gaiser wrote: > >> > >> > On Wed, Apr 13, 2022 at 5:21 PM Mark Gaiser > wrote: > >> > > >> >>

Re: [FFmpeg-devel] [PATCH v1] avformat/ipfsgateway: define PATH_MAX

2022-04-14 Thread Martin Storsjö
On Thu, 14 Apr 2022, Mark Gaiser wrote: On Thu, Apr 14, 2022 at 10:25 AM Martin Storsjö wrote: On Wed, 13 Apr 2022, Mark Gaiser wrote: > On Wed, Apr 13, 2022 at 5:21 PM Mark Gaiser wrote: > >> PATH_MAX is posix. Some compilers (MSVC) don't define this >> thus failing to compile the

Re: [FFmpeg-devel] [PATCH v1] avformat/ipfsgateway: define PATH_MAX

2022-04-14 Thread Mark Gaiser
On Thu, Apr 14, 2022 at 10:25 AM Martin Storsjö wrote: > On Wed, 13 Apr 2022, Mark Gaiser wrote: > > > On Wed, Apr 13, 2022 at 5:21 PM Mark Gaiser wrote: > > > >> PATH_MAX is posix. Some compilers (MSVC) don't define this > >> thus failing to compile the ipfsgateway file. > >> Defining it fixes

Re: [FFmpeg-devel] [PATCH v2 5/5] avcodec/x86/hevc_mc: add qpel_h64_8_avx512icl

2022-04-14 Thread Wu Jianhua
Ping! Wu Jianhua: >Henrik Gramner: >> Sent: Friday, March 11, 2022 10:51 PM >> To: FFmpeg development discussions and patches > devel at ffmpeg.org> >> Subject: Re: [FFmpeg-devel] [PATCH v2 5/5] avcodec/x86/hevc_mc: add >> qpel_h64_8_avx512icl >> >> All 5/5 LGTM. >> > >Hi there, > >Are there any

Re: [FFmpeg-devel] [PATCH 1/2] libavutil/hwcontext_vulkan: Fix VK_FORMAT_R8G8_UNORM and VK_FORMAT_R16G16_UNORM map problem on Vulkan

2022-04-14 Thread Lynne
13 Apr 2022, 08:05 by haihao.xiang-at-intel@ffmpeg.org: > On Wed, 2022-03-30 at 11:41 +0800, Wenbin Chen wrote: > >> Vulkan map both DRM_FORMAT_RG88 and DRM_FORMAT_GR88 to >> VK_FORMAT_R8G8_UNORM. This cannot distinguish nv12/nv24 and nv21/nv42. >> Vulkan also map both DRM_FORMAT_RG1616 and

Re: [FFmpeg-devel] [PATCH] avfilter: add feedback video filter

2022-04-14 Thread Paul B Mahol
Will apply soon, as there are no comments against. ___ 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

Re: [FFmpeg-devel] [PATCH] avfilter: add pixelize video filter

2022-04-14 Thread Paul B Mahol
relatively trivial filter, gonna apply 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".

Re: [FFmpeg-devel] [RFC] Switching ffmpeg.c to a threaded architecture

2022-04-14 Thread Paul B Mahol
On Wed, Apr 13, 2022 at 12:43 AM Soft Works wrote: > > > > -Original Message- > > From: ffmpeg-devel On Behalf Of Paul > > B Mahol > > Sent: Tuesday, April 12, 2022 11:29 AM > > To: FFmpeg development discussions and patches > de...@ffmpeg.org> > > Subject: Re: [FFmpeg-devel] [RFC]

Re: [FFmpeg-devel] [PATCH v1] avformat/ipfsgateway: define PATH_MAX

2022-04-14 Thread Martin Storsjö
On Wed, 13 Apr 2022, Mark Gaiser wrote: On Wed, Apr 13, 2022 at 5:21 PM Mark Gaiser wrote: PATH_MAX is posix. Some compilers (MSVC) don't define this thus failing to compile the ipfsgateway file. Defining it fixes the compile. Signed-off-by: Mark Gaiser --- libavformat/ipfsgateway.c | 6

Re: [FFmpeg-devel] [PATCH] libavcodec/videotoolboxenc.c: add option to hevc encoder to prioritize speed.

2022-04-14 Thread Thilo Borgmann
Hi, Am 14.04.22 um 08:48 schrieb Simone Karin Lehmann: Somehow it seems, that this patch never got onto the mailing list. So may I kindly ask you to review it. -- Stay hungry, stay foolish Am 26.01.2022 um 17:31 schrieb Simone Karin Lehmann :  The patch adds an option to the

Re: [FFmpeg-devel] [PATCH 2/2] lavfi: Add blurriness filter

2022-04-14 Thread Thilo Borgmann
Am 05.04.22 um 10:56 schrieb Thilo Borgmann: Hi, v3 updated to current HEAD. Named blurdetect filter now. Minor fixes on allocation and removed -f option. Please make this per plane filtering, with default to measure only first plane. done in v4. (Will add Changelog, version.h and fate

Re: [FFmpeg-devel] [PATCH v2] avformat/mpegts: set data broadcast streams as such

2022-04-14 Thread Jan Ekström
On Mon, Apr 11, 2022 at 1:50 PM Jan Ekström wrote: > > From: Jan Ekström > > Additionally, they should not be probed, as this is essentially > various types of binary data. > > Signed-off-by: Jan Ekström > --- Ping. Basically this checks if we have an unknown stream with a private stream type

Re: [FFmpeg-devel] [PATCH] libavcodec/videotoolboxenc.c: add option to hevc encoder to prioritize speed.

2022-04-14 Thread Simone Karin Lehmann
Somehow it seems, that this patch never got onto the mailing list. So may I kindly ask you to review it. -- Stay hungry, stay foolish > Am 26.01.2022 um 17:31 schrieb Simone Karin Lehmann : > >  > The patch adds an option to the hevc_videotoolbox encoder to prioritize speed > for Macs with

[FFmpeg-devel] avcodec/audiotoolboxdec: fix missing decoder out properties

2022-04-14 Thread MCC CS
Resubmit: fix title. This is a repost of the patch by Chris Snowhill who originally sent this patch on 31 December - which didn't receive any replies. As you can see on https://trac.ffmpeg.org/ticket/9564 the patch is needed and works perfectly. I have rebased it and recompiled it. Original