Re: [FFmpeg-devel] [PATCH] avformat/concatdec: don't call open_file when seek position within a file

2016-09-20 Thread raymond zheng
Submit reasons: I have enabled concat and async protocol,but ffmpeg will flush async buffer when doing seek. I don't want flush async buffer when doing seek within the same segment file. I found, the concatdev will call open_file everytime I try to do seek, which will cause call async_open,and the

[FFmpeg-devel] [PATCH] ffmpeg: switch to codecpar

2016-09-20 Thread Clément Bœsch
From: Clément Bœsch This commit is largely based on commit 15e84ed3 from Anton Khirnov which was previously skipped in bbf5ef9d. There are still a bunch of things raising codecpar related warnings that need fixing, such as: - the use of codec->debug in the interactive debug mode - read_ffserver

[FFmpeg-devel] [PATCH] lavc/parser: export field order if not already set

2016-09-20 Thread Rodger Combs
Some codecs set this in the parser, but not the decoder --- libavcodec/parser.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/parser.c b/libavcodec/parser.c index 2c8fc69..30cfc55 100644 --- a/libavcodec/parser.c +++ b/libavcodec/parser.c @@ -182,6 +182,11 @@ int av_parser_pa

[FFmpeg-devel] [PATCH] avcodec/avrndec: Remove obsolete FIXME

2016-09-20 Thread Michael Niedermayer
This FIXME probably meant to suggest to use a AVFrame that no longer exists in the context. Found-by: durandal117 Signed-off-by: Michael Niedermayer --- libavcodec/avrndec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/avrndec.c b/libavcodec/avrndec.c index 695c

Re: [FFmpeg-devel] [PATCH] ffmpeg: switch to codecpar

2016-09-20 Thread Hendrik Leppkes
On Tue, Sep 20, 2016 at 12:28 PM, Clément Bœsch wrote: > From: Clément Bœsch > > This commit is largely based on commit 15e84ed3 from Anton Khirnov > which was previously skipped in bbf5ef9d. > > There are still a bunch of things raising codecpar related warnings that > need fixing, such as: > -

Re: [FFmpeg-devel] Dropping SDL1 support [VOTE]

2016-09-20 Thread Ronald S. Bultje
Hi, On Mon, Sep 19, 2016 at 9:47 PM, Lukasz Marek wrote: > On Sep 18, 2016 04:57, "Ronald S. Bultje" wrote: > > > > Hi, > > > > On Sat, Sep 17, 2016 at 4:26 PM, Marton Balint wrote: > > > > > I think we should wait until Lukasz does the port of the opengl device > to > > > SDL2, and do the swi

Re: [FFmpeg-devel] [PATCH] avcodec/avrndec: Remove obsolete FIXME

2016-09-20 Thread Paul B Mahol
On 9/20/16, Michael Niedermayer wrote: > This FIXME probably meant to suggest to use a AVFrame that no longer > exists in the context. > > Found-by: durandal117 > Signed-off-by: Michael Niedermayer > --- > libavcodec/avrndec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > ok ___

Re: [FFmpeg-devel] [PATCH] ffmpeg: switch to codecpar

2016-09-20 Thread Michael Niedermayer
On Tue, Sep 20, 2016 at 12:28:41PM +0200, Clément Bœsch wrote: > From: Clément Bœsch > > This commit is largely based on commit 15e84ed3 from Anton Khirnov > which was previously skipped in bbf5ef9d. > > There are still a bunch of things raising codecpar related warnings that > need fixing, suc

Re: [FFmpeg-devel] [PATCH] hwcontext: add a QSV implementation

2016-09-20 Thread Hendrik Leppkes
On Mon, Sep 19, 2016 at 12:00 PM, Ivan Uskov wrote: > > Hello nablet, > > Saturday, September 17, 2016, 11:32:26 AM, you wrote: > >> From: Anton Khirnov > >> Signed-off-by: nablet developer >> --- >> libavutil/Makefile | 3 + >> libavutil/hwcontext.c | 3 + >> libavutil

Re: [FFmpeg-devel] [PATCH] ffmpeg: switch to codecpar

2016-09-20 Thread Clément Bœsch
On Tue, Sep 20, 2016 at 01:53:05PM +0200, Michael Niedermayer wrote: > On Tue, Sep 20, 2016 at 12:28:41PM +0200, Clément Bœsch wrote: > > From: Clément Bœsch > > > > This commit is largely based on commit 15e84ed3 from Anton Khirnov > > which was previously skipped in bbf5ef9d. > > > > There ar

Re: [FFmpeg-devel] [PATCH] ffmpeg: switch to codecpar

2016-09-20 Thread Clément Bœsch
On Tue, Sep 20, 2016 at 01:46:11PM +0200, Hendrik Leppkes wrote: [...] > > +/* > > + * For subtitles, this is required by the decoding process in > > order to > > + * rescale the timestamps: in the current API the decoded subtitles > > + * have their pts expressed i

[FFmpeg-devel] [PATCH] avformat/movenc: write pasp atom even if sar.num == sar.den

2016-09-20 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavformat/movenc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index aa4a076..0382309 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -1837,8 +1837,7 @@ static int mov_write_vid

Re: [FFmpeg-devel] [PATCH] ffmpeg: switch to codecpar

2016-09-20 Thread Michael Niedermayer
On Tue, Sep 20, 2016 at 12:28:41PM +0200, Clément Bœsch wrote: > From: Clément Bœsch > > This commit is largely based on commit 15e84ed3 from Anton Khirnov > which was previously skipped in bbf5ef9d. > > There are still a bunch of things raising codecpar related warnings that > need fixing, suc

[FFmpeg-devel] [PATCH] fate: add aac_adtstoasc test (Ticket 3715)

2016-09-20 Thread Michael Niedermayer
--- tests/fate-run.sh | 3 +- tests/fate/ffmpeg.mak | 5 ++ tests/ref/fate/adtstoasc_ticket3715 | 94 + 3 files changed, 101 insertions(+), 1 deletion(-) create mode 100644 tests/ref/fate/adtstoasc_ticket3715 diff --git a/test

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Put correct display aspect ratio in ARES atom

2016-09-20 Thread Carl Eugen Hoyos
2016-09-19 13:28 GMT+02:00 Carl Eugen Hoyos : > Hi! > > We read a display aspect ratio from ARES atom because of several > user-provided samples, I believe it makes sense to also write the > width value into the ares atom depending on the aspect ratio. Ping. Carl Eugen ___

Re: [FFmpeg-devel] [PATCH] fate: add aac_adtstoasc test (Ticket 3715)

2016-09-20 Thread Michael Niedermayer
On Tue, Sep 20, 2016 at 04:02:05PM +0200, Michael Niedermayer wrote: > --- > tests/fate-run.sh | 3 +- > tests/fate/ffmpeg.mak | 5 ++ > tests/ref/fate/adtstoasc_ticket3715 | 94 > + > 3 files changed, 101 insertions(+), 1 dele

Re: [FFmpeg-devel] [PATCH] lavc/parser: export field order if not already set

2016-09-20 Thread Carl Eugen Hoyos
2016-09-20 13:13 GMT+02:00 Rodger Combs : > Some codecs set this in the parser, but not the decoder > +#define FILL(name) if(s->name > 0 && avctx->name <= 0) avctx->name = s->name > +if (avctx->codec_type == AVMEDIA_TYPE_VIDEO) { > +FILL(field_order); How is this a useful macro? Carl

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Put correct display aspect ratio in ARES atom

2016-09-20 Thread Paul B Mahol
On 9/20/16, Carl Eugen Hoyos wrote: > 2016-09-19 13:28 GMT+02:00 Carl Eugen Hoyos : >> Hi! >> >> We read a display aspect ratio from ARES atom because of several >> user-provided samples, I believe it makes sense to also write the >> width value into the ares atom depending on the aspect ratio. >

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Put correct display aspect ratio in ARES atom

2016-09-20 Thread Carl Eugen Hoyos
2016-09-20 17:26 GMT+02:00 Paul B Mahol : > On 9/20/16, Carl Eugen Hoyos wrote: >> 2016-09-19 13:28 GMT+02:00 Carl Eugen Hoyos : >>> Hi! >>> >>> We read a display aspect ratio from ARES atom because of several >>> user-provided samples, I believe it makes sense to also write the >>> width value in

Re: [FFmpeg-devel] [PATCH] lavc/parser: export field order if not already set

2016-09-20 Thread Rodger Combs
> On Sep 20, 2016, at 10:18, Carl Eugen Hoyos wrote: > > 2016-09-20 13:13 GMT+02:00 Rodger Combs : >> Some codecs set this in the parser, but not the decoder > >> +#define FILL(name) if(s->name > 0 && avctx->name <= 0) avctx->name = s->name >> +if (avctx->codec_type == AVMEDIA_TYPE_VIDEO) {

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Put correct display aspect ratio in ARES atom

2016-09-20 Thread Paul B Mahol
On 9/20/16, Carl Eugen Hoyos wrote: > 2016-09-20 17:26 GMT+02:00 Paul B Mahol : >> On 9/20/16, Carl Eugen Hoyos wrote: >>> 2016-09-19 13:28 GMT+02:00 Carl Eugen Hoyos : Hi! We read a display aspect ratio from ARES atom because of several user-provided samples, I believe it mak

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Put correct display aspect ratio in ARES atom

2016-09-20 Thread Carl Eugen Hoyos
2016-09-20 17:42 GMT+02:00 Paul B Mahol : > On 9/20/16, Carl Eugen Hoyos wrote: >> 2016-09-20 17:26 GMT+02:00 Paul B Mahol : >>> On 9/20/16, Carl Eugen Hoyos wrote: 2016-09-19 13:28 GMT+02:00 Carl Eugen Hoyos : > Hi! > > We read a display aspect ratio from ARES atom because of se

Re: [FFmpeg-devel] [PATCH]lavf/mov: Read aspect ratio from ares atom for dnxhd

2016-09-20 Thread Carl Eugen Hoyos
2016-09-19 14:57 GMT+02:00 Paul B Mahol : > On 9/19/16, Paul B Mahol wrote: >> Ignore that one above, patck with corrected text is ok. > > title should mentioned what kind of aspect ratio this is about. As I > thought it is about sar. Log message changed locally to "lavf/mov: Read display aspect

Re: [FFmpeg-devel] Patches related to bug 4988

2016-09-20 Thread Carl Eugen Hoyos
2016-09-10 10:22 GMT+02:00 Oliver Collyer : > One workaround is to issue > “modprobe -r uvcvideo && modprobe uvcvideo” to re-load the > uvcvideo module each time before running FFmpeg. This works > for both devices and the captures then proceed as normal. Doesn't this indicate that there really i

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Put correct display aspect ratio in ARES atom

2016-09-20 Thread Paul B Mahol
On 9/20/16, Carl Eugen Hoyos wrote: > 2016-09-20 17:42 GMT+02:00 Paul B Mahol : >> On 9/20/16, Carl Eugen Hoyos wrote: >>> 2016-09-20 17:26 GMT+02:00 Paul B Mahol : On 9/20/16, Carl Eugen Hoyos wrote: > 2016-09-19 13:28 GMT+02:00 Carl Eugen Hoyos : >> Hi! >> >> We read a dis

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Put correct display aspect ratio in ARES atom

2016-09-20 Thread Carl Eugen Hoyos
2016-09-20 18:47 GMT+02:00 Paul B Mahol : > On 9/20/16, Carl Eugen Hoyos wrote: >> 2016-09-20 17:42 GMT+02:00 Paul B Mahol : >>> On 9/20/16, Carl Eugen Hoyos wrote: 2016-09-20 17:26 GMT+02:00 Paul B Mahol : > On 9/20/16, Carl Eugen Hoyos wrote: >> 2016-09-19 13:28 GMT+02:00 Carl Eug

Re: [FFmpeg-devel] [PATCH]lavf/movenc: Put correct display aspect ratio in ARES atom

2016-09-20 Thread Paul B Mahol
On 9/20/16, Carl Eugen Hoyos wrote: > 2016-09-20 18:47 GMT+02:00 Paul B Mahol : >> On 9/20/16, Carl Eugen Hoyos wrote: >>> 2016-09-20 17:42 GMT+02:00 Paul B Mahol : On 9/20/16, Carl Eugen Hoyos wrote: > 2016-09-20 17:26 GMT+02:00 Paul B Mahol : >> On 9/20/16, Carl Eugen Hoyos wrote

[FFmpeg-devel] [PATCH] avfilter: add nlmeans filter

2016-09-20 Thread Clément Bœsch
Fixes Ticket #4910 --- I actually tried to implement the better defaults suggestion from ipol (see @todo) but it wasn't convincing; probably because of different scales, so I need to investigate. Also, integral is still inplace in the filter for now as I didn't find a clean way of testing it outsi

[FFmpeg-devel] [German] Workshop bei der OpenRheinRuhr

2016-09-20 Thread Gerion Entrup
Hello/Hallo, I'll write the rest of the mail in German, because the subject is the German Conference OpenRheinRuhr. Ich wollte auf der OpenRheinRuhr wieder einen Workshop zu der Benutzung von FFmpeg halten, diesmal aber mit Fokus auf Filtern (ich habe vor einem Jahr auf den Chemnitzer Linuxtag

[FFmpeg-devel] [PATCH] ffmpeg: switch to codecpar

2016-09-20 Thread Clément Bœsch
From: Clément Bœsch This commit is largely based on commit 15e84ed3 from Anton Khirnov which was previously skipped in bbf5ef9d. There are still a bunch of things raising codecpar related warnings that need fixing, such as: - the use of codec->debug in the interactive debug mode - read_ffserver

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Make audio timestamps strictly monotonically increasing inside an edit list. Fixes gapless decoding.

2016-09-20 Thread Sasi Inguva
On Mon, Sep 19, 2016 at 9:22 PM, Michael Niedermayer wrote: > On Mon, Sep 19, 2016 at 06:31:31PM -0700, Sasi Inguva wrote: > > Signed-off-by: Sasi Inguva > > --- > > libavcodec/utils.c | 15 +++--- > > libavformat/mov.c| 75 > +++

[FFmpeg-devel] [PATCH] lavf/mov.c: Make audio timestamps strictly monotonically increasing inside an edit list. Fixes gapless decoding.

2016-09-20 Thread Sasi Inguva
Signed-off-by: Sasi Inguva --- libavcodec/utils.c | 15 +++--- libavformat/mov.c| 81 tests/ref/fate/gaplessenc-itunes-to-ipod-aac | 2 +- tests/ref/fate/gaplessenc-pcm-to-mov-aac | 2 +- 4 files changed, 78

Re: [FFmpeg-devel] [PATCH] avcodec/avrndec: Remove obsolete FIXME

2016-09-20 Thread Michael Niedermayer
On Tue, Sep 20, 2016 at 01:51:36PM +0200, Paul B Mahol wrote: > On 9/20/16, Michael Niedermayer wrote: > > This FIXME probably meant to suggest to use a AVFrame that no longer > > exists in the context. > > > > Found-by: durandal117 > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/a

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxfdec: use first valid sourceclip found if material track has multiple components

2016-09-20 Thread Michael Niedermayer
On Sun, Sep 18, 2016 at 02:11:34PM -0700, Mark Reid wrote: > hi, > This patch fixes a issue with mxf footage having multiple > components on a material track. > > here is a link to the sample footage for the second patches fate test > https://dl.dropboxusercontent.com/u/170952/fate/mxf/multiple_co

[FFmpeg-devel] [PATCH] avcodec/h264: Use ptrdiff_t for (bi)width functions

2016-09-20 Thread Michael Niedermayer
Might fix some mysterious asm related issues like Ticket5579 Signed-off-by: Michael Niedermayer --- libavcodec/h264dsp.h | 5 +++-- libavcodec/h264dsp_template.c | 4 ++-- libavcodec/mips/h264dsp_mips.h | 26 +- libavcodec/mips/h264dsp_mmi.c | 12 ++-

[FFmpeg-devel] [PATCH 1/3] cuvid: Add MIT licenced nvcuid headers from Video SDK 7.0

2016-09-20 Thread Philip Langdale
For unknown reasons, the only accurately descriptive version of cuviddec.h is in the Video SDK - the one in CUDA 7.5 lacks vp8 PICPARAMS and the vp9 struct definition is inaccurate. The CUDA 8 RC includes an ancient version of this file from many many years go. However, the one in the Video SDK is

[FFmpeg-devel] [PATCH 2/3] cuvid: Modify cuvid headers to be usable

2016-09-20 Thread Philip Langdale
We remove the dynlink fanciness with normal function prototypes and update the include paths. Signed-off-by: Philip Langdale --- compat/cuda/cuviddec.h | 53 +- compat/cuda/nvcuvid.h | 34 +++- 2 files changed, 29 inser

[FFmpeg-devel] [PATCH 3/3] cuvid: Use the compat headers for nvcuvid

2016-09-20 Thread Philip Langdale
Signed-off-by: Philip Langdale --- libavcodec/cuvid.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cuvid.c b/libavcodec/cuvid.c index f2e92cf..7fd0b0d 100644 --- a/libavcodec/cuvid.c +++ b/libavcodec/cuvid.c @@ -30,7 +30,7 @@ #include "avcodec.h" #include "inte

[FFmpeg-devel] [PATCH 0/3] Include headers for cuvid

2016-09-20 Thread Philip Langdale
The cuvid header situation is a mess - the only feature-complete headers are in the Video SDK and not in the cuda header collection. The headers in the Video SDK are MIT licenced, and so we can bundle them like we do the nvenc header. Once bundled, we do need to tweak them to remove the use of a d

Re: [FFmpeg-devel] [PATCH 0/3] Include headers for cuvid

2016-09-20 Thread Matt Oliver
On 21 September 2016 at 14:38, Philip Langdale wrote: > The cuvid header situation is a mess - the only feature-complete headers > are > in the Video SDK and not in the cuda header collection. The headers in the > Video SDK are MIT licenced, and so we can bundle them like we do the nvenc > header

Re: [FFmpeg-devel] [PATCH 0/3] Include headers for cuvid

2016-09-20 Thread Philip Langdale
On Wed, 21 Sep 2016 15:09:36 +1000 Matt Oliver wrote: > On 21 September 2016 at 14:38, Philip Langdale > wrote: > > > The cuvid header situation is a mess - the only feature-complete > > headers are > > in the Video SDK and not in the cuda header collection. The headers > > in the Video SDK are