[FFmpeg-devel] [GSOC]Inquiry Regarding "Update SITI Filter to Match with Latest Specification" Project and Qualification Task

2024-03-03 Thread Divyansh Khatri
I hope this message finds you well. My name is Divyansh, and I am reaching out to express my interest in the project titled "Update SITI Filter to Match with Latest Specification" listed in Ideas list of FFmpeg 2024 GSOC( link

[FFmpeg-devel] [PATCH] avcodec/mediacodecenc: Add a hint message to use nv12 pix_fmt

2024-03-03 Thread Zhao Zhili
From: Zhao Zhili --- libavcodec/mediacodecenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/mediacodecenc.c b/libavcodec/mediacodecenc.c index 086b545590..984014f1b1 100644 --- a/libavcodec/mediacodecenc.c +++ b/libavcodec/mediacodecenc.c @@ -319,6 +319,9 @@ static av_cold

Re: [FFmpeg-devel] [RFC] clarifying the TC conflict of interest rule

2024-03-03 Thread Michael Niedermayer
On Sun, Mar 03, 2024 at 10:57:43PM +0100, Anton Khirnov wrote: > Quoting Michael Niedermayer (2024-03-03 03:49:33) > > Hi > > > > On Fri, Mar 01, 2024 at 06:33:12PM +0100, Anton Khirnov wrote: > > > Hi all, > > > it seems the discussion died down, > > > > There are patches pending, which i will

Re: [FFmpeg-devel] [PATCH 2/3] doc/community: Conflict of interest recusal requirement (Similar to mid part of Antons proposal)

2024-03-03 Thread Michael Niedermayer
On Sun, Mar 03, 2024 at 10:19:03AM +0200, Rémi Denis-Courmont wrote: > Le tiistaina 27. helmikuuta 2024, 0.44.37 EET Michael Niedermayer a écrit : > > Signed-off-by: Michael Niedermayer > > --- > > doc/community.texi | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git

Re: [FFmpeg-devel] [RFC] clarifying the TC conflict of interest rule

2024-03-03 Thread Michael Niedermayer
On Mon, Mar 04, 2024 at 12:36:21AM +0100, Michael Niedermayer wrote: > On Sun, Mar 03, 2024 at 10:57:43PM +0100, Anton Khirnov wrote: > > Quoting Michael Niedermayer (2024-03-03 03:49:33) [...] > > > > > Please add the vote options: (I belive this gives the people a more > > > complete set of

Re: [FFmpeg-devel] [PATCH 1/3] doc/community: Vote in the interest of the project (first part of Antons proposal)

2024-03-03 Thread Michael Niedermayer
On Sun, Mar 03, 2024 at 10:05:49AM +0200, Rémi Denis-Courmont wrote: > Le sunnuntaina 3. maaliskuuta 2024, 4.53.01 EET Michael Niedermayer a écrit : > > On Mon, Feb 26, 2024 at 11:44:36PM +0100, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > > --- > > > > > >

Re: [FFmpeg-devel] [RFC] clarifying the TC conflict of interest rule

2024-03-03 Thread Michael Niedermayer
On Sun, Mar 03, 2024 at 03:49:33AM +0100, Michael Niedermayer wrote: [...] > +If a TC member is aware of a conflict of interest with regards to the case, > they must announce it > +and recuse themselves from the TC discussion and vote. please replace this in my proposal by this (as it clearer

Re: [FFmpeg-devel] [PATCH 1/4] lavc/vp9dsp: R-V V ipred dc

2024-03-03 Thread Rémi Denis-Courmont
Le sunnuntaina 3. maaliskuuta 2024, 3.59.00 EET flow gg a écrit : > updated a little improve in this reply As noted eaerlier, I don't understand why you have two size parameters. It seems that \size is always either the same as (1 << (\size2 - 1)) a.k.a. ((1 << \size2) / 2), or unused. The

Re: [FFmpeg-devel] [PATCH 1/3] lavc/vp8dsp: R-V V put_bilin_h

2024-03-03 Thread flow gg
Sorry since I did not send the emails all at once, so cannot apply all 4 patches together with git am *.patch. Instead, it needs to first apply the patch with 'git am '[PATCH] lavc/vp8dsp: R-V V put_vp8_pixels'', and then apply the patches 1-3 in the series with 'git am *.patch'. Rémi

Re: [FFmpeg-devel] [PATCH 2/2] doc/ffprobe.xsd: add and

2024-03-03 Thread Stefano Sabatini
On date Wednesday 2024-02-28 14:23:50 +0100, Niklas Haas wrote: > From: Niklas Haas > > Originally introduced for dovi side data printing, but not properly > reflected in the xsd. > --- > doc/ffprobe.xsd | 26 ++ > 1 file changed, 26 insertions(+) > > diff --git

[FFmpeg-devel] [PATCH 3/8] fate/iamf: print stream group information

2024-03-03 Thread James Almer
Signed-off-by: James Almer --- tests/fate/iamf.mak | 8 ++-- tests/ref/fate/iamf-5_1_4 | 60 + tests/ref/fate/iamf-7_1_4 | 68 + tests/ref/fate/iamf-ambisonic_1 | 44 +

[FFmpeg-devel] [PATCH 4/8] avformat/mov: fix setting disposition for the first iamf stream

2024-03-03 Thread James Almer
st->disposition will be overwritten with disposition later in the function. Signed-off-by: James Almer --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index e07f2a1ada..7a09017020 100644 --- a/libavformat/mov.c +++

[FFmpeg-devel] [PATCH 5/8] fate/mov: print stream group information for iamf tests

2024-03-03 Thread James Almer
Signed-off-by: James Almer --- tests/fate/mov.mak | 12 +- tests/ref/fate/mov-mp4-iamf-5_1_4 | 318 + tests/ref/fate/mov-mp4-iamf-7_1_4 | 362 tests/ref/fate/mov-mp4-iamf-ambisonic_1 | 230 +++

[FFmpeg-devel] [PATCH 6/8] avformat/mov: don't mark an item referenced by a grid as dependent if it's the primary item

2024-03-03 Thread James Almer
If it's the primary item, then it's expected to be ready for presentation even outside of the grid it belongs to. Signed-off-by: James Almer --- libavformat/mov.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index

[FFmpeg-devel] [PATCH 7/8] fate/mov: print stream group information for avif/heic tests

2024-03-03 Thread James Almer
Signed-off-by: James Almer --- tests/fate-run.sh | 13 ++ tests/fate/mov.mak| 12 +- .../ref/fate/mov-heic-demux-still-image-grid | 133 -- .../ref/fate/mov-heic-demux-still-image-iovl | 81 +-- 4 files changed,

[FFmpeg-devel] [PATCH 8/8] fate/mov: use framecrc for the remaining avif/heic tests

2024-03-03 Thread James Almer
Put them in sync with the other tests. Signed-off-by: James Almer --- tests/fate/mov.mak| 21 ++- .../fate/mov-avif-demux-still-image-1-item| 8 ++- .../mov-avif-demux-still-image-multiple-items | 8 ++-

[FFmpeg-devel] [PATCH 2/8] fftools/ffprobe: print some basic Tile Grid Stream Group parameters

2024-03-03 Thread James Almer
Signed-off-by: James Almer --- fftools/ffprobe.c | 8 1 file changed, 8 insertions(+) diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index 548c5e0948..fef2a37d4c 100644 --- a/fftools/ffprobe.c +++ b/fftools/ffprobe.c @@ -3367,7 +3367,15 @@ static void

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/vulkan_decode: Un-sparse extensions table

2024-03-03 Thread Lynne
Mar 3, 2024, 12:35 by andreas.rheinha...@outlook.com: > Only three of the 226 (== AV_CODEC_ID_AV1) entries > have been used. Unsparsing this table is especially > important given that this array lives in .data.rel.ro. > > Signed-off-by: Andreas Rheinhardt > --- > Instead of parallel tables, one

Re: [FFmpeg-devel] [PATCH] fftools/ffprobe: Print dispositions generically

2024-03-03 Thread James Almer
On 3/2/2024 4:37 PM, Andreas Rheinhardt wrote: Signed-off-by: Andreas Rheinhardt --- fftools/ffprobe.c | 39 +++ 1 file changed, 15 insertions(+), 24 deletions(-) diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index ea225f14ab..ac6b92f5d6 100644 ---

Re: [FFmpeg-devel] [PATCH] avcodec/x86/vvc/vvcdsp_init: fix linking error when configuring with --disable-ssse3 --disable-optimizations options

2024-03-03 Thread Nuo Mi
Thank you, Jianhua. This patch mixes many things. Could you help split it into smaller, more atomic patches? For example, one for moving code blocks and another for fixing --disable-ssse3. On Fri, Mar 1, 2024 at 1:42 AM wrote: > From: Wu Jianhua > > Signed-off-by: Wu Jianhua > --- >

Re: [FFmpeg-devel] [PATCH 15/38] lavu/opt: add array options

2024-03-03 Thread Diederick C. Niehorster
On Sun, Mar 3, 2024 at 3:55 PM Anton Khirnov wrote: > Quoting Marton Balint (2024-02-26 20:38:46) > > The more I think about it, it is actually a broader problem. > > > > You are changing the semantics of existing AV_OPT_TYPE_xxx types. So > > previously an option with AV_OPT_TYPE_STRING used to

Re: [FFmpeg-devel] [PATCH 1/2] avutil/tests/opt: test negative values for INT and INT64 types

2024-03-03 Thread Stefano Sabatini
On date Tuesday 2024-02-27 10:59:46 -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavutil/tests/opt.c | 11 +-- > tests/ref/fate/opt| 35 ++- > 2 files changed, 39 insertions(+), 7 deletions(-) > > diff --git a/libavutil/tests/opt.c

Re: [FFmpeg-devel] [PATCH 2/2] avutil/opt: add an unsigned option type

2024-03-03 Thread Stefano Sabatini
On date Tuesday 2024-02-27 10:59:47 -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavutil/opt.c | 20 > libavutil/opt.h | 1 + > libavutil/tests/opt.c | 9 + > tests/ref/fate/opt| 32 ++-- > 4 files

Re: [FFmpeg-devel] [PATCH] web: Add spi.txt

2024-03-03 Thread Stefano Sabatini
On date Tuesday 2024-02-27 20:01:26 +0100, Michael Niedermayer wrote: > This explains how to request refunds and what can be funded by SPI > > Co-Author: Stefano Sabatini > --- > Makefile | 2 +- > src/spi| 98 ++ > src/spi_js

Re: [FFmpeg-devel] [PATCH 1/2] doc/ffprobe.xsd: add frameSideDatumType entry

2024-03-03 Thread Stefano Sabatini
On date Wednesday 2024-02-28 14:23:49 +0100, Niklas Haas wrote: > From: Niklas Haas > > Frame-level side data attributes are printed with the same key/value > structure as packet-level side data attributes, but this is not > reflected in the XSD. > --- > doc/ffprobe.xsd | 6 ++ > 1 file

Re: [FFmpeg-devel] [PATCH v3] avcodec/jpeg2000dec: support of 2 fields in 1 AVPacket

2024-03-03 Thread Jerome Martinez
On 01/03/2024 23:29, Tomas Härdin wrote: sön 2024-02-25 klockan 05:14 +0100 skrev Jerome Martinez: [...] I also needed to add a dedicated AVStream field for saying that the decoder is able to manage this functionality (and is needed there). What is the added value to call the decoder twice

[FFmpeg-devel] [PATCH] avcodec/vvcdec: fix undefined behaviours for derive_affine_mvc

2024-03-03 Thread Nuo Mi
libavcodec/vvc/vvc_inter.c:823:18: runtime error: signed integer overflow: 1426128896 + 1426128896 cannot be represented in type 'int' SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior libavcodec/vvc/vvc_inter.c:823:18 Suggested-by: Andreas Rheinhardt --- libavcodec/vvc/vvc_inter.c | 11

Re: [FFmpeg-devel] [PATCH] avcodec/vvcdec: check pred flag to fix undefined beavhiours

2024-03-03 Thread Nuo Mi
On Sun, Mar 3, 2024 at 2:43 PM Andreas Rheinhardt < andreas.rheinha...@outlook.com> wrote: > Nuo Mi: > > libavcodec/vvc/vvc_inter.c:823:18: runtime error: signed integer > overflow: 1426128896 + 1426128896 cannot be represented in type 'int' > > SUMMARY: UndefinedBehaviorSanitizer:

Re: [FFmpeg-devel] [PATCH 1/3] lavc/vp8dsp: R-V V put_bilin_h

2024-03-03 Thread Rémi Denis-Courmont
Le perjantaina 23. helmikuuta 2024, 16.45.46 EET flow gg a écrit : > Looks like this needs rebasing, or otherwise does not apply. -- Rémi Denis-Courmont http://www.remlab.net/ ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 15/38] lavu/opt: add array options

2024-03-03 Thread Anton Khirnov
Quoting Marton Balint (2024-02-26 20:38:46) > The more I think about it, it is actually a broader problem. > > You are changing the semantics of existing AV_OPT_TYPE_xxx types. So > previously an option with AV_OPT_TYPE_STRING used to have default value in > default_val.str. After your patch,

Re: [FFmpeg-devel] [PATCH 1/4] lavc/vp9dsp: R-V V ipred dc

2024-03-03 Thread flow gg
> As noted eaerlier, I don't understand why you have two size parameters. It seems that \size is always either the same as (1 << (\size2 - 1)) a.k.a. ((1 << \size2) / 2), or unused. The assembler *can* compute arithmetic constants. Thanks , I didn't know that before > Similarly, you can use

Re: [FFmpeg-devel] [PATCH 1/2] avfilter/vf_showinfo: add AVFilmGrainAOMParams support

2024-03-03 Thread Stefano Sabatini
On date Tuesday 2024-02-27 14:43:12 +0100, Niklas Haas wrote: > From: Niklas Haas > > For my own testing purposes. You can drop "my own". > --- > libavfilter/vf_showinfo.c | 42 +-- > 1 file changed, 40 insertions(+), 2 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH 15/38] lavu/opt: add array options

2024-03-03 Thread James Almer
On 3/3/2024 12:53 PM, Diederick C. Niehorster wrote: On Sun, Mar 3, 2024 at 3:55 PM Anton Khirnov wrote: Quoting Marton Balint (2024-02-26 20:38:46) The more I think about it, it is actually a broader problem. You are changing the semantics of existing AV_OPT_TYPE_xxx types. So previously

Re: [FFmpeg-devel] [PATCH 2/2] fftools/ffprobe: add AVFilmGrainParams printing

2024-03-03 Thread Stefano Sabatini
On date Tuesday 2024-02-27 14:43:13 +0100, Niklas Haas wrote: > From: Niklas Haas > > So we can add a FATE test for this. > --- > fftools/ffprobe.c | 118 ++ > 1 file changed, 118 insertions(+) Looks good to me, probably you can add an entry to the

[FFmpeg-devel] [PATCH 1/8 v2] fftools/ffprobe: add support for Stream Groups

2024-03-03 Thread James Almer
Signed-off-by: James Almer --- doc/ffprobe.xsd | 1 + fftools/ffprobe.c | 145 -- tests/ref/fate/cavs-demux | 2 +- tests/ref/fate/ffprobe_compact| 2 +- tests/ref/fate/ffprobe_csv

Re: [FFmpeg-devel] FFmpeg 7.0 blocking issues

2024-03-03 Thread James Almer
On 3/3/2024 4:35 AM, Jean-Baptiste Kempf wrote: n Sat, 2 Mar 2024, at 23:55, Michael Niedermayer wrote: On Tue, Jan 23, 2024 at 08:22:41PM +0100, Michael Niedermayer wrote: Hi all As it was a little difficult for me to not loose track of what is blocking a release. I suggest that for all

Re: [FFmpeg-devel] [PATCH 1/3] doc/community: Vote in the interest of the project (first part of Antons proposal)

2024-03-03 Thread Rémi Denis-Courmont
Le sunnuntaina 3. maaliskuuta 2024, 4.53.01 EET Michael Niedermayer a écrit : > On Mon, Feb 26, 2024 at 11:44:36PM +0100, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > > > doc/community.texi | 2 ++ > > 1 file changed, 2 insertions(+) > > I intend to apply the

[FFmpeg-devel] [PATCH 2/3] avcodec/vulkan: Make FFVkCodecMap decoder-only

2024-03-03 Thread Andreas Rheinhardt
All the fields of FFVkCodecMap are either decoder-only or encoder-only (with the latter being unused and unset for now). This commit therefore makes this struct decoder-only and moves it to vulkan_decode.c. Also use designated initializers while just at it. Signed-off-by: Andreas Rheinhardt ---

[FFmpeg-devel] [PATCH 3/3] avcodec/vulkan_decode: Un-sparse extensions table

2024-03-03 Thread Andreas Rheinhardt
Only three of the 226 (== AV_CODEC_ID_AV1) entries have been used. Unsparsing this table is especially important given that this array lives in .data.rel.ro. Signed-off-by: Andreas Rheinhardt --- Instead of parallel tables, one could also merge VkCodecMap and VkExtensionProperties (i.e. putting

[FFmpeg-devel] [PATCH 1/3] avcodec/vulkan_video: Don't use sparse table

2024-03-03 Thread Andreas Rheinhardt
ff_vk_codec_map currently is an array indexed by AVCodecID; it has AV_CODEC_ID_FIRST_AUDIO (= 65536) entries, but uses only three of them; only 24B of 1MiB were actually used This commit fixes this by adding an AVCodecID field to the table and making it non-sparse. Signed-off-by: Andreas

Re: [FFmpeg-devel] [PATCH 14/38] lavu/opt: factor per-type dispatch out of av_opt_set()

2024-03-03 Thread Anton Khirnov
Quoting Anton Khirnov (2024-03-01 17:07:22) > Quoting Michael Niedermayer (2024-02-23 23:50:10) > > Hi Anton > > > > On Fri, Feb 23, 2024 at 02:58:36PM +0100, Anton Khirnov wrote: > > > Will be useful in following commits. > > > --- > > > > breaks: > > > > ./ffmpeg -y -request_channel_layout 3

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/vulkan_decode: Un-sparse extensions table

2024-03-03 Thread Andreas Rheinhardt
Lynne: > Mar 3, 2024, 12:35 by andreas.rheinha...@outlook.com: > >> Only three of the 226 (== AV_CODEC_ID_AV1) entries >> have been used. Unsparsing this table is especially >> important given that this array lives in .data.rel.ro. >> >> Signed-off-by: Andreas Rheinhardt >> --- >> Instead of

[FFmpeg-devel] [PATCH v2] avformat/dvdvideodec: fix -pgc and -pg options

2024-03-03 Thread Marth64
Rebased on top of recently merged fixes (should apply correctly now). In merged DVD patch, -pgc and -pg options were broken. While these are rather advanced options, they are the only means to get content for some strangely authored discs. Signed-off-by: Marth64 --- libavformat/dvdvideodec.c |

[FFmpeg-devel] [PATCH] Changelog: restore new line

2024-03-03 Thread Marth64
A new line was removed in a1304272c3a0ef7c3d77c702dd466b3f11bf300f which breaks consistency in between versions. Signed-off-by: Marth64 --- Changelog | 1 + 1 file changed, 1 insertion(+) diff --git a/Changelog b/Changelog index fedad5782c..f388d41b02 100644 --- a/Changelog +++ b/Changelog @@

Re: [FFmpeg-devel] [PATCH 2/3] doc/community: Conflict of interest recusal requirement (Similar to mid part of Antons proposal)

2024-03-03 Thread Rémi Denis-Courmont
Le tiistaina 27. helmikuuta 2024, 0.44.37 EET Michael Niedermayer a écrit : > Signed-off-by: Michael Niedermayer > --- > doc/community.texi | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/doc/community.texi b/doc/community.texi > index 8a38c6aca0..fc22a8fa61 100644 > ---

Re: [FFmpeg-devel] [PATCH 1/3] doc/community: Vote in the interest of the project (first part of Antons proposal)

2024-03-03 Thread Andreas Rheinhardt
Michael Niedermayer: > On Mon, Feb 26, 2024 at 11:44:36PM +0100, Michael Niedermayer wrote: >> Signed-off-by: Michael Niedermayer >> --- >> doc/community.texi | 2 ++ >> 1 file changed, 2 insertions(+) > > I intend to apply the patchset tomorrow if there are no objections > > thx > Gyan sent

Re: [FFmpeg-devel] FFmpeg 7.0 blocking issues

2024-03-03 Thread Sebastian Ramacher
On 2024-03-02 20:39:08 -0500, Sean McGovern wrote: > On Sat, Mar 2, 2024, 18:19 Michael Niedermayer > wrote: > > > On Sun, Mar 03, 2024 at 12:06:14AM +0100, Sebastian Ramacher wrote: > > > On 2024-03-02 23:55:38 +0100, Michael Niedermayer wrote: > > > > On Tue, Jan 23, 2024 at 08:22:41PM +0100,

Re: [FFmpeg-devel] [PATCH] avcodec/vvcdec: check pred flag to fix undefined beavhiours

2024-03-03 Thread Frank Plowman
On 03/03/2024 03:31, Nuo Mi wrote: > libavcodec/vvc/vvc_inter.c:823:18: runtime error: signed integer overflow: > 1426128896 + 1426128896 cannot be represented in type 'int' > SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior > libavcodec/vvc/vvc_inter.c:823:18 > --- >

[FFmpeg-devel] [PATCH 1/8] avutil/vulkan: Don't autoinclude vulkan_loader.h

2024-03-03 Thread Andreas Rheinhardt
Only include it where necessary. Signed-off-by: Andreas Rheinhardt --- libavcodec/vulkan_decode.c | 1 + libavfilter/vulkan_filter.c | 1 + libavutil/vulkan.c | 1 - libavutil/vulkan.h | 1 - 4 files changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/lpc: Don't use AAC defines directly

2024-03-03 Thread Andreas Rheinhardt
Andreas Rheinhardt: > It leads to defines for the AAC decoder being included > outside of the AAC decoder. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/aac_defines.h | 11 +-- > libavcodec/aacdec_fixed.c| 2 +- > libavcodec/aacdec_template.c | 2 +- >

Re: [FFmpeg-devel] [PATCH 1/4] avcodec/lpc: Don't use AAC defines directly

2024-03-03 Thread Lynne
Mar 3, 2024, 19:43 by andreas.rheinha...@outlook.com: > Andreas Rheinhardt: > >> It leads to defines for the AAC decoder being included >> outside of the AAC decoder. >> >> Signed-off-by: Andreas Rheinhardt >> --- >> libavcodec/aac_defines.h | 11 +-- >> libavcodec/aacdec_fixed.c

Re: [FFmpeg-devel] [PATCH v2 1/5] avcodec/parser: merge packets from the same frame

2024-03-03 Thread Michael Niedermayer
On Fri, Mar 01, 2024 at 02:39:19PM +0100, Nicolas Gaullier wrote: > The mpegts demuxer splits packets according to its max_packet_size. > This currently fills the AVCodecParserContext s->cur_frame_* arrays with > kind of 'empty' entries: no pts/dts. > This patch merges these entries, so the parser

[FFmpeg-devel] [PATCH 7/8 v2] fate/mov: print stream group information for avif/heic tests

2024-03-03 Thread James Almer
Signed-off-by: James Almer --- tests/fate-run.sh | 13 ++ tests/fate/mov.mak| 12 +- .../ref/fate/mov-heic-demux-still-image-grid | 140 -- .../ref/fate/mov-heic-demux-still-image-iovl | 88 ++- 4 files changed,

Re: [FFmpeg-devel] [PATCH 15/38] lavu/opt: add array options

2024-03-03 Thread Marton Balint
On Sun, 3 Mar 2024, James Almer wrote: On 3/3/2024 12:53 PM, Diederick C. Niehorster wrote: On Sun, Mar 3, 2024 at 3:55 PM Anton Khirnov wrote: Quoting Marton Balint (2024-02-26 20:38:46) The more I think about it, it is actually a broader problem. You are changing the semantics of

Re: [FFmpeg-devel] [PATCH] web: Add spi.txt

2024-03-03 Thread Michael Niedermayer
On Sun, Mar 03, 2024 at 05:36:41PM +0100, Stefano Sabatini wrote: > On date Tuesday 2024-02-27 20:01:26 +0100, Michael Niedermayer wrote: > > This explains how to request refunds and what can be funded by SPI > > > > Co-Author: Stefano Sabatini > > --- > > Makefile | 2 +- > >

Re: [FFmpeg-devel] [PATCH] web: Add spi.txt

2024-03-03 Thread Lynne
Mar 3, 2024, 22:08 by mich...@niedermayer.cc: > On Sun, Mar 03, 2024 at 05:36:41PM +0100, Stefano Sabatini wrote: > >> On date Tuesday 2024-02-27 20:01:26 +0100, Michael Niedermayer wrote: >> > This explains how to request refunds and what can be funded by SPI >> > >> > Co-Author: Stefano

Re: [FFmpeg-devel] [PATCH v4 1/2] lavu/hashtable: create generic robin hood hash table

2024-03-03 Thread Connor Worley
Any objections to this patchset? On Sat, Feb 24, 2024 at 12:06 PM Connor Worley wrote: > Signed-off-by: Connor Worley > --- > libavutil/Makefile | 2 + > libavutil/hashtable.c | 192 > libavutil/hashtable.h | 40 >

[FFmpeg-devel] [PATCH] fftools/cmdutils: Inline print_error()

2024-03-03 Thread Andreas Rheinhardt
It is only used by ffprobe (once) and ffplay (twice); inlining it avoids including it unnecessarily into ffmpeg. Signed-off-by: Andreas Rheinhardt --- fftools/cmdutils.c | 5 - fftools/cmdutils.h | 5 - 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/fftools/cmdutils.c

[FFmpeg-devel] [PATCH 5/8] avutil/vulkan: Avoid shadowing

2024-03-03 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavutil/vulkan.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/libavutil/vulkan.c b/libavutil/vulkan.c index 76b61dcaaa..4392a77d6c 100644 --- a/libavutil/vulkan.c +++ b/libavutil/vulkan.c @@ -1308,13 +1308,15 @@ void

[FFmpeg-devel] [PATCH 6/8] avutil/vulkan: Make ff_vk_set_descriptor_image() static

2024-03-03 Thread Andreas Rheinhardt
Only used in vulkan.c. Signed-off-by: Andreas Rheinhardt --- libavutil/vulkan.c | 11 ++- libavutil/vulkan.h | 3 --- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/libavutil/vulkan.c b/libavutil/vulkan.c index 4392a77d6c..c725634fef 100644 --- a/libavutil/vulkan.c +++

[FFmpeg-devel] [PATCH 7/8] avutil/vulkan: Remove unused ff_vk_set_descriptor_sampler()

2024-03-03 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- libavutil/vulkan.c | 27 --- libavutil/vulkan.h | 4 2 files changed, 31 deletions(-) diff --git a/libavutil/vulkan.c b/libavutil/vulkan.c index c725634fef..67b9526255 100644 --- a/libavutil/vulkan.c +++ b/libavutil/vulkan.c @@

[FFmpeg-devel] [PATCH 8/8] avutil/vulkan: Move functions only used by lavfi to it

2024-03-03 Thread Andreas Rheinhardt
lavu/vulkan.c is duplicated into lavfi, lavc and lavu, yet lots of functions in it are only used by lavfi. This commit moves them to lavfi, saving 6544B of .text from both lavc and lavu as well as some .rodata and .data.rel.ro (in total 7880B each for lavc and lavu). Signed-off-by: Andreas

[FFmpeg-devel] [PATCH 2/8] avutil/vulkan_loader: Avoid redundant strings and relocations

2024-03-03 Thread Andreas Rheinhardt
There are three possible names for the functions requested; they only differ in an extension: "", "EXT" or "KHR". Yet vk_load_info contained pointers to all these strings. This is wasteful and this commit changes it to avoid the latter two strings. This saves 6353B of strings, 1776 B of

[FFmpeg-devel] [PATCH 3/8] avutil/vulkan_loader: Use smaller types

2024-03-03 Thread Andreas Rheinhardt
Saves 16B per entry here (four of these 16 bytes are padding); leads to 1776 B of savings in each file that uses ff_vk_load_functions(). Signed-off-by: Andreas Rheinhardt --- libavutil/vulkan_loader.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[FFmpeg-devel] [PATCH 4/8] avutil/vulkan_loader: Avoid relocations for strings

2024-03-03 Thread Andreas Rheinhardt
To do so, concatenate all the names together to one big string name1\0name2\0lastname\0\0. This avoids the pointer in the FunctionLoadInfo structure and thereby moves vk_load_info into .rodata (and makes it smaller by 888B). Signed-off-by: Andreas Rheinhardt --- libavutil/vulkan_loader.h |

[FFmpeg-devel] Inquiry Regarding "Update SITI Filter to Match with Latest Specification" Project and Qualification Task

2024-03-03 Thread Divyansh Khatri
I hope this message finds you well. My name is Divyansh, and I am reaching out to express my interest in the project titled "Update SITI Filter to Match with Latest Specification" listed in Ideas list of FFmpeg 2024 GSOC( link

[FFmpeg-devel] [PATCH] doc/demuxers: emphasize that DVD demuxer does not do decryption

2024-03-03 Thread Marth64
Signed-off-by: Marth64 --- doc/demuxers.texi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 905828d852..f4bac8f3b3 100644 --- a/doc/demuxers.texi +++ b/doc/demuxers.texi @@ -295,6 +295,9 @@ into a conversion pipeline. Menus and seeking are not

Re: [FFmpeg-devel] [PATCH] web: Add spi.txt

2024-03-03 Thread Michael Niedermayer
On Sun, Mar 03, 2024 at 10:29:59PM +0100, Lynne wrote: > Mar 3, 2024, 22:08 by mich...@niedermayer.cc: > > > On Sun, Mar 03, 2024 at 05:36:41PM +0100, Stefano Sabatini wrote: > > > >> On date Tuesday 2024-02-27 20:01:26 +0100, Michael Niedermayer wrote: > >> > This explains how to request refunds

Re: [FFmpeg-devel] [PATCH 8/8] avutil/vulkan: Move functions only used by lavfi to it

2024-03-03 Thread Lynne
Mar 3, 2024, 19:42 by andreas.rheinha...@outlook.com: > lavu/vulkan.c is duplicated into lavfi, lavc and lavu, > yet lots of functions in it are only used by lavfi. > This commit moves them to lavfi, saving 6544B of .text > from both lavc and lavu as well as some .rodata and > .data.rel.ro (in

Re: [FFmpeg-devel] [RFC] clarifying the TC conflict of interest rule

2024-03-03 Thread Anton Khirnov
Quoting Michael Niedermayer (2024-03-03 03:49:33) > Hi > > On Fri, Mar 01, 2024 at 06:33:12PM +0100, Anton Khirnov wrote: > > Hi all, > > it seems the discussion died down, > > There are patches pending, which i will apply soon if no objections > have been raised. I object to these patches.

[FFmpeg-devel] [PATCH] avformat/options: Deprecate av_fmt_ctx_get_duration_estimation_method()

2024-03-03 Thread Andreas Rheinhardt
Forgotten in b7785d10b00c88029d8af7dbddac09ab0d6f2b7f. Signed-off-by: Andreas Rheinhardt --- doc/APIchanges | 4 libavformat/avformat.h | 4 libavformat/options.c | 2 ++ libavformat/version_major.h | 1 + 4 files changed, 11 insertions(+) diff --git