Re: [FFmpeg-devel] [PATCH] configure: link to libatomic when it's present

2022-01-21 Thread Brad Smith
On 1/19/2022 10:23 AM, James Almer wrote: On 1/19/2022 10:48 AM, Anton Khirnov wrote: C11 atomics in some configurations (e.g. 64bit operations on ppc64 with GCC) require linking to libatomic. Fixes #9275 ---   configure | 25 -   1 file changed, 16 insertions(+), 9 dele

[FFmpeg-devel] [PATCH] fate/ffmpeg: Add test for autorotating video

2022-01-21 Thread Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt --- tests/fate/ffmpeg.mak | 6 tests/ref/fate/autorotate | 61 +++ 2 files changed, 67 insertions(+) create mode 100644 tests/ref/fate/autorotate diff --git a/tests/fate/ffmpeg.mak b/tests/fate/ffmpeg.mak index b804

[FFmpeg-devel] [PATCH] swscale/x86/swscale: Remove superfluous and invalid '; '

2022-01-21 Thread Andreas Rheinhardt
Inside a function an unnecessary ';' is just a null statement; yet outside of it it is actually illegal (but compilers happen to accept it without warning except when using -pedantic). So modify the macros to always expect the user to add a ';'. Signed-off-by: Andreas Rheinhardt --- libswscale/x

[FFmpeg-devel] [PATCH v5] avformat/imf: add IMF CPL with repeated resources to FATE

2022-01-21 Thread pal
From: Pierre-Anthony Lemieux Signed-off-by: Pierre-Anthony Lemieux --- Notes: The `fate-imf-cpl-with-repeat` target requires following files (<256 kB) to placed under $(TARGET_SAMPLES)/imf/countdown/: http://ffmpeg-imf-samples-public.s3-website-us-west-1.amazonaws.com/countdown

Re: [FFmpeg-devel] [PATCH 091/281] matroska: convert to new channel layout API

2022-01-21 Thread James Almer
On 1/21/2022 4:52 PM, Andreas Rheinhardt wrote: James Almer: From: Vittorio Giovara Signed-off-by: Vittorio Giovara Signed-off-by: Anton Khirnov Signed-off-by: James Almer --- libavformat/matroskadec.c | 8 ++-- libavformat/matroskaenc.c | 10 +- 2 files changed, 11 inserti

Re: [FFmpeg-devel] [PATCH 091/281] matroska: convert to new channel layout API

2022-01-21 Thread Andreas Rheinhardt
James Almer: > From: Vittorio Giovara > > Signed-off-by: Vittorio Giovara > Signed-off-by: Anton Khirnov > Signed-off-by: James Almer > --- > libavformat/matroskadec.c | 8 ++-- > libavformat/matroskaenc.c | 10 +- > 2 files changed, 11 insertions(+), 7 deletions(-) > > diff --g

[FFmpeg-devel] [PATCH] avformat/matroskadec: Export rectangular projection as displaymatrix

2022-01-21 Thread Andreas Rheinhardt
This is the officially supported way to express rotations and flips in Matroska. Signed-off-by: Andreas Rheinhardt --- No muxing support and tests for now; for the former, I first have to write a proper function that actually checks for whether a given display matrix is a rotation (even if not a

Re: [FFmpeg-devel] [PATCH 2/2] libfdk-aacdec: Flush delayed samples at the end

2022-01-21 Thread Andreas Rheinhardt
Martin Storsjö: > On Fri, 21 Jan 2022, Andreas Rheinhardt wrote: > >> Martin Storsjö: >>> Also trim off delay samples at the start instead of adjusting pts >>> to compensate for them; this avoids unwanted offsets if working >>> with raw samples without considering their pts. >>> --- >>>  libavcode

Re: [FFmpeg-devel] [PATCH 2/2] libfdk-aacdec: Flush delayed samples at the end

2022-01-21 Thread Martin Storsjö
On Fri, 21 Jan 2022, Andreas Rheinhardt wrote: Martin Storsjö: Also trim off delay samples at the start instead of adjusting pts to compensate for them; this avoids unwanted offsets if working with raw samples without considering their pts. --- libavcodec/libfdk-aacdec.c | 80 +

Re: [FFmpeg-devel] [PATCH 2/2] libfdk-aacdec: Flush delayed samples at the end

2022-01-21 Thread Andreas Rheinhardt
Martin Storsjö: > Also trim off delay samples at the start instead of adjusting pts > to compensate for them; this avoids unwanted offsets if working > with raw samples without considering their pts. > --- > libavcodec/libfdk-aacdec.c | 80 +++--- > 1 file changed,

Re: [FFmpeg-devel] [PATCH v2 1/6] avformat/mux: Remove assert based on faulty assumptions

2022-01-21 Thread Andreas Rheinhardt
Andreas Rheinhardt: > This assert is based upon the wrong assumption that > the noninterleaved codepath is never used; if it is used, > max_interleave_delta is irrelevant. It furthermore > ignores audio_preload. > > Signed-off-by: Andreas Rheinhardt > --- > libavformat/mux.c | 1 - > 1 file chan

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: fix (a)buffer src names

2022-01-21 Thread James Almer
On 1/21/2022 5:15 AM, Anton Khirnov wrote: --- Grmbl, could have sworn I tested this --- fftools/ffmpeg_filter.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c index 1f6cba2c04..7b6fcc36de 100644 --- a/fftools/ffmpeg

[FFmpeg-devel] [PATCH v1] avfilter/vf_gblur_vulkan: add sizeV option

2022-01-21 Thread Wu, Jianhua
[PATCH 1/5] avfilter/vf_gblur_vulkan: add sizeV option [PATCH 2/5] avfilter:add shader_vulkan filter [PATCH 3/5] avfilter/vf_blend_vulkan: add multiply blend mode [PATCH 4/5] avutil/vulkan: don't use strlen as loop condition [PATCH 5/5] avfilter/scale_vulkan: use RET for checking return value P

Re: [FFmpeg-devel] [PATCH] avutil/pixfmt.h: typo

2022-01-21 Thread Andreas Rheinhardt
Diederick Niehorster: > Signed-off-by: Diederick Niehorster > --- > libavutil/pixfmt.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/pixfmt.h b/libavutil/pixfmt.h > index 02e355eb4d..66913f606b 100644 > --- a/libavutil/pixfmt.h > +++ b/libavutil/pixfmt.h > @@

Re: [FFmpeg-devel] [PATCH] avformat/mov: Fix endian-dependent parsing

2022-01-21 Thread Martin Storsjö
On Fri, 21 Jan 2022, Andreas Rheinhardt wrote: MOVAtom.type is always read as a little-endian number (despite MOV/ISOBMFF being big-endian). Fixes the matroska-dovi-write-config8 FATE-test on big-endian arches (which runs into the "index out of range" warning message). Signed-off-by: Andreas Rh

[FFmpeg-devel] [PATCH] avformat/mov: Fix endian-dependent parsing

2022-01-21 Thread Andreas Rheinhardt
MOVAtom.type is always read as a little-endian number (despite MOV/ISOBMFF being big-endian). Fixes the matroska-dovi-write-config8 FATE-test on big-endian arches (which runs into the "index out of range" warning message). Signed-off-by: Andreas Rheinhardt --- A few days ago I sent a patch using

Re: [FFmpeg-devel] [PATCH 1/2] lavu/tx: rewrite internal code as a tree-based codelet constructor

2022-01-21 Thread Lynne
21 Jan 2022, 09:33 by d...@lynne.ee: > This commit rewrites the internal transform code into a constructor > that stitches transforms (codelets). > This allows for transforms to reuse arbitrary parts of other > transforms, and allows transforms to be stacked onto one > another (such as a full iMDC

[FFmpeg-devel] [PATCH 2/2] lavu/tx: add an RDFT implementation

2022-01-21 Thread Lynne
RDFTs are full of conventions that vary between implementations. What I've gone for here is what's most common between both fftw, avcodec's rdft and what we use, the equivalent of which is DFT_R2C for forward and IDFT_C2R for inverse. The other 2 conventions (IDFT_R2C and DFT_C2R) were not used at

[FFmpeg-devel] [PATCH 1/2] lavu/tx: rewrite internal code as a tree-based codelet constructor

2022-01-21 Thread Lynne
This commit rewrites the internal transform code into a constructor that stitches transforms (codelets). This allows for transforms to reuse arbitrary parts of other transforms, and allows transforms to be stacked onto one another (such as a full iMDCT using a half-iMDCT which in turn uses an FFT).

[FFmpeg-devel] [PATCH] fftools/ffmpeg: fix (a)buffer src names

2022-01-21 Thread Anton Khirnov
--- Grmbl, could have sworn I tested this --- fftools/ffmpeg_filter.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c index 1f6cba2c04..7b6fcc36de 100644 --- a/fftools/ffmpeg_filter.c +++ b/fftools/ffmpeg_filter.c @@ -949,8