Re: [FFmpeg-devel] [PATCH] vp9: change type of tile_size from unsigned to int64_t

2015-06-08 Thread Clément Bœsch
On Mon, Jun 08, 2015 at 09:45:07PM +0200, Andreas Cadhalpun wrote: On 07.06.2015 22:30, Ronald S. Bultje wrote: Hi, On Sun, Jun 7, 2015 at 1:02 PM, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Otherwise the check 'tile_size size' treats a negative size as unsigned,

[FFmpeg-devel] [PATCH]Use libopenjpeg by default if it was enabled

2015-06-08 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #3619 for me. Please comment, Carl Eugen diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c index ce97746..6f40b51 100644 --- a/libavcodec/allcodecs.c +++ b/libavcodec/allcodecs.c @@ -104,6 +104,9 @@ void avcodec_register_all(void)

Re: [FFmpeg-devel] [PATCH] vp9: change type of tile_size from unsigned to int64_t

2015-06-08 Thread Andreas Cadhalpun
On 07.06.2015 22:30, Ronald S. Bultje wrote: Hi, On Sun, Jun 7, 2015 at 1:02 PM, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Otherwise the check 'tile_size size' treats a negative size as unsigned, causing the check to pass. This subsequently leads to segmentation faults.

[FFmpeg-devel] [PATCH] mpjpegdec: don't try to alloc an AVIOContext when probe is guaranteed to fail

2015-06-08 Thread James Almer
The first check is done without the AVIOContext, so alloc it only if said check succeeds Signed-off-by: James Almer jamr...@gmail.com --- libavformat/mpjpegdec.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavformat/mpjpegdec.c b/libavformat/mpjpegdec.c index

Re: [FFmpeg-devel] [libav-devel] [PATCH] vp8: check for too large dimensions

2015-06-08 Thread Andreas Cadhalpun
On 07.06.2015 22:39, Michael Niedermayer wrote: On Sun, Jun 07, 2015 at 04:27:42PM -0400, Ronald S. Bultje wrote: So what happens when you change mv_max/min to be integers (instead of int16_t) without further touching VP56mv, e.g. by making mv_max/min a VP8intminmaxmv (a newly created type

Re: [FFmpeg-devel] [libav-devel] [PATCH] vp8: check for too large dimensions

2015-06-08 Thread Ronald S. Bultje
Hi, On Mon, Jun 8, 2015 at 5:08 PM, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: On 07.06.2015 22:39, Michael Niedermayer wrote: On Sun, Jun 07, 2015 at 04:27:42PM -0400, Ronald S. Bultje wrote: So what happens when you change mv_max/min to be integers (instead of int16_t)

Re: [FFmpeg-devel] [libav-devel] [PATCH] vp8: check for too large dimensions

2015-06-08 Thread Andreas Cadhalpun
Hi Ronald, On 08.06.2015 23:18, Ronald S. Bultje wrote: On Mon, Jun 8, 2015 at 5:08 PM, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: On 07.06.2015 22:39, Michael Niedermayer wrote: On Sun, Jun 07, 2015 at 04:27:42PM -0400, Ronald S. Bultje wrote: So what happens when you

Re: [FFmpeg-devel] [PATCH] mpjpegdec: don't try to alloc an AVIOContext when probe is guaranteed to fail

2015-06-08 Thread James Almer
On 08/06/15 6:23 PM, Michael Niedermayer wrote: On Mon, Jun 08, 2015 at 05:40:55PM -0300, James Almer wrote: The first check is done without the AVIOContext, so alloc it only if said check succeeds Signed-off-by: James Almer jamr...@gmail.com --- libavformat/mpjpegdec.c | 7 +++ 1

Re: [FFmpeg-devel] [FFmpeg-cvslog] vp8: change mv_{min, max}.{x, y} type to int

2015-06-08 Thread Ronald S. Bultje
On Mon, Jun 8, 2015 at 5:33 PM, Clément Bœsch u...@pkh.me wrote: On Mon, Jun 08, 2015 at 11:31:15PM +0200, Andreas Cadhalpun wrote: ffmpeg | branch: master | Andreas Cadhalpun andreas.cadhal...@googlemail.com | Mon Jun 8 22:38:29 2015 +0200| [6fdbaa2b7fb56623ab2163f861952bc1408c39b3] |

Re: [FFmpeg-devel] [FFmpeg-cvslog] vp8: change mv_{min, max}.{x, y} type to int

2015-06-08 Thread Andreas Cadhalpun
On 08.06.2015 23:37, Ronald S. Bultje wrote: On Mon, Jun 8, 2015 at 5:33 PM, Clément Bœsch u...@pkh.me wrote: On Mon, Jun 08, 2015 at 11:31:15PM +0200, Andreas Cadhalpun wrote: ffmpeg | branch: master | Andreas Cadhalpun andreas.cadhal...@googlemail.com | Mon Jun 8 22:38:29 2015 +0200|

[FFmpeg-devel] [PATCH] lavf/tls_securetransport: fix SNI support when not verifying

2015-06-08 Thread Rodger Combs
--- libavformat/tls_securetransport.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/tls_securetransport.c b/libavformat/tls_securetransport.c index c90eab7..cdc7953 100644 --- a/libavformat/tls_securetransport.c +++ b/libavformat/tls_securetransport.c @@

Re: [FFmpeg-devel] [FFmpeg-cvslog] vp8: change mv_{min, max}.{x, y} type to int

2015-06-08 Thread Clément Bœsch
On Mon, Jun 08, 2015 at 11:31:15PM +0200, Andreas Cadhalpun wrote: ffmpeg | branch: master | Andreas Cadhalpun andreas.cadhal...@googlemail.com | Mon Jun 8 22:38:29 2015 +0200| [6fdbaa2b7fb56623ab2163f861952bc1408c39b3] | committer: Andreas Cadhalpun vp8: change mv_{min,max}.{x,y} type to

Re: [FFmpeg-devel] [PATCH]Use libopenjpeg by default if it was enabled

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 10:01:18PM +0200, Carl Eugen Hoyos wrote: Hi! Attached patch fixes ticket #3619 for me. should have become unneeded [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Awnsering whenever a program halts or runs forever is On a turing

Re: [FFmpeg-devel] [PATCH] mpjpegdec: don't try to alloc an AVIOContext when probe is guaranteed to fail

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 05:40:55PM -0300, James Almer wrote: The first check is done without the AVIOContext, so alloc it only if said check succeeds Signed-off-by: James Almer jamr...@gmail.com --- libavformat/mpjpegdec.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-)

Re: [FFmpeg-devel] [FFmpeg-cvslog] vp8: change mv_{min, max}.{x, y} type to int

2015-06-08 Thread Clément Bœsch
On Mon, Jun 08, 2015 at 05:37:22PM -0400, Ronald S. Bultje wrote: On Mon, Jun 8, 2015 at 5:33 PM, Clément Bœsch u...@pkh.me wrote: On Mon, Jun 08, 2015 at 11:31:15PM +0200, Andreas Cadhalpun wrote: ffmpeg | branch: master | Andreas Cadhalpun andreas.cadhal...@googlemail.com | Mon Jun 8

[FFmpeg-devel] [PATCH] lavf/tls_securetransport: fix SNI support when not verifying

2015-06-08 Thread Rodger Combs
--- libavformat/segment.c | 4 +++- libavformat/tls_securetransport.c | 6 +++--- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 953140f..66e28b2 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@

Re: [FFmpeg-devel] [PATCH]Mark experimental j2k decoder as experimental if libopenjpeg is available

2015-06-08 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: it seems libopenjpeg is not able to decode the files generated with libopenjpeg. That is with the libopenjpeg from ubuntu 12.04 (1.3+dfsg-4+squeeze2build0.12.04.1) but ubuntu 14.04 ships a variant of 1.3 too test done with: ./ffmpeg

[FFmpeg-devel] [PATCH] avcodec/libopenjpegdec: Mark as experimental if = 1.3

2015-06-08 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/libopenjpegdec.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavcodec/libopenjpegdec.c b/libavcodec/libopenjpegdec.c index 8fe7a50..7f28e87 100644 --- a/libavcodec/libopenjpegdec.c +++

Re: [FFmpeg-devel] [PATCH] lavf/tls_securetransport: fix SNI support when not verifying

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 04:59:32PM -0500, Rodger Combs wrote: --- libavformat/tls_securetransport.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The educated differ from the

[FFmpeg-devel] [PATCH 0/2] Fix silent audio when channel layout specified with trailing c

2015-06-08 Thread Simon Thelen
When specifying the output channel layout with a trailing 'c' such as '1c' or '2c', the pad filter created silent output files. Leaving the trailing 'c' away resulted in correct output files, but generated a warning that the syntax was deprecated. The first patch fixes two bugs in

[FFmpeg-devel] [PATCH 1/2] libavfilter/formats: Fix parsing of channel specifications with a trailing 'c'.

2015-06-08 Thread Simon Thelen
Fix an off-by-one in checking tail for trailing characters and ensure that the parsing helper is only called for unknown channel layouts. Note: This removes the check ensuring that the channel layout is 0 and 63. Signed-off-by: Simon Thelen ffmpeg-...@c-14.de --- If the check ensuring 0

Re: [FFmpeg-devel] [PATCH] configure: autodetect OpenCL headers and ICD loader

2015-06-08 Thread Hendrik Leppkes
On Mon, Jun 8, 2015 at 5:53 AM, Gupta, Maneesh maneesh.gu...@amd.com wrote: -Original Message- From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel- boun...@ffmpeg.org] On Behalf Of James Almer Sent: Thursday, May 28, 2015 10:06 AM To: FFmpeg development discussions and patches

Re: [FFmpeg-devel] [PATCH] libavutil/doc: Changes in documentation due to changes to fixed_dsp

2015-06-08 Thread Nedeljko Babic
On Fri, Jun 05, 2015 at 08:05:33PM +0200, Michael Niedermayer wrote: On Fri, Jun 05, 2015 at 07:59:47PM +0200, Michael Niedermayer wrote: On Fri, Jun 05, 2015 at 10:04:54AM -0400, Ronald S. Bultje wrote: Hi, On Fri, Jun 5, 2015 at 5:54 AM, Nedeljko Babic nedeljko.ba...@imgtec.com wrote:

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Allow overriding /manual setting of the signal standard

2015-06-08 Thread tim nicholson
On 06/06/15 02:23, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/mxfenc.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index c612747..f2a7f0a 100644 --- a/libavformat/mxfenc.c

Re: [FFmpeg-devel] [PATCH] Reenable GHA phase inversion in Atrac3+

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 02:17:16AM +0200, Max Pole wrote: Hello crews, I attached two simple patches reenabling a decoder feature been initially disabled due to lack of real-world samples. I've recently got a couple of appropriate samples from the PPSSPP emulator project so I coud test

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Allow overriding /manual setting of the signal standard

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 09:48:33AM +0100, tim nicholson wrote: On 06/06/15 02:23, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/mxfenc.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavformat/mxfenc.c

Re: [FFmpeg-devel] [PATCH] Reenable GHA phase inversion in Atrac3+

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 03:33:07AM +0200, Max Pole wrote: I've recently got a couple of appropriate samples from the PPSSPP emulator project so I coud test this feature extensively. where can i find these samples ? are they online somewhere? can you upload them ? I've just

Re: [FFmpeg-devel] [PATCH] configure: autodetect OpenCL headers and ICD loader

2015-06-08 Thread Gupta, Maneesh
-Original Message- From: ffmpeg-devel-boun...@ffmpeg.org [mailto:ffmpeg-devel- boun...@ffmpeg.org] On Behalf Of Hendrik Leppkes Sent: Monday, June 08, 2015 1:00 PM To: FFmpeg development discussions and patches Subject: Re: [FFmpeg-devel] [PATCH] configure: autodetect OpenCL headers

[FFmpeg-devel] [PATCH] lavf/tls: let the user specify what name to verify against

2015-06-08 Thread Rodger Combs
This can be useful for debugging, or in scenarios where the user doesn't want to use the system's DNS settings for whatever reason. --- libavformat/tls.c | 13 - libavformat/tls.h | 7 +-- 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/libavformat/tls.c

Re: [FFmpeg-devel] [PATCH 3/6] avcodec/apng: Dispose previous frame properly

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 02:28:05PM +0200, Paul B Mahol wrote: On 6/7/15, Donny Yang w...@kota.moe wrote: The spec specifies the dispose operation as how the current (i.e., currently being rendered) frame should be disposed when the next frame is blended onto it This is contrary to

Re: [FFmpeg-devel] [PATCH 5/6] avcodec/apng: Add support for blending with GRAY8A pixel format

2015-06-08 Thread Michael Niedermayer
On Tue, Jun 02, 2015 at 08:21:36PM +0200, Paul B Mahol wrote: On 6/2/15, Donny Yang w...@kota.moe wrote: On 3 June 2015 at 03:38, Paul B Mahol one...@gmail.com wrote: Dana 2. 6. 2015. 17:50 osoba Donny Yang w...@kota.moe napisala je: Signed-off-by: Donny Yang w...@kota.moe ---

[FFmpeg-devel] Test-please ignore

2015-06-08 Thread tim nicholson
Test of address munging on new server -- Tim. Key Fingerprint 38CF DB09 3ED0 F607 8B67 6CED 0C0B FC44 8B0B FC83 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 4/6] avcodec/apng: Add blending support for non-alpha pixel formats

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 02:38:39PM +0200, Paul B Mahol wrote: On 6/2/15, Donny Yang w...@kota.moe wrote: Signed-off-by: Donny Yang w...@kota.moe --- libavcodec/pngdec.c | 15 +++ 1 file changed, 15 insertions(+) probably ok applied thanks [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 6/6] avcodec/apng: Add partial support for blending with PAL8 pixel format

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 02:35:28PM +0200, Paul B Mahol wrote: On 6/2/15, Donny Yang w...@kota.moe wrote: Currently restricted to blending pixels that only contain either 0 or 255 in their alpha components Signed-off-by: Donny Yang w...@kota.moe --- libavcodec/pngdec.c | 15

Re: [FFmpeg-devel] [PATCH 4/6] avcodec/apng: Add blending support for non-alpha pixel formats

2015-06-08 Thread Paul B Mahol
On 6/2/15, Donny Yang w...@kota.moe wrote: Signed-off-by: Donny Yang w...@kota.moe --- libavcodec/pngdec.c | 15 +++ 1 file changed, 15 insertions(+) probably ok ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 6/6] avcodec/apng: Add partial support for blending with PAL8 pixel format

2015-06-08 Thread Paul B Mahol
On 6/2/15, Donny Yang w...@kota.moe wrote: Currently restricted to blending pixels that only contain either 0 or 255 in their alpha components Signed-off-by: Donny Yang w...@kota.moe --- libavcodec/pngdec.c | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) lgtm

Re: [FFmpeg-devel] [PATCH] lavf/tls: let the user specify what name to verify against

2015-06-08 Thread Michael Niedermayer
On Mon, Jun 08, 2015 at 04:50:48AM -0500, Rodger Combs wrote: This can be useful for debugging, or in scenarios where the user doesn't want to use the system's DNS settings for whatever reason. --- libavformat/tls.c | 13 - libavformat/tls.h | 7 +-- 2 files changed, 13

Re: [FFmpeg-devel] [PATCH 3/6] avcodec/apng: Dispose previous frame properly

2015-06-08 Thread Paul B Mahol
On 6/7/15, Donny Yang w...@kota.moe wrote: The spec specifies the dispose operation as how the current (i.e., currently being rendered) frame should be disposed when the next frame is blended onto it This is contrary to ffmpeg's current behaviour of interpreting the dispose operation as how