[FFmpeg-devel] [PATCH v1] lavc/av1dec: drop redundant frame

2021-10-28 Thread Fei Wang
For some non-standard clip which contains multiple display frame data in a ivf packet, drop the redundant frame can make the decode continuously with some warning message shown. Otherwise decode will interrupt with unexpected error. Signed-off-by: Fei Wang --- libavcodec/av1dec.c | 10

Re: [FFmpeg-devel] [PATCH] fftools/ffplay: don't disable x11 compositing

2021-10-28 Thread Marton Balint
On Fri, 29 Oct 2021, Marton Balint wrote: On Wed, 27 Oct 2021, Zane van Iperen wrote: Prevents desktop stutters caused by the change (specifically on KDE). We're not a game, we don't actually need it disabled. Signed-off-by: Zane van Iperen --- fftools/ffplay.c | 4 1 file

Re: [FFmpeg-devel] [PATCH] fftools/ffplay: don't disable x11 compositing

2021-10-28 Thread Marton Balint
On Wed, 27 Oct 2021, Zane van Iperen wrote: Prevents desktop stutters caused by the change (specifically on KDE). We're not a game, we don't actually need it disabled. Signed-off-by: Zane van Iperen --- fftools/ffplay.c | 4 1 file changed, 4 insertions(+) diff --git

[FFmpeg-devel] [PATCH v7 4/4] avformat/movenc: Refactor mov_write_dvcc_dvvc_tag to use ff_isom_put_dvcc_dvvc

2021-10-28 Thread quietvoid
Improves code legibility by not using bit shifts. Also avoids duplicating the dvcC/dvvC ISOM box writing code. Signed-off-by: quietvoid --- libavformat/movenc.c | 26 +- 1 file changed, 9 insertions(+), 17 deletions(-) diff --git a/libavformat/movenc.c

[FFmpeg-devel] [PATCH v7 3/4] avformat/mov: Refactor mov_read_dvcc_dvvc to use ff_isom_parse_dvcc_dvvc

2021-10-28 Thread quietvoid
To avoid duplicating code. The implementation in dovi_isom is identical. Signed-off-by: quietvoid --- libavformat/mov.c | 50 +-- 1 file changed, 9 insertions(+), 41 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index

[FFmpeg-devel] [PATCH v7 2/4] avformat/matroska{dec, enc}: Parse BlockAdditionMapping elements

2021-10-28 Thread quietvoid
Adds handling of dvcC/dvvC block addition mappings. The parsing creates AVDOVIDecoderConfigurationRecord side data. The configuration block is written when muxing into Matroska, if DOVI side data is present for the track. Most of the Matroska element parsing is based on Plex's FFmpeg source

[FFmpeg-devel] [PATCH v7 1/4] avformat/dovi_isom: Implement Dolby Vision configuration parsing/writing

2021-10-28 Thread quietvoid
Both parse/write implementations are based on mov/movenc. This only adds support for the "Dolby Vision configuration box". Other configuration boxes, such as "Dolby Vision enhancement layer configuration box" are not supported. The new functions will be used to implement parsing/writing the DOVI

[FFmpeg-devel] [PATCH v7 0/4] Add support for Matroska BlockAdditionMapping elements

2021-10-28 Thread quietvoid
This patch set adds support for reading/writing the Matroska BlockAdditionMapping elements, as well as for reading/writing dvcC/dvvC blocks in Matroska. Created utility functions to read/write Dolby Vision boxes for ISOM. This was done to avoid duplicating the code, as the Matroska blocks and

Re: [FFmpeg-devel] [PATCH 1/2] avfilter/vf_zscale: deduplicate output color information setting

2021-10-28 Thread Jan Ekström
On Thu, Oct 28, 2021 at 1:24 AM Paul B Mahol wrote: > > lgtm Thanks for the reviews, applied these patches as cd1d09e81b53d47380b494acd3432fd4abb3c17b and 27c0dd55609daf440a7744e96ac20c119bbeb80f . Jan ___ ffmpeg-devel mailing list

[FFmpeg-devel] [PATCH] avfilter: add wpsnr video filter

2021-10-28 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_wpsnr.c | 894 +++ 3 files changed, 896 insertions(+) create mode 100644 libavfilter/vf_wpsnr.c diff --git a/libavfilter/Makefile

Re: [FFmpeg-devel] [PATCH] mfxenc add jpeg2000 frame field interlacing support

2021-10-28 Thread Erwann RENAN
Ok, i will split it -Message d'origine- De : ffmpeg-devel De la part de Tomas Härdin Envoyé : jeudi 28 octobre 2021 16:33 À : FFmpeg development discussions and patches Objet : Re: [FFmpeg-devel] [PATCH] mfxenc add jpeg2000 frame field interlacing support Looks like you messed up the

Re: [FFmpeg-devel] [PATCH] mfxenc add jpeg2000 frame field interlacing support

2021-10-28 Thread Tomas Härdin
Looks like you messed up the subject. You need two newlines between the title of the patch and the description. This patch also mixes a lot of different changes. Please split it up. The bigger a patch is the more rounds of review it tends to have to go through. > +    { 0x840B, >

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

2021-10-28 Thread Tomas Härdin
ons 2021-10-27 klockan 15:50 +0200 skrev Marc-Antoine Arnaud: > +if(channel_ordering_ptr->service_type != > AV_AUDIO_SERVICE_TYPE_NB) { > +ast = (enum > AVAudioServiceType*)av_stream_new_side_data(st, > AV_PKT_DATA_AUDIO_SERVICE_TYPE,

[FFmpeg-devel] [PATCH] mfxenc add jpeg2000 frame field interlacing support. SPONSORED BY INA (Institut National de l'Audiovisuel) * the use of a jpeg2000 frame/field input is signaled by the option fl

2021-10-28 Thread Erwann Renan
--- libavformat/mxf.h| 1 + libavformat/mxfenc.c | 371 +-- 2 files changed, 361 insertions(+), 11 deletions(-) diff --git a/libavformat/mxf.h b/libavformat/mxf.h index fe9c52732c..2ce637d4a8 100644 --- a/libavformat/mxf.h +++ b/libavformat/mxf.h @@

Re: [FFmpeg-devel] [PATCH] flvenc: flush after write header

2021-10-28 Thread Steven Liu
> 2021年10月28日 下午2:32,Lingjiang Fang 写道: > > keep align with movenc, or it seems a little weired when debug > --- > libavformat/flvenc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c > index 3f24c7e192..f4798dc9a6 100644 > ---

Re: [FFmpeg-devel] [PATCH 2/2] avfilter/vf_zscale: fix mapping of zimg_chroma_location_e to AVChromaLocation

2021-10-28 Thread Jan Ekström
On Thu, Oct 28, 2021 at 1:25 AM Paul B Mahol wrote: > > probably fine This can be verified by specifying - say - chromal topleft, and seeing that the ffmpeg.c Output: bit right now would end up being "left" (most likely). While if you apply this, you should get "topleft". Technically if the

Re: [FFmpeg-devel] [PATCH 3/5] avformat/imf: Demuxer implementation

2021-10-28 Thread Paul B Mahol
On Thu, Oct 28, 2021 at 6:32 AM Pierre-Anthony Lemieux wrote: > On Wed, Oct 27, 2021 at 12:54 AM Paul B Mahol wrote: > > > > > > > > On Fri, Oct 8, 2021 at 1:42 AM wrote: > >> > >> From: Pierre-Anthony Lemieux > >> > >> Signed-off-by: Pierre-Anthony Lemieux > >> --- > >> > >> Notes: > >>

Re: [FFmpeg-devel] [PATCH 2/5] avformat/imf: CPL processor

2021-10-28 Thread Paul B Mahol
On Thu, Oct 28, 2021 at 6:34 AM Pierre-Anthony Lemieux wrote: > On Wed, Oct 27, 2021 at 12:57 AM Paul B Mahol wrote: > > > > > > > > On Wed, Oct 20, 2021 at 4:55 PM wrote: > >> > >> From: Pierre-Anthony Lemieux > >> > >> Signed-off-by: Pierre-Anthony Lemieux > >> --- > >> > >> Notes: > >>

[FFmpeg-devel] [PATCH] flvenc: flush after write header

2021-10-28 Thread Lingjiang Fang
keep align with movenc, or it seems a little weired when debug --- libavformat/flvenc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c index 3f24c7e192..f4798dc9a6 100644 --- a/libavformat/flvenc.c +++ b/libavformat/flvenc.c @@ -767,6 +767,8 @@