Re: [FFmpeg-devel] [PATCH 1/3] nutdec: initialize has_keyframe[0]

2015-05-22 Thread Michael Niedermayer
On Fri, May 22, 2015 at 11:36:17PM +0200, Andreas Cadhalpun wrote: If x is 2 it is not initialized before it is used. This fixes 'Conditional jump or move depends on uninitialised value(s)' valgrind warnings. Signed-off-by: Andreas Cadhalpun andreas.cadhal...@googlemail.com ---

[FFmpeg-devel] [PATCH] Revert: mpegvideo: Drop err_recognition AND mpegvideo: Drop flags and flags2

2015-05-22 Thread Michael Niedermayer
this increases mpeg4 and msmpeg4v3 decoding speed by about 0.3-0.5% overall This reverts the merges of 848e86f74d3e6e87fa592ee8ba8c184cc5fd9a42 and 9c1db92ad372d4cd69e0490e691c56e4097cb193 Speed tested with decoding matrixbench_mpg4.avi and matrixbench_msmpeg4v3.avi Test files where generated

Re: [FFmpeg-devel] [PATCH] Revert: mpegvideo: Drop err_recognition AND mpegvideo: Drop flags and flags2

2015-05-22 Thread James Almer
On 22/05/15 6:59 PM, Michael Niedermayer wrote: this increases mpeg4 and msmpeg4v3 decoding speed by about 0.3-0.5% overall This reverts the merges of 848e86f74d3e6e87fa592ee8ba8c184cc5fd9a42 and 9c1db92ad372d4cd69e0490e691c56e4097cb193 Speed tested with decoding matrixbench_mpg4.avi and

[FFmpeg-devel] [PATCH 3/3] nutdec: fix various memleaks on failure

2015-05-22 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun andreas.cadhal...@googlemail.com --- libavformat/nutdec.c | 77 +--- 1 file changed, 55 insertions(+), 22 deletions(-) diff --git a/libavformat/nutdec.c b/libavformat/nutdec.c index 969f446..fc40f57 100644 ---

Re: [FFmpeg-devel] [PATCH] avcodec: loongson3 optimized h264dsp weighted mc with mmi

2015-05-22 Thread 周晓勇
i am working on debuging the two patchs i send u before. mpegvideo patch has two incorrect functions which cause failures in fate-vsynth1(2)-mpeg2(-422). i will repatch the mpegvideo optimization fist after resolving the bugs. thanks. -原始邮件- 发件人: Michael Niedermayer michae...@gmx.at

[FFmpeg-devel] patch ffserver

2015-05-22 Thread ill
So can the ffserver patch be committed? From 5b681fdaf7971f841b84cfe06412c67ba417fb00 Mon Sep 17 00:00:00 2001 From: Caligula useraccount calig...@sarijopen.student.utwente.nl Date: Mon, 11 May 2015 17:42:42 -0600 Subject: [PATCH] ffserver jpg patch --- ffserver.c | 4

[FFmpeg-devel] [PATCH] x86/cpu: add AV_CPU_FLAG_AVXSLOW flag

2015-05-22 Thread James Almer
Signed-off-by: James Almer jamr...@gmail.com --- doc/APIchanges | 3 +++ libavutil/cpu.c | 5 + libavutil/cpu.h | 1 + libavutil/version.h | 4 ++-- libavutil/x86/cpu.c | 17 ++--- 5 files changed, 25 insertions(+), 5 deletions(-) diff --git a/doc/APIchanges

Re: [FFmpeg-devel] [PATCH] avcodec[/format]/webpenc: use WebPAnimEncoder API to generate animated WebP

2015-05-22 Thread James Almer
On 22/05/15 10:43 PM, Michael Niedermayer wrote: applied i think the AVOption table and AVClass name could still be improved to match each encoder more individually, i dont think both support all options Thanks I thought classes couldn't be shared between different components, as it

Re: [FFmpeg-devel] [PATCH] avcodec[/format]/webpenc: use WebPAnimEncoder API to generate animated WebP

2015-05-22 Thread James Almer
On 23/05/15 12:34 AM, James Almer wrote: On 22/05/15 10:43 PM, Michael Niedermayer wrote: applied i think the AVOption table and AVClass name could still be improved to match each encoder more individually, i dont think both support all options Thanks I thought classes couldn't be

Re: [FFmpeg-devel] [PATCH] avcodec[/format]/webpenc: use WebPAnimEncoder API to generate animated WebP

2015-05-22 Thread Michael Niedermayer
On Fri, May 22, 2015 at 05:28:06PM +, Urvang Joshi wrote: On Thu, May 21, 2015 at 6:48 PM Michael Niedermayer michae...@gmx.at wrote: On Wed, May 20, 2015 at 01:54:59AM +, Urvang Joshi wrote: On Thu, May 14, 2015 at 7:21 PM Michael Niedermayer michae...@gmx.at wrote: On

Re: [FFmpeg-devel] [PATCH 2/3] nutdec: always check the get_str return value

2015-05-22 Thread Michael Niedermayer
On Fri, May 22, 2015 at 11:36:32PM +0200, Andreas Cadhalpun wrote: If it fails, the buffers can be (partially) uninitialized. This fixes 'Conditional jump or move depends on uninitialised value(s)' valgrind warnings. Signed-off-by: Andreas Cadhalpun andreas.cadhal...@googlemail.com ---

[FFmpeg-devel] [PATCH] configure: simplify libwebp checks

2015-05-22 Thread James Almer
The libwebpmux pkg-config file already has an explicit dependecy on libwebp = 0.2.0. Also remove the warning and silently disable the anim encoder when libwebpmux is not new enough. This is more in line with other library components, like libvpx-vp9 Signed-off-by: James Almer jamr...@gmail.com

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Add color siting element

2015-05-22 Thread Michael Niedermayer
On Fri, May 22, 2015 at 10:37:36AM +0100, tim nicholson wrote: On 21/05/15 15:04, tim nicholson wrote: Please don't waste any more time fiddling with this, I expect it will suddenly click in my brain when I least expect it. Oh hang on I think it just did, you superimpose the second art on

[FFmpeg-devel] [PATCH 1/2] lavf: split tls.c

2015-05-22 Thread wm4
Move the OpenSSL and GnuTLS implementations to their own files. Other than the connection code (including options) and some boilerplate, no code is actually shared. --- This is on top of commit 29216d7fd14d1cec16821867d17c90b5c49e8c07 (aka Libav). Sending this to ffmpeg-devel for review as

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: Add color siting element

2015-05-22 Thread Michael Niedermayer
On Thu, May 21, 2015 at 03:06:11PM +0100, tim nicholson wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 21/05/15 11:59, Michael Niedermayer wrote: On Thu, May 21, 2015 at 09:00:56AM +0200, Tomas Härdin wrote: [..] Why is this guessing code in mxfenc? This should be something

[FFmpeg-devel] [PATCH 2/2] lavf: remove TLS related ifdeffery to library specific files

2015-05-22 Thread wm4
There is no need to have this mess in network.c. --- libavformat/network.c | 83 +++ libavformat/tls_common.h | 6 libavformat/tls_gnutls.c | 26 +++ libavformat/tls_openssl.c | 64 4 files

[FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: Add mxf muxer avclass

2015-05-22 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/mxfenc.c | 12 1 file changed, 12 insertions(+) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index adac69e..644ca56 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@ -2615,6 +2615,17 @@

Re: [FFmpeg-devel] [PATCH] Add support for large boxes(32 bit)

2015-05-22 Thread Philip Langdale
On Thu, 21 May 2015 21:45:52 +0530 Niklesh Lalwani lalwani1...@gmail.com wrote: Updated patch. Thanks, Niklesh Pushed. Thanks. --phil ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/2] lavf: split tls.c

2015-05-22 Thread wm4
On Fri, 22 May 2015 15:14:29 -0300 James Almer jamr...@gmail.com wrote: On 22/05/15 10:54 AM, wm4 wrote: Move the OpenSSL and GnuTLS implementations to their own files. Other than the connection code (including options) and some boilerplate, no code is actually shared. --- This is on

Re: [FFmpeg-devel] [PATCH 1/2] lavf: split tls.c

2015-05-22 Thread James Almer
On 22/05/15 10:54 AM, wm4 wrote: Move the OpenSSL and GnuTLS implementations to their own files. Other than the connection code (including options) and some boilerplate, no code is actually shared. --- This is on top of commit 29216d7fd14d1cec16821867d17c90b5c49e8c07 (aka Libav). Sending