[FFmpeg-devel] [PATCH]lavc/x264:Refuse RGB24 as input pix_fmt

2015-06-08 Thread Carl Eugen Hoyos
Hi! Attached patch works around ticket #4287 here. Please comment, Carl Eugen diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 9020a40..5736c6c 100644 --- a/libavcodec/libx264.c +++ b/libavcodec/libx264.c @@ -755,7 +755,6 @@ static const enum AVPixelFormat pix_fmts_8bit_rgb[] = { #

[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 --- If the check ensuring 0 < chlayout < 63 is nece

[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 ff_parse_channe

[FFmpeg-devel] [PATCH 2/2] libavutil/channel_layout: Correctly return layout when channel specification ends with a trailing 'c'.

2015-06-08 Thread Simon Thelen
Return layout when FF_API_GET_CHANNEL_LAYOUT_COMPAT is set even if the layout itself is not in the deprecated style. Signed-off-by: Simon Thelen --- libavutil/channel_layout.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/channel_layout.c b/libavutil/channel_layou

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

2015-06-08 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- 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 +++ b/libavcodec/libopenjpegdec.c @@ -433,

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 un

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

2015-06-08 Thread Carl Eugen Hoyos
Michael Niedermayer 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 -threads

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

[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 @@ -2

[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 @@ -113,6

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 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| >> [

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 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

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 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] | committe

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 > | Mon Jun 8 22:38:29 2015 +0200| > [6fdbaa2b7fb56623ab2163f861952bc1408c39b3] | committer: Andreas Cadhalpun > > vp8: change mv_{min,max}.{x,y} type to int > > If one of the dime

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

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 >> --- >> libavformat/mpjpegdec.c | 7 +++ >> 1 file

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 > --- > libavformat/mpjpegdec.c | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) LGTM thanks

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 > >>

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 ty

[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 --- libavformat/mpjpegdec.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavformat/mpjpegdec.c b/libavformat/mpjpegdec.c index 56fe159..845e95c 100

[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) REGISTER_HWACCEL(W

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

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 >> segment

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 21:03, Luca Barbato wrote: > On 07/06/15 19:02, Andreas Cadhalpun wrote: >> Otherwise the check 'tile_size < size' treats a negative size as >> unsigned, causing the check to pass. This subsequently leads to >> segmentation faults. >> >> This was originally fixed as part of Libav comm

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 wrote: > > Currently restricted to blending pixels that only contain either > > 0 or 255 in their alpha components > > > > Signed-off-by: Donny Yang > > --- > > libavcodec/pngdec.c | 15 ++- > > 1 f

[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 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 wrote: > > On 3 June 2015 at 03:38, Paul B Mahol wrote: > > > >> Dana 2. 6. 2015. 17:50 osoba "Donny Yang" napisala je: > >> > > >> > Signed-off-by: Donny Yang > >> > --- > >> > libavcodec/pngdec.c | 8 ++

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 wrote: > > Signed-off-by: Donny Yang > > --- > > libavcodec/pngdec.c | 15 +++ > > 1 file changed, 15 insertions(+) > > > > probably ok applied thanks [...] -- Michael GnuPG fingerprint: 9F

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 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 ffmp

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, 1

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 wrote: > Signed-off-by: Donny Yang > --- > libavcodec/pngdec.c | 15 +++ > 1 file changed, 15 insertions(+) > probably ok ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg

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 wrote: > Currently restricted to blending pixels that only contain either > 0 or 255 in their alpha components > > Signed-off-by: Donny Yang > --- > libavcodec/pngdec.c | 15 ++- > 1 file changed, 14 insertions(+), 1 deletion(-) > lgtm _

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 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 the pr

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 > > --- > > libavformat/mxfenc.c | 10 ++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/libavformat/mxfenc.c b/libavformat/m

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 t

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 ?

[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 b/libavfor

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 > --- > 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 > +++ b/

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 hea

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 > wrote:

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 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 >> Subject: