Re: [FFmpeg-devel] [PATCH v10 0/6] Support enhanced flv in FFmpeg

2023-06-09 Thread Tristan Matthews
On Wed, Jun 7, 2023 at 8:02 PM Neal Gompa wrote: > > On Tue, Jun 6, 2023 at 2:43 AM Steven Liu wrote: > > > > Neal Gompa 于2023年5月31日周三 13:47写道: > > > > > > On Mon, May 15, 2023 at 10:41 PM Neal Gompa wrote: > > > > > > > > On Mon, May 15, 2023 at 4:32 AM Steven Liu wrote: > > > > > > > > > >

[FFmpeg-devel] [PATCH] avformat/mov: Add support for uuid-based senc box

2023-06-09 Thread Hendi
Attached.From eca7929274b9bfd7c5aa8a76177dafaa57ab91a5 Mon Sep 17 00:00:00 2001 From: Hendi Date: Fri, 9 Jun 2023 01:46:30 +0200 Subject: [PATCH] avformat/mov: Add support for uuid-based senc box In PIFF-branded mp4 files, the senc data comes in a uuid box. Signed-off-by: Hendi ---

Re: [FFmpeg-devel] [PATCH 4/5] avformat/jpegxl_probe: inline various ret < 0 checks

2023-06-09 Thread Michael Niedermayer
On Thu, Jun 08, 2023 at 11:44:38PM -0300, James Almer wrote: > On 6/8/2023 11:32 PM, Anton Khirnov wrote: > > Quoting Leo Izen (2023-06-08 16:26:36) > > > Inlines some ret < 0 checks to look like: > > > if ((ret = func()) < 0) > > > return ret; > > > > > > which clarifies code

Re: [FFmpeg-devel] [External Sender] Re: [PATCH] fftools/ffmpeg_enc: Prevent duplicate A53 Closed Captions at frame rate up-conversion

2023-06-09 Thread Artem Smorodin
Hi David, I think the reworking of the "-r" option is a breaking change. I understand that output won't conform to the spec, but at least CC won't be broken. For example libx264->a53cc option is 1 by default, so any h264 output with CC with frame rate up-conversion will be broken. The

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg_enc: Prevent duplicate A53 Closed Captions at frame rate up-conversion

2023-06-09 Thread Devin Heitmueller
Hi Artem, On Fri, Jun 9, 2023 at 9:19 AM Artem Smorodin wrote: > > When up-converting frame rate, ffmpeg duplicates frames including side data. > This causes duplicated characters in the output. This patch removes A53 side > data from duplicated frames. > > Signed-off-by: Artem Smorodin > ---

Re: [FFmpeg-devel] [PATCH] avformat/mov: Don't allocate unnecessarily large blocks of memory

2023-06-09 Thread Hendi
On 09.06.2023 04:47, "zhilizhao(赵志立)" wrote: It’s a quick fix, but I’m afraid the two tickets are caused by more deep pitfalls. It would be helpful if someone can provide a sample for test. I've uploaded a sample called bear-640x360-v_frag-cenc-senc.mp4. You can play it with

[FFmpeg-devel] [PATCH] fftools/ffmpeg_enc: Prevent duplicate A53 Closed Captions at frame rate up-conversion

2023-06-09 Thread Artem Smorodin
When up-converting frame rate, ffmpeg duplicates frames including side data. This causes duplicated characters in the output. This patch removes A53 side data from duplicated frames. Signed-off-by: Artem Smorodin --- fftools/ffmpeg_enc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git

[FFmpeg-devel] [PATCH 3/3] avcodec/videotoolboxenc: inherit profile from AVCodecContext

2023-06-09 Thread Zhao Zhili
From: Zhao Zhili This can happen when user set the avctx->profile field directly instead of specify profile via option. --- libavcodec/videotoolboxenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index

[FFmpeg-devel] [PATCH 2/3] avcodec/videotoolboxenc: remove AUTO_PROFILE define

2023-06-09 Thread Zhao Zhili
From: Zhao Zhili Use FF_PROFILE_UNKNOWN as auto mode. --- libavcodec/videotoolboxenc.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 26485a3dea..9f65519700 100644 ---

[FFmpeg-devel] [PATCH 1/3] avcodec/videotoolboxenc: use int as the type of profile

2023-06-09 Thread Zhao Zhili
From: Zhao Zhili Other than save a few bytes, it also has the benefit to show the AV_OPT_TYPE_CONST value in help, e.g., -profile E..V... Profile (from 0 to INT_MAX) (default 0) baseline66 E..V... Baseline Profile ... ---

[FFmpeg-devel] [PATCH] avcodec/mediacodecenc: add profile names to options

2023-06-09 Thread Zhao Zhili
From: Zhao Zhili H.264 high10/high422/high44 are unlikely supported by devices. It's there for developers to do the experiment. H.265 main10 works on my device with AV_PIX_FMT_MEDIACODEC. OMX_COLOR_FormatYUV420Planar16 doesn't work. Signed-off-by: Zhao Zhili --- libavcodec/mediacodecenc.c |

Re: [FFmpeg-devel] [PATCH] avcodec: add VMX1 decoder

2023-06-09 Thread Paul B Mahol
On Fri, Jun 9, 2023 at 7:57 AM Jean-Baptiste Kempf wrote: > On Thu, 8 Jun 2023, at 19:39, Paul B Mahol wrote: > > Attached. > > Missing version bumping? > That is done when committing. To prevent potential need for rebase if in meantime someone changes version. > -- > Jean-Baptiste Kempf -

[FFmpeg-devel] [PATCH] lavc/h264chroma: RISC-V V add motion compensation for 4xH and 2xH chroma blocks

2023-06-09 Thread Arnie Chang
Optimize the put and avg filtering for 4xH and 2xH blocks Signed-off-by: Arnie Chang --- checkasm: using random seed 3475799765 RVVi32: - h264chroma.chroma_mc [OK] checkasm: all 6 tests passed avg_h264_chroma_mc1_8_c: 1821.5 avg_h264_chroma_mc1_8_rvv_i32: 466.5 avg_h264_chroma_mc2_8_c: 939.2