Re: [FFmpeg-devel] [PATCH v4 2/7] lavutil: add DOVI related header

2020-04-21 Thread Anton Khirnov
Quoting myp...@gmail.com (2020-04-21 04:47:42) > On Mon, Apr 20, 2020 at 9:08 PM Anton Khirnov wrote: > > > > Quoting Jean-Baptiste Kempf (2020-04-19 10:25:07) > > > I'd like to ask opinions whether a installed header for just one > > > structure is a good idea. > > > > I see no problem with it.

Re: [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: fix the infinite loop while framerate lower than input

2020-04-21 Thread Fu, Linjie
> From: Zhong Li > Sent: Tuesday, April 21, 2020 17:10 > To: Fu, Linjie > Cc: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: fix the infinite loop > while framerate lower than input > > > > I can't see the benefit to use

Re: [FFmpeg-devel] [PATCH] lavfi/vf_vpp_qsv: fix the infinite loop while framerate lower than input

2020-04-21 Thread Zhong Li
> > I can't see the benefit to use MSDK framerate conversion function. Is > > it a good idea to drop it and use ffmpeg native fps filter instead? > > The implementation of FRC inside MSDK is quite straight-forward or simple > currently since it just drops or repeats frames, hence I think using

Re: [FFmpeg-devel] [PATCH 6/8] avcodec/dump_extradata_bsf: Remove unnecessary header

2020-04-21 Thread Anton Khirnov
Quoting Andreas Rheinhardt (2020-04-21 04:31:51) > Since 33d18982fa03feb061c8f744a4f0a9175c1f63ab (the commit that > introduced the new bsf API) allocating an enlarged buffer in case > extradata needs to be added to a packet is done via av_new_packet(), > so that libavutil/mem.h is no longer

[FFmpeg-devel] [PATCH 1/3] lavu/pix_fmt: add new pixel format x2rgb10

2020-04-21 Thread Fei Wang
The format is packed RGB with each channel 10 bits available and include 2 bits unused. Signed-off-by: Fei Wang --- libavutil/pixdesc.c | 24 libavutil/pixfmt.h | 3 +++ libavutil/version.h | 2 +-

[FFmpeg-devel] [PATCH 2/3] swscale: Add swscale input/output support for X2RGB10LE

2020-04-21 Thread Fei Wang
Signed-off-by: Fei Wang --- libswscale/input.c | 13 - libswscale/output.c | 14 ++ libswscale/utils.c | 1 + libswscale/yuv2rgb.c | 22 ++ 4 files changed, 49 insertions(+), 1 deletion(-) diff --git a/libswscale/input.c b/libswscale/input.c

[FFmpeg-devel] [PATCH 3/3] lavu/hwcontext_vaapi: add vaapi_format_map support for x2rgb10

2020-04-21 Thread Fei Wang
Signed-off-by: Fei Wang --- libavutil/hwcontext_vaapi.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavutil/hwcontext_vaapi.c b/libavutil/hwcontext_vaapi.c index b306965..50727fb 100644 --- a/libavutil/hwcontext_vaapi.c +++ b/libavutil/hwcontext_vaapi.c @@ -136,6 +136,9 @@ static

Re: [FFmpeg-devel] [PATCH 1/3] lavu/pix_fmt: add new pixel format x2rgb10

2020-04-21 Thread Hendrik Leppkes
On Tue, Apr 21, 2020 at 9:53 AM Fei Wang wrote: > > The format is packed RGB with each channel 10 bits available and > include 2 bits unused. > The name of this pix_fmt seems very confusing, and the entire format seems very specific. What is such a packed RGB format really good for in ffmpeg? -

Re: [FFmpeg-devel] [PATCH v2 1/4] fftools/ffmpeg: set AV_PKT_FLAG_KEY for the subtitle packet

2020-04-21 Thread Nicolas George
lance.lmw...@gmail.com (12020-04-21): > From: Limin Wang > > For better code review, the patch is helping to update the fate tests result > so the change of ffmpeg.c will be deleted in the last patch. > > Signed-off-by: Limin Wang You did not need to post it. You only needed to make it, run

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Nicolas George
Limin Wang (12020-04-21): > I have checked with one of non-empty-moov.mp4, and have observed one byte > difference > only(02 and 01). I haven't tool to dump every box in detail. If you think > it's necessary, > I can looked at the movenc muxer further and figure out which field it's. You can

Re: [FFmpeg-devel] [PATCH] avfilter/vf_v360: check the limit of in_pad and out_pad

2020-04-21 Thread Nicolas George
Steven Liu (12020-04-21): > When i set out_pad=1 it will Segmentation fault > so i think it should check the limit value of the in_pad and out_pad > > Signed-off-by: Steven Liu > --- > libavfilter/vf_v360.c | 4 > 1 file changed, 4 insertions(+) Please don't guess. Find the exact cause of

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Jan Ekström
On Tue, Apr 21, 2020 at 4:19 PM Limin Wang wrote: > > On Tue, Apr 21, 2020 at 01:08:05PM +0200, Nicolas George wrote: > > Limin Wang (12020-04-21): > > > OK, below is the commit log, please help to review whether it's your > > > expected log: > > > > > > NOTE about the testing effect: > > >

Re: [FFmpeg-devel] [PATCH v2 1/4] fftools/ffmpeg: set AV_PKT_FLAG_KEY for the subtitle packet

2020-04-21 Thread Limin Wang
On Tue, Apr 21, 2020 at 11:26:14AM +0200, Nicolas George wrote: > lance.lmw...@gmail.com (12020-04-21): > > From: Limin Wang > > > > For better code review, the patch is helping to update the fate tests result > > so the change of ffmpeg.c will be deleted in the last patch. > > > >

[FFmpeg-devel] [PATCH] avfilter/vf_v360: check the limit of in_pad and out_pad

2020-04-21 Thread Steven Liu
When i set out_pad=1 it will Segmentation fault so i think it should check the limit value of the in_pad and out_pad Signed-off-by: Steven Liu --- libavfilter/vf_v360.c | 4 1 file changed, 4 insertions(+) diff --git a/libavfilter/vf_v360.c b/libavfilter/vf_v360.c index

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Limin Wang
On Tue, Apr 21, 2020 at 04:52:16PM +0300, Jan Ekström wrote: > On Tue, Apr 21, 2020 at 4:19 PM Limin Wang wrote: > > > > On Tue, Apr 21, 2020 at 01:08:05PM +0200, Nicolas George wrote: > > > Limin Wang (12020-04-21): > > > > OK, below is the commit log, please help to review whether it's your >

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Limin Wang
On Tue, Apr 21, 2020 at 01:08:05PM +0200, Nicolas George wrote: > Limin Wang (12020-04-21): > > OK, below is the commit log, please help to review whether it's your > > expected log: > > > > NOTE about the testing effect: > > subtitle packet will be marked as keyframe, it'll change below testing

Re: [FFmpeg-devel] [PATCH v10 0/2] Pro Pinball Series Soundbank demuxer + decoder.

2020-04-21 Thread Zane van Iperen
On Sat, 18 Apr 2020 00:20:30 + "Zane van Iperen" wrote: > > Zane van Iperen (2): > avcodec: add support for Cunning Developments' ADPCM > avformat: add demuxer for Pro Pinball Series' Soundbanks > > Changelog| 2 + > doc/general.texi | 1 + >

Re: [FFmpeg-devel] [PATCH v2 1/4] fftools/qsv: enabling d3d11va/dxva2 device selection

2020-04-21 Thread Steve Lhomme
Hi, On 2020-04-15 15:07, artem.ga...@gmail.com wrote: From: Artem Galin child_device_type argument is responsible for selection. Usage examples: -init_hw_device qsv:hw,child_device_type=d3d11va -init_hw_device qsv:hw,child_device_type=dxva2 Signed-off-by: Artem Galin ---

Re: [FFmpeg-devel] [PATCH v2 1/4] fftools/ffmpeg: set AV_PKT_FLAG_KEY for the subtitle packet

2020-04-21 Thread Nicolas George
Limin Wang (12020-04-21): > OK, Got it, I'll merge patch #1,#2,#4 into one patch and update the patch set. You forgot the part about writing the result of your examination of the tests in the commit message. A commit that changes tests reference files should explain what changed. Regards, --

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Nicolas George
Limin Wang (12020-04-21): > OK, below is the commit log, please help to review whether it's your expected > log: > > NOTE about the testing effect: > subtitle packet will be marked as keyframe, it'll change below testing > reference > files: > tests/ref/fate/binsub-movtextenc and

Re: [FFmpeg-devel] [PATCH v2 4/4] libavutil/qsv: enabling d3d11va support

2020-04-21 Thread Steve Lhomme
On 2020-04-15 15:07, artem.ga...@gmail.com wrote: From: Artem Galin Makes selection of d3d11va device type by default and over DirectX 9, which is still supported but requires explicit selection. This enables usage of non-powered/headless GPU, better HDR support. Pool of resources is allocated

Re: [FFmpeg-devel] [PATCH v2 3/4] libavcodec/qsv: enabling d3d11va support, added mfxhdlpair

2020-04-21 Thread Steve Lhomme
Mostly the same remarks as for 2/4 regarding the uneeded loop and uninitialized handle_type, plus comments below. On 2020-04-15 15:07, artem.ga...@gmail.com wrote: From: Artem Galin Adding DX11 relevant device type checks and adjusting callbacks with proper MediaSDK pair type support.

Re: [FFmpeg-devel] [PATCH v2 2/4] libavfilter/qsvvpp: enabling d3d11va support

2020-04-21 Thread Steve Lhomme
Comments below. On 2020-04-15 15:07, artem.ga...@gmail.com wrote: From: Artem Galin Adding DX11 relevant device type checks and adjusting callback with proper MediaSDK pair type support. Signed-off-by: Artem Galin --- libavfilter/qsvvpp.c | 40 ++-- 1

[FFmpeg-devel] [PATCH v3 1/2] avformat/utils: move is_intra_only() to header and rename to ff_is_intra_only()

2020-04-21 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/internal.h | 2 ++ libavformat/utils.c| 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/internal.h b/libavformat/internal.h index 329b2e9..716e42c 100644 --- a/libavformat/internal.h +++

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Nicolas George
lance.lmw...@gmail.com (12020-04-21): > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavformat/internal.h | 2 + > libavformat/mux.c| 7 +++- > tests/ref/fate/binsub-movtextenc | 2 +- > tests/ref/fate/movenc| 50 +++ >

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Limin Wang
On Tue, Apr 21, 2020 at 12:35:39PM +0200, Nicolas George wrote: > lance.lmw...@gmail.com (12020-04-21): > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > libavformat/internal.h | 2 + > > libavformat/mux.c| 7 +++- > >

[FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/internal.h | 2 + libavformat/mux.c| 7 +++- tests/ref/fate/binsub-movtextenc | 2 +- tests/ref/fate/movenc| 50 +++ tests/ref/fate/sub2video | 86

Re: [FFmpeg-devel] [PATCH] avfilter/vf_v360: check the limit of in_pad and out_pad

2020-04-21 Thread Steven Liu
> 2020年4月21日 下午10:28,Paul B Mahol 写道: > > On 4/21/20, Steven Liu wrote: >> When i set out_pad=1 it will Segmentation fault >> so i think it should check the limit value of the in_pad and out_pad > > So huge pads do not make sense. Yes, just don’t know how to use it, and try to set it to 1,

Re: [FFmpeg-devel] [PATCH 6/8] avcodec/dump_extradata_bsf: Remove unnecessary header

2020-04-21 Thread Andreas Rheinhardt
Anton Khirnov: > Quoting Andreas Rheinhardt (2020-04-21 04:31:51) >> Since 33d18982fa03feb061c8f744a4f0a9175c1f63ab (the commit that >> introduced the new bsf API) allocating an enlarged buffer in case >> extradata needs to be added to a packet is done via av_new_packet(), >> so that

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Limin Wang
On Tue, Apr 21, 2020 at 04:21:38PM +0200, Nicolas George wrote: > Limin Wang (12020-04-21): > > Thanks, I'll try to build the tool to use. By the ffmpeg trace log, the > > difference is frag flags of tfhd: > > The trace log is still seeing things from the inside. If there is a bug > after the

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Nicolas George
Limin Wang (12020-04-21): > Thanks, I'll try to build the tool to use. By the ffmpeg trace log, the > difference is frag flags of tfhd: The trace log is still seeing things from the inside. If there is a bug after the trace point, it will not be detected. It is important to test the RESULTING

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Limin Wang
On Tue, Apr 21, 2020 at 05:34:09PM +0300, Jan Ekström wrote: > On Tue, Apr 21, 2020 at 5:19 PM Limin Wang wrote: > > > > On Tue, Apr 21, 2020 at 04:52:16PM +0300, Jan Ekström wrote: > > > On Tue, Apr 21, 2020 at 4:19 PM Limin Wang wrote: > > > > > > > > On Tue, Apr 21, 2020 at 01:08:05PM +0200,

Re: [FFmpeg-devel] [PATCH] avfilter/vf_v360: check the limit of in_pad and out_pad

2020-04-21 Thread Paul B Mahol
On 4/21/20, Steven Liu wrote: > When i set out_pad=1 it will Segmentation fault > so i think it should check the limit value of the in_pad and out_pad So huge pads do not make sense. Limit it to 1/10 in via options. ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Jan Ekström
On Tue, Apr 21, 2020 at 5:19 PM Limin Wang wrote: > > On Tue, Apr 21, 2020 at 04:52:16PM +0300, Jan Ekström wrote: > > On Tue, Apr 21, 2020 at 4:19 PM Limin Wang wrote: > > > > > > On Tue, Apr 21, 2020 at 01:08:05PM +0200, Nicolas George wrote: > > > > Limin Wang (12020-04-21): > > > > > OK,

Re: [FFmpeg-devel] [PATCH] avfilter/vf_v360: check the limit of in_pad and out_pad

2020-04-21 Thread Paul B Mahol
On 4/21/20, Steven Liu wrote: > > >> 2020年4月21日 下午10:28,Paul B Mahol 写道: >> >> On 4/21/20, Steven Liu wrote: >>> When i set out_pad=1 it will Segmentation fault >>> so i think it should check the limit value of the in_pad and out_pad >> >> So huge pads do not make sense. > Yes, just don’t know

[FFmpeg-devel] [PATCH v2] avfilter/vf_v360: adjustment out_pad and in_pad maximum value to 1/10

2020-04-21 Thread Steven Liu
Because not every user know about in_pad and out_pad reasonable value range so maybe try to set 1.0, but setting 1.0 is so hugh to get an fatal error. Suggested-by: Paul B Mahol Signed-off-by: Steven Liu --- doc/filters.texi | 1 + libavfilter/vf_v360.c | 4 ++-- 2 files changed, 3

[FFmpeg-devel] [PATCH v4 1/2] avformat/utils: move is_intra_only() to header and rename to ff_is_intra_only()

2020-04-21 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- libavformat/internal.h | 2 ++ libavformat/utils.c| 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/internal.h b/libavformat/internal.h index 329b2e9..716e42c 100644 --- a/libavformat/internal.h +++

[FFmpeg-devel] [PATCH v4 2/2] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread lance . lmwang
From: Limin Wang The patch will make audio and subtitle packets be marked as AV_PKT_FLAG_KEY. For audio, it'll caused the audio sample to be sync sample. To verify ref/fate/movenc results: 1. Get the movenc test data [lmwang@vpn ffmpeg]$ libavformat/tests/movenc -w && mkdir -p audio_old && mv

Re: [FFmpeg-devel] [PATCH] lavf/tls_gnutls: check for interrupt inside handshake loop

2020-04-21 Thread Jan Ekström
On Mon, Apr 20, 2020 at 8:07 PM Jan Ekström wrote: > > On Thu, Sep 5, 2019 at 6:13 PM Michael Niedermayer > wrote: > > > > On Fri, Aug 16, 2019 at 10:38:46AM +0200, Błażej Szczygieł wrote: > > > fixes #8080 > > > > > > Signed-off-by: Błażej Szczygieł > > > --- > > > libavformat/tls_gnutls.c |

Re: [FFmpeg-devel] [PATCH v1] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Jan Ekström
On Tue, Apr 21, 2020 at 9:05 PM Nicolas George wrote: > > Derek Buitenhuis (12020-04-17): > > Can you elaborate on why it's an abomination? It's so far been the only > > thing > > that has kept people from breaking movenc API use several times, > > specifically > > in relation to how it behaves

Re: [FFmpeg-devel] [PATCH v2 4/4] avformat/mpegtsenc: use the correct stream_types and write HDMV descriptors for m2ts

2020-04-21 Thread Carl Eugen Hoyos
Am Di., 21. Apr. 2020 um 21:44 Uhr schrieb Marton Balint : > > > > On Mon, 20 Apr 2020, Marton Balint wrote: > > > > > > > On Thu, 16 Apr 2020, Carl Eugen Hoyos wrote: > > > >> Am Mo., 13. Apr. 2020 um 02:17 Uhr schrieb Marton Balint : > >>> > >>> > >>> > >>> On Mon, 13 Apr 2020, Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH v4 2/2] libavcodec/libaomenc.c: Support lossless encoding

2020-04-21 Thread James Zern
On Fri, Apr 17, 2020 at 9:57 AM Ryo Hirafuji wrote: > > I see, Thanks. > I'm looking forward to the answer. > There is some benefit to using cq-level as many of the encoder decisions are based on this (reference frame structure, speed features, etc) and so it's better tuned than quality mode. In

Re: [FFmpeg-devel] [PATCH v2 4/4] avformat/mpegtsenc: use the correct stream_types and write HDMV descriptors for m2ts

2020-04-21 Thread Marton Balint
On Mon, 20 Apr 2020, Marton Balint wrote: On Thu, 16 Apr 2020, Carl Eugen Hoyos wrote: Am Mo., 13. Apr. 2020 um 02:17 Uhr schrieb Marton Balint : On Mon, 13 Apr 2020, Carl Eugen Hoyos wrote: > Am Mo., 13. Apr. 2020 um 00:38 Uhr schrieb Marton Balint : >> >> Fixes ticket #2622. >>

[FFmpeg-devel] [PATCH] avformat/gopher: Support 'g' type

2020-04-21 Thread parazyd
This adds support for the gif gopher type. Since ffmpeg can work with GIF files, it makes sense to support it in the Gopher protocol. Signed-off-by: parazyd --- libavformat/gopher.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/gopher.c b/libavformat/gopher.c index

Re: [FFmpeg-devel] [PATCH] avformat/gopher: Support 'g' type

2020-04-21 Thread Carl Eugen Hoyos
Am Di., 21. Apr. 2020 um 22:13 Uhr schrieb Marton Balint : > > > > On Tue, 21 Apr 2020, parazyd wrote: > > > This adds support for the gif gopher type. Since ffmpeg can work with GIF > > files, it makes sense to support it in the Gopher protocol. > > > > Signed-off-by: parazyd > > --- > >

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/jpeg2000dec.c: ROI marker support

2020-04-21 Thread Michael Niedermayer
On Tue, Apr 21, 2020 at 01:07:39AM +0530, gautamr...@gmail.com wrote: > From: Gautam Ramakrishnan > > This patch adds support for decoding images > with a Region of Interest. Allows decoding > samples such as p0_03.j2k. This patch should > fix ticket #4681. > --- > libavcodec/jpeg2000.h| 1

Re: [FFmpeg-devel] [PATCH] avformat/mpegtsenc: don't print warning for DVB text streams

2020-04-21 Thread Marton Balint
On Mon, 20 Apr 2020, Gyan Doshi wrote: They can be demuxed by ffmpeg. --- libavformat/mpegtsenc.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index ccb631d746..3818001976 100644 --- a/libavformat/mpegtsenc.c +++

Re: [FFmpeg-devel] [PATCH] avformat/gopher: Support 'g' type

2020-04-21 Thread Marton Balint
On Tue, 21 Apr 2020, parazyd wrote: This adds support for the gif gopher type. Since ffmpeg can work with GIF files, it makes sense to support it in the Gopher protocol. Signed-off-by: parazyd --- libavformat/gopher.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/gopher.c

[FFmpeg-devel] [PATCH v5 4/7] lavf/mov: support dvcC/dvvC box for DOVI

2020-04-21 Thread Jun Zhao
From: vacingfang support dvcC/dvcC box from spec Dolby Vision Streams Within the ISO Base MediaFile Format Version 2.1.2 (https://www.dolby.com/in/en/technologies/dolby-vision/dolby-vision\ -bitstreams-within-the-iso-base-media-file-format-v2.1.2.pdf) export the DOVI information to sidedata.

[FFmpeg-devel] [PATCH v5 5/7] lavf/dump: dump DOVI side data

2020-04-21 Thread Jun Zhao
From: Jun Zhao dump DOVI side data. Signed-off-by: Jun Zhao --- libavformat/dump.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/libavformat/dump.c b/libavformat/dump.c index 220f404..9578b88 100644 --- a/libavformat/dump.c +++ b/libavformat/dump.c @@ -27,6

Re: [FFmpeg-devel] [PATCH 1/3] lavu/pix_fmt: add new pixel format x2rgb10

2020-04-21 Thread Wang, Fei W
> -Original Message- > From: ffmpeg-devel On Behalf Of > Vittorio Giovara > Sent: Wednesday, April 22, 2020 11:02 AM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 1/3] lavu/pix_fmt: add new pixel format > x2rgb10 > > On Tue, Apr

[FFmpeg-devel] [PATCH v5 7/7] lavf/movenc: enable dvcC/dvvC box support from DOVI sidedata

2020-04-21 Thread Jun Zhao
From: Jun Zhao enable dvcC/dvvC box support from DOVI sidedata. Signed-off-by: Jun Zhao --- libavformat/movenc.c | 35 +++ 1 file changed, 35 insertions(+) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 253cff8..556dd06 100644 ---

[FFmpeg-devel] [PATCH v5 6/7] fftools/ffprobe: support DOVI sidedata

2020-04-21 Thread Jun Zhao
From: Jun Zhao support DOVI sidedata. Signed-off-by: Jun Zhao --- fftools/ffprobe.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index e3f221f..840fcb7 100644 --- a/fftools/ffprobe.c +++ b/fftools/ffprobe.c @@ -36,6 +36,7 @@ #include

Re: [FFmpeg-devel] [PATCH v1] avcodec/utils: use av_rescale()

2020-04-21 Thread Limin Wang
On Tue, Apr 21, 2020 at 11:05:59PM +0200, Michael Niedermayer wrote: > On Mon, Apr 20, 2020 at 07:00:55PM +0800, lance.lmw...@gmail.com wrote: > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > libavcodec/utils.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > >

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/jpeg2000dec.c: ROI marker support

2020-04-21 Thread Gautam Ramakrishnan
On Wed, Apr 22, 2020 at 3:04 AM Michael Niedermayer wrote: > > On Tue, Apr 21, 2020 at 01:07:39AM +0530, gautamr...@gmail.com wrote: > > From: Gautam Ramakrishnan > > > > This patch adds support for decoding images > > with a Region of Interest. Allows decoding > > samples such as p0_03.j2k.

[FFmpeg-devel] [PATCH] mpeg4videoenc: Don't crash with -fsanitize=bounds

2020-04-21 Thread Vitaly Buka
Also the patch makes this code consistent with mpeg4videodec.c --- libavcodec/mpeg4videoenc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/mpeg4videoenc.c b/libavcodec/mpeg4videoenc.c index f6a5992df7..2cd5a8c015 100644 --- a/libavcodec/mpeg4videoenc.c +++

[FFmpeg-devel] [PATCH v5 2/7] lavutil: add DOVI related header

2020-04-21 Thread Jun Zhao
From: vacingfang add DOVI related struct Signed-off-by: vacingfang --- libavutil/Makefile| 1 + libavutil/dovi_meta.h | 80 +++ 2 files changed, 81 insertions(+) create mode 100644 libavutil/dovi_meta.h diff --git a/libavutil/Makefile

Re: [FFmpeg-devel] [PATCH] lavf/mux: bypass interleave delay early when waiting on subtitle streams

2020-04-21 Thread Jan Ekström
On Wed, Mar 11, 2020 at 11:36 PM rcombs wrote: > > This avoids long delays when converting live streams that have sparse > subtitles > --- > libavformat/avformat.h | 9 + > libavformat/mux.c | 25 + > libavformat/options_table.h | 1 + >

Re: [FFmpeg-devel] [PATCH v1] avcodec/utils: use av_rescale()

2020-04-21 Thread Michael Niedermayer
On Mon, Apr 20, 2020 at 07:00:55PM +0800, lance.lmw...@gmail.com wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > libavcodec/utils.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index

[FFmpeg-devel] [PATCH v5 0/7] Support Dolby Vision

2020-04-21 Thread Jun Zhao
From: Jun Zhao V5: - add a constructor for DOVI (tks Anton Khirnov) - some minor coding style clean V4: - change the DOVI dump message (tks Jan Ekström) - fix the file order in Makefile V3: - rename the file name for Dolby Vision trademark risk(tks Jean-Baptiste Kempf) - fix other typo issue

[FFmpeg-devel] [PATCH v5 1/7] lavc: add a new sidedata type for DOVI

2020-04-21 Thread Jun Zhao
From: Jun Zhao add a new sidedata type for DOVI. Signed-off-by: Jun Zhao --- libavcodec/avpacket.c | 1 + libavcodec/packet.h | 9 + 2 files changed, 10 insertions(+) diff --git a/libavcodec/avpacket.c b/libavcodec/avpacket.c index ad020ee..1b53451 100644 ---

Re: [FFmpeg-devel] [PATCH 1/3] lavu/pix_fmt: add new pixel format x2rgb10

2020-04-21 Thread Wang, Fei W
> -Original Message- > From: ffmpeg-devel On Behalf Of > Hendrik Leppkes > Sent: Tuesday, April 21, 2020 4:08 PM > To: FFmpeg development discussions and patches de...@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 1/3] lavu/pix_fmt: add new pixel format > x2rgb10 > > On Tue, Apr

Re: [FFmpeg-devel] [PATCH v2 1/3] lavu/pix_fmt: add new pixel format x2rgb10

2020-04-21 Thread Wang, Fei W
> -Original Message- > From: Wang, Fei W > Sent: Wednesday, April 22, 2020 1:23 PM > To: ffmpeg-devel@ffmpeg.org > Cc: Wang, Fei W > Subject: [PATCH v2 1/3] lavu/pix_fmt: add new pixel format x2rgb10 > > The format is packed RGB with each channel 10 bits available and include 2 > bits

Re: [FFmpeg-devel] [PATCH v4 2/7] lavutil: add DOVI related header

2020-04-21 Thread myp...@gmail.com
On Tue, Apr 21, 2020 at 2:23 PM Anton Khirnov wrote: > > Quoting myp...@gmail.com (2020-04-21 04:47:42) > > On Mon, Apr 20, 2020 at 9:08 PM Anton Khirnov wrote: > > > > > > Quoting Jean-Baptiste Kempf (2020-04-19 10:25:07) > > > > I'd like to ask opinions whether a installed header for just one

Re: [FFmpeg-devel] [PATCH 1/3] lavu/pix_fmt: add new pixel format x2rgb10

2020-04-21 Thread Vittorio Giovara
On Tue, Apr 21, 2020 at 10:32 PM Wang, Fei W wrote: > > > > -Original Message- > > From: ffmpeg-devel On Behalf Of > > Hendrik Leppkes > > Sent: Tuesday, April 21, 2020 4:08 PM > > To: FFmpeg development discussions and patches > de...@ffmpeg.org> > > Subject: Re: [FFmpeg-devel] [PATCH

Re: [FFmpeg-devel] [PATCH v2] lavf/tls_mbedtls: fix resource leak

2020-04-21 Thread myp...@gmail.com
On Mon, Apr 20, 2020 at 4:24 PM Jun Zhao wrote: > > From: Jun Zhao > > fix resource leak in mbedtls part. > > fix #8614 > > Signed-off-by: Jun Zhao > --- > libavformat/tls_mbedtls.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/tls_mbedtls.c b/libavformat/tls_mbedtls.c >

[FFmpeg-devel] [PATCH v5 3/7] lavf/mpegts: support DOVI Video Stream Descriptor

2020-04-21 Thread Jun Zhao
From: vacingfang support DOVI Video Stream Descriptor from Dolby Vision Streams Within the MPEG-2 Transport Stream Format V1.2 From the spec: https://www.dolby.com/us/en/technologies/\ dolby-vision/dolby-vision-bitstreams-in-mpeg-2-transport-\ stream-multiplex-v1.2.pdf. export the DOVI

[FFmpeg-devel] [PATCH v2 2/3] swscale: Add swscale input/output support for X2RGB10LE

2020-04-21 Thread Fei Wang
Signed-off-by: Fei Wang --- libswscale/input.c | 13 - libswscale/output.c | 14 ++ libswscale/utils.c | 1 + libswscale/yuv2rgb.c | 22 ++

[FFmpeg-devel] [PATCH v2 1/3] lavu/pix_fmt: add new pixel format x2rgb10

2020-04-21 Thread Fei Wang
The format is packed RGB with each channel 10 bits available and include 2 bits unused. Signed-off-by: Fei Wang --- libavutil/pixdesc.c | 24 libavutil/pixfmt.h | 3 +++ libavutil/version.h | 2 +-

[FFmpeg-devel] [PATCH v2 3/3] lavu/hwcontext_vaapi: add vaapi_format_map support for x2rgb10

2020-04-21 Thread Fei Wang
Signed-off-by: Fei Wang --- libavutil/hwcontext_vaapi.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavutil/hwcontext_vaapi.c b/libavutil/hwcontext_vaapi.c index b306965..50727fb 100644 --- a/libavutil/hwcontext_vaapi.c +++ b/libavutil/hwcontext_vaapi.c @@ -136,6 +136,9 @@ static

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/cbs_h265: add missing support for reserved_payload_extension_data SEI bits

2020-04-21 Thread James Almer
On 4/20/2020 6:45 PM, James Almer wrote: > Fixes ticket #8622 > > Signed-off-by: James Almer > --- > It fixes it assuming the Picture Timing in that sample is not being misparsed > by cbs_h265. > We're compliant with the latest revision of the spec, so any > reserved_payload_extension_data found

Re: [FFmpeg-devel] [PATCH v1] avformat/mux: Set AV_PKT_FLAG_KEY for is_intra_only packet

2020-04-21 Thread Nicolas George
Derek Buitenhuis (12020-04-17): > Can you elaborate on why it's an abomination? It's so far been the only thing > that has kept people from breaking movenc API use several times, specifically > in relation to how it behaves for outputting fragments - and ffmpeg.c does not > necessarily exercise