[FFmpeg-devel] [PATCH 3/4] avformat/mov: fix audio last packet durations

2016-03-20 Thread Marton Balint
Stream duration might not be reliable, let's not use if the frame size is known. Signed-off-by: Marton Balint --- libavformat/mov.c| 2 ++ tests/ref/fate/gaplessenc-itunes-to-ipod-aac | 4 ++-- tests/ref/fate/gaplessenc-pcm-to-mov-aac | 4 ++-- 3 files changed, 6

[FFmpeg-devel] [PATCH 4/4] avformat/mov: add support for reading end padding from gapless itunes metadata

2016-03-20 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mov.c| 13 ++--- tests/ref/fate/gaplessenc-itunes-to-ipod-aac | 12 ++-- tests/ref/fate/gaplessinfo-itunes1 | 2 +- tests/ref/fate/gaplessinfo-itunes2 | 2 +- 4 files changed, 18 inserti

[FFmpeg-devel] [PATCH 2/4] avcodec/utils: fix packet duration of frames with discarded paddings

2016-03-20 Thread Marton Balint
Signed-off-by: Marton Balint --- libavcodec/utils.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index c625bbc..073c6fa 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -2337,8 +2337,7 @@ int attribute_align_arg avcodec_

[FFmpeg-devel] [PATCH 1/4] avformat/mov: set start_pad in samples and not in stream time_scale

2016-03-20 Thread Marton Balint
This usually does not matter, probably better this way. Signed-off-by: Marton Balint --- libavformat/mov.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 3fa8fcc..ee3fb96 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Gerion Entrup
On Sonntag, 20. März 2016 16:52:36 CET Clément Bœsch wrote: > On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote: > > Good day, > > > > I attached the patch. The MPEG7 video signature is a method to calculate a > > fingerprint of a video and to compare two or more of this fingerprints.

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Gerion Entrup
On Sonntag, 20. März 2016 17:01:17 CET Thilo Borgmann wrote: > > On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote: > [...] > > > This filter does not implement all features of MPEG7. Missing features: > > > > - binary output > > - compression of signature files > > I assume these fe

Re: [FFmpeg-devel] [PATCH] lavf/matroskaenc.c: add early support for colour elements

2016-03-20 Thread Michael Niedermayer
On Mon, Mar 07, 2016 at 02:42:47PM -0800, Neil Birkbeck wrote: > Adding early support for a subset of the proposed colour elements > according to the latest version of spec: > https://mailarchive.ietf.org/arch/search/?email_list=cellar&gbt=1&index=hIKLhMdgTMTEwUTeA4ct38h0tmE > > Like matroskadec,

Re: [FFmpeg-devel] [PATCH] Added a selftest to libavutil/display.c

2016-03-20 Thread Michael Niedermayer
On Sun, Mar 20, 2016 at 09:29:04AM +, Petru Rares Sincraian wrote: > > - Check if av_display_rotation_get() gets the correct degrees > - Check if av_display_rotation_set() sets the correct matrix > - Check if av_display_matrix_flip() changes correct the matrix > --- > libavutil/Ma

[FFmpeg-devel] [PATCH 2/2] fate: Remove vsync drop from some h264 tests

2016-03-20 Thread Michael Niedermayer
Note some tests need vsync drop to produce exact timestamps, these seem not to need it. quite likely many more dont need it either, ive not checked beyond finding one that needs it and the ones which have it removed Signed-off-by: Michael Niedermayer --- tests/fate/h264.mak | 32 +

[FFmpeg-devel] [PATCH 1/2] fate: Modify a random h264 test to also test the -framerate option

2016-03-20 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- tests/fate/h264.mak |5 - tests/ref/fate/h264-conformance-ba1_ft_c |2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/tests/fate/h264.mak b/tests/fate/h264.mak index 164fc9e..d1a0165 100644 --- a/tests/fate

Re: [FFmpeg-devel] [PATCHv5 3/3] avformat/mov: read start_pad from edit list start time if codec is aac

2016-03-20 Thread Marton Balint
On Sat, 19 Mar 2016, Marton Balint wrote: On Mon, 14 Mar 2016, Marton Balint wrote: On Sun, 13 Mar 2016, Marton Balint wrote: Related to ticket #2324, #2325. Stream duration still need to be fixed... Signed-off-by: Marton Balint --- libavformat/mov.c| 3 +++ t

Re: [FFmpeg-devel] [PATCH] lavf/dv: use c->sys->frame_size in dv_frame_offset()

2016-03-20 Thread Clément Bœsch
On Sun, Mar 20, 2016 at 07:18:43PM +0100, Michael Niedermayer wrote: > On Sun, Mar 20, 2016 at 04:20:32PM +0100, Clément Bœsch wrote: > > dv_frame_offset() is static and called only from dv_read_seek(), where > > c->sys->frame_size is already used. > > > > This simplifies the incoming codecpar mer

Re: [FFmpeg-devel] [PATCH] lavf/dv: do not check for c->sys

2016-03-20 Thread Clément Bœsch
On Sun, Mar 20, 2016 at 03:09:51PM +0100, Michael Niedermayer wrote: > On Sun, Mar 20, 2016 at 02:28:05PM +0100, Clément Bœsch wrote: > > dv_extract_video_info() is a static function called only from > > avpriv_dv_produce_packet(), where c->sys is made sure to be set. > > --- > > libavformat/dv.c

[FFmpeg-devel] [PATCH] Added a selftest to libavutil/display.c

2016-03-20 Thread Petru Rares Sincraian
- Check if av_display_rotation_get() gets the correct degrees - Check if av_display_rotation_set() sets the correct matrix - Check if av_display_matrix_flip() changes correct the matrix --- libavutil/Makefile | 1 + libavutil/display.c | 47

Re: [FFmpeg-devel] [PATCH] lavf/dv: use c->sys->frame_size in dv_frame_offset()

2016-03-20 Thread Michael Niedermayer
On Sun, Mar 20, 2016 at 04:20:32PM +0100, Clément Bœsch wrote: > dv_frame_offset() is static and called only from dv_read_seek(), where > c->sys->frame_size is already used. > > This simplifies the incoming codecpar merge where > avctx->{coded_width,coded_height,time_base} are not accessible anymo

Re: [FFmpeg-devel] [PATCH 3/3] movenc: simplify the 'tfxd' fragment start timestamps

2016-03-20 Thread Jan Ekstrom
On Sun, Mar 20, 2016 at 3:28 PM, Michael Niedermayer wrote: > On Sat, Mar 19, 2016 at 07:39:07PM +0200, Jan Ekström wrote: >> As far as can be seen, this value is supposed to be the DTS of a >> fragment in smooth streaming. Thus, don't take b-picture delay and >> such into mention when calculating

[FFmpeg-devel] [PATCH] lavc/videotoolboxenc: Fix crash when closing codec after error

2016-03-20 Thread Rick Kern
Fixes crash in #5352. VTCompressionSessionInvalidate() crashes if the internal encoder hasn't completed, but hasn't experienced an error. The function call isn't needed since the encoder is invalidated when the reference count reaches 0 anyway. Signed-off-by: Rick Kern --- libavcodec/videotoo

Re: [FFmpeg-devel] [PATCH] lavc/videotoolbox: Consistent fallback to external software encoding/decoding.

2016-03-20 Thread Richard Kern
> On Mar 20, 2016, at 11:49 PM, Thilo Borgmann wrote: > > Hi, > > trying to handle software fallback more consistently for videotoolbox and > probably other hardware accelerations. > > Addresses ticket #5352 where software fallback is demanded which has been > removed on purpose before. With t

Re: [FFmpeg-devel] [PATCH] added support for hardware assist H264 video encoding for the Raspberry Pi

2016-03-20 Thread Lou Logan
On Fri, 18 Mar 2016 12:12:24 -0700, Amancio Hasty wrote: > The patch was against: > 2016-02-26-raspbian-jessie.img > > In my Raspberry Pi system: > /opt/vc/include/bcm_host.h > > in ffmpeg/configure: > if enabled vc264_encoder && [ -n "$target_os_default"="linux" ] ; then > add_cflags

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Thilo Borgmann
> On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote: [...] > This filter does not implement all features of MPEG7. Missing features: > - binary output > - compression of signature files I assume these features are optional? > - work only on (cropped) parts of the video How useful i

Re: [FFmpeg-devel] [PATCH] lavc/videotoolbox: Consistent fallback to external software encoding/decoding.

2016-03-20 Thread Carl Eugen Hoyos
Thilo Borgmann mail.de> writes: > h264_videotoolbox_encoder_deps="videotoolbox_encoder pthreads" > -h264_videotoolbox_encoder_select="bzlib zlib iconv" > +#h264_videotoolbox_encoder_select="bzlib zlib iconv" If nobody beats me (or objects) I'll remove this line: Compilation succeeds without zli

Re: [FFmpeg-devel] [PATCH] lavc/videotoolbox: Consistent fallback to external software encoding/decoding.

2016-03-20 Thread Thilo Borgmann
Am 20.03.16 um 16:49 schrieb Thilo Borgmann: > Hi, > > trying to handle software fallback more consistently for videotoolbox and > probably other hardware accelerations. > > Addresses ticket #5352 where software fallback is demanded which has been > removed on purpose before. With this patch the

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Clément Bœsch
On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote: > Good day, > > I attached the patch. The MPEG7 video signature is a method to calculate a > fingerprint of a video and to compare two or more of this fingerprints. Most > of > the standard is implemented. I've done this work as par

[FFmpeg-devel] [PATCH] lavc/videotoolbox: Consistent fallback to external software encoding/decoding.

2016-03-20 Thread Thilo Borgmann
Hi, trying to handle software fallback more consistently for videotoolbox and probably other hardware accelerations. Addresses ticket #5352 where software fallback is demanded which has been removed on purpose before. With this patch the user can configure the desired behaviour. -Thilo From b7d6

[FFmpeg-devel] [PATCH] lavf/dv: use c->sys->frame_size in dv_frame_offset()

2016-03-20 Thread Clément Bœsch
dv_frame_offset() is static and called only from dv_read_seek(), where c->sys->frame_size is already used. This simplifies the incoming codecpar merge where avctx->{coded_width,coded_height,time_base} are not accessible anymore. --- libavformat/dv.c | 7 +++ 1 file changed, 3 insertions(+), 4

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Gerion Entrup
On Sonntag, 20. März 2016 15:46:12 CET Gerion Entrup wrote: > On Sonntag, 20. März 2016 12:57:33 CET Michael Niedermayer wrote: > > On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote: > > [...] > > > > > +++ b/libavfilter/signature.h > > > @@ -0,0 +1,569 @@ > > > +/* > > > + * Copyright

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc/dxva2_h264: Fix incorrect assert statement.

2016-03-20 Thread Matt Oliver
On 16 March 2016 at 23:10, Hendrik Leppkes wrote: > On Wed, Mar 16, 2016 at 1:04 PM, wm4 wrote: > > On Wed, 16 Mar 2016 22:55:09 +1100 > > Matt Oliver wrote: > > > >> On 16 March 2016 at 22:32, Clément Bœsch wrote: > >> > >> > On Wed, Mar 16, 2016 at 12:31:35PM +0100, Matt Oliver wrote: > >> >

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Gerion Entrup
On Sonntag, 20. März 2016 12:57:33 CET Michael Niedermayer wrote: > On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote: > [...] > > > +++ b/libavfilter/signature.h > > @@ -0,0 +1,569 @@ > > +/* > > + * Copyright (c) 2016 Gerion Entrup > > + * > > + * This file is part of FFmpeg. > > + *

Re: [FFmpeg-devel] [PATCH] lavf/dv: do not check for c->sys

2016-03-20 Thread Michael Niedermayer
On Sun, Mar 20, 2016 at 02:28:05PM +0100, Clément Bœsch wrote: > dv_extract_video_info() is a static function called only from > avpriv_dv_produce_packet(), where c->sys is made sure to be set. > --- > libavformat/dv.c | 6 +- > 1 file changed, 1 insertion(+), 5 deletions(-) should be ok thx

Re: [FFmpeg-devel] [PATCH 3/3] movenc: simplify the 'tfxd' fragment start timestamps

2016-03-20 Thread Michael Niedermayer
On Sat, Mar 19, 2016 at 07:39:07PM +0200, Jan Ekström wrote: > As far as can be seen, this value is supposed to be the DTS of a > fragment in smooth streaming. Thus, don't take b-picture delay and > such into mention when calculating the start timestamp. The duration > calculation requires PTS valu

[FFmpeg-devel] [PATCH] lavf/dv: do not check for c->sys

2016-03-20 Thread Clément Bœsch
dv_extract_video_info() is a static function called only from avpriv_dv_produce_packet(), where c->sys is made sure to be set. --- libavformat/dv.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/libavformat/dv.c b/libavformat/dv.c index 84c3061..b1f8759 100644 --- a/libav

[FFmpeg-devel] [PATCH] ffplay: remove redundant silence buffer

2016-03-20 Thread Lukasz Marek
Signed-off-by: Lukasz Marek --- ffplay.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/ffplay.c b/ffplay.c index 2cfdf26..731ae25 100644 --- a/ffplay.c +++ b/ffplay.c @@ -243,7 +243,6 @@ typedef struct VideoState { AVStream *audio_st; PacketQueue audi

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Michael Niedermayer
On Sun, Mar 20, 2016 at 12:00:13PM +0100, Gerion Entrup wrote: [...] > +++ b/libavfilter/signature.h > @@ -0,0 +1,569 @@ > +/* > + * Copyright (c) 2016 Gerion Entrup > + * > + * This file is part of FFmpeg. > + * > + * FFmpeg is free software; you can redistribute it and/or modify > + * it under th

Re: [FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Paul B Mahol
On 3/20/16, Gerion Entrup wrote: > Good day, > > I attached the patch. The MPEG7 video signature is a method to calculate a > fingerprint of a video and to compare two or more of this fingerprints. Most > of > the standard is implemented. I've done this work as part of my bachelor > thesis. > > Th

[FFmpeg-devel] [PATCH] add signature filter for MPEG7 video signature

2016-03-20 Thread Gerion Entrup
Good day, I attached the patch. The MPEG7 video signature is a method to calculate a fingerprint of a video and to compare two or more of this fingerprints. Most of the standard is implemented. I've done this work as part of my bachelor thesis. The method is designed mainly for fast caculation

Re: [FFmpeg-devel] libavcodec/exr : add B44 and B44A compression

2016-03-20 Thread Martin Vignali
2016-03-20 9:31 GMT+01:00 Carl Eugen Hoyos : > Martin Vignali gmail.com> writes: > > > > > The B44/B44A block decoding function (unpack_14 and unpack_3) > > > > come from the official library. > > > > > > Then please add an appropriate copyright notice to your patch. > > > What form the copyright

Re: [FFmpeg-devel] [PATCH 1/3] configure: Force mingw's ld to keep the reloc section

2016-03-20 Thread Christophe Gisquet
Hi, 2016-03-19 19:08 GMT+01:00 Ismail Donmez : >> 2016-03-11 8:57 GMT+01:00 Christophe Gisquet : It should either be reverted or made dependent on --enable/disable-debug (I would favor the first, honestly, since its a rather ugly hack in itself). [...] >> I don't have a strong opin

Re: [FFmpeg-devel] [PATCH] ffplay: Remove "&& 0" from already disabled debug code

2016-03-20 Thread Michael Niedermayer
On Sat, Mar 19, 2016 at 07:09:01PM +0100, Marton Balint wrote: > On Fri, 18 Mar 2016, Michael Niedermayer wrote: > > >Signed-off-by: Michael Niedermayer > >--- > >ffplay.c |2 +- > >1 file changed, 1 insertion(+), 1 deletion(-) > > > >diff --git a/ffplay.c b/ffplay.c > >index 2cfdf26..ade5159

Re: [FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

2016-03-20 Thread Thilo Borgmann
Am 16.03.16 um 22:24 schrieb Thilo Borgmann: > Am 16.03.16 um 21:10 schrieb Moritz Barsnick: >> [...] > > All addressed in newest patch, thanks! Any more comments from anyone? @Clement: Do you like it better this way? -Thilo ___ ffmpeg-devel mailing l

Re: [FFmpeg-devel] libavcodec/exr : add B44 and B44A compression

2016-03-20 Thread Carl Eugen Hoyos
Martin Vignali gmail.com> writes: > > > The B44/B44A block decoding function (unpack_14 and unpack_3) > > > come from the official library. > > > > Then please add an appropriate copyright notice to your patch. > What form the copyright need to have ? and where i need to put it ? The easy way i

[FFmpeg-devel] My own project idea for GSOC 2016

2016-03-20 Thread Pawan Pal
Hi, I am 3rd year undergraduate student pursuing B.Tech in Information Technology and Mathematical innovation from Cluster Innovation Centre, University of Delhi. I have been developing android application from 2 years and a good experience on various libraries. This is my github

[FFmpeg-devel] [PATCH 1/2] Fix detelecine filter for patterns containing 1

2016-03-20 Thread Benjamin Steffes
Signed-off-by: Benjamin Steffes --- libavfilter/vf_detelecine.c | 78 ++--- 1 file changed, 53 insertions(+), 25 deletions(-) diff --git a/libavfilter/vf_detelecine.c b/libavfilter/vf_detelecine.c index f5ae350..16230f1 100644 --- a/libavfilter/vf_deteleci

Re: [FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

2016-03-20 Thread Clément Bœsch
On Wed, Mar 16, 2016 at 10:24:09PM +0100, Thilo Borgmann wrote: [...] Not sure if I'm commenting on the last version... > From b6b889c06edc7872f0a31fd0482793a199ddd28e Mon Sep 17 00:00:00 2001 > From: Thilo Borgmann > Date: Wed, 16 Mar 2016 22:18:20 +0100 > Subject: [PATCH 2/2] lavf: Add coreim

Re: [FFmpeg-devel] [PATCH 1/3] ffmpeg: set tty mode and signals after parsing options.

2016-03-20 Thread Michael Niedermayer
On Wed, Mar 16, 2016 at 11:01:02PM +0100, Nicolas George wrote: > Le septidi 27 ventôse, an CCXXIV, Michael Niedermayer a écrit : > > iam not saying its necessary (or that its not), just that its done > > currently and would not be done anymore in the future. > > But what? What is done now and not

Re: [FFmpeg-devel] [PATCH 1/2] libwebpenc_animencoder: zero initialize the WebPAnimEncoderOptions struct

2016-03-20 Thread James Almer
On 3/17/2016 1:20 PM, wm4 wrote: > On Thu, 17 Mar 2016 13:15:23 -0300 > James Almer wrote: > >> On 3/17/2016 1:02 PM, Michael Niedermayer wrote: >>> On Thu, Mar 17, 2016 at 11:06:29AM -0300, James Almer wrote: On 3/17/2016 4:32 AM, wm4 wrote: > On Thu, 17 Mar 2016 01:03:49 -0300

Re: [FFmpeg-devel] [PATCH 2/2] lavf: Add coreimage filter for GPU based image filtering on OSX.

2016-03-20 Thread Thilo Borgmann
Am 16.03.16 um 14:44 schrieb Thilo Borgmann: > Am 16.03.16 um 14:43 schrieb Thilo Borgmann: >> Am 12.03.16 um 15:14 schrieb Thilo Borgmann: >>> Add coreimage filter for OSX. >> >> Included all comments so far. >> Also features a "coreimagesrc" video source like suggested by Nicolas. > > Actually s