[FFmpeg-devel] [PATCH] mpegdec: fix redundant dummy frames issue of interlaced clips

2017-09-29 Thread Zhong Li
It is to fix https://trac.ffmpeg.org/ticket/6677. Actucally it is a regression of commit 99e07a4453732058df90885f80b3db3b4f37cb3c which always inserts a dummy frame when decode the first key field picture. Signed-off-by: Zhong Li --- libavcodec/mpegvideo.c | 6 +- 1 file changed, 1 insertion

Re: [FFmpeg-devel] [PATCH] maintainers: add myself to the general developers list

2017-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2017 at 03:03:23AM +0300, Jan Ekström wrote: > --- > > It was recommended to me that I do this, as I do not maintain > any specific functionality/module as such. > > Jan > > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) will apply thx [...] -- Michael GnuPG finger

Re: [FFmpeg-devel] [PATCH v4] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2017 at 08:33:03PM -0300, James Almer wrote: > On 9/29/2017 8:01 PM, Michael Niedermayer wrote: > > On Fri, Sep 29, 2017 at 12:30:48AM -0300, James Almer wrote: > >> On 9/28/2017 7:09 PM, Michael Niedermayer wrote: > >>> On Wed, Sep 27, 2017 at 08:48:42PM -0500, Brian Matherly wrote

Re: [FFmpeg-devel] [PATCH]lavf/bit: Fix the G.729 bit auto-detection

2017-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2017 at 07:23:17PM +0200, Carl Eugen Hoyos wrote: > Hi! > > The G.729 bit auto-detection has never worked, patch attached. > > Please comment, Carl Eugen > bit.c | 23 +-- > 1 file changed, 13 insertions(+), 10 deletions(-) > 1eaabbb2e56d350949d0c8c439a2ba4

Re: [FFmpeg-devel] [PATCH] movenc: take packet dts shifting into mention in check_pkt

2017-09-29 Thread Michael Niedermayer
On Sat, Sep 30, 2017 at 01:04:00AM +0300, Jan Ekström wrote: > This FFmpeg-specific "fuzzer fix" was never perfect, but now it > stopped encoding of actual content with a big enough DTS shift. > This returns the function to its original state of results > before negative CTS offsets were added. >

Re: [FFmpeg-devel] [PATCH 2/2] avformat/wavenc: skip writing peak-of-peaks position when writing peaks only

2017-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2017 at 05:08:16PM +0200, Tobias Rapp wrote: > Signed-off-by: Tobias Rapp > --- > libavformat/wavenc.c | 5 - > tests/ref/lavf/wav_peak_only | 2 +- > 2 files changed, 5 insertions(+), 2 deletions(-) The commit message doest say why this chnage is done [...] -- Mich

Re: [FFmpeg-devel] [PATCH 1/2] avformat/wavenc: replace literal numbers with enum constants

2017-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2017 at 05:08:15PM +0200, Tobias Rapp wrote: > Signed-off-by: Tobias Rapp > --- > libavformat/wavenc.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Let us carefully ob

Re: [FFmpeg-devel] [PATCH 2/2] mov: fix non-monotonous DTS when fragments overlap in time

2017-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2017 at 08:54:09AM -0700, John Stebbins wrote: > --- > libavformat/mov.c | 16 ++-- > 1 file changed, 14 insertions(+), 2 deletions(-) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index c7422cd9ed..871f10b240 100644 > --- a/libavformat/mov.c > +++ b/libavfo

Re: [FFmpeg-devel] [PATCH v4] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-29 Thread James Almer
On 9/29/2017 8:01 PM, Michael Niedermayer wrote: > On Fri, Sep 29, 2017 at 12:30:48AM -0300, James Almer wrote: >> On 9/28/2017 7:09 PM, Michael Niedermayer wrote: >>> On Wed, Sep 27, 2017 at 08:48:42PM -0500, Brian Matherly wrote: From: Brian Matherly Correctly set the interlaced_f

Re: [FFmpeg-devel] [PATCH v4] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2017 at 12:30:48AM -0300, James Almer wrote: > On 9/28/2017 7:09 PM, Michael Niedermayer wrote: > > On Wed, Sep 27, 2017 at 08:48:42PM -0500, Brian Matherly wrote: > >> From: Brian Matherly > >> > >> Correctly set the interlaced_frame and top_field_first fields when > >> pic_struc

Re: [FFmpeg-devel] [PATCH 1/3] lavf/mov: Allow reading very large files.

2017-09-29 Thread Michael Niedermayer
On Sat, Sep 30, 2017 at 12:58:27AM +0200, Michael Niedermayer wrote: > From: Pablo Montilla > > The Sample count in the time-to-sample table is defined as > 32-bit unsigned integer by the QT specification. > > Fixes ticket #6700. > --- > libavformat/isom.h | 2 +- > libavformat/mov.c | 6 +

[FFmpeg-devel] [PATCH 2/3] avcodec/x86/lossless_videoencdsp: Fix handling of small widths

2017-09-29 Thread Michael Niedermayer
Fixes out of array access Fixes: crash-huf.avi Regression since: 6b41b4414934cc930468ccd5db598dd6ef643987 This could also be fixed by adding checks in the C code that calls the dsp Found-by: Zhibin Hu and 连一汉 Signed-off-by: Michael Niedermayer --- libavcodec/x86/lossless_videoencdsp.asm | 13

[FFmpeg-devel] [PATCH 3/3] avcodec/x86/lossless_videoencdsp: Fix warning: signed dword value exceeds bounds

2017-09-29 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/x86/lossless_videoencdsp.asm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/x86/lossless_videoencdsp.asm b/libavcodec/x86/lossless_videoencdsp.asm index a9c7a0a73c..89f20dc4e2 100644 --- a/libavcodec/x86/lossless_v

[FFmpeg-devel] [PATCH 1/3] lavf/mov: Allow reading very large files.

2017-09-29 Thread Michael Niedermayer
From: Pablo Montilla The Sample count in the time-to-sample table is defined as 32-bit unsigned integer by the QT specification. Fixes ticket #6700. --- libavformat/isom.h | 2 +- libavformat/mov.c | 6 +- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/libavformat/isom.h b/

[FFmpeg-devel] [PATCH] movenc: take packet dts shifting into mention in check_pkt

2017-09-29 Thread Jan Ekström
This FFmpeg-specific "fuzzer fix" was never perfect, but now it stopped encoding of actual content with a big enough DTS shift. This returns the function to its original state of results before negative CTS offsets were added. I remember dealing with this function before, but somehow had forgotten

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread James Almer
On 9/29/2017 6:56 PM, Hendrik Leppkes wrote: > On Fri, Sep 29, 2017 at 8:39 PM, Clément Bœsch wrote: >> On Fri, Sep 29, 2017 at 08:12:14PM +0200, Hendrik Leppkes wrote: >> [...] Just to be clear (sorry if I make you repeat yourself), we currently have the following: - Systems w

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Hendrik Leppkes
On Fri, Sep 29, 2017 at 8:39 PM, Clément Bœsch wrote: > On Fri, Sep 29, 2017 at 08:12:14PM +0200, Hendrik Leppkes wrote: > [...] >> > Just to be clear (sorry if I make you repeat yourself), we currently have >> > the following: >> > >> > - Systems without VDA (that is, almost all of them) do not h

Re: [FFmpeg-devel] How to tell CFR/VFR video apart, and determine frame rate

2017-09-29 Thread Lou Logan
On Fri, Sep 29, 2017, at 04:32 AM, Richard Ling wrote: > How can I determine whether a filter input is VFR or CFR? > > static int is_vfr(AVFilterLink *inlink) > { > return ???; > } [...] ffmpeg-devel is only for discussions regarding the development of FFmpeg and for submitting patches. Ques

Re: [FFmpeg-devel] [PATCH]lavd/decklink_dec: Do not claim to output transparency information

2017-09-29 Thread Marton Balint
On Fri, 29 Sep 2017, Carl Eugen Hoyos wrote: 2017-09-29 17:35 GMT+02:00 Maksym Veremeyenko : 29.09.2017 0:20, Carl Eugen Hoyos пише: I don't have decklink hardware but I assume it never outputs actual transparency. Or does it? dual link SDI could provide fill/key, some boards could suppor

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Hendrik Leppkes
On Fri, Sep 29, 2017 at 8:25 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Sep 29, 2017 at 2:12 PM, Hendrik Leppkes > wrote: > >> On Fri, Sep 29, 2017 at 7:08 PM, Clément Bœsch wrote: >> > On Fri, Sep 29, 2017 at 06:15:57PM +0200, Hendrik Leppkes wrote: >> >> On Fri, Sep 29, 2017 at 2:40 PM, Clé

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Clément Bœsch
On Fri, Sep 29, 2017 at 08:12:14PM +0200, Hendrik Leppkes wrote: [...] > > Just to be clear (sorry if I make you repeat yourself), we currently have > > the following: > > > > - Systems without VDA (that is, almost all of them) do not have access to > > either VDA or the stub: for them, we are ex

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Ronald S. Bultje
Hi, On Fri, Sep 29, 2017 at 2:12 PM, Hendrik Leppkes wrote: > On Fri, Sep 29, 2017 at 7:08 PM, Clément Bœsch wrote: > > On Fri, Sep 29, 2017 at 06:15:57PM +0200, Hendrik Leppkes wrote: > >> On Fri, Sep 29, 2017 at 2:40 PM, Clément Bœsch wrote: > >> > On Fri, Sep 29, 2017 at 11:38:42AM +0200, H

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Hendrik Leppkes
On Fri, Sep 29, 2017 at 7:08 PM, Clément Bœsch wrote: > On Fri, Sep 29, 2017 at 06:15:57PM +0200, Hendrik Leppkes wrote: >> On Fri, Sep 29, 2017 at 2:40 PM, Clément Bœsch wrote: >> > On Fri, Sep 29, 2017 at 11:38:42AM +0200, Hendrik Leppkes wrote: >> >> On Fri, Sep 29, 2017 at 11:11 AM, Clément B

[FFmpeg-devel] [PATCH]lavf/bit: Fix the G.729 bit auto-detection

2017-09-29 Thread Carl Eugen Hoyos
Hi! The G.729 bit auto-detection has never worked, patch attached. Please comment, Carl Eugen From 2521a9bd86d807a3e18fe930b35039beb7674b3e Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Fri, 29 Sep 2017 19:10:46 +0200 Subject: [PATCH] lavf/bit: Fix the G.729 bit auto-detection. --- lib

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Carl Eugen Hoyos
2017-09-29 11:11 GMT+02:00 Clément Bœsch : > From: Clément Bœsch > > Deprecated (aka removed) in OSX 10.11, and we have a > replacement for it (VideoToolbox). Does this mean binaries built on older systems do not start on 10.11? If yes, my suggestion would be to disable auto-detection until remo

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Clément Bœsch
On Fri, Sep 29, 2017 at 06:15:57PM +0200, Hendrik Leppkes wrote: > On Fri, Sep 29, 2017 at 2:40 PM, Clément Bœsch wrote: > > On Fri, Sep 29, 2017 at 11:38:42AM +0200, Hendrik Leppkes wrote: > >> On Fri, Sep 29, 2017 at 11:11 AM, Clément Bœsch wrote: > >> > From: Clément Bœsch > >> > > >> > Depre

Re: [FFmpeg-devel] Added HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-09-29 Thread wm4
On Fri, 29 Sep 2017 15:04:00 + "Mironov, Mikhail" wrote: > I would like to understand better the nature of the concern. The license is > MIT. The paragraph in question is a notice, not limiting the usage of the SDK. > I can definitely reduce number of headers. I can merge all necessary > in

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Hendrik Leppkes
On Fri, Sep 29, 2017 at 2:40 PM, Clément Bœsch wrote: > On Fri, Sep 29, 2017 at 11:38:42AM +0200, Hendrik Leppkes wrote: >> On Fri, Sep 29, 2017 at 11:11 AM, Clément Bœsch wrote: >> > From: Clément Bœsch >> > >> > Deprecated (aka removed) in OSX 10.11, and we have a replacement for it >> > (Vide

[FFmpeg-devel] [PATCH 1/2] mov: fix decode of fragments that overlap in time

2017-09-29 Thread John Stebbins
When keyframe intervals of dash segments are not perfectly aligned, fragments in the stream can overlap in time. Append new "trun" index entries to the end of the index instead of sorting by timestamp. Sorting by timestamp causes packets to be read out of decode order and results in decode errors.

[FFmpeg-devel] [PATCH 2/2] mov: fix non-monotonous DTS when fragments overlap in time

2017-09-29 Thread John Stebbins
--- libavformat/mov.c | 16 ++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index c7422cd9ed..871f10b240 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -4267,6 +4267,7 @@ static int mov_read_trun(MOVContext *c, AVIOCo

Re: [FFmpeg-devel] [PATCH]lavd/decklink_dec: Do not claim to output transparency information

2017-09-29 Thread Carl Eugen Hoyos
2017-09-29 17:35 GMT+02:00 Maksym Veremeyenko : > 29.09.2017 0:20, Carl Eugen Hoyos пише: >> I don't have decklink hardware but I assume it never outputs actual >> transparency. >> Or does it? > > dual link SDI could provide fill/key, some boards could support this (Do they or don't they?) Thank

Re: [FFmpeg-devel] Added HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-09-29 Thread Carl Eugen Hoyos
2017-09-29 17:04 GMT+02:00 Mironov, Mikhail : > I would like to understand better the nature of the > concern. The license is MIT. (I don't think this is correct, but see below.) > The paragraph in question is a notice, not limiting > the usage of the SDK. I don't think this is true (I am not a

Re: [FFmpeg-devel] [PATCH]lavd/decklink_dec: Do not claim to output transparency information

2017-09-29 Thread Maksym Veremeyenko
29.09.2017 0:20, Carl Eugen Hoyos пише: Hi! I don't have decklink hardware but I assume it never outputs actual transparency. Or does it? dual link SDI could provide fill/key, some boards could support this -- Maksym Veremeyenko ___ ffmpeg-devel mai

Re: [FFmpeg-devel] Added HW accelerated H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-09-29 Thread Mironov, Mikhail
I would like to understand better the nature of the concern. The license is MIT. The paragraph in question is a notice, not limiting the usage of the SDK. I can definitely reduce number of headers. I can merge all necessary interfaces into one header, though maintenance will take more resources.

[FFmpeg-devel] [PATCH 2/2] avformat/wavenc: skip writing peak-of-peaks position when writing peaks only

2017-09-29 Thread Tobias Rapp
Signed-off-by: Tobias Rapp --- libavformat/wavenc.c | 5 - tests/ref/lavf/wav_peak_only | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/libavformat/wavenc.c b/libavformat/wavenc.c index adb20cb..7a4acf9 100644 --- a/libavformat/wavenc.c +++ b/libavformat/wavenc.

[FFmpeg-devel] [PATCH 1/2] avformat/wavenc: replace literal numbers with enum constants

2017-09-29 Thread Tobias Rapp
Signed-off-by: Tobias Rapp --- libavformat/wavenc.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/wavenc.c b/libavformat/wavenc.c index 7f3059e..adb20cb 100644 --- a/libavformat/wavenc.c +++ b/libavformat/wavenc.c @@ -331,7 +331,7 @@ static int wav_write_

Re: [FFmpeg-devel] [PATCH] avfilter: add vmafmotion filter

2017-09-29 Thread Ronald S. Bultje
Hi, On Fri, Sep 15, 2017 at 4:47 PM, Ashish Pratap Singh wrote: > From: Ashish Singh > > Hi, this patch addresses the previous issues and changes it to a single > input filter. > > Signed-off-by: Ashish Singh > --- > Changelog | 1 + > doc/filters.texi| 14 ++

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Clément Bœsch
On Fri, Sep 29, 2017 at 11:38:42AM +0200, Hendrik Leppkes wrote: > On Fri, Sep 29, 2017 at 11:11 AM, Clément Bœsch wrote: > > From: Clément Bœsch > > > > Deprecated (aka removed) in OSX 10.11, and we have a replacement for it > > (VideoToolbox). > > --- > > Changes since RFC: rebase, ifdefery dep

[FFmpeg-devel] How to tell CFR/VFR video apart, and determine frame rate

2017-09-29 Thread Richard Ling
How can I determine whether a filter input is VFR or CFR? static int is_vfr(AVFilterLink *inlink) { return ???; } If I determine the input is CFR, how do I get the frame rate in FPS? Is it always the reciprocal of timebase, if not what is the correct implementation? static float get_constant

Re: [FFmpeg-devel] [PATCH] libavformat: Add format context parameter to ff_id3v2_read_dict

2017-09-29 Thread Moritz Barsnick
On Tue, Sep 26, 2017 at 08:49:44 +0200, Lukas Stabe wrote: > - * Chapters are not currently read by this variant. > + * Chapters are not currently only read by this variant when s is not NULL. I believe you forgot to drop the first "not" from that sentence. Moritz

Re: [FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Hendrik Leppkes
On Fri, Sep 29, 2017 at 11:11 AM, Clément Bœsch wrote: > From: Clément Bœsch > > Deprecated (aka removed) in OSX 10.11, and we have a replacement for it > (VideoToolbox). > --- > Changes since RFC: rebase, ifdefery depreciation dance for the pix fmt, > Changelog entry. > vda.h is an installed he

[FFmpeg-devel] [PATCH] build: make h264 VT encoder select the VT encoder dependency

2017-09-29 Thread Clément Bœsch
From: Clément Bœsch Otherwise, it's never enabled unless the user explicitely enables it. --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 8e4775be8c..6a935a5833 100755 --- a/configure +++ b/configure @@ -2936,7 +2936,8 @@ pcm_alaw_a

[FFmpeg-devel] [PATCH] lavc: drop VDA

2017-09-29 Thread Clément Bœsch
From: Clément Bœsch Deprecated (aka removed) in OSX 10.11, and we have a replacement for it (VideoToolbox). --- Changes since RFC: rebase, ifdefery depreciation dance for the pix fmt, Changelog entry. I'll apply soon since the RFC seemed to raise acceptance. --- Changelog

Re: [FFmpeg-devel] libavcodec/hap : add HapAlphaOnly decoding/encoding

2017-09-29 Thread Martin Vignali
2017-09-28 23:24 GMT+02:00 Carl Eugen Hoyos : > 2017-09-28 21:50 GMT+02:00 Martin Vignali : > > > After comments about the previous patchs > > Please explain how this encoder and decoder can be used > from the FFmpeg cli: What kind of input could be used, what > could be done with the output? > >

Re: [FFmpeg-devel] Decklink question

2017-09-29 Thread Kieran O Leary
On 29 Sep 2017 03:22, wrote: I will continue to do testing.. > This works and captures full range (data levels, but the captured data happens to be video levels 64-940) to the output file and the out.mov file looks good/clean. Not sure how to flag (metadata) output color space and levels, but