Re: [FFmpeg-devel] [PATCH 1/4] lavc/h264: move green meta logging out of the sei decoding

2016-06-14 Thread Clément Bœsch
On Tue, Jun 14, 2016 at 12:01:25AM +0200, Hendrik Leppkes wrote: > On Mon, Jun 13, 2016 at 11:02 PM, Clément Bœsch wrote: > > This will simplify the next Libav merge where SEI decoding doesn't have > > access to the debug level anymore. > > This whole business looks rather fragile

Re: [FFmpeg-devel] [PATCH] libavformat/mpegtsenc: improved cbr mode pcr accuracy

2016-06-14 Thread Alexander Shumsky
08.06.2016 18:48, Michael Niedermayer пишет: did you test this ? if so please provide information & test results from it also has this been tested with multiple programs ? We have tested this with StreamGuru MPEG Analyzer. With default 20ms PCR interval patched version gives max PCR interval

Re: [FFmpeg-devel] [PATCH] libavformat/mpegtsenc: improved cbr mode pcr accuracy

2016-06-14 Thread Michael Niedermayer
On Tue, Jun 14, 2016 at 04:25:50PM +0300, Alexander Shumsky wrote: > 08.06.2016 18:48, Michael Niedermayer пишет: > >did you test this ? > >if so please provide information & test results from it > > > >also has this been tested with multiple programs ? > > We have tested this with StreamGuru

Re: [FFmpeg-devel] [PATCH] lavf/utils: fix timestamp trace log in estimate_timings() function

2016-06-14 Thread Stefano Sabatini
On date Sunday 2016-06-05 15:42:05 +0200, Michael Niedermayer encoded: > On Sun, Jun 05, 2016 at 12:36:30PM +0200, Stefano Sabatini wrote: > > Use the correct timebase, also replace "stream" with "format" in the > > context of the format. > > --- > > libavformat/utils.c | 8 > > 1 file

Re: [FFmpeg-devel] [PATCH] ffprobe: Support adding av_log output to frames

2016-06-14 Thread Stefano Sabatini
On date Wednesday 2016-06-08 18:20:39 +0200, Michael Niedermayer encoded: > On Sun, Jun 05, 2016 at 12:56:08PM +0200, Stefano Sabatini wrote: > > On date Tuesday 2016-05-31 21:23:27 +0200, Michael Niedermayer encoded: > > > adding demuxer and other logs should be easy > > > This forces single

Re: [FFmpeg-devel] warnings: Truncating likely oversized PPS

2016-06-14 Thread Stefano Sabatini
On date Sunday 2016-06-05 16:16:15 +0200, Robert Krüger encoded: > Hi, > > I'm observing tons of these messages in my logs for mxf files from a number > of professional cameras. > > Would it help to submit a sample, so someone could check if the warning > makes sense? I'm close to hacking the

Re: [FFmpeg-devel] [PATCH v2] Add support for vp9 in iso-bmff

2016-06-14 Thread KongQun Yang
Patchset v2 addresses review comments. Thanks everyone for the review. Here is a summary of the changes in patchset v2: 1. Add "-strict -2" requirement, since the feature is still being reviewed and may subject to change. 2. Rename vpx.c/vpx.h to vpc.c/vpc.h to align with avc and hevc. 3. Use

Re: [FFmpeg-devel] [PATCH] fate: add afade tes

2016-06-14 Thread Petru Rares Sincraian
I look the code for some time but I don't know how to express the dependency. I think it's ok because it uses FATE_AFILTER_SAMPLES and then this variable is added to FATE_SAMPLES_AVCONV. Muhammad which command did you execute to reproduce the error? Thanks, Petru Rares.

[FFmpeg-devel] [PATCH 1/3] avformat/mpegts: enhance logging in trace mode

2016-06-14 Thread Aman Gupta
From: Aman Gupta --- libavformat/mpegts.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index 4d1bc6d..6acb797 100644 --- a/libavformat/mpegts.c +++ b/libavformat/mpegts.c @@ -446,7 +446,7 @@ static

[FFmpeg-devel] [PATCH 2/3] avformat/utils: in debug mode, print number of streams found before avformat_find_stream_info()

2016-06-14 Thread Aman Gupta
From: Aman Gupta --- libavformat/utils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index 25c9a1b..494379a 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -3297,8 +3297,8 @@ int

[FFmpeg-devel] [PATCH] avcodec/utils: ensure conversion from ass to ass_with_timings works with unknown durations

2016-06-14 Thread Aman Gupta
From: Aman Gupta the eia_608 decoder with real_time=1 emits subtitles with an unknown duration, which were getting converted into ass incorrectly before this patch. --- libavcodec/utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/utils.c

Re: [FFmpeg-devel] [PATCH] lavf: add textdata virtual demuxer and demuxer

2016-06-14 Thread Stefano Sabatini
On date Sunday 2016-06-05 12:59:48 +0200, Stefano Sabatini encoded: > On date Thursday 2016-05-19 18:50:17 +0200, Stefano Sabatini encoded: > > On date Thursday 2016-05-19 18:45:22 +0200, Stefano Sabatini encoded: > > > This format is useful to inject custom user data into streams. > > > --- > > >

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

2016-06-14 Thread Aman Gupta
From: Aman Gupta this removes the need to probe to discover mpeg2 and aac streams inside mpegts containers, thus speeding up initial playback. --- libavformat/mpegts.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index

Re: [FFmpeg-devel] [PATCH 1/5] rtpdec_vp9: Make sure to free the temp buffer on close

2016-06-14 Thread Thomas Volkert
On 13.06.2016 21:03, Thomas Volkert wrote: From: Martin Storsjö Signed-off-by: Martin Storsjö --- libavformat/rtpdec_vp9.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/rtpdec_vp9.c b/libavformat/rtpdec_vp9.c index

Re: [FFmpeg-devel] [PATCH] os_support: use the appropriate stat functions matching the stat type

2016-06-14 Thread Michael Niedermayer
On Tue, Jun 14, 2016 at 05:09:01PM +0200, Hendrik Leppkes wrote: > The stat struct is defined to stati64, which requires using the appropriate > wstati/stati functions as well. > Fixes a whole bunch of compiler warnings as well as build breakage with the > decklink avdevice. > --- >

Re: [FFmpeg-devel] [PATCH v2] Add support for vp9 in iso-bmff

2016-06-14 Thread Ronald S. Bultje
Hi, On Mon, Jun 13, 2016 at 5:26 PM, Kongqun Yang wrote: > > @@ -5369,6 +5384,17 @@ static int mov_write_header(AVFormatContext *s) > pix_fmt == AV_PIX_FMT_MONOWHITE || > pix_fmt == AV_PIX_FMT_MONOBLACK; > } >

[FFmpeg-devel] [PATCH 1/5] avcodec/ccaption_dec: change write_char() to void as return value is unused

2016-06-14 Thread Aman Gupta
From: Aman Gupta --- libavcodec/ccaption_dec.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c index 3b15149..a6d879e 100644 --- a/libavcodec/ccaption_dec.c +++ b/libavcodec/ccaption_dec.c @@ -315,7

[FFmpeg-devel] [PATCH 4/5] fate: update sub-cc tests for closed caption positioning

2016-06-14 Thread Aman Gupta
From: Aman Gupta --- tests/ref/fate/sub-cc | 4 +-- tests/ref/fate/sub-cc-realtime | 60 +- 2 files changed, 32 insertions(+), 32 deletions(-) diff --git a/tests/ref/fate/sub-cc b/tests/ref/fate/sub-cc index 0d5bc77..4cc02d1

[FFmpeg-devel] [PATCH 3/5] avcodec/ccaption_dec: implement positioning for closed captions

2016-06-14 Thread Aman Gupta
From: Aman Gupta Positioning math is based on the guidelines in https://dvcs.w3.org/hg/text-tracks/raw-file/default/608toVTT/608toVTT.html#positioning-in-cea-608 --- libavcodec/ccaption_dec.c | 29 ++--- 1 file changed, 26 insertions(+), 3 deletions(-)

[FFmpeg-devel] [PATCH 2/5] avcodec/ccaption_dec: implement tab offset commands

2016-06-14 Thread Aman Gupta
From: Aman Gupta --- libavcodec/ccaption_dec.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c index a6d879e..0b4a061 100644 --- a/libavcodec/ccaption_dec.c +++ b/libavcodec/ccaption_dec.c @@ -713,6 +713,11 @@ static

Re: [FFmpeg-devel] [PATCH 2/3] avformat/utils: in debug mode, print number of streams found before avformat_find_stream_info()

2016-06-14 Thread Michael Niedermayer
On Tue, Jun 14, 2016 at 11:08:54AM -0700, Aman Gupta wrote: > From: Aman Gupta > > --- > libavformat/utils.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Why not

[FFmpeg-devel] [PATCH 5/5] avcodec/ccaption_dec: default rollup to row 10

2016-06-14 Thread Aman Gupta
From: Aman Gupta This ensures that captions are written towards the bottom of the screen when tuning into mid-stream. The row will be reset on the receipt of the next PAC command. Row 10 was chosen as it corresponds to the value of "0" in a PAC (see row_map in handle_pac()). ---

Re: [FFmpeg-devel] [PATCH 1/3] avformat/mpegts: enhance logging in trace mode

2016-06-14 Thread Michael Niedermayer
On Tue, Jun 14, 2016 at 11:08:53AM -0700, Aman Gupta wrote: > From: Aman Gupta > > --- > libavformat/mpegts.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Concerning

Re: [FFmpeg-devel] [PATCH] libopusenc: Add channel mapping family argument

2016-06-14 Thread Michael Graczyk
Has anybody had a chance to look at this patch? Any thoughts or ways I can make it easier to look over? On Fri, Jun 10, 2016 at 10:39 PM, Michael Graczyk wrote: > The libopus encoder has a parameter called "channel mapping family" which > is used to specify inter-channel

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

2016-06-14 Thread Dan Parrot
On Tue, 2016-06-14 at 18:56 -0500, Dan Parrot wrote: > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel Please disregard this attempted patch. Made wrong choice of using email client rather than

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

2016-06-14 Thread Dan Parrot
This is the first commit addressing Trac ticket #5570. Functions defined in libswscale/input.c have corresponding definitions in libswscale/ppc/input_vsx.h The corresponding function names in the latter contain the suffix "_vsx". --- libswscale/input.c | 44 +--

Re: [FFmpeg-devel] [PATCH] libopusenc: Add channel mapping family argument

2016-06-14 Thread Michael Graczyk
Michael, Thanks for the input. I have refactored the patch into two patches. The first refactors to make the second patch simpler. I have attached both new patches. As for the approach, I can see why this may not be an ideal approach. I based this patch on a discussion I had on #ffmpeg-devel a

Re: [FFmpeg-devel] [PATCH v4] Add experimental support for vp9 in iso-bmff

2016-06-14 Thread Ronald S. Bultje
Hi, On Tue, Jun 14, 2016 at 5:52 PM, Kongqun Yang wrote: > +if (profile == FF_PROFILE_UNKNOWN) { > + if (vpx_chroma_subsampling == VPX_SUBSAMPLING_420_VERTICAL || > Indent is still 2 spaces in the second block (relative to first). Ronald

[FFmpeg-devel] [PATCH v5] Add experimental support for vp9 in iso-bmff

2016-06-14 Thread Kongqun Yang
Implemented according to the draft specification "VP Codec ISO Media File Format Binding": http://www.webmproject.org/vp9/#draft-vp-codec-iso-media-file-format-binding '-strict -2' is required to use this feature. Change-Id: Iaa7ddf5524b17e8d79cd1923b26f096d6e91 --- libavformat/Makefile |

Re: [FFmpeg-devel] Where and how to add a custom hardware decoder in libavcodec ?

2016-06-14 Thread Sunny Shukla
Hi can anyone please help ! On Jun 9, 2016 3:37 PM, "Sunny Shukla" wrote: > Hi, > > I am new to ffmpeg, am working on a custom* board* which has its own > hardware decoder. > > I have a linux application for hardware decoder which take a h264 file as > input and provides a

Re: [FFmpeg-devel] [PATCH v5] Add experimental support for vp9 in iso-bmff

2016-06-14 Thread Ronald S. Bultje
Hi, On Tue, Jun 14, 2016 at 6:05 PM, Kongqun Yang wrote: > +default: > +av_log(NULL, AV_LOG_ERROR, "Unsupported color space (%d)\n", > + color_space); > +return -1; > [..] > +default: > +av_log(NULL, AV_LOG_ERROR,

Re: [FFmpeg-devel] [PATCH v2] Add support for vp9 in iso-bmff

2016-06-14 Thread KongQun Yang
Thanks for the review. Please see patchset v4 for the new change. (Please ignore patchset v3 which is uploaded incorrectly) -- KongQun Yang (KQ) On Tue, Jun 14, 2016 at 1:11 PM, Ronald S. Bultje wrote: > Hi, > > On Mon, Jun 13, 2016 at 5:26 PM, Kongqun Yang

Re: [FFmpeg-devel] [PATCH v5] Add experimental support for vp9 in iso-bmff

2016-06-14 Thread Hendrik Leppkes
On Wed, Jun 15, 2016 at 12:05 AM, Kongqun Yang wrote: > Implemented according to the draft specification > "VP Codec ISO Media File Format Binding": > http://www.webmproject.org/vp9/#draft-vp-codec-iso-media-file-format-binding > > '-strict -2' is required to use this

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

2016-06-14 Thread Dan Parrot
From e38eb7af05be27d8f36058373557d86e5a481db8 Mon Sep 17 00:00:00 2001 From: Dan Parrot Date: Tue, 14 Jun 2016 23:19:21 + Subject: [PATCH] PPC64: IBM POWER8 SIMD Implementation This is the first commit addressing Trac ticket #5570. Functions defined in

Re: [FFmpeg-devel] [PATCH v5] Add experimental support for vp9 in iso-bmff

2016-06-14 Thread Ronald S. Bultje
Hi, On Tue, Jun 14, 2016 at 7:34 PM, Hendrik Leppkes wrote: > On Wed, Jun 15, 2016 at 12:05 AM, Kongqun Yang > wrote: > > Implemented according to the draft specification > > "VP Codec ISO Media File Format Binding": > > >

Re: [FFmpeg-devel] [PATCH v5] Add experimental support for vp9 in iso-bmff

2016-06-14 Thread KongQun Yang
-- KongQun Yang (KQ) On Tue, Jun 14, 2016 at 4:20 PM, Ronald S. Bultje wrote: > Hi, > > On Tue, Jun 14, 2016 at 6:05 PM, Kongqun Yang > wrote: > >> +default: >> +av_log(NULL, AV_LOG_ERROR, "Unsupported color space (%d)\n", >> +

[FFmpeg-devel] [PATCH v3] Add support for vp9 in iso-bmff

2016-06-14 Thread Kongqun Yang
Implemented according to the draft specification "VP Codec ISO Media File Format Binding": http://www.webmproject.org/vp9/#draft-vp-codec-iso-media-file-format-binding Change-Id: Iaa7ddf5524b17e8d79cd1923b26f096d6e91 --- libavformat/Makefile | 2 +- libavformat/isom.c | 3 +

Re: [FFmpeg-devel] [PATCH] os_support: use the appropriate stat functions matching the stat type

2016-06-14 Thread Hendrik Leppkes
On Tue, Jun 14, 2016 at 10:06 PM, Michael Niedermayer wrote: > On Tue, Jun 14, 2016 at 05:09:01PM +0200, Hendrik Leppkes wrote: >> The stat struct is defined to stati64, which requires using the appropriate >> wstati/stati functions as well. >> Fixes a whole bunch of

Re: [FFmpeg-devel] [PATCH v5] Add experimental support for vp9 in iso-bmff

2016-06-14 Thread KongQun Yang
-- KongQun Yang (KQ) On Tue, Jun 14, 2016 at 4:34 PM, Hendrik Leppkes wrote: > On Wed, Jun 15, 2016 at 12:05 AM, Kongqun Yang > wrote: > > Implemented according to the draft specification > > "VP Codec ISO Media File Format Binding": > > >

[FFmpeg-devel] [PATCH v4] Add experimental support for vp9 in iso-bmff

2016-06-14 Thread Kongqun Yang
Implemented according to the draft specification "VP Codec ISO Media File Format Binding": http://www.webmproject.org/vp9/#draft-vp-codec-iso-media-file-format-binding '-strict -2' is required to use this feature. Change-Id: Iaa7ddf5524b17e8d79cd1923b26f096d6e91 --- libavformat/Makefile |

Re: [FFmpeg-devel] [PATCH] libopusenc: Add channel mapping family argument

2016-06-14 Thread Michael Niedermayer
On Fri, Jun 10, 2016 at 10:39:23PM -0700, Michael Graczyk wrote: > The libopus encoder has a parameter called "channel mapping family" which > is used to specify inter-channel relationships for the purpose of encoding. > I have added a new command line argument which makes it possible to forward >