Re: [FFmpeg-devel] [PATCH 1/4] libavutil/hwcontext_d3d11va: Add nb_surfaces to AVD3D11VAFramesContext

2021-11-10 Thread Chen, Wenbin
> > -Original Message- > > From: ffmpeg-devel On Behalf Of > > Chen, Wenbin > > Sent: Wednesday, November 10, 2021 4:03 AM > > To: ffmpeg-devel@ffmpeg.org > > Subject: Re: [FFmpeg-devel] [PATCH 1/4] libavutil/hwcontext_d3d11va: > > Add nb_surfaces to AVD3D11VAFramesContext > > > > > Adding

Re: [FFmpeg-devel] [PATCH 2/4] libavutil/hwcontext_qsv: fix a bug when malloc handle_pairs_internal

2021-11-10 Thread Chen, Wenbin
> > -Original Message- > > From: ffmpeg-devel On Behalf Of > > Chen, Wenbin > > Sent: Wednesday, November 10, 2021 4:03 AM > > To: ffmpeg-devel@ffmpeg.org > > Subject: Re: [FFmpeg-devel] [PATCH 2/4] libavutil/hwcontext_qsv: fix > > a bug when malloc handle_pairs_internal > > > > > This com

Re: [FFmpeg-devel] [PATCH 6/7] libavutil/hwcontext_vulkan: fix a sem_wait bug when export drm

2021-11-10 Thread Chen, Wenbin
> 9 Nov 2021, 10:18 by wenbin.c...@intel.com: > > > sem_sig_val is wrongly assigned to pWaitSemaphoreValues when export > drm. Now fix > > it. > > > > Signed-off-by: Wenbin Chen <> wenbin.c...@intel.com> > > > > > Thanks for spotting this, I fixed that in my patchset and updated branch. > frame->

Re: [FFmpeg-devel] [PATCH 1/2] avformat/hlsenc: Remove nonsense memset

2021-11-10 Thread Steven Liu
> 2021年11月10日 下午8:59,Andreas Rheinhardt 写道: > > The memset here is both unnecessary (avio_read() ignores the previous > content of the destination buffer) as well as nonsense (for a char > buf[BUFSIZE] sizeof(buf) and sizeof(BUFSIZE) are not the same; the > latter is sizeof(int)). > Fixes Cover

[FFmpeg-devel] [PATCH 2/2] avformat/mxf: add documentation for eia608_extract parameter

2021-11-10 Thread Marc-Antoine Arnaud
--- doc/demuxers.texi | 3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index fecfeadb47..06f666b5f1 100644 --- a/doc/demuxers.texi +++ b/doc/demuxers.texi @@ -776,6 +776,9 @@ MXF demuxer. @table @option +@item -eia608_extract @var{bool} +Extract EIA

[FFmpeg-devel] [PATCH 1/2] avformat/mxf: support MCA audio information

2021-11-10 Thread Marc-Antoine Arnaud
--- doc/demuxers.texi | 10 ++ libavformat/mxf.h | 1 + libavformat/mxfdec.c | 284 +- libavformat/version.h | 2 +- 4 files changed, 290 insertions(+), 7 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 1c9575b2e8..fecfea

[FFmpeg-devel] [PATCH v2] avutil/opt: handle whole range of int64_t in av_opt_get_int

2021-11-10 Thread Zhao Zhili
Make get_int/set_int symetric. The int64_t to double to int64_t conversion is unprecise for large value. --- libavutil/opt.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavutil/opt.c b/libavutil/opt.c index c7001dbcd3..cfda31ea2f 100644 --- a/libavutil/opt.c +++ b/lib

Re: [FFmpeg-devel] [PATCH 03/17] fate/mxf: Add tests for H.264 remuxing

2021-11-10 Thread Tomas Härdin
ons 2021-11-10 klockan 14:52 +0100 skrev Andreas Rheinhardt: > Tomas Härdin: > > tis 2021-11-09 klockan 19:01 +0100 skrev Andreas Rheinhardt: > > > These tests exhibit two bugs: Instead of using the in-band > > > extradata > > > the demuxer makes up some extradata designed for AVC intra tracks > >

Re: [FFmpeg-devel] [PATCH 01/17] avformat/mxfenc: Auto-insert h264_mp4toannexb BSF if needed

2021-11-10 Thread Tomas Härdin
ons 2021-11-10 klockan 14:21 +0100 skrev Andreas Rheinhardt: > Tomas Härdin: > > tis 2021-11-09 klockan 22:07 +0100 skrev Andreas Rheinhardt: > > > Tomas Härdin: > > > > tis 2021-11-09 klockan 18:34 +0100 skrev Andreas Rheinhardt: > > > > > The mxf and mxf_opatom muxer expect H.264 in Annex B forma

Re: [FFmpeg-devel] [PATCH] avutil/opt: handle whole range of int64_t in av_opt_get_int

2021-11-10 Thread Michael Niedermayer
On Wed, Nov 10, 2021 at 08:46:26PM +0800, Zhao Zhili wrote: > Make get_int/set_int symetric. The int64_t to double to int64_t > conversion is unprecise for large value. > --- > libavutil/opt.c | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavutil/opt.c b/libavutil

[FFmpeg-devel] [PATCH v3 3/3] fate: Add test for Dolby Vision RPU side data

2021-11-10 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- tests/fate/hevc.mak| 3 + tests/ref/fate/hevc-dv-rpu | 512 + 2 files changed, 515 insertions(+) create mode 100644 tests/ref/fate/hevc-dv-rpu diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak index f6ea1df9a

[FFmpeg-devel] [PATCH v3 2/3] avcodec/hevcdec: Export Dolby Vision RPUs as side data

2021-11-10 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- libavcodec/hevcdec.c | 39 +++ libavcodec/hevcdec.h | 2 ++ libavcodec/version.h | 2 +- 3 files changed, 42 insertions(+), 1 deletion(-) diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c index 246ffd7d80..3e19f4ebe

[FFmpeg-devel] [PATCH v3 1/3] avutil: Add Dolby Vision RPU side data type

2021-11-10 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- doc/APIchanges | 3 +++ libavutil/frame.c | 1 + libavutil/frame.h | 7 +++ libavutil/version.h | 2 +- 4 files changed, 12 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index 99e185ee4e..86c7cae4bc 100644 --- a/doc/API

[FFmpeg-devel] [PATCH v3 0/3] Dolby Vision RPU Side Data

2021-11-10 Thread Derek Buitenhuis
Only patch 2 ha`s changed, as per Andreas' review. Derek Buitenhuis (3): avutil: Add Dolby Vision RPU side data type avcodec/hevcdec: Export Dolby Vision RPUs as side data fate: Add test for Dolby Vision RPU side data doc/APIchanges | 3 + libavcodec/hevcdec.c | 39 +++

[FFmpeg-devel] android_camera libx264 rtmps : Audio not capturing

2021-11-10 Thread Hemant K
Hello , I am using the below command to broadcast from android phone. I am able to view the Video but it is not capturing the audio from phone. Could you please let me know what is missing to all capturing the audio from phone? FFmpegKit.executeAsync("-f android_camera -r 15 -i 0:0 -vf scale=-1

Re: [FFmpeg-devel] [PATCH 03/17] fate/mxf: Add tests for H.264 remuxing

2021-11-10 Thread Andreas Rheinhardt
Tomas Härdin: > tis 2021-11-09 klockan 19:01 +0100 skrev Andreas Rheinhardt: >> These tests exhibit two bugs: Instead of using the in-band extradata >> the demuxer makes up some extradata designed for AVC intra tracks >> that lack in-band extradata; these files are nevertheless decodable >> because

Re: [FFmpeg-devel] [PATCH 01/17] avformat/mxfenc: Auto-insert h264_mp4toannexb BSF if needed

2021-11-10 Thread Andreas Rheinhardt
Tomas Härdin: > tis 2021-11-09 klockan 22:07 +0100 skrev Andreas Rheinhardt: >> Tomas Härdin: >>> tis 2021-11-09 klockan 18:34 +0100 skrev Andreas Rheinhardt: The mxf and mxf_opatom muxer expect H.264 in Annex B format. Signed-off-by: Andreas Rheinhardt --- The check

Re: [FFmpeg-devel] [PATCH 08/17] avformat/mxfenc: Improve returned error codes

2021-11-10 Thread Andreas Rheinhardt
Tomas Härdin: > tis 2021-11-09 klockan 19:01 +0100 skrev Andreas Rheinhardt: >> Signed-off-by: Andreas Rheinhardt >> --- >>  libavformat/mxfenc.c | 16 >>  1 file changed, 8 insertions(+), 8 deletions(-) > > Aha, this one does exactly what I just suggested with return values. > D'

[FFmpeg-devel] [PATCH 2/2] avformat/hlsenc: Only write something if there is something to write

2021-11-10 Thread Andreas Rheinhardt
This is especially important in case avio_write() would be switched to an unsigned type like size_t, then a potential error from avio_read() (with negative return value) would no longer be handled gracefully by avio_write(). Signed-off-by: Andreas Rheinhardt --- libavformat/hlsenc.c | 2 +- 1 fi

[FFmpeg-devel] [PATCH 1/2] avformat/hlsenc: Remove nonsense memset

2021-11-10 Thread Andreas Rheinhardt
The memset here is both unnecessary (avio_read() ignores the previous content of the destination buffer) as well as nonsense (for a char buf[BUFSIZE] sizeof(buf) and sizeof(BUFSIZE) are not the same; the latter is sizeof(int)). Fixes Coverity issue #1465863. Signed-off-by: Andreas Rheinhardt ---

[FFmpeg-devel] [PATCH] avutil/opt: handle whole range of int64_t in av_opt_get_int

2021-11-10 Thread Zhao Zhili
Make get_int/set_int symetric. The int64_t to double to int64_t conversion is unprecise for large value. --- libavutil/opt.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavutil/opt.c b/libavutil/opt.c index c7001dbcd3..b2be63828d 100644 --- a/libavutil/opt.c +++ b/lib

[FFmpeg-devel] Collection Ping (softworkz)

2021-11-10 Thread Soft Works
Hi, I thought it might be more friendly to ping for my pending patches in a single message. Here’s the list of my pending (mostly unresponded) patch sets: [FFmpeg-devel,v2] DXVA2: Add ARGB format https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=4491 [FFmpeg-devel,v3] do

Re: [FFmpeg-devel] [PATCH 17/17] fftools/ffmpeg_opt: Apply copyinkf for all stream types

2021-11-10 Thread Jan Ekström
On Tue, Nov 9, 2021 at 8:05 PM Andreas Rheinhardt wrote: > > The earlier code has ignored it for all stream types except > video and subtitles, probably because audio was presumed > to only consist of keyframes. Yet this assumption is not true > for e.g. TrueHD. > > Signed-off-by: Andreas Rheinhar

Re: [FFmpeg-devel] [PATCH 16/17] fftools/ffmpeg_opt: Fix copyinkf option name in warning message

2021-11-10 Thread Jan Ekström
On Tue, Nov 9, 2021 at 8:05 PM Andreas Rheinhardt wrote: > > Signed-off-by: Andreas Rheinhardt LGTM. Jan ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email

Re: [FFmpeg-devel] [PATCH 2/4] libavutil/hwcontext_qsv: fix a bug when malloc handle_pairs_internal

2021-11-10 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Chen, Wenbin > Sent: Wednesday, November 10, 2021 4:03 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH 2/4] libavutil/hwcontext_qsv: fix > a bug when malloc handle_pairs_internal > > > This commandline cause

Re: [FFmpeg-devel] [PATCH 1/4] libavutil/hwcontext_d3d11va: Add nb_surfaces to AVD3D11VAFramesContext

2021-11-10 Thread Soft Works
> -Original Message- > From: ffmpeg-devel On Behalf Of > Chen, Wenbin > Sent: Wednesday, November 10, 2021 4:03 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH 1/4] libavutil/hwcontext_d3d11va: > Add nb_surfaces to AVD3D11VAFramesContext > > > Adding nb_surfaces i