Re: [FFmpeg-devel] [PATCH] lavf/os_support: Add safe win32 dlopen/dlclose/dlsym functions.

2016-11-01 Thread Stephen Hutchinson
On 10/30/2016 11:51 PM, Matt Oliver wrote: On 31 October 2016 at 05:30, James Almer wrote: On 10/30/2016 8:31 AM, Matt Oliver wrote: From 1dc1f59158cfb12d9160ee47342f5742d67ad864 Mon Sep 17 00:00:00 2001 From: Matt Oliver Date: Sun, 30 Oct 2016 15:13:47 +1100 Subject: [PATCH] avformat/avisyn

Re: [FFmpeg-devel] [PATCH 1/2] img2: added support for %t output pattern

2016-11-01 Thread James Almer
On 11/1/2016 6:43 PM, James Almer wrote: > On 10/25/2016 9:38 PM, Roger Pack wrote: >> From e8cac5c7de18766ce0f8f286f7dc140b82129df2 Mon Sep 17 00:00:00 2001 >> From: rogerdpack >> Date: Tue, 25 Oct 2016 18:33:12 -0600 >> Subject: [PATCH 1/2] img2 encoder: allow %t in filename, based on patch from

Re: [FFmpeg-devel] [PATCH] mov: add option to ignore moov atoms which are detected in free atoms, so apps can have flexibility to use moov atom not in free atoms as default.

2016-11-01 Thread Carl Eugen Hoyos
2016-10-31 21:33 GMT+01:00 Zhenni Huang : > Thanks Carl, I think setting strict_std_compliance is fine. Could please > you apply the change? Done. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpe

Re: [FFmpeg-devel] [OPW] OPW Project Proposal

2016-11-01 Thread Michael Niedermayer
On Wed, Nov 02, 2016 at 05:00:09AM +0530, Pallavi Kumari wrote: > Hi Michael, > > I have attached a working patch with the mail. PFA. > > Usage: > > ./ffmpeg -i kpg.mp3 -filter_complex peakpoints=input=kpg.mp3:wsize=16 I realize now, theres a mistake in this, you must provide a output as in ./

Re: [FFmpeg-devel] [PATCH]lavfi/mergeplanes: Fix little endian formats >8 bit.

2016-11-01 Thread Carl Eugen Hoyos
2016-10-30 17:15 GMT+01:00 Paul B Mahol : > OK, try with non 444 cases, like with 420 high bit depth and if it > works apply both patches. Tested and applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org

Re: [FFmpeg-devel] [PATCH] pixblockdsp: disable altivec optimizations on ppc64be

2016-11-01 Thread Carl Eugen Hoyos
2016-11-01 0:27 GMT+01:00 Andreas Cadhalpun : > -#if HAVE_ALTIVEC > +#if HAVE_ALTIVEC && !(ARCH_PPC64 && HAVE_BIGENDIAN) > if (!PPC_ALTIVEC(av_get_cpu_flags())) Do the dnxhd tests work for you on 32bit BE powerpc? (It fails here afaict) Carl Eugen ___

Re: [FFmpeg-devel] [OPW] OPW Project Proposal

2016-11-01 Thread Michael Niedermayer
On Wed, Nov 02, 2016 at 05:00:09AM +0530, Pallavi Kumari wrote: > Hi Michael, > > I have attached a working patch with the mail. PFA. > > Usage: > > ./ffmpeg -i kpg.mp3 -filter_complex peakpoints=input=kpg.mp3:wsize=16 > > > Regards, > Atana > > On Fri, Oct 28, 2016 at 4:38 AM, Michael Nieder

[FFmpeg-devel] [PATCH] avformat/mpegtsenc: Add option to mark stream begin as discontinuous

2016-11-01 Thread Michael Niedermayer
This avoids continuity check failures in concatenated streams TODO: docs Signed-off-by: Michael Niedermayer --- libavformat/mpegtsenc.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index 3ad3de7..71a6642 100644

Re: [FFmpeg-devel] [OPW] OPW Project Proposal

2016-11-01 Thread Pallavi Kumari
Hi Michael, I have attached a working patch with the mail. PFA. Usage: ./ffmpeg -i kpg.mp3 -filter_complex peakpoints=input=kpg.mp3:wsize=16 Regards, Atana On Fri, Oct 28, 2016 at 4:38 AM, Michael Niedermayer wrote: > On Thu, Oct 27, 2016 at 11:38:27PM +0530, Pallavi Kumari wrote: > > Hi Mi

Re: [FFmpeg-devel] [PATCH 2/2] Add experimental demuxing support for FLAC in ISO BMFF (MP4).

2016-11-01 Thread Matthew Gregan
At 2016-11-01T18:07:07-0300, James Almer wrote: > Just include libavcodec/flac.h which already defines it. > > With the above header you'll also be able to use the inline function > flac_parse_block_header() for this. See flac and matroska demuxers. > > INVALIDDATA errors would ideally also throw

Re: [FFmpeg-devel] [PATCH 1/2] Add experimental muxing support for FLAC in ISO BMFF (MP4).

2016-11-01 Thread Matthew Gregan
At 2016-11-01T00:56:45-0300, James Almer wrote: > The muxer gets the extradata as created by the encoder only during init(). > Even if the encoder changes it in any way afterwards, it will never make > it to the muxer. That's why packet side data is used for this. > > The reason the encoder used t

[FFmpeg-devel] [PATCH] lavc: Add hevc main10 profile to ffmpeg cli

2016-11-01 Thread Vittorio Giovara
--- Please CC. Vittorio libavcodec/options_table.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h index 995906b..4323ae9 100644 --- a/libavcodec/options_table.h +++ b/libavcodec/options_table.h @@ -394,6 +394,7 @@ static const AVOption

Re: [FFmpeg-devel] [PATCH] avformat/apngenc: use the stream parameters extradata if no updated one is made available

2016-11-01 Thread Hendrik Leppkes
Am 01.11.2016 17:17 schrieb "Andreas Cadhalpun" < andreas.cadhal...@googlemail.com>: > > On 01.11.2016 17:06, James Almer wrote: > > On 11/1/2016 12:54 PM, Andreas Cadhalpun wrote: > >> What I like about the approach of using the private extra_data context buffer is > >> that is is quite obvious wh

Re: [FFmpeg-devel] [PATCH 2/3] vf_colorspace: Add support for ycgco color space

2016-11-01 Thread Ronald S. Bultje
Hi, On Sun, Oct 30, 2016 at 9:14 AM, Ronald S. Bultje wrote: > Hi, > > On Sun, Oct 30, 2016 at 3:07 AM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > >> Signed-off-by: Vittorio Giovara >> --- >> This is a little hackish, not sure how to represent the matrix otherwise. > > > Haha :)

Re: [FFmpeg-devel] [PATCH 1/2] img2: added support for %t output pattern

2016-11-01 Thread James Almer
On 10/25/2016 9:38 PM, Roger Pack wrote: > From e8cac5c7de18766ce0f8f286f7dc140b82129df2 Mon Sep 17 00:00:00 2001 > From: rogerdpack > Date: Tue, 25 Oct 2016 18:33:12 -0600 > Subject: [PATCH 1/2] img2 encoder: allow %t in filename, based on patch from > Yuval Adam > > Signed-off-by: rogerdpack

Re: [FFmpeg-devel] [PATCH] vf_colorspace: Add support for film primaries

2016-11-01 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 2:49 PM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 31, 2016 at 2:43 PM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > >> Signed-off-by: Vittorio Giovara >> --- >> This is the last easy picking, the remaining trc/prm require more >> convoluted >> chang

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-11-01 Thread Ronald S. Bultje
Hi, On Tue, Nov 1, 2016 at 8:09 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 31, 2016 at 5:51 PM, Vittorio Giovara < > vittorio.giov...@gmail.com> wrote: > >> On Mon, Oct 31, 2016 at 4:08 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Mon, Oct 31, 2016 at 3:31 PM, Vittorio Giovara >>

Re: [FFmpeg-devel] [PATCH 1/2] img2: added support for %t output pattern

2016-11-01 Thread Michael Niedermayer
On Tue, Nov 01, 2016 at 11:23:47AM -0600, Roger Pack wrote: > On 10/26/16, Michael Niedermayer wrote: > > On Tue, Oct 25, 2016 at 06:38:12PM -0600, Roger Pack wrote: > >> On 10/25/16, Michael Niedermayer wrote: > >> > On Mon, Oct 24, 2016 at 03:20:14PM -0600, Roger Pack wrote: > >> >> On 10/16/16

Re: [FFmpeg-devel] [PATCH 2/2] Add experimental demuxing support for FLAC in ISO BMFF (MP4).

2016-11-01 Thread James Almer
On 10/31/2016 3:27 AM, Matthew Gregan wrote: > Based on the draft spec at > https://git.xiph.org/?p=flac.git;a=blob;f=doc/isoflac.txt > > Signed-off-by: Matthew Gregan > --- > libavformat/mov.c | 41 + > 1 file changed, 41 insertions(+) > > diff --git a/

[FFmpeg-devel] [PATCHv3 2/3] lavfi/loudnorm: add an internal libebur128 library

2016-11-01 Thread Marton Balint
Also contains the following changes to the library: - add ff_ prefix to functions - remove cplusplus defines. - add FF_ prefix to contants and some structs - remove true peak calculation feature, since it uses its own resampler, and af_loudnorm does not need it. - remove version info and some fpr

Re: [FFmpeg-devel] [PATCH] avcodec/dnxhdenc: Fix alignment of edge_buf*

2016-11-01 Thread Michael Niedermayer
On Tue, Nov 01, 2016 at 02:31:10AM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/dnxhdenc.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Many thi

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

2016-11-01 Thread Andreas Cadhalpun
On 01.11.2016 18:27, James Almer wrote: > On 11/1/2016 1:42 PM, Andreas Cadhalpun wrote: >> Signed-off-by: Andreas Cadhalpun >> --- >> >> To be applied after the fix. >> >> --- >> >> tests/lavf-regression.sh | 3 +++ >> tests/ref/lavf/apng | 3 +++ >> 2 files changed, 6 insertions(+) >> >> d

Re: [FFmpeg-devel] [PATCH] apngdec: use side data to pass extradata to the decoder

2016-11-01 Thread Andreas Cadhalpun
On 01.11.2016 18:26, James Almer wrote: > On 11/1/2016 1:18 PM, Andreas Cadhalpun wrote: >> Fixes remuxing apng streams coming from the apng demuxer. >> This is a regression since 940b8908b94404a65f9f55e33efb4ccc6c81383c. >> >> Found-by: James Almer >> Signed-off-by: Andreas Cadhalpun >> --- >>

Re: [FFmpeg-devel] [PATCH] mov: only read e_old if there were any old streams

2016-11-01 Thread Andreas Cadhalpun
On 01.11.2016 06:32, Sasi Inguva wrote: > patch looks good to me. Thanks for the fix. Pushed. Best regards, Andreas ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-11-01 Thread James Almer
On 11/1/2016 1:42 PM, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > > To be applied after the fix. > > --- > > tests/lavf-regression.sh | 3 +++ > tests/ref/lavf/apng | 3 +++ > 2 files changed, 6 insertions(+) > > diff --git a/tests/lavf-regression.sh b/tests/lavf-

Re: [FFmpeg-devel] [PATCH] apngdec: use side data to pass extradata to the decoder

2016-11-01 Thread James Almer
On 11/1/2016 1:18 PM, Andreas Cadhalpun wrote: > Fixes remuxing apng streams coming from the apng demuxer. > This is a regression since 940b8908b94404a65f9f55e33efb4ccc6c81383c. > > Found-by: James Almer > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/pngdec.c | 23 ++-- > l

Re: [FFmpeg-devel] [PATCH 1/2] img2: added support for %t output pattern

2016-11-01 Thread Roger Pack
On 10/26/16, Michael Niedermayer wrote: > On Tue, Oct 25, 2016 at 06:38:12PM -0600, Roger Pack wrote: >> On 10/25/16, Michael Niedermayer wrote: >> > On Mon, Oct 24, 2016 at 03:20:14PM -0600, Roger Pack wrote: >> >> On 10/16/16, Michael Niedermayer wrote: >> >> > On Mon, Oct 10, 2016 at 02:56:24

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the correct timescale when seeking for audio.

2016-11-01 Thread Derek Buitenhuis
On 11/1/2016 5:01 PM, Sasi Inguva wrote: > When we add up (last pts + 0.023 ) it amounts to that . We fix the last > packet duration according to edit list segment duration in our edit list > patch. Here last pts = 1324.652 . I figured this was the case. IIRC, the OGG code works like this to

Re: [FFmpeg-devel] [PATCH] fix warnings due to misleading indentation

2016-11-01 Thread Michael Niedermayer
On Tue, Nov 01, 2016 at 01:51:04PM +0100, Adriano Pallavicino wrote: > > libavcodec/mpegvideo.c | 54 > ++-- > libavfilter/vf_paletteuse.c |2 - > libavformat/matroskaenc.c | 20 > 3 files changed, 38 insertions(+), 38 dele

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the correct timescale when seeking for audio.

2016-11-01 Thread Sasi Inguva
Yes. This is expected. The audio has an edit list with duration 7294805 which at a timescale of 600 means 1324.675 . Entry[0].SegmentDuration 794805 Entry[0].MediaTime 2048 ~/ffmpeg_new/ffprobe -show_frames -print_format compact -select_streams 1 ~/Downloads/edit_list_sample.mp4 When we add up

[FFmpeg-devel] [PATCH] fate: add streamcopy test for apng

2016-11-01 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- To be applied after the fix. --- tests/lavf-regression.sh | 3 +++ tests/ref/lavf/apng | 3 +++ 2 files changed, 6 insertions(+) diff --git a/tests/lavf-regression.sh b/tests/lavf-regression.sh index 941f4d1..12954d5 100755 --- a/tests/lavf-regression

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: factor out sending frame to filters

2016-11-01 Thread Michael Niedermayer
On Tue, Nov 01, 2016 at 03:05:16PM +0100, Clément Bœsch wrote: > Video doesn't exit ffmpeg on error anymore, and audio now prints an > error. > --- > ffmpeg.c | 64 > ++-- > 1 file changed, 30 insertions(+), 34 deletions(-) i think this

[FFmpeg-devel] [PATCH] apngdec: use side data to pass extradata to the decoder

2016-11-01 Thread Andreas Cadhalpun
Fixes remuxing apng streams coming from the apng demuxer. This is a regression since 940b8908b94404a65f9f55e33efb4ccc6c81383c. Found-by: James Almer Signed-off-by: Andreas Cadhalpun --- libavcodec/pngdec.c | 23 ++-- libavformat/apngdec.c | 73 +++--

Re: [FFmpeg-devel] [PATCH] avformat/apngenc: use the stream parameters extradata if no updated one is made available

2016-11-01 Thread Andreas Cadhalpun
On 01.11.2016 17:06, James Almer wrote: > On 11/1/2016 12:54 PM, Andreas Cadhalpun wrote: >> What I like about the approach of using the private extra_data context >> buffer is >> that is is quite obvious where it is set. On the other hand the codecpar >> extradata can be set anywhere, which makes

Re: [FFmpeg-devel] Fate : add metadata test for aphasemeter

2016-11-01 Thread Michael Niedermayer
On Mon, Oct 24, 2016 at 11:06:13PM +0200, Martin Vignali wrote: > Hello, > > In attach a patch to add 2 test for aphasemeter metadata > > The sample can be found here : > > https://we.tl/AP7SY8eNSe > > As aphasemeter use float, i choose two "extreme" cases : > - a mono file, with result of 1. >

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-11-01 Thread Steven Liu
Rostislav Pehlivanov 于2016年11月1日 周二下午11:09写道: > On 1 November 2016 at 02:00, Steven Liu wrote: > > > 2016-11-01 3:16 GMT+08:00 Jan Ekstrom : > > > > > On Mon, Oct 31, 2016 at 5:30 PM, Nicolas George > wrote: > > > > Le nonidi 9 brumaire, an CCXXV, Steven Liu a écrit : > > > >> I saw ffmpeg

Re: [FFmpeg-devel] [PATCH] avformat/apngenc: use the stream parameters extradata if no updated one is made available

2016-11-01 Thread James Almer
On 11/1/2016 12:54 PM, Andreas Cadhalpun wrote: > On 01.11.2016 05:09, James Almer wrote: >> On 10/31/2016 11:32 PM, James Almer wrote: >>> Fixes remuxing apng streams coming from the apng demuxer. >>> This is a regression since 97792e85c338d129342f5812e2a52048373e57d6. >>> >>> Signed-off-by: James

Re: [FFmpeg-devel] [PATCH] avformat/apngenc: use the stream parameters extradata if no updated one is made available

2016-11-01 Thread Andreas Cadhalpun
On 01.11.2016 05:09, James Almer wrote: > On 10/31/2016 11:32 PM, James Almer wrote: >> Fixes remuxing apng streams coming from the apng demuxer. >> This is a regression since 97792e85c338d129342f5812e2a52048373e57d6. >> >> Signed-off-by: James Almer >> --- >> libavformat/apngenc.c | 7 +++ >>

Re: [FFmpeg-devel] [PATCH] add hds demuxer

2016-11-01 Thread Rostislav Pehlivanov
On 1 November 2016 at 02:00, Steven Liu wrote: > 2016-11-01 3:16 GMT+08:00 Jan Ekstrom : > > > On Mon, Oct 31, 2016 at 5:30 PM, Nicolas George wrote: > > > Le nonidi 9 brumaire, an CCXXV, Steven Liu a écrit : > > >> I saw ffmpeg have no HDS and DASH demuxer, and all of them's > > format is

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-11-01 Thread Michael Niedermayer
On Tue, Nov 01, 2016 at 02:45:00AM -0300, James Almer wrote: > On 10/31/2016 3:30 PM, Nicolas George wrote: > > Le decadi 10 brumaire, an CCXXV, James Almer a écrit : [...] > > > > * What benefits do you see for the separate tool approach? > > > > * Can you negate any of these drawbacks? > > I'll

[FFmpeg-devel] [PATCH 2/2] ffmpeg: factor out sending frame to filters

2016-11-01 Thread Clément Bœsch
Video doesn't exit ffmpeg on error anymore, and audio now prints an error. --- ffmpeg.c | 64 ++-- 1 file changed, 30 insertions(+), 34 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index 7e366f2..36a921a 100644 --- a/ffmpeg.c +++ b/ffmp

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Use the correct timescale when seeking for audio.

2016-11-01 Thread Derek Buitenhuis
On 10/31/2016 1:48 PM, Rostislav Pehlivanov wrote: > The tests pass on my machine, pushed after Derek said it looks correct. Hi Sasi, I notice there's still one difference after this patch. Could you explain if this is intended: -"duration": 3072, -"duration_time"

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: factor out sending frame to filters

2016-11-01 Thread Michael Niedermayer
On Tue, Nov 01, 2016 at 10:55:31AM +0100, Clément Bœsch wrote: > On Mon, Oct 24, 2016 at 11:05:07PM +0200, Clément Bœsch wrote: > > Video doesn't exit ffmpeg on error anymore, and audio now prints an > > error. > > --- > > ffmpeg.c | 64 > > ++--

[FFmpeg-devel] [PATCH] fix warnings due to misleading indentation

2016-11-01 Thread Adriano Pallavicino
From 796426819c0dfb41666a39e94dca78db20d69ee9 Mon Sep 17 00:00:00 2001 From: Adriano Pallavicino Date: Tue, 1 Nov 2016 13:42:27 +0100 Subject: [PATCH] Fix build warnings due to misleading indentation Signed-off-by: Adriano Pallavicino Changes to be committed: modified: libavcodec/mpegvideo

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-11-01 Thread Stefano Sabatini
On date Tuesday 2016-11-01 02:45:00 -0300, James Almer encoded: > On 10/31/2016 3:30 PM, Nicolas George wrote: > > Le decadi 10 brumaire, an CCXXV, James Almer a écrit : [...] > > * What benefits do you see for the separate tool approach? > > > > * Can you negate any of these drawbacks? > > I'll

Re: [FFmpeg-devel] [PATCH 3/3] vf_colorspace: Add support for smpte 431/432 (dci/display p3) primaries

2016-11-01 Thread Ronald S. Bultje
Hi, On Mon, Oct 31, 2016 at 5:51 PM, Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > On Mon, Oct 31, 2016 at 4:08 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Mon, Oct 31, 2016 at 3:31 PM, Vittorio Giovara > > wrote: > >> > >> On Mon, Oct 31, 2016 at 2:53 PM, Ronald S. Bultje > >>

Re: [FFmpeg-devel] [PATCH 2/2] ffmpeg: factor out sending frame to filters

2016-11-01 Thread Clément Bœsch
On Mon, Oct 24, 2016 at 11:05:07PM +0200, Clément Bœsch wrote: > Video doesn't exit ffmpeg on error anymore, and audio now prints an > error. > --- > ffmpeg.c | 64 > ++-- > 1 file changed, 30 insertions(+), 34 deletions(-) > ping (no

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: move some unrelated code out of a filter loop

2016-11-01 Thread Clément Bœsch
On Sat, Oct 29, 2016 at 05:35:41PM +0200, Andreas Cadhalpun wrote: > On 29.10.2016 12:03, Clément Bœsch wrote: > > On Mon, Oct 24, 2016 at 11:05:06PM +0200, Clément Bœsch wrote: > >> --- > >> Not sure if the chunk is even needed > > I'm not sure either. > Now that I look a bit above, I see:

[FFmpeg-devel] [PATCH] ffmpeg: remove dead code out of the video filter loop

2016-11-01 Thread Clément Bœsch
It makes no sense for this code to be inside the video filter loop, and the frame sample aspect ratio is even set unconditionally a few lines above. --- ffmpeg.c | 5 - 1 file changed, 5 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index 3b91710..7e366f2 100644 --- a/ffmpeg.c +++ b/ffmpeg.c