Re: [FFmpeg-devel] [PATCH 2/2] ffplay: ensure that we buffer at least 1 second of content

2016-06-21 Thread Adrian Cruceru
Hi Marton, I apologize for the delay, I've tested with provided patches and: - With 1 second duration I'm still encountering issue - a lot later though - With 2 second duration - haven't seen issue in ~6 hour runs - will let it run longer next few days. Notes: - Running with debug logs which m

Re: [FFmpeg-devel] [PATCH] avcodec/h264_sei: Do not skip subsequent SEIs on error unless explode is set

2016-06-21 Thread Michael Niedermayer
On Sun, Jun 12, 2016 at 06:15:55PM +0200, Hendrik Leppkes wrote: > On Sun, Jun 12, 2016 at 12:49 PM, Michael Niedermayer > wrote: > > Signed-off-by: Michael Niedermayer > > --- > > libavcodec/h264_sei.c |6 +- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > One argument here

[FFmpeg-devel] [PATCH] avcodec/h264_sei: Do not skip subsequent SEIs on errors potentially caused by missing parameter sets.

2016-06-21 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/h264_sei.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/libavcodec/h264_sei.c b/libavcodec/h264_sei.c index d0596dc..30eade5 100644 --- a/libavcodec/h264_sei.c +++ b/libavcodec/h264_sei.c @@ -32,6 +32,8 @@

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

2016-06-21 Thread Carl Eugen Hoyos
Dan Parrot mail.com> writes: > "/usr/bin/time -p make -j 4 fate SAMPLES=fate-suite/" I would expect that this depends on many factors outside of your patch. You could test with the START_TIMER and STOP_TIMER macros. (Or look at the asm code that is produced...) Carl Eugen

Re: [FFmpeg-devel] [PATCH] lavc/mediacodecdec_h264: use ff_h264_decode_extradata to extract PPS/SPS

2016-06-21 Thread Matthieu Bouron
On Mon, Jun 20, 2016 at 06:25:19PM +0200, Matthieu Bouron wrote: > On Mon, Jun 20, 2016 at 12:03:58PM +0200, Matthieu Bouron wrote: > > On Sun, Jun 19, 2016 at 07:44:44PM +0200, Matthieu Bouron wrote: > > > On Mon, Jun 13, 2016 at 02:37:29PM +0200, Matthieu Bouron wrote: > > > > On Mon, Jun 13, 201

[FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Benoit Fouet
Hi, Unless I totally missed something, the FIXME in H264ParamSets structure should be fixed by attached patch. -- Ben From 28ae10498f81070539bdb8f40236326743350101 Mon Sep 17 00:00:00 2001 From: Benoit Fouet Date: Tue, 21 Jun 2016 14:17:13 +0200 Subject: [PATCH] h264: make H264ParamSets sps

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Clément Bœsch
On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: > Hi, > > Unless I totally missed something, the FIXME in H264ParamSets structure > should be fixed by attached patch. > > -- > Ben > > From 28ae10498f81070539bdb8f40236326743350101 Mon Sep 17 00:00:00 2001 > From: Benoit Fouet > D

[FFmpeg-devel] [PATCH 2/4] lavc/mediacodec: re-indent after previous commit

2016-06-21 Thread Matthieu Bouron
From: Matthieu Bouron --- libavcodec/mediacodecdec.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c index 6b7cdb4..75933fc 100644 --- a/libavcodec/mediacodecdec.c +++ b/libavcodec/mediacodecde

[FFmpeg-devel] lavc/mediacodec: improve end of stream handling

2016-06-21 Thread Matthieu Bouron
Hello, The following patchset improve handling of EOS (End Of Stream) in the mediacodec decoder. The decoder now relies on the relevant buffer info flags to detect EOS instead of counting queued/dequeued buffers. It also fixes a potential loss of frames while flushing (draining) the remaining fr

[FFmpeg-devel] [PATCH 1/4] lavc/mediacodec: discard 0-sized buffers

2016-06-21 Thread Matthieu Bouron
From: Matthieu Bouron Their only purpose is to carry the end of stream flag. --- libavcodec/mediacodecdec.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c index 68df885..6b7cdb4 100644 --- a/libavcodec/mediacodecdec.c +++ b/liba

[FFmpeg-devel] [PATCH 3/4] lavc/mediacodec: rely on buffer flags to detect end of stream

2016-06-21 Thread Matthieu Bouron
From: Matthieu Bouron --- libavcodec/mediacodecdec.c | 17 - libavcodec/mediacodecdec.h | 3 +-- 2 files changed, 9 insertions(+), 11 deletions(-) diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c index 75933fc..7cef90b 100644 --- a/libavcodec/mediacodecdec.c

[FFmpeg-devel] [PATCH 4/4] lavc/mediacodec: increase buffer dequeue timeout when the codec needs to be drained

2016-06-21 Thread Matthieu Bouron
From: Matthieu Bouron Increase buffer dequeue timeout when the codec needs to be drained as it could happen that no input buffer is available when we receive a null packet for the first time (meaning we are unable to signal end of stream and mark the codec as draining). Fixes potential loss of l

[FFmpeg-devel] [PATCH] fate: add test for amerge

2016-06-21 Thread Petru Rares Sincraian
Hi there, Here is test for amerge. Regards, Petru Rares.From 4882601b6eb5a825bd5ddf837b1dce3977496314 Mon Sep 17 00:00:00 2001 From: Petru Rares Sincraian Date: Tue, 21 Jun 2016 14:43:01 +0200 Subject: [PATCH] fate: add test for amerge --- tests/fate/filter-audio.mak | 5 ++ tests/ref/fat

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Hendrik Leppkes
On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: > On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: >> Hi, >> >> Unless I totally missed something, the FIXME in H264ParamSets structure >> should be fixed by attached patch. >> >> -- >> Ben >> > >> From 28ae10498f81070539bdb8f40236

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Benoit Fouet
Hi, On 21/06/2016 14:52, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: Hi, Unless I totally missed something, the FIXME in H264ParamSets structure should be fixed by attached patch. -- Ben From 28

Re: [FFmpeg-devel] PATCH: dshow: don't add two instances of same device to graphs

2016-06-21 Thread Roger Pack
On 6/17/16, Roger Pack wrote: > On 6/15/16, Clément Bœsch wrote: >> On Wed, Jun 15, 2016 at 02:54:09AM -0600, Roger Pack wrote: >>> A handful of devices don't support this, and the rest work fine with it >>> :) >>> >>> -roger- >> >>> From e724d7f169bcae3217455cd88f9c023d275d367a Mon Sep 17 00:00:

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Hendrik Leppkes
On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet wrote: > Hi, > > > On 21/06/2016 14:52, Hendrik Leppkes wrote: >> >> On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: >>> >>> On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: Hi, Unless I totally missed something,

Re: [FFmpeg-devel] [PATCH] h264: make H264ParamSets sps const

2016-06-21 Thread Benoit Fouet
Hi, On 21/06/2016 16:29, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 4:20 PM, Benoit Fouet wrote: Hi, On 21/06/2016 14:52, Hendrik Leppkes wrote: On Tue, Jun 21, 2016 at 2:40 PM, Clément Bœsch wrote: On Tue, Jun 21, 2016 at 02:34:33PM +0200, Benoit Fouet wrote: Hi, Unless I totally m

[FFmpeg-devel] [PATCH] lavc/videotoolbox: fix avcC extradata creation

2016-06-21 Thread Matthieu Bouron
From: Matthieu Bouron Fixes properly ticket #5638. Since 772ad7142dff590c7bb30370acf5d3c9a8fb512b PPS and SPS contain the nal type byte. This reverts commit b8d754c5d0a2e99022c27bd1159ea3f597f67452. This reverts commit 8e47a99f576da10b2a36e33b6b9acbf1c2da7485. --- libavcodec/videotoolbox.c | 8

Re: [FFmpeg-devel] [PATCH] fate: add test for amerge

2016-06-21 Thread Michael Niedermayer
On Tue, Jun 21, 2016 at 12:53:05PM +, Petru Rares Sincraian wrote: > > Hi there, > > Here is test for amerge. > > > Regards, > Petru Rares. > fate/filter-audio.mak |5 + > ref/fate/filter-amerge | 135 > + > 2 files changed, 140 inser

Re: [FFmpeg-devel] [PATCH] lavc/videotoolbox: fix avcC extradata creation

2016-06-21 Thread Richard Kern
> On Jun 21, 2016, at 11:48 AM, Matthieu Bouron > wrote: > > From: Matthieu Bouron > > Fixes properly ticket #5638. > > Since 772ad7142dff590c7bb30370acf5d3c9a8fb512b PPS and SPS contain the > nal type byte. > > This reverts commit b8d754c5d0a2e99022c27bd1159ea3f597f67452. > This reverts co

[FFmpeg-devel] [PATCH] YUY2 Lossless Codec decoder

2016-06-21 Thread Paul B Mahol
Hi, patch attached. From fe0a15460b5ceb13ae7af9bdd3593cbad4b7 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Mon, 13 Jun 2016 12:17:44 +0200 Subject: [PATCH] avcodec: add YUY2 Lossless Codec decoder Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 1 + libavcodec/allcodecs.c

Re: [FFmpeg-devel] [PATCH] YUY2 Lossless Codec decoder

2016-06-21 Thread Lou Logan
On Tue, 21 Jun 2016 20:02:20 +0200, Paul B Mahol wrote: > Hi, > > patch attached. A few trivial warnings: libavcodec/ylc.c: In function ‘build_vlc’: libavcodec/ylc.c:104:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] int cur_node = 256; ^~~ li

Re: [FFmpeg-devel] [PATCH] YUY2 Lossless Codec decoder

2016-06-21 Thread Paul B Mahol
On 6/21/16, Lou Logan wrote: > On Tue, 21 Jun 2016 20:02:20 +0200, Paul B Mahol wrote: > >> Hi, >> >> patch attached. > > A few trivial warnings: > > libavcodec/ylc.c: In function `build_vlc': > libavcodec/ylc.c:104:5: warning: ISO C90 forbids mixed declarations and code > [-Wdeclaration-after-sta

Re: [FFmpeg-devel] [PATCH 3/3] avformat/mpegts: include stream types for mpeg2 and aac

2016-06-21 Thread Michael Niedermayer
On Sat, Jun 18, 2016 at 12:56:50PM +0200, Hendrik Leppkes wrote: > On Tue, Jun 14, 2016 at 8:08 PM, Aman Gupta wrote: > > From: Aman Gupta > > > > this removes the need to probe to discover mpeg2 and aac streams > > inside mpegts containers, thus speeding up initial playback. > > --- > > libavfo

[FFmpeg-devel] [PATCH] movenc: Add an option for offsetting the ISM fragment info timestamps

2016-06-21 Thread Thomas Volkert
From: Martin Storsjö --- libavformat/movenc.c | 2 ++ libavformat/movenc.h | 1 + 2 files changed, 3 insertions(+) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 837e1e5..589c2ea 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -87,6 +87,7 @@ static const AVOption

Re: [FFmpeg-devel] [PATCH] libavformat/matroskaenc: omit segment UID for webm

2016-06-21 Thread Michael Niedermayer
On Mon, Jun 20, 2016 at 08:20:47PM -0700, Michael Bradshaw wrote: > Hi, > > Attached patch removes SegmentUID element from WebM files, as it's > unsupported in WebM[1]. > > Please review/apply. > > Thanks, > > Michael > > [1]: https://www.webmproject.org/docs/container/#SegmentUID what does "

Re: [FFmpeg-devel] [PATCH] libavformat/matroskaenc: omit segment UID for webm

2016-06-21 Thread Dave Rice
> On Jun 21, 2016, at 3:22 PM, Michael Niedermayer > wrote: > > On Mon, Jun 20, 2016 at 08:20:47PM -0700, Michael Bradshaw wrote: >> Hi, >> >> Attached patch removes SegmentUID element from WebM files, as it's >> unsupported in WebM[1]. >> >> Please review/apply. >> >> Thanks, >> >> Michael

Re: [FFmpeg-devel] [PATCH] PPC64: Add IBM POWER8 SIMD Implementation

2016-06-21 Thread Dan Parrot
On Tue, 2016-06-21 at 00:04 -0500, Dan Parrot wrote: > On Tue, 2016-06-21 at 02:22 +0200, Michael Niedermayer wrote: > > On Mon, Jun 20, 2016 at 06:38:18PM -0500, Dan Parrot wrote: > > > On Tue, 2016-06-21 at 01:06 +0200, Michael Niedermayer wrote: > > > > On Mon, Jun 20, 2016 at 05:55:47PM -0500,