Re: [FFmpeg-devel] [PATCH 3/3] avformat/movenc: Add support for AVIF muxing

2022-05-13 Thread Gyan Doshi
On 2022-05-12 09:53 pm, Vignesh Venkatasubramanian wrote: Add an AVIF muxer by re-using the existing the mov/mp4 muxer. Bumped lavf minor and pushed set as    ab05e9a7f2...84241e63cf Regards, Gyan AVIF Specification: https://aomediacodec.github.io/av1-avif Sample usage for still image:

Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move ff_nlmeans_init into a header

2022-05-13 Thread Andreas Rheinhardt
Soft Works: > > >> -Original Message- >> From: ffmpeg-devel On Behalf Of >> Andreas Rheinhardt >> Sent: Tuesday, May 3, 2022 8:38 AM >> To: ffmpeg-devel@ffmpeg.org >> Cc: Andreas Rheinhardt >> Subject: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move >> ff_nlmeans_init into a head

Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move ff_nlmeans_init into a header

2022-05-13 Thread Hendrik Leppkes
On Fri, May 13, 2022 at 10:27 AM Andreas Rheinhardt wrote: > > Soft Works: > > > > > >> -Original Message- > >> From: ffmpeg-devel On Behalf Of > >> Andreas Rheinhardt > >> Sent: Tuesday, May 3, 2022 8:38 AM > >> To: ffmpeg-devel@ffmpeg.org > >> Cc: Andreas Rheinhardt > >> Subject: [FFmp

Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move ff_nlmeans_init into a header

2022-05-13 Thread Soft Works
> -Original Message- > From: Andreas Rheinhardt > Sent: Friday, May 13, 2022 10:27 AM > To: Soft Works ; FFmpeg development discussions > and patches > Subject: Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move > ff_nlmeans_init into a header > > Soft Works: > > > > > >>

Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move ff_nlmeans_init into a header

2022-05-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Hendrik Leppkes > Sent: Friday, May 13, 2022 11:00 AM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move > ff_nlmeans_init into a header > > O

Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move ff_nlmeans_init into a header

2022-05-13 Thread Hendrik Leppkes
On Fri, May 13, 2022 at 11:01 AM Soft Works wrote: > > > > > -Original Message- > > From: Andreas Rheinhardt > > Sent: Friday, May 13, 2022 10:27 AM > > To: Soft Works ; FFmpeg development discussions > > and patches > > Subject: Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move

[FFmpeg-devel] [PATCH] avfilter, avcodec/*_init.h: Use #if to disable code

2022-05-13 Thread Andreas Rheinhardt
Should fix the compilation issue reported in https://ffmpeg.org/pipermail/ffmpeg-devel/2022-May/296373.html and should also prevent such issues on arches other than X86. Signed-off-by: Andreas Rheinhardt --- libavcodec/v210dec_init.h | 3 ++- libavcodec/v210enc_init.h | 3 ++- libavf

Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move ff_nlmeans_init into a header

2022-05-13 Thread Andreas Rheinhardt
Soft Works: > > >> -Original Message- >> From: Andreas Rheinhardt >> Sent: Friday, May 13, 2022 10:27 AM >> To: Soft Works ; FFmpeg development discussions >> and patches >> Subject: Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move >> ff_nlmeans_init into a header >> >> Soft W

Re: [FFmpeg-devel] [PATCH] avfilter, avcodec/*_init.h: Use #if to disable code

2022-05-13 Thread Hendrik Leppkes
On Fri, May 13, 2022 at 11:19 AM Andreas Rheinhardt wrote: > > Should fix the compilation issue reported in > https://ffmpeg.org/pipermail/ffmpeg-devel/2022-May/296373.html > and should also prevent such issues on arches other than X86. > Since this doesn't actually affect actual compilers, but c

Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move ff_nlmeans_init into a header

2022-05-13 Thread Hendrik Leppkes
On Fri, May 13, 2022 at 11:26 AM Andreas Rheinhardt wrote: > > Soft Works: > > > > > >> -Original Message- > >> From: Andreas Rheinhardt > >> Sent: Friday, May 13, 2022 10:27 AM > >> To: Soft Works ; FFmpeg development discussions > >> and patches > >> Subject: Re: [FFmpeg-devel] [PATCH

Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move ff_nlmeans_init into a header

2022-05-13 Thread Soft Works
> -Original Message- > From: Andreas Rheinhardt > Sent: Friday, May 13, 2022 11:26 AM > To: Soft Works ; FFmpeg development discussions > and patches > Subject: Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move > ff_nlmeans_init into a header > > Soft Works: > > > > > >>

Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move ff_nlmeans_init into a header

2022-05-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Hendrik Leppkes > Sent: Friday, May 13, 2022 11:27 AM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 07/10] avfilter/vf_nlmeans: Move > ff_nlmeans_init into a header > > O

[FFmpeg-devel] [PATCH 2/3] avcodec/libuavs3d: use output_reorder_delay as has_b_frames

2022-05-13 Thread Zhao Zhili
has_b_frames is more than a bool, it's the size of the frame reordering buffer in the decoder. --- libavcodec/libuavs3d.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/libuavs3d.c b/libavcodec/libuavs3d.c index 23de4c8cd5..e911963a41 100644 --- a/libavcodec/libuavs

[FFmpeg-devel] [PATCH 1/3] avcodec/avs3_parser: set has_b_frames properly

2022-05-13 Thread Zhao Zhili
has_b_frames should be output_reorder_delay field in AVS3 sequence header and larger than 1. The parser implementation doesn't parse that field. Decoder can set has_b_frames properly, so use FFMAX here to avoid resetting has_b_frames from output_reorder_delay to 1. --- libavcodec/avs3_parser.c | 2

[FFmpeg-devel] [PATCH 3/3] avcodec/libdavs2: export has_b_frames info

2022-05-13 Thread Zhao Zhili
More precisely, we should use picture_reorder_delay, but it's unavailable yet. --- libavcodec/libdavs2.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavcodec/libdavs2.c b/libavcodec/libdavs2.c index a47027d300..bc31745a4f 100644 --- a/libavcodec/libdavs2.c +++ b/libavcodec/libdavs2

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/avs3_parser: set has_b_frames properly

2022-05-13 Thread Andreas Rheinhardt
Zhao Zhili: > has_b_frames should be output_reorder_delay field in AVS3 sequence > header and larger than 1. The parser implementation doesn't parse > that field. Decoder can set has_b_frames properly, so use FFMAX > here to avoid resetting has_b_frames from output_reorder_delay to 1. > --- > liba

Re: [FFmpeg-devel] [PATCH] avfilter, avcodec/*_init.h: Use #if to disable code

2022-05-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Andreas Rheinhardt > Sent: Friday, May 13, 2022 11:19 AM > To: ffmpeg-devel@ffmpeg.org > Cc: Andreas Rheinhardt > Subject: [FFmpeg-devel] [PATCH] avfilter, avcodec/*_init.h: Use #if to > disable code > > Should fix the compilati

[FFmpeg-devel] [PATCH] fftools/opt_common: add missing include of avf/version.h

2022-05-13 Thread softworkz
From: softworkz required for PRINT_LIB_INFO(avfilter... Signed-off-by: softworkz --- fftools/opt_common: add missing include of avf/version.h MSVC compiler complains without this include Published-As: https://github.com/ffstaging/FFmpeg/releases/tag/pr-ffstaging-27%2Fsoftworkz%2F

[FFmpeg-devel] [PATCH 0/2] Support long file names on Windows

2022-05-13 Thread ffmpegagent
This patchset adds support for long file and directory paths on Windows. The implementation follows the same logic that .NET is using internally, with the only exception that it doesn't expand short path components in 8.3 format. .NET does this as the same function is also used for other purposes,

[FFmpeg-devel] [PATCH 1/2] avutil/wchar_filename, file_open: Support long file names on Windows

2022-05-13 Thread softworkz
From: softworkz Signed-off-by: softworkz --- libavutil/file_open.c | 2 +- libavutil/wchar_filename.h | 123 + 2 files changed, 124 insertions(+), 1 deletion(-) diff --git a/libavutil/file_open.c b/libavutil/file_open.c index cc302f2f76..57c5e78d51 10

[FFmpeg-devel] [PATCH 2/2] avformat/os_support: Support long file names on Windows

2022-05-13 Thread softworkz
From: softworkz Signed-off-by: softworkz --- libavformat/os_support.h | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/os_support.h b/libavformat/os_support.h index 5e6b32d2dc..6c1e6c3851 100644 --- a/libavformat/os_support.h +++ b/libavformat/os_support.h

Re: [FFmpeg-devel] [PATCH 0/2] Support long file names on Windows

2022-05-13 Thread Soft Works
> -Original Message- > From: ffmpegagent > Sent: Friday, May 13, 2022 11:53 AM > To: ffmpeg-devel@ffmpeg.org > Cc: softworkz > Subject: [PATCH 0/2] Support long file names on Windows > > This patchset adds support for long file and directory paths on > Windows. The > implementation fo

[FFmpeg-devel] [PATCH v2 1/3] avcodec/avs3_parser: set has_b_frames properly

2022-05-13 Thread Zhao Zhili
has_b_frames should be output_reorder_delay field in AVS3 sequence header and larger than 1. The parser implementation doesn't parse that field. Decoder can set has_b_frames properly, so use FFMAX here to avoid resetting has_b_frames from output_reorder_delay to 1. --- libavcodec/avs3_parser.c | 5

Re: [FFmpeg-devel] [PATCH 1/1] Add Mono/2D option to stereo3d filter

2022-05-13 Thread James Lancaster
Is there anything that needs to be done on this to get it accepted and merged? On Mon, May 9, 2022 at 3:23 PM James Lancaster wrote: > The following patch adds the option for mono input to the stereo3d filter, > allowing a 2D video source to be duplicated to any of the output formats. > > This e

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/avs3_parser: set has_b_frames properly

2022-05-13 Thread zhilizhao(赵志立)
> On May 13, 2022, at 5:43 PM, Andreas Rheinhardt > wrote: > > Zhao Zhili: >> has_b_frames should be output_reorder_delay field in AVS3 sequence >> header and larger than 1. The parser implementation doesn't parse >> that field. Decoder can set has_b_frames properly, so use FFMAX >> here to a

Re: [FFmpeg-devel] [PATCH 1/1] Add Mono/2D option to stereo3d filter

2022-05-13 Thread Paul B Mahol
fix commit log to be consistent with other commits already in repo. ___ 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 sub

Re: [FFmpeg-devel] [PATCH] fftools/opt_common: add missing include of avf/version.h

2022-05-13 Thread Andreas Rheinhardt
softworkz: > From: softworkz > > required for PRINT_LIB_INFO(avfilter... > > Signed-off-by: softworkz > --- > fftools/opt_common: add missing include of avf/version.h > > MSVC compiler complains without this include > > Published-As: > https://github.com/ffstaging/FFmpeg/releases

[FFmpeg-devel] [PATCH] avutil/csp: create public API for colorspace structs

2022-05-13 Thread Leo Izen
This commit moves some of the functionality from avfilter/colorspace into avutil/csp and exposes it as a public API so it can be used by libavcodec and/or libavformat. --- libavfilter/colorspace.c| 88 libavfilter/colorspace.h| 25 +--- libavfilter/fflcms

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/ac3: Remove declaration of inexistent function

2022-05-13 Thread Andreas Rheinhardt
Andreas Rheinhardt: > Removed in 7b11eead1b4e08728561595e6b610cf8fe2b7122. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/ac3.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/libavcodec/ac3.h b/libavcodec/ac3.h > index dabd49f4cc..8e66d47629 100644 > --- a/libavcodec/ac3.h

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wrapped_avframe: Don't attach FrameDecodeData unnecessarily

2022-05-13 Thread Andreas Rheinhardt
Andreas Rheinhardt: > It is unneeded, as this decoder does not call ff_get_buffer(). > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/wrapped_avframe.c | 6 -- > 1 file changed, 6 deletions(-) > > diff --git a/libavcodec/wrapped_avframe.c b/libavcodec/wrapped_avframe.c > index 3af00

Re: [FFmpeg-devel] [PATCH] avutil/csp: create public API for colorspace structs

2022-05-13 Thread Ronald S. Bultje
Hi, On Fri, May 13, 2022 at 11:42 AM Leo Izen wrote: > This commit moves some of the functionality from avfilter/colorspace > into avutil/csp and exposes it as a public API so it can be used by > libavcodec and/or libavformat. > --- > libavfilter/colorspace.c| 88 --

Re: [FFmpeg-devel] [PATCH] avutil/csp: create public API for colorspace structs

2022-05-13 Thread James Almer
On 5/13/2022 12:42 PM, Leo Izen wrote: This commit moves some of the functionality from avfilter/colorspace into avutil/csp and exposes it as a public API so it can be used by libavcodec and/or libavformat. If it's of no use for library users, then make them avpriv_ and not installed. ---

[FFmpeg-devel] [PATCH] Add FourCC codes for Hap R and Hap HDR

2022-05-13 Thread Malcolm Bechard
Signed-off-by: Malcolm Bechard --- libavformat/isom_tags.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/isom_tags.c b/libavformat/isom_tags.c index 62e60470a8..24953b9821 100644 --- a/libavformat/isom_tags.c +++ b/libavformat/isom_tags.c @@ -237,6 +237,8 @@ const AVCodecTag f

Re: [FFmpeg-devel] [PATCH] avutil/csp: create public API for colorspace structs

2022-05-13 Thread Michael Niedermayer
On Fri, May 13, 2022 at 11:42:08AM -0400, Leo Izen wrote: > This commit moves some of the functionality from avfilter/colorspace > into avutil/csp and exposes it as a public API so it can be used by > libavcodec and/or libavformat. [...] > diff --git a/libavutil/csp.h b/libavutil/csp.h > new file m

Re: [FFmpeg-devel] [PATCH] fftools/opt_common: add missing include of avf/version.h

2022-05-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Andreas Rheinhardt > Sent: Friday, May 13, 2022 3:33 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] fftools/opt_common: add missing > include of avf/version.h > > softworkz: > > From: softworkz > > > > re

Re: [FFmpeg-devel] [PATCH v11 1/6] libavutil/wchar_filename.h: Add whcartoutf8, wchartoansi and utf8toansi

2022-05-13 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of nil- > admir...@mailo.com > Sent: Wednesday, May 11, 2022 10:57 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v11 1/6] > libavutil/wchar_filename.h: Add whcartoutf8, wchartoansi and > utf8toansi > > > I think

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/libuavs3d: use output_reorder_delay as has_b_frames

2022-05-13 Thread myp...@gmail.com
On Fri, May 13, 2022 at 5:41 PM Zhao Zhili wrote: > > has_b_frames is more than a bool, it's the size of the frame > reordering buffer in the decoder. > --- > libavcodec/libuavs3d.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/libuavs3d.c b/libavcodec/libuav

Re: [FFmpeg-devel] [PATCH] avutil/csp: create public API for colorspace structs

2022-05-13 Thread Leo Izen
On 5/13/22 17:22, Michael Niedermayer wrote: On Fri, May 13, 2022 at 11:42:08AM -0400, Leo Izen wrote: + +struct WhitepointCoefficients { +double xw, yw; +}; I think we should avoid floating point so as to ensure reproduceable results and simplify regerssion testing thx This code already e

Re: [FFmpeg-devel] [PATCH] avutil/csp: create public API for colorspace structs

2022-05-13 Thread Leo Izen
On 5/13/22 15:18, James Almer wrote: On 5/13/2022 12:42 PM, Leo Izen wrote: This commit moves some of the functionality from avfilter/colorspace into avutil/csp and exposes it as a public API so it can be used by libavcodec and/or libavformat. If it's of no use for library users, then make the

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/libuavs3d: use output_reorder_delay as has_b_frames

2022-05-13 Thread zhilizhao(赵志立)
> On May 14, 2022, at 9:04 AM, myp...@gmail.com wrote: > > On Fri, May 13, 2022 at 5:41 PM Zhao Zhili wrote: >> >> has_b_frames is more than a bool, it's the size of the frame >> reordering buffer in the decoder. >> --- >> libavcodec/libuavs3d.c | 2 +- >> 1 file changed, 1 insertion(+), 1 dele