Re: [FFmpeg-devel] [PATCH 3/4] libavdevice/avfoundation: add capture_screen_mouse_clicks option

2015-03-18 Thread Matthieu Bouron
On Fri, Mar 13, 2015 at 8:16 PM, matthieu.bou...@gmail.com wrote: From: Matthieu Bouron matthieu.bou...@gmail.com Support mouse clicks capture while recording a screen on OSX = 10.7. --- libavdevice/avfoundation.m | 15 +++ 1 file changed, 15 insertions(+) diff --git

Re: [FFmpeg-devel] [PATCH 2/4] libavdevice/avfoundation: add capture_screen_cursor option

2015-03-18 Thread Matthieu Bouron
On Sat, Mar 14, 2015 at 11:11 AM, Matthieu Bouron matthieu.bou...@gmail.com wrote: On Sat, Mar 14, 2015 at 10:50 AM, Matthieu Bouron matthieu.bou...@gmail.com wrote: On Sat, Mar 14, 2015 at 8:29 AM, Carl Eugen Hoyos ceho...@ag.or.at wrote: matthieu.bouron at gmail.com writes:

Re: [FFmpeg-devel] [PATCH 4/4] libavdevice/avfoundation: use pts/dts provided by the CMSampleBuffer API

2015-03-18 Thread Matthieu Bouron
On Fri, Mar 13, 2015 at 8:16 PM, matthieu.bou...@gmail.com wrote: From: Matthieu Bouron matthieu.bou...@gmail.com --- libavdevice/avfoundation.m | 21 +++-- 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/libavdevice/avfoundation.m

Re: [FFmpeg-devel] [PATCH 1/4] libavdevice/avfoundation: add framerate and video size options

2015-03-18 Thread Matthieu Bouron
On Sat, Mar 14, 2015 at 11:09 AM, Matthieu Bouron matthieu.bou...@gmail.com wrote: On Sat, Mar 14, 2015 at 11:01 AM, Matthieu Bouron matthieu.bou...@gmail.com wrote: On Sat, Mar 14, 2015 at 8:31 AM, Carl Eugen Hoyos ceho...@ag.or.at wrote: matthieu.bouron at gmail.com writes: +

[FFmpeg-devel] [PATCH]Do not set bitrate for DTS-HD Master and High Resolution

2015-03-18 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4375 for me. Please comment, Carl Eugen diff --git a/libavcodec/dcadec.c b/libavcodec/dcadec.c index 548a5f9..0339eef 100644 --- a/libavcodec/dcadec.c +++ b/libavcodec/dcadec.c @@ -1472,7 +1472,6 @@ static int dca_decode_frame(AVCodecContext *avctx, void *data,

Re: [FFmpeg-devel] [PATCH] avfilter/showwaves: add single_pic option

2015-03-18 Thread Stefano Sabatini
On date Thursday 2014-12-25 23:57:47 +0100, Clément Bœsch encoded: On Thu, Dec 25, 2014 at 09:07:04PM +0100, Lars Kiesow wrote: [...] One improvement that's left is to choose the colors, but that's something for another patch and you can archive that quite easily with some of the other

[FFmpeg-devel] [PATCH] mips/float_dsp: fix vector_fmul_window_mips on mips64

2015-03-18 Thread James Cowgill
Commit dfa920807494 (mips/float_dsp: fix a bug in vector_fmul_window_mips) fixed vector_fmul_window_mips by unrolling the loop only 4 times, but also removed the outer C loop and replaced it with assembly branches and pointer arithmetic. When submitting my 64-bit porting patch I missed this new

Re: [FFmpeg-devel] [PATCH]Do not set bitrate for DTS-HD Master and High Resolution

2015-03-18 Thread Hendrik Leppkes
On Wed, Mar 18, 2015 at 1:06 PM, Michael Niedermayer michae...@gmx.at wrote: On Wed, Mar 18, 2015 at 07:49:46AM -0400, Marcus Johnson wrote: This reminds me of another bug with DTS files, it estimates the file duration by counting each frame I assume, including the HD ones resulting in it

Re: [FFmpeg-devel] [PATCH]Do not set bitrate for DTS-HD Master and High Resolution

2015-03-18 Thread Marcus Johnson
This reminds me of another bug with DTS files, it estimates the file duration by counting each frame I assume, including the HD ones resulting in it being massively incorrect for example here's the ffmpeg output of a DTS-HD MA file that's actually 98 minutes long Log: ffmpeg -i

Re: [FFmpeg-devel] [PATCH]Do not set bitrate for DTS-HD Master and High Resolution

2015-03-18 Thread Michael Niedermayer
On Wed, Mar 18, 2015 at 07:49:46AM -0400, Marcus Johnson wrote: This reminds me of another bug with DTS files, it estimates the file duration by counting each frame I assume, including the HD ones resulting in it being massively incorrect for example here's the ffmpeg output of a DTS-HD MA

Re: [FFmpeg-devel] [PATCH]Do not set bitrate for DTS-HD Master and High Resolution

2015-03-18 Thread Michael Niedermayer
On Wed, Mar 18, 2015 at 01:21:25PM +0100, Hendrik Leppkes wrote: On Wed, Mar 18, 2015 at 1:06 PM, Michael Niedermayer michae...@gmx.at wrote: On Wed, Mar 18, 2015 at 07:49:46AM -0400, Marcus Johnson wrote: This reminds me of another bug with DTS files, it estimates the file duration by

Re: [FFmpeg-devel] [PATCH] avfilter/showwaves: add single_pic option

2015-03-18 Thread Stefano Sabatini
On date Tuesday 2015-03-17 21:23:27 +0100, Clément Bœsch encoded: On Mon, Mar 16, 2015 at 04:11:24PM +0100, Stefano Sabatini wrote: [...] From 5ecb0190806d39bdd869de951634000b1d1e1c0f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Cl=C3=A9ment=20B=C5=93sch?= u...@pkh.me Date: Wed, 24 Dec 2014

Re: [FFmpeg-devel] [PATCH v2] mov: Add option to keep exact packet sequence after seeking

2015-03-18 Thread Derek Buitenhuis
On 3/17/2015 2:19 PM, Derek Buitenhuis wrote: [...] --- libavformat/isom.h| 1 + libavformat/mov.c | 46 +++--- libavformat/version.h | 4 ++-- 3 files changed, 38 insertions(+), 13 deletions(-) Ping. -Derek

Re: [FFmpeg-devel] [PATCH v2] mov: Add option to keep exact packet sequence after seeking

2015-03-18 Thread Michael Niedermayer
On Wed, Mar 18, 2015 at 01:28:05PM +, Derek Buitenhuis wrote: On 3/17/2015 2:19 PM, Derek Buitenhuis wrote: [...] --- libavformat/isom.h| 1 + libavformat/mov.c | 46 +++--- libavformat/version.h | 4 ++-- 3 files changed, 38

Re: [FFmpeg-devel] [PATCH v2] mov: Add option to keep exact packet sequence after seeking

2015-03-18 Thread Michael Niedermayer
On Wed, Mar 18, 2015 at 02:04:29PM +, Derek Buitenhuis wrote: On 3/18/2015 2:01 PM, Michael Niedermayer wrote: LGTM but like the previous version i did not verify that it actually produces the same sequence of packets, i dont think its critical to verify that ... I've verified this

[FFmpeg-devel] Patch for libavcodec/exr.c implementing B44 compression technique

2015-03-18 Thread greeshma
The .diff file is also attached henceforth.Please have a look. diff --git a/libavcodec/exr.c b/libavcodec/exr.c index 6251fb7..cc15a3a 100644 --- a/libavcodec/exr.c +++ b/libavcodec/exr.c @@ -770,6 +770,163 @@ static int piz_uncompress(EXRContext *s, const uint8_t *src, int ssize, return 0;

Re: [FFmpeg-devel] [PATCH]Do not set bitrate for DTS-HD Master and High Resolution

2015-03-18 Thread Marcus Johnson
I see, I thought it counted frames and not just multiplied the bitrate by the number of frames. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] mips/float_dsp: fix vector_fmul_window_mips on mips64

2015-03-18 Thread Nedeljko Babic
LGTM Thanks, - Nedeljko Od: ffmpeg-devel-boun...@ffmpeg.org [ffmpeg-devel-boun...@ffmpeg.org] u ime korisnika James Cowgill [james...@cowgill.org.uk] Poslato: 18. mart 2015 14:02 Za: ffmpeg-devel@ffmpeg.org Cc: Nedeljko Babic; James Cowgill Tema:

Re: [FFmpeg-devel] [PATCH v2] mov: Add option to keep exact packet sequence after seeking

2015-03-18 Thread Derek Buitenhuis
On 3/18/2015 2:01 PM, Michael Niedermayer wrote: LGTM but like the previous version i did not verify that it actually produces the same sequence of packets, i dont think its critical to verify that ... I've verified this patch on several pathological samples I have, and it works. - Derek

Re: [FFmpeg-devel] [PATCH]Do not set bitrate for DTS-HD Master and High Resolution

2015-03-18 Thread Hendrik Leppkes
On Wed, Mar 18, 2015 at 3:40 PM, Marcus Johnson bumblebritche...@gmail.com wrote: I see, I thought it counted frames and not just multiplied the bitrate by the number of frames. It just divides the file size by the bitrate. It doesn't count frames at all, its not equipped for that. If it were,

Re: [FFmpeg-devel] Patch for libavcodec/exr.c implementing B44 compression technique

2015-03-18 Thread Derek Buitenhuis
On 3/18/2015 2:08 PM, greeshma wrote: The .diff file is also attached henceforth.Please have a look. This is still not a git-format patch as required and asked for by Michael. Also, the style and formatting is all over the place - that is, there is no consistent style or formatting at all.

Re: [FFmpeg-devel] [PATCH]Do not set bitrate for DTS-HD Master and High Resolution

2015-03-18 Thread Marcus Johnson
That's even worse, is there any way we can fix it? how much effort would it require? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Patch for libavcodec/exr.c implementing B44 compression technique

2015-03-18 Thread Derek Buitenhuis
On 3/18/2015 2:20 PM, Derek Buitenhuis wrote: Also, if you wrote this code, this copyright is both incorrect and makes it non-free. Hi, So after checking, the vast majority of this patch is copy pasted from OpenEXR. On that note, a few points: 1. You have no properly recreated the license

Re: [FFmpeg-devel] [PATCH] mxfenc: ensure mxf-body_partition_offset is not NULL before using it

2015-03-18 Thread tomas . hardin
On 2015-03-17 16:13, Andreas Cadhalpun wrote: On 17.03.2015 10:17, tomas.har...@codemill.se wrote: On 2015-03-14 18:03, Andreas Cadhalpun wrote: [PATCH 2/2] mxfenc: don't try to write footer without header: +if (!mxf-header_written || +(s-oformat == ff_mxf_opatom_muxer

Re: [FFmpeg-devel] [PATCH] web/index: add link to ff-facebook

2015-03-18 Thread Alexander Strasser
On 2015-03-18 20:10 +0100, Michael Niedermayer wrote: --- src/index |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/index b/src/index index d4fbc37..fbb3c05 100644 --- a/src/index +++ b/src/index @@ -31,7 +31,8 @@ a href=main.rssstrongi class=fa

[FFmpeg-devel] [PATCH]Add an ignore_delay option to the gif demuxer

2015-03-18 Thread Carl Eugen Hoyos
Hi! Attached patch allows users to force a delay for an animated gif. Related to ticket #4369 (I don't think there is a bug, if there is a bug, it is reproducible with avi etc.) Please comment, Carl Eugen diff --git a/libavformat/gifdec.c b/libavformat/gifdec.c index 7db5a27..4e0467e 100644 ---

Re: [FFmpeg-devel] Ticket 3530 : Fix proposal

2015-03-18 Thread Michael Niedermayer
On Thu, Mar 19, 2015 at 12:21:11AM +0100, Martin Vignali wrote: In attach a patch for ticket 3530 : https://trac.ffmpeg.org/ticket/3530 the file Apache_cut.aif of ticket 3530, works after the patch But i'm not sure, this is the best way to correct it, because with this sample file,

Re: [FFmpeg-devel] [PATCH] mxfenc: ensure mxf-body_partition_offset is not NULL before using it

2015-03-18 Thread Michael Niedermayer
On Wed, Mar 18, 2015 at 10:01:36PM +0100, Andreas Cadhalpun wrote: On 18.03.2015 16:59, tomas.har...@codemill.se wrote: On 2015-03-17 16:13, Andreas Cadhalpun wrote: On 17.03.2015 10:17, tomas.har...@codemill.se wrote: On 2015-03-14 18:03, Andreas Cadhalpun wrote: [PATCH 2/2] mxfenc:

[FFmpeg-devel] Ticket 3530 : Fix proposal

2015-03-18 Thread Martin Vignali
In attach a patch for ticket 3530 : https://trac.ffmpeg.org/ticket/3530 the file Apache_cut.aif of ticket 3530, works after the patch But i'm not sure, this is the best way to correct it, because with this sample file, avio_feof return 1 before the end of the file. (tested with Mac os 10.9,

[FFmpeg-devel] Blackmagic Design Intensity Pro 4K - Decklink - Problem to select RGB color space

2015-03-18 Thread Gonzalo Berdeal
Dear ffmpeg developers, First of all, congrats for your incredible software. I am using ffmpeg compiled with option Decklink and linked with Deckling API to capture hdmi video. I have been using the Intensity Pro with success. But l have just bought the new Intensity Pro 4K and I can not make it

Re: [FFmpeg-devel] GSoc Qualification Task - Implementing Compression technique B44 as a part of exr format (libavcodec/exr.c)

2015-03-18 Thread greeshma
Greeshma On Wed, Mar 18, 2015 at 4:28 AM, Michael Niedermayer michae...@gmx.at wrote: On Mon, Mar 16, 2015 at 11:49:09PM +0530, greeshma wrote: Hello , I have implemented B44 lossy compression technique.The first patch is hereby attached.Please have a look. The diff file is also

[FFmpeg-devel] [PATCH 1/2] libavutil: add av_mod_uintp2

2015-03-18 Thread James Almer
Signed-off-by: James Almer jamr...@gmail.com --- libavutil/common.h | 14 ++ 1 file changed, 14 insertions(+) diff --git a/libavutil/common.h b/libavutil/common.h index 852c1de..3e62b6d 100644 --- a/libavutil/common.h +++ b/libavutil/common.h @@ -211,6 +211,17 @@ static

[FFmpeg-devel] [PATCH 2/2] libavutil: add bmi2 optimized av_mod_uintp2

2015-03-18 Thread James Almer
Signed-off-by: James Almer jamr...@gmail.com --- libavutil/x86/intmath.h | 24 ++-- 1 file changed, 22 insertions(+), 2 deletions(-) diff --git a/libavutil/x86/intmath.h b/libavutil/x86/intmath.h index 7aa6bc4..c42fa83 100644 --- a/libavutil/x86/intmath.h +++

Re: [FFmpeg-devel] Patch for libavcodec/exr.c implementing B44 compression technique

2015-03-18 Thread greeshma
On Wed, Mar 18, 2015 at 9:37 PM, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 3/18/2015 2:20 PM, Derek Buitenhuis wrote: Also, if you wrote this code, this copyright is both incorrect and makes it non-free. Hi, Thanks for pointing out the mistakes. So after checking, the vast

Re: [FFmpeg-devel] [PATCH] replace cookies with updated values

2015-03-18 Thread Michael Niedermayer
On Tue, Mar 17, 2015 at 08:55:11PM +1100, Micah Galizia wrote: Hi, The current Set-Cookie handling appends cookies to the string without checking to see if a cookie with the same name should have its value updated. This is my second attempt to fix this issue -- last time it was a bit of

[FFmpeg-devel] [PATCH] web/index: add link to ff-facebook

2015-03-18 Thread Michael Niedermayer
--- src/index |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/index b/src/index index d4fbc37..fbb3c05 100644 --- a/src/index +++ b/src/index @@ -31,7 +31,8 @@ a href=main.rssstrongi class=fa fa-rss/i/strong/a nbsp; a

Re: [FFmpeg-devel] [PATCH] mips/float_dsp: fix vector_fmul_window_mips on mips64

2015-03-18 Thread Michael Niedermayer
On Wed, Mar 18, 2015 at 02:50:29PM +, Nedeljko Babic wrote: LGTM applied thanks Thanks, - Nedeljko Od: ffmpeg-devel-boun...@ffmpeg.org [ffmpeg-devel-boun...@ffmpeg.org] u ime korisnika James Cowgill [james...@cowgill.org.uk] Poslato: 18.

Re: [FFmpeg-devel] [PATCH] web/index: add link to ff-facebook

2015-03-18 Thread Michael Niedermayer
On Wed, Mar 18, 2015 at 10:00:21PM +0100, Alexander Strasser wrote: On 2015-03-18 20:10 +0100, Michael Niedermayer wrote: --- src/index |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/index b/src/index index d4fbc37..fbb3c05 100644 --- a/src/index +++