Re: [FFmpeg-devel] [PATCH v1] avcodec/av1dec: Add tile list OBU to decompose list

2023-12-21 Thread Andreas Rheinhardt
Wang, Fei W: > On Thu, 2023-12-21 at 20:14 +0100, Andreas Rheinhardt wrote: >> Wang, Fei W: >>> On Wed, 2023-12-20 at 17:11 +0100, Andreas Rheinhardt wrote: fei.w.wang-at-intel@ffmpeg.org: > From: Fei Wang > > Show the unsupported message and return unsupported for clips

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Anton Khirnov
Hi Paul, Quoting Paul B Mahol (2023-12-21 21:05:18) > Say what serious feature you contributed ? - Nothing. this is a personal attach. Personal attacks are not allowed on this mailing list. Please refrain from them in the future. -- Anton Khirnov (CC member)

Re: [FFmpeg-devel] [PATCH] checkasm: Generalize crash handling

2023-12-21 Thread Rémi Denis-Courmont
Le 21 décembre 2023 22:16:09 GMT+02:00, "Rémi Denis-Courmont" a écrit : >Le tiistaina 19. joulukuuta 2023, 14.02.00 EET Martin Storsjö a écrit : >> This replaces the riscv specific handling from >> 7212466e735aa187d82f51dadbce957fe3da77f0 (which essentially is >> reverted, together with

Re: [FFmpeg-devel] [PATCH] checkasm: Generalize crash handling

2023-12-21 Thread Rémi Denis-Courmont
Le 22 décembre 2023 00:03:59 GMT+02:00, Henrik Gramner via ffmpeg-devel a écrit : >On Thu, Dec 21, 2023 at 9:16 PM Rémi Denis-Courmont wrote: >> > +checkasm_fail_func("%s", >> > + s == SIGFPE ? "fatal arithmetic error" : >> > + s ==

Re: [FFmpeg-devel] [PATCH 1/2] configure: Remove a redundant check for UWP mode

2023-12-21 Thread Wu, Tong1
>Subject: [FFmpeg-devel] [PATCH 1/2] configure: Remove a redundant check for >UWP mode > >The check for UWP mode was duplicated from right above, in >d54127c41a81cf2078a3504f78e0e4232cfe11b7. > >Also, instead of several lines with "enabled uwp && ...", make one >"if enabled uwp; then" block.

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Michael Niedermayer
On Thu, Dec 21, 2023 at 08:43:16PM +0100, Tomas Härdin wrote: > ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: > > On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: > > > tis 2023-12-19 klockan 15:02 +0100 skrev Nicolas George: > > [...] > > > [...] , but every line of

Re: [FFmpeg-devel] [PATCH] libavfilter/af_afir: R-V V dcmul_add

2023-12-21 Thread flow gg
It's at c908 According to the benchmark results, if vlseg2e64 is used, the speed is almost as slow as C language (dcmul_add_rvv_f64: 86.2), if vsseg2e64 is used, it will be only a bit slower (dcmul_add_rvv_f64: 50.2). Rémi Denis-Courmont 于2023年12月22日周五 04:52写道: > Le tiistaina 19. joulukuuta

Re: [FFmpeg-devel] Temporary delays in patch reviewing and testing by me

2023-12-21 Thread Kieran Kunhya
On Fri, 22 Dec 2023, 00:36 Michael Niedermayer, wrote: > On Thu, Dec 21, 2023 at 03:22:20AM +0100, Michael Niedermayer wrote: > > Hi all > > > > My main development machiene died today, so expect increased delays > > about things that involve me/it until thats fixed > > > > Rest of the mail is

Re: [FFmpeg-devel] [PATCH 6/6] lavc/takdsp: R-V V decorrelate_sm

2023-12-21 Thread flow gg
func ff_decorrelate_sm_rvv, zve32x 1: vsetvli t0, a2, e32, m8, ta, ma vle32.v v8, (a1) sub a2, a2, t0 vle32.v v0, (a0) vssra.vi v8, v8, 1 vsub.vv v16, v0, v8 vse32.v v16, (a0) sh2add a0, t0, a0 vadd.vv v16, v0, v8

Re: [FFmpeg-devel] [PATCH v1] avcodec/av1dec: Add tile list OBU to decompose list

2023-12-21 Thread Wang, Fei W
On Thu, 2023-12-21 at 20:14 +0100, Andreas Rheinhardt wrote: > Wang, Fei W: > > On Wed, 2023-12-20 at 17:11 +0100, Andreas Rheinhardt wrote: > > > fei.w.wang-at-intel@ffmpeg.org: > > > > From: Fei Wang > > > > > > > > Show the unsupported message and return unsupported for clips > > > >

[FFmpeg-devel] [PATCH 2/2] x86/takdsp: add avx2 versions of all functions

2023-12-21 Thread James Almer
On an Intel Core i7 12700k: decorrelate_ls_c: 814.3 decorrelate_ls_sse2: 165.8 decorrelate_ls_avx2: 101.3 decorrelate_sf_c: 1602.6 decorrelate_sf_sse4: 640.1 decorrelate_sf_avx2: 324.6 decorrelate_sm_c: 1564.8 decorrelate_sm_sse2: 379.3 decorrelate_sm_avx2: 203.3 decorrelate_sr_c: 785.3

[FFmpeg-devel] [PATCH 1/2] checkasm/takdsp: add decorrelate_sf test

2023-12-21 Thread James Almer
Signed-off-by: James Almer --- tests/checkasm/takdsp.c | 36 +--- 1 file changed, 33 insertions(+), 3 deletions(-) diff --git a/tests/checkasm/takdsp.c b/tests/checkasm/takdsp.c index 495b7242c5..8df93cfd52 100644 --- a/tests/checkasm/takdsp.c +++

Re: [FFmpeg-devel] Backporting Clang 14 fix to the 4.4 branch

2023-12-21 Thread Brad Smith
On 2023-12-21 7:34 p.m., Michael Niedermayer wrote: On Thu, Dec 21, 2023 at 01:41:20AM -0500, Brad Smith wrote: https://github.com/FFmpeg/FFmpeg/commit/ab792634197e364ca1bb194f9abe36836e42f12d But the function is in libavformat/utils.c for 4.4 instead of libavformat/seek.c. Could the Clang 14

Re: [FFmpeg-devel] Temporary delays in patch reviewing and testing by me

2023-12-21 Thread Michael Niedermayer
On Thu, Dec 21, 2023 at 03:22:20AM +0100, Michael Niedermayer wrote: > Hi all > > My main development machiene died today, so expect increased delays > about things that involve me/it until thats fixed > > Rest of the mail is just about the issue, and you can skip reading that > > Timeline: >

Re: [FFmpeg-devel] Backporting Clang 14 fix to the 4.4 branch

2023-12-21 Thread Michael Niedermayer
On Thu, Dec 21, 2023 at 01:41:20AM -0500, Brad Smith wrote: > > https://github.com/FFmpeg/FFmpeg/commit/ab792634197e364ca1bb194f9abe36836e42f12d > > But the function is in libavformat/utils.c for 4.4 instead of > libavformat/seek.c. > > Could the Clang 14 fix please be backported to the 4.4

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Michael Niedermayer
On Thu, Dec 21, 2023 at 05:37:58PM -0500, Vittorio Giovara wrote: > On Thu, Dec 21, 2023 at 5:22 PM Nicolas George wrote: > > > Vittorio Giovara (12023-12-21): > > > Oh so you know how it feels! > > > > I am not surprised to see your animosity against me is stronger than > > your care about the

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 6:14 PM Nicolas George wrote: > Vittorio Giovara (12023-12-21): > > I agree, I hope you can work on your lack of self-awareness. > > Oh, the “Ah ?… Et moi, Cyrano-Savinien-Hercule De Bergerac.” comeback. > You're really scrapping the bottom of the barrel, aren't your. >

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Nicolas George
Vittorio Giovara (12023-12-21): > I agree, I hope you can work on your lack of self-awareness. Oh, the “Ah ?… Et moi, Cyrano-Savinien-Hercule De Bergerac.” comeback. You're really scrapping the bottom of the barrel, aren't your. -- Nicolas George

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 5:41 PM Nicolas George wrote: > Vittorio Giovara (12023-12-21): > > Not sure what gave you the idea > > Mails like this one for example. The lack of self-awareness is > hilarious. > I agree, I hope you can work on your lack of self-awareness. -- Vittorio

[FFmpeg-devel] [PATCH 2/2] avcodec/libjxlenc: Set AV_CODEC_CAP_DR1

2023-12-21 Thread Andreas Rheinhardt
This encoder uses ff_get_encode_buffer() to allocate the packet buffer. Signed-off-by: Andreas Rheinhardt --- libavcodec/libjxlenc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/libjxlenc.c b/libavcodec/libjxlenc.c index 055c9b7bb1..021e558687 100644 ---

[FFmpeg-devel] [PATCH 1/2] avcodec/libjxlenc: Don't refer to decoder in comments

2023-12-21 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavcodec/libjxlenc.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/libjxlenc.c b/libavcodec/libjxlenc.c index d707f3a61b..055c9b7bb1 100644 --- a/libavcodec/libjxlenc.c +++ b/libavcodec/libjxlenc.c @@ -85,8 +85,8 @@

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Nicolas George
Vittorio Giovara (12023-12-21): > Not sure what gave you the idea Mails like this one for example. The lack of self-awareness is hilarious. -- Nicolas George ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 5:22 PM Nicolas George wrote: > Vittorio Giovara (12023-12-21): > > Oh so you know how it feels! > > I am not surprised to see your animosity against me is stronger than > your care about the project. > Not sure what gave you the idea, but I have no animosity towards you

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Nicolas George
Vittorio Giovara (12023-12-21): > Oh so you know how it feels! I am not surprised to see your animosity against me is stronger than your care about the project. -- Nicolas George signature.asc Description: PGP signature ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH v8 2/5] libavcodec/webp: add support for animated WebP decoding

2023-12-21 Thread Andreas Rheinhardt
Thilo Borgmann via ffmpeg-devel: > From: Josef Zlomek > > Fixes: 4907 > > Adds support for decoding of animated WebP. > > The WebP decoder adds the animation related features according to the specs: > https://developers.google.com/speed/webp/docs/riff_container#animation > The frames of the

[FFmpeg-devel] [PATCH] configure: Don't pass -mfp16-format to MSVC on ARM

2023-12-21 Thread Martin Storsjö
The check for this option does succeed - MSVC accepts the option, but prints a warning about it being unknown and ignored, for each compiled object file: cl : Command line warning D9002 : ignoring unknown option '-mfp16-format=ieee' The configure script only attempts to add this option on

Re: [FFmpeg-devel] [PATCH] checkasm: Generalize crash handling

2023-12-21 Thread Henrik Gramner via ffmpeg-devel
On Thu, Dec 21, 2023 at 9:16 PM Rémi Denis-Courmont wrote: > > +checkasm_fail_func("%s", > > + s == SIGFPE ? "fatal arithmetic error" : > > + s == SIGILL ? "illegal instruction" : > > + s == SIGBUS ? "bus error"

Re: [FFmpeg-devel] [PATCH] Add new vf_tiltandshift filter

2023-12-21 Thread Vittorio Giovara
On Wed, Dec 20, 2023 at 2:56 PM Vittorio Giovara wrote: > > > On Wed, Dec 20, 2023 at 2:50 PM Nicolas George wrote: > >> Vittorio Giovara (12023-12-20): >> > If there are no more comments, I'll push this today or tomorrow. >> >> I think the change you made after the last request might go too

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 3:05 PM Paul B Mahol wrote: > On Thu, Dec 21, 2023 at 8:43 PM Tomas Härdin wrote: > > > ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: > > > On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: > > > > tis 2023-12-19 klockan 15:02 +0100 skrev

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Vittorio Giovara
On Thu, Dec 21, 2023 at 3:31 PM Nicolas George wrote: > Paul B Mahol (12023-12-21): > > Say what serious feature you contributed ? - Nothing. > > I did not want to say it, but since it is now in the open: Not nothing: > negative. His naysaying discouraged me from working further on the >

[FFmpeg-devel] [PATCH 2/2] configure: Fix linking d3d12va in UWP mode

2023-12-21 Thread Martin Storsjö
--- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 69b755f274..18fe8eaa5c 100755 --- a/configure +++ b/configure @@ -3922,7 +3922,7 @@ swscale_suggest="libm stdatomic" avcodec_extralibs="pthreads_extralibs iconv_extralibs

[FFmpeg-devel] [PATCH 1/2] configure: Remove a redundant check for UWP mode

2023-12-21 Thread Martin Storsjö
The check for UWP mode was duplicated from right above, in d54127c41a81cf2078a3504f78e0e4232cfe11b7. Also, instead of several lines with "enabled uwp && ...", make one "if enabled uwp; then" block. --- configure | 30 +++--- 1 file changed, 11 insertions(+), 19

[FFmpeg-devel] [PATCH v3 0/1] avcodec/libjxlenc: Add libjxl_anim encoder

2023-12-21 Thread Zsolt Vadász via ffmpeg-devel
This patchset adds support for encoding animated JPEG XL images via a new encoder (libjxl_anim). When using the encoder, the output format needs to be set to raw video, as shown below: `ffmpeg -i sample.gif -c:v libjxl_anim -f rawvideo out.jxl` V3 changes: - Renamed encoder to libjxl_anim for

[FFmpeg-devel] [PATCH v3 1/1] avcodec/libjxlenc: Add libjxl_animated encoder

2023-12-21 Thread Zsolt Vadász via ffmpeg-devel
--- configure | 1 + libavcodec/allcodecs.c | 1 + libavcodec/libjxlenc.c | 226 ++--- 3 files changed, 189 insertions(+), 39 deletions(-) diff --git a/configure b/configure index 7d2ee66000..2d27d8015a 100755 --- a/configure +++ b/configure

Re: [FFmpeg-devel] [PATCH] libavfilter/af_afir: R-V V dcmul_add

2023-12-21 Thread Rémi Denis-Courmont
Le tiistaina 19. joulukuuta 2023, 4.53.12 EET flow gg a écrit : > c908: > dcmul_add_c: 88.0 > dcmul_add_rvv_f64: 46.2 > > Did not use vlseg2e64, because it is much slower than vlse64 > Did not use vsseg2e64, because it is slightly slower than vsse64 Is this about C910 or C908? I have not checked

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Nicolas George
Paul B Mahol (12023-12-21): > Say what serious feature you contributed ? - Nothing. I did not want to say it, but since it is now in the open: Not nothing: negative. His naysaying discouraged me from working further on the built-in documentation system. -- Nicolas George signature.asc

Re: [FFmpeg-devel] [PATCH] checkasm: Generalize crash handling

2023-12-21 Thread Rémi Denis-Courmont
Le tiistaina 19. joulukuuta 2023, 14.02.00 EET Martin Storsjö a écrit : > This replaces the riscv specific handling from > 7212466e735aa187d82f51dadbce957fe3da77f0 (which essentially is > reverted, together with 286d6742218ba0235c32876b50bf593cb1986353) > with a different implementation of the

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Paul B Mahol
On Thu, Dec 21, 2023 at 8:43 PM Tomas Härdin wrote: > ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: > > On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: > > > tis 2023-12-19 klockan 15:02 +0100 skrev Nicolas George: > > [...] > > > [...] , but every line of code > >

Re: [FFmpeg-devel] [PATCH] lavc: remove the QOA decoder

2023-12-21 Thread Tomas Härdin
ons 2023-12-20 klockan 20:11 +0100 skrev Michael Niedermayer: > On Wed, Dec 20, 2023 at 05:57:40PM +0100, Tomas Härdin wrote: > > tis 2023-12-19 klockan 15:02 +0100 skrev Nicolas George: > [...] > > [...] , but every line of code > > carries with it a non-zero maintenance burden > > Assuming you

[FFmpeg-devel] FFmpeg 6.1.1

2023-12-21 Thread Michael Niedermayer
Hi all I will probably make a 6.1.1 release in maybe 1-3 weeks due to bug/fixes in it. if you want something in it, please backport it now! thx -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB If you drop bombs on a foreign country and kill a hundred thousand

Re: [FFmpeg-devel] [PATCH v1] avcodec/av1dec: Add tile list OBU to decompose list

2023-12-21 Thread Andreas Rheinhardt
Wang, Fei W: > On Wed, 2023-12-20 at 17:11 +0100, Andreas Rheinhardt wrote: >> fei.w.wang-at-intel@ffmpeg.org: >>> From: Fei Wang >>> >>> Show the unsupported message and return unsupported for clips >>> contain >>> tile list OBU since it hasn't been implemented. Otherwise, decoding >>> maybe

Re: [FFmpeg-devel] [PATCH 6/6] lavc/takdsp: R-V V decorrelate_sm

2023-12-21 Thread Rémi Denis-Courmont
Le torstaina 21. joulukuuta 2023, 18.07.55 EET Rémi Denis-Courmont a écrit : > You can use VSSRA, and then VADD won't need to depend on the output of VSUB. P.S.: I have NOT checked which approach is actually faster. -- Rémi Denis-Courmont http://www.remlab.net/

Re: [FFmpeg-devel] [PATCH 6/6] lavc/takdsp: R-V V decorrelate_sm

2023-12-21 Thread Rémi Denis-Courmont
Le maanantaina 18. joulukuuta 2023, 17.16.27 EET flow gg a écrit : > C908: > decorrelate_sm_c: 130.0 > decorrelate_sm_rvv_i32: 43.7 + +func ff_decorrelate_sm_rvv, zve32x +1: +vsetvli t0, a2, e32, m8, ta, ma +vle32.v v0, (a0) +sub a2, a2, t0 +vle32.v v8, (a1) +

Re: [FFmpeg-devel] [PATCH v2 1/2] avformat/flvenc: support enhanced flv PacketTypeMetadata

2023-12-21 Thread zhupengfei via ffmpeg-devel
> 2023年12月18日 02:22,James Almer 写道: > > On 12/17/2023 1:24 PM, zhupengfei via ffmpeg-devel wrote: >> From: Zhu Pengfei <411294...@qq.com> >> Signed-off-by: Zhu Pengfei <411294...@qq.com> >> --- >> libavformat/flvenc.c | 155 +++ >> 1 file changed, 155

Re: [FFmpeg-devel] [PATCH 3/3] avutil/hwcontext_d3d12va: remove unused variables

2023-12-21 Thread Wu, Tong1
>-Original Message- >From: ffmpeg-devel On Behalf Of James >Almer >Sent: Thursday, December 21, 2023 8:47 PM >To: ffmpeg-devel@ffmpeg.org >Subject: Re: [FFmpeg-devel] [PATCH 3/3] avutil/hwcontext_d3d12va: remove >unused variables > >On 12/21/2023 9:42 AM, Martin Storsjö wrote: >> On Thu,

Re: [FFmpeg-devel] [PATCH 3/3] avutil/hwcontext_d3d12va: remove unused variables

2023-12-21 Thread James Almer
On 12/21/2023 9:42 AM, Martin Storsjö wrote: On Thu, 21 Dec 2023, James Almer wrote: Removes -Wunused-variable warnings. Signed-off-by: James Almer --- Were this meant to be used? Or are just a remnant of a previous interation of the set? libavutil/hwcontext_d3d12va.c | 4 1 file

Re: [FFmpeg-devel] [PATCH 3/3] avutil/hwcontext_d3d12va: remove unused variables

2023-12-21 Thread Martin Storsjö
On Thu, 21 Dec 2023, James Almer wrote: Removes -Wunused-variable warnings. Signed-off-by: James Almer --- Were this meant to be used? Or are just a remnant of a previous interation of the set? libavutil/hwcontext_d3d12va.c | 4 1 file changed, 4 deletions(-) diff --git

[FFmpeg-devel] [PATCH 3/3] avutil/hwcontext_d3d12va: remove unused variables

2023-12-21 Thread James Almer
Removes -Wunused-variable warnings. Signed-off-by: James Almer --- Were this meant to be used? Or are just a remnant of a previous interation of the set? libavutil/hwcontext_d3d12va.c | 4 1 file changed, 4 deletions(-) diff --git a/libavutil/hwcontext_d3d12va.c

[FFmpeg-devel] [PATCH 2/3] avutil/hwcontext_d3d12va: fix indentation in d3d12va_transfer_data()

2023-12-21 Thread James Almer
Removes -Wmisleading-indentation warnings. Signed-off-by: James Almer --- libavutil/hwcontext_d3d12va.c | 54 +-- 1 file changed, 27 insertions(+), 27 deletions(-) diff --git a/libavutil/hwcontext_d3d12va.c b/libavutil/hwcontext_d3d12va.c index

[FFmpeg-devel] [PATCH 1/3] avutil/hwcontext_d3d12va: cast the input pointer array argument on av_image_copy calls

2023-12-21 Thread James Almer
Removes -Wincompatible-pointer-types warnings. Signed-off-by: James Almer --- libavutil/hwcontext_d3d12va.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavutil/hwcontext_d3d12va.c b/libavutil/hwcontext_d3d12va.c index 1600d94cb0..052de3472f 100644 ---

Re: [FFmpeg-devel] [PATCH] checkasm: Generalize crash handling

2023-12-21 Thread Martin Storsjö
On Thu, 21 Dec 2023, Rémi Denis-Courmont wrote: diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c index 6318d9296b..668034c67f 100644 --- a/tests/checkasm/checkasm.c +++ b/tests/checkasm/checkasm.c @@ -23,8 +23,10 @@ #include "config.h" #include "config_components.h" -#ifndef

Re: [FFmpeg-devel] [PATCH v2] avutil/hwcontext_d3d12va: remove an unused function

2023-12-21 Thread Martin Storsjö
On Thu, 21 Dec 2023, Tong Wu wrote: It caused lacking a public declaration build error with -Werror=missing-prototypes. Since DXGI_FORMAT is moved to public since patch set V10, this function is no longer useful. Now remove it. Signed-off-by: Tong Wu --- libavutil/hwcontext_d3d12va.c | 9

Re: [FFmpeg-devel] [PATCH] avutil/hwcontext_d3d12va: remove an unused function

2023-12-21 Thread Wu, Tong1
> >On Thu, 21 Dec 2023, Tong Wu wrote: > >> Signed-off-by: Tong Wu >> --- >> libavutil/hwcontext_d3d12va.c | 9 - >> 1 file changed, 9 deletions(-) >> >> diff --git a/libavutil/hwcontext_d3d12va.c b/libavutil/hwcontext_d3d12va.c >> index 1600d94cb0..4995518dbd 100644 >> ---

[FFmpeg-devel] [PATCH v2] avutil/hwcontext_d3d12va: remove an unused function

2023-12-21 Thread Tong Wu
It caused lacking a public declaration build error with -Werror=missing-prototypes. Since DXGI_FORMAT is moved to public since patch set V10, this function is no longer useful. Now remove it. Signed-off-by: Tong Wu --- libavutil/hwcontext_d3d12va.c | 9 - 1 file changed, 9 deletions(-)

Re: [FFmpeg-devel] [PATCH] checkasm: Generalize crash handling

2023-12-21 Thread Rémi Denis-Courmont
Le 19 décembre 2023 14:02:00 GMT+02:00, "Martin Storsjö" a écrit : >This replaces the riscv specific handling from >7212466e735aa187d82f51dadbce957fe3da77f0 (which essentially is >reverted, together with 286d6742218ba0235c32876b50bf593cb1986353) >with a different implementation of the same

Re: [FFmpeg-devel] [PATCH] avutil/hwcontext_d3d12va: remove an unused function

2023-12-21 Thread Martin Storsjö
On Thu, 21 Dec 2023, Tong Wu wrote: Signed-off-by: Tong Wu --- libavutil/hwcontext_d3d12va.c | 9 - 1 file changed, 9 deletions(-) diff --git a/libavutil/hwcontext_d3d12va.c b/libavutil/hwcontext_d3d12va.c index 1600d94cb0..4995518dbd 100644 --- a/libavutil/hwcontext_d3d12va.c +++

Re: [FFmpeg-devel] [PATCH 13/13 v3] fftools/ffmpeg: convert to a threaded architecture

2023-12-21 Thread Paul B Mahol
On Thu, Dec 7, 2023 at 6:26 PM Paul B Mahol wrote: > > > On Wed, Dec 6, 2023 at 2:38 PM Nicolas George wrote: > >> James Almer (12023-12-06): >> > I honestly can't believe you're arguing this. >> >> Yet I do, so I suggest you think a little harder to understand why I do. >> >> > And being

Re: [FFmpeg-devel] [PATCH v12 1/9] libavutil: add hwcontext_d3d12va and AV_PIX_FMT_D3D12

2023-12-21 Thread Wu, Tong1
>There are still no public external API users to actually test the API, >which is concerning. It took months for the Vulkan API to be portable >enough to work with the major Vulkan users out there. Granted, Vulkan >is special. > >Is there any public test code out there? If not, I'd prefer the API

Re: [FFmpeg-devel] [PATCH] checkasm: Generalize crash handling

2023-12-21 Thread Henrik Gramner via ffmpeg-devel
On Tue, Dec 19, 2023 at 1:02 PM Martin Storsjö wrote: > This replaces the riscv specific handling from > 7212466e735aa187d82f51dadbce957fe3da77f0 (which essentially is > reverted, together with 286d6742218ba0235c32876b50bf593cb1986353) > with a different implementation of the same (plus a bit

Re: [FFmpeg-devel] [PATCH] d3d12va: Add a missing include for the declaration of ff_d3d12va_get_surface_index

2023-12-21 Thread Wu, Tong1
>From: ffmpeg-devel On Behalf Of >Martin Storsjö >Sent: Thursday, December 21, 2023 5:48 PM >To: ffmpeg-devel@ffmpeg.org >Cc: Xiang, Haihao >Subject: Re: [FFmpeg-devel] [PATCH] d3d12va: Add a missing include for the >declaration of ff_d3d12va_get_surface_index > >On Thu, 21 Dec 2023, Martin

Re: [FFmpeg-devel] [PATCH] d3d12va: Add a missing include for the declaration of ff_d3d12va_get_surface_index

2023-12-21 Thread Wu, Tong1
>-Original Message- >From: ffmpeg-devel On Behalf Of >Martin Storsjö >Sent: Thursday, December 21, 2023 5:45 PM >To: ffmpeg-devel@ffmpeg.org >Cc: Xiang, Haihao >Subject: [FFmpeg-devel] [PATCH] d3d12va: Add a missing include for the >declaration of ff_d3d12va_get_surface_index > >This

[FFmpeg-devel] [PATCH] avutil/hwcontext_d3d12va: remove an unused function

2023-12-21 Thread Tong Wu
Signed-off-by: Tong Wu --- libavutil/hwcontext_d3d12va.c | 9 - 1 file changed, 9 deletions(-) diff --git a/libavutil/hwcontext_d3d12va.c b/libavutil/hwcontext_d3d12va.c index 1600d94cb0..4995518dbd 100644 --- a/libavutil/hwcontext_d3d12va.c +++ b/libavutil/hwcontext_d3d12va.c @@ -71,15

Re: [FFmpeg-devel] [PATCH] [MXF] - Add jpeg2000 subdescriptor in MXF file (V2).

2023-12-21 Thread Tomas Härdin
ons 2023-12-20 klockan 18:04 +0100 skrev Cédric Le Barz: > Le 19/12/2023 à 14:36, Tomas Härdin a écrit : > > > +    for ( comp = 0; comp < component_count; comp++ ) { > > > +    avio_write(pb, >j2k_info.j2k_comp_desc[3*comp] , 3); > > > +    } > > Looks like this could be simplified to just >

[FFmpeg-devel] [PATCH v2] avformat/riffenc: don't force WAVEFORMATEXTENSIBLE for flt/dbl LPCM

2023-12-21 Thread Gyan Doshi
2c2a167ca7 forced WAVEFORMATEXTENSIBLE for all LPCM streams with greater than 16 bits per sample. However, WAVEFORMATEX allows IEEE Float samples or any depth where raw depth == coded depth, see Remarks section at https://learn.microsoft.com/en-us/windows/win32/api/mmreg/ns-mmreg-waveformatex and

Re: [FFmpeg-devel] [PATCH] d3d12va: Add a missing include for the declaration of ff_d3d12va_get_surface_index

2023-12-21 Thread Martin Storsjö
On Thu, 21 Dec 2023, Martin Storsjö wrote: This fixes the following build error: src/libavcodec/d3d12va_decode.c:49:10: error: no previous prototype for function 'ff_d3d12va_get_surface_index' [-Werror,-Wmissing-prototypes] 49 | unsigned ff_d3d12va_get_surface_index(const AVCodecContext

[FFmpeg-devel] [PATCH] d3d12va: Add a missing include for the declaration of ff_d3d12va_get_surface_index

2023-12-21 Thread Martin Storsjö
This fixes the following build error: src/libavcodec/d3d12va_decode.c:49:10: error: no previous prototype for function 'ff_d3d12va_get_surface_index' [-Werror,-Wmissing-prototypes] 49 | unsigned ff_d3d12va_get_surface_index(const AVCodecContext *avctx, | ^ ---

Re: [FFmpeg-devel] [PATCH v12 1/9] libavutil: add hwcontext_d3d12va and AV_PIX_FMT_D3D12

2023-12-21 Thread Lynne
There are still no public external API users to actually test the API, which is concerning. It took months for the Vulkan API to be portable enough to work with the major Vulkan users out there. Granted, Vulkan is special. Is there any public test code out there? If not, I'd prefer the API to be

Re: [FFmpeg-devel] [PATCH v12 1/9] libavutil: add hwcontext_d3d12va and AV_PIX_FMT_D3D12

2023-12-21 Thread Xiang, Haihao
On Ma, 2023-12-18 at 06:28 +, Xiang, Haihao wrote: > On Di, 2023-12-05 at 14:46 +0800, Tong Wu wrote: > > From: Wu Jianhua > > > > Signed-off-by: Wu Jianhua > > Signed-off-by: Tong Wu > > --- > > > > Compared to v11, v12 set the initial value to 0 for the fence value, > > fixing a

Re: [FFmpeg-devel] [PATCH v8 0/5] webp: add support for animated WebP decoding

2023-12-21 Thread Thilo Borgmann via ffmpeg-devel
Am 15.12.23 um 20:59 schrieb Thilo Borgmann via ffmpeg-devel: Still images fixed, includes FATE tests, VP8 decoder decoupled so there are no more data races, fixed more asserts, fixed ffprobe regression, removed unnecessary parser changes, put the whole animated sequence into one packet.