Re: [FFmpeg-devel] Hardware purchase request: AVX512-capable laptop

2024-01-17 Thread Lynne
Jan 18, 2024, 04:29 by mich...@niedermayer.cc: > On Wed, Jan 17, 2024 at 04:39:21PM +0100, Lynne wrote: > >> Jan 17, 2024, 14:37 by mich...@niedermayer.cc: >> > [...] > >> > also what exactly are your requirements for the new machiene >> > and what are the "nice to have" things ? >> > >> > also

[FFmpeg-devel] configure doesn't seem to use pkg-config flags?

2024-01-17 Thread Roger Pack
Hello. After compiling libx265 as a "static library" (mingw cross compiling targeting win64) $ pkg-config --libs --static x265 -L/home/rdp/new/sandbox/win64_static/build_files/lib -lx265 /usr/lib/gcc/x86_64-w64-mingw32/10-win32/libstdc++.a -lgcc I get this failure at configure time: ./configure

Re: [FFmpeg-devel] Hardware purchase request: AVX512-capable laptop

2024-01-17 Thread Michael Niedermayer
On Wed, Jan 17, 2024 at 04:39:21PM +0100, Lynne wrote: > Jan 17, 2024, 14:37 by mich...@niedermayer.cc: [...] > > also what exactly are your requirements for the new machiene > > and what are the "nice to have" things ? > > > > also what is the expected lifetime of this, how long will the > > new

[FFmpeg-devel] [PATCH] Add DXV encoder with support for DXT1 texture format

2024-01-17 Thread Connor Worley
Signed-off-by: Connor Worley --- Changelog | 1 + configure | 1 + doc/general_contents.texi | 3 +- libavcodec/Makefile | 1 + libavcodec/allcodecs.c| 1 + libavcodec/dxvenc.c | 358 ++

Re: [FFmpeg-devel] [PATCH] Add DXV encoder with support for DXT1 texture format

2024-01-17 Thread Connor Worley
On 1/17/24 08:55, Connor Worley wrote: Signed-off-by: Connor Worley ---  Changelog |   1 +  configure |   1 +  doc/general_contents.texi |   3 +-  libavcodec/Makefile   |   1 +  libavcodec/allcodecs.c    |   1 +  libavcodec/dxvenc.c   | 362

[FFmpeg-devel] [PATCH 2/2] avformat: add a Tile Grid stream group type

2024-01-17 Thread James Almer
This will be used to support tiled image formats like HEIF. Signed-off-by: James Almer --- libavformat/avformat.c | 5 + libavformat/avformat.h | 3 +++ libavformat/dump.c | 36 libavformat/options.c | 9 + 4 files changed, 53

[FFmpeg-devel] [PATCH 1/2] avutil: add a Tile Grid API

2024-01-17 Thread James Almer
This includes a struct and helpers. It will be used to support container level tiled image formats like HEIF, but should be generic enough for general usage. Signed-off-by: James Almer --- libavutil/Makefile | 2 + libavutil/tile.c | 72 + libavutil/tile.h | 132

[FFmpeg-devel] [PATCH] avformat/dump: only print streams within a group in verbose levels

2024-01-17 Thread James Almer
Signed-off-by: James Almer --- libavformat/dump.c | 278 +++-- 1 file changed, 143 insertions(+), 135 deletions(-) diff --git a/libavformat/dump.c b/libavformat/dump.c index cc179f284f..aff51b43f6 100644 --- a/libavformat/dump.c +++ b/libavformat/dump.c

Re: [FFmpeg-devel] [PATCH 7/7] fftools/ffplay: support applying container level cropping

2024-01-17 Thread Dmitrii Ovchinnikov
I tested these patches with the AMF AV1 encoder(by adding AV_PKT_DATA_FRAME_CROPPING) It works fine. I suggest a small change to the code for better safety. Change this part: >>if (sd->data && sd->size == sizeof(uint32_t) * 4) { to: if (sd && sd->data && sd->size == sizeof(uint32_t) * 4) { This

[FFmpeg-devel] [PATCH] Add DXV encoder with support for DXT1 texture format

2024-01-17 Thread Connor Worley
Signed-off-by: Connor Worley --- Changelog | 1 + configure | 1 + doc/general_contents.texi | 3 +- libavcodec/Makefile | 1 + libavcodec/allcodecs.c| 1 + libavcodec/dxvenc.c | 362 ++

Re: [FFmpeg-devel] Hardware purchase request: AVX512-capable laptop

2024-01-17 Thread Lynne
Jan 17, 2024, 14:37 by mich...@niedermayer.cc: > On Tue, Jan 16, 2024 at 03:59:43PM +0100, Lynne wrote: > >> Jan 16, 2024, 11:06 by kier...@obe.tv: >> >> >> >> >> A ticket doesn't have durability. >> >> >> > >> > A Ryzen 5 vs Ryzen 7 in the same laptop chassis doesn't change its >> > durability,

Re: [FFmpeg-devel] [PATCH 2/8] fftools/cmdutils: add a flag for per-stream options

2024-01-17 Thread Anton Khirnov
Quoting Michael Niedermayer (2024-01-17 14:01:49) > On Wed, Jan 17, 2024 at 01:40:29PM +0100, Anton Khirnov wrote: > > Not all OPT_SPEC options are per-stream, this will allow identifying > > those that are, which will be useful in following commits. > > --- > > fftools/cmdutils.h | 13 +++-- >

Re: [FFmpeg-devel] [PATCH 4/6] fftools/cmdutils: add option syntax for loading arbitrary arguments from a file

2024-01-17 Thread Anton Khirnov
Quoting Michael Niedermayer (2024-01-17 14:08:29) > On Wed, Jan 17, 2024 at 10:22:31AM +0100, Anton Khirnov wrote: > > Aligned with analogous feature for filter options in ffmpeg CLI. > > --- > > Changelog| 2 + > > doc/fftools-common-opts.texi | 9 > >

Re: [FFmpeg-devel] Hardware purchase request: AVX512-capable laptop

2024-01-17 Thread Michael Niedermayer
On Tue, Jan 16, 2024 at 03:59:43PM +0100, Lynne wrote: > Jan 16, 2024, 11:06 by kier...@obe.tv: > > >> > >> A ticket doesn't have durability. > >> > > > > A Ryzen 5 vs Ryzen 7 in the same laptop chassis doesn't change its > > durability, it only doubles the laptop's price. > > > > It doubles the

Re: [FFmpeg-devel] [PATCH] Add DXV encoder with support for DXT1 texture format

2024-01-17 Thread Michael Niedermayer
On Wed, Jan 17, 2024 at 12:27:40AM -0800, Connor Worley wrote: > Signed-off-by: Connor Worley > --- > Changelog | 1 + > configure | 1 + > doc/general_contents.texi | 3 +- > libavcodec/Makefile | 1 + > libavcodec/allcodecs.c| 1 + >

Re: [FFmpeg-devel] [PATCH 4/6] fftools/cmdutils: add option syntax for loading arbitrary arguments from a file

2024-01-17 Thread Michael Niedermayer
On Wed, Jan 17, 2024 at 10:22:31AM +0100, Anton Khirnov wrote: > Aligned with analogous feature for filter options in ffmpeg CLI. > --- > Changelog| 2 + > doc/fftools-common-opts.texi | 9 > fftools/cmdutils.c | 99 ++-- >

Re: [FFmpeg-devel] [PATCH 2/8] fftools/cmdutils: add a flag for per-stream options

2024-01-17 Thread Michael Niedermayer
On Wed, Jan 17, 2024 at 01:40:29PM +0100, Anton Khirnov wrote: > Not all OPT_SPEC options are per-stream, this will allow identifying > those that are, which will be useful in following commits. > --- > fftools/cmdutils.h | 13 +++-- > fftools/ffmpeg_opt.c | 118

[FFmpeg-devel] [PATCH 8/8] fftools/ffmpeg_opt: improve wording in option descriptions

2024-01-17 Thread Anton Khirnov
--- fftools/ffmpeg_opt.c | 33 + 1 file changed, 17 insertions(+), 16 deletions(-) diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c index bdb71d4283..6ff1cb229d 100644 --- a/fftools/ffmpeg_opt.c +++ b/fftools/ffmpeg_opt.c @@ -1476,7 +1476,7 @@ const

[FFmpeg-devel] [PATCH 4/8] fftools/ffmpeg_opt: add more structure to long help output

2024-01-17 Thread Anton Khirnov
Split the "Per-file main options" section into * per-file input and output * per-file input-only * per-file output-only * per-stream --- fftools/ffmpeg_opt.c | 43 ++- 1 file changed, 38 insertions(+), 5 deletions(-) diff --git a/fftools/ffmpeg_opt.c

[FFmpeg-devel] [PATCH 6/8] fftools/ffmpeg_opt: briefly mention stream specifier syntax in help output

2024-01-17 Thread Anton Khirnov
--- fftools/ffmpeg_opt.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c index 4abdd457d3..bdb71d4283 100644 --- a/fftools/ffmpeg_opt.c +++ b/fftools/ffmpeg_opt.c @@ -1194,6 +1194,9 @@ void show_help_default(const char *opt, const char *arg)

[FFmpeg-devel] [PATCH 2/8] fftools/cmdutils: add a flag for per-stream options

2024-01-17 Thread Anton Khirnov
Not all OPT_SPEC options are per-stream, this will allow identifying those that are, which will be useful in following commits. --- fftools/cmdutils.h | 13 +++-- fftools/ffmpeg_opt.c | 118 +-- 2 files changed, 68 insertions(+), 63 deletions(-) diff

[FFmpeg-devel] [PATCH 7/8] fftools/cmdutils: surround option arguments by <> in help output

2024-01-17 Thread Anton Khirnov
Makes it more clear that they are placeholders for actual values. --- fftools/cmdutils.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index f652820184..daf7673adb 100644 --- a/fftools/cmdutils.c +++ b/fftools/cmdutils.c @@

[FFmpeg-devel] [PATCH 5/8] fftools/cmdutils: indicate specifiers in option syntax in help output

2024-01-17 Thread Anton Khirnov
--- fftools/cmdutils.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index be01372743..f652820184 100644 --- a/fftools/cmdutils.c +++ b/fftools/cmdutils.c @@ -126,6 +126,12 @@ void show_help_options(const OptionDef *options, const char *msg,

[FFmpeg-devel] [PATCH 1/8] fftools/cmdutils: hide some information listing options from basic help

2024-01-17 Thread Anton Khirnov
They are for advanced usage only and should not clutter basic output. --- fftools/ffmpeg_opt.c | 7 +-- fftools/opt_common.h | 24 2 files changed, 17 insertions(+), 14 deletions(-) diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c index

[FFmpeg-devel] [PATCH 3/8] fftools/cmdutils: drop alt_flags parameter from show_help_options()

2024-01-17 Thread Anton Khirnov
No user sets it to more than one flag, so it is redundant with req_flags. --- fftools/cmdutils.c | 3 +-- fftools/cmdutils.h | 3 +-- fftools/ffmpeg_opt.c | 28 ++-- fftools/ffplay.c | 4 ++-- fftools/ffprobe.c| 2 +- 5 files changed, 19 insertions(+), 21

Re: [FFmpeg-devel] [PATCH 1/6] fftools/ffprobe: make option strings dynamically allocated

2024-01-17 Thread Anton Khirnov
Quoting Zhao Zhili (2024-01-17 12:22:10) > > > > On Jan 17, 2024, at 17:22, Anton Khirnov wrote: > > > > Do not store the supplied arg pointer directly. While that is valid for > > now, arg will become ephemeral in the future commits. > > --- > > fftools/ffprobe.c | 18 +- > > 1

Re: [FFmpeg-devel] [PATCH 1/6] fftools/ffprobe: make option strings dynamically allocated

2024-01-17 Thread Zhao Zhili
> On Jan 17, 2024, at 17:22, Anton Khirnov wrote: > > Do not store the supplied arg pointer directly. While that is valid for > now, arg will become ephemeral in the future commits. > --- > fftools/ffprobe.c | 18 +- > 1 file changed, 13 insertions(+), 5 deletions(-) > > diff

[FFmpeg-devel] [PATCH 5/6] fftools/ffmpeg: deprecate -filter_complex_script

2024-01-17 Thread Anton Khirnov
It is equivalent to -/filter_complex. --- doc/ffmpeg.texi | 9 + doc/filters.texi| 3 +-- fftools/ffmpeg.h| 1 + fftools/ffmpeg_opt.c| 9 - tests/fate/ffmpeg.mak | 2 +- tests/fate/filter-audio.mak | 8

[FFmpeg-devel] [PATCH 6/6] fftools/ffmpeg: deprecate -filter_script

2024-01-17 Thread Anton Khirnov
It is equivalent to -/filter. --- doc/ffmpeg.texi | 6 -- doc/filters.texi| 3 --- fftools/ffmpeg.h| 2 ++ fftools/ffmpeg_mux_init.c | 36 ++-- fftools/ffmpeg_opt.c| 4 +++- tests/fate/filter-audio.mak | 2 +-

[FFmpeg-devel] [PATCH 3/6] fftools/ffmpeg: make attachment filenames dynamically allocated

2024-01-17 Thread Anton Khirnov
Do not store the supplied arg pointer directly. While that is valid for now, it will become ephemeral in the future commits. --- fftools/ffmpeg_opt.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c index ea995f2b5f..563f443bd8

[FFmpeg-devel] [PATCH 1/6] fftools/ffprobe: make option strings dynamically allocated

2024-01-17 Thread Anton Khirnov
Do not store the supplied arg pointer directly. While that is valid for now, arg will become ephemeral in the future commits. --- fftools/ffprobe.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index

[FFmpeg-devel] [PATCH 4/6] fftools/cmdutils: add option syntax for loading arbitrary arguments from a file

2024-01-17 Thread Anton Khirnov
Aligned with analogous feature for filter options in ffmpeg CLI. --- Changelog| 2 + doc/fftools-common-opts.texi | 9 fftools/cmdutils.c | 99 ++-- fftools/cmdutils.h | 3 ++ fftools/ffmpeg.h | 1 -

[FFmpeg-devel] [PATCH 2/6] fftools/ffplay: make option strings dynamically allocated

2024-01-17 Thread Anton Khirnov
Do not store the supplied arg pointer directly. While that is valid for now, it will become ephemeral in the future commits. --- fftools/ffplay.c | 30 +++--- 1 file changed, 23 insertions(+), 7 deletions(-) diff --git a/fftools/ffplay.c b/fftools/ffplay.c index

Re: [FFmpeg-devel] [PATCH 7/8] fftools/ffmpeg_demux: implement -bsf for input

2024-01-17 Thread Anton Khirnov
Quoting Stefano Sabatini (2024-01-06 13:12:19) > > This looks spurious, since this suggests the example is about the > listing, and it's applying a weird order of example/explanation > (rather than the opposite). I see nothing weird about this order, it's the standard way it is done in most

[FFmpeg-devel] [PATCH] Add DXV encoder with support for DXT1 texture format

2024-01-17 Thread Connor Worley
Signed-off-by: Connor Worley --- Changelog | 1 + configure | 1 + doc/general_contents.texi | 3 +- libavcodec/Makefile | 1 + libavcodec/allcodecs.c| 1 + libavcodec/dxvenc.c | 362 ++