Re: [FFmpeg-devel] [PATCH] Fixed PNG decoding with Paeth filter for RGBA 64-bits.

2014-12-06 Thread Michael Niedermayer
On Fri, Dec 05, 2014 at 12:20:31AM +, Dominique Leroux wrote: > Found and fixed an artifact in the last column of decoded RGBA 64-bits PNG > images. > > The code was dealing with a SIMD-optimized version of the function without > taking into account that we can have RGB/RGBA images that are

Re: [FFmpeg-devel] [PATCH 2/2] fate: add libavcodec/options-test

2014-12-06 Thread Lukasz Marek
On 06.12.2014 19:48, Michael Niedermayer wrote: On Thu, Dec 04, 2014 at 12:50:50AM +0100, Lukasz Marek wrote: Signed-off-by: Lukasz Marek --- tests/fate/libavcodec.mak | 4 + tests/ref/fate/libavcodec-options | 161 ++ 2 files changed, 165 inser

Re: [FFmpeg-devel] [PATCH]Slightly improve configure --help output

2014-12-06 Thread Michael Niedermayer
On Sat, Dec 06, 2014 at 11:14:49PM +0100, Carl Eugen Hoyos wrote: > On Saturday 06 December 2014 02:27:30 pm Michael Niedermayer wrote: > > On Sat, Dec 06, 2014 at 01:52:26PM +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Users currently cannot easily know how to enable filters like the > > >

[FFmpeg-devel] [PATCH] Fix for xcb based x11grab, trac ticket #4164

2014-12-06 Thread Daniel Moran
Hi, This is the first time I've created/submitted a GIT patch, so I apologise in advance if I have made a mistake. This is to address an error when using show_region, which would cause part of the captured area to become static. It looks like the rectangle specifying the capture area was relat

Re: [FFmpeg-devel] [PATCH]Slightly improve configure --help output

2014-12-06 Thread Carl Eugen Hoyos
On Saturday 06 December 2014 02:27:30 pm Michael Niedermayer wrote: > On Sat, Dec 06, 2014 at 01:52:26PM +0100, Carl Eugen Hoyos wrote: > > Hi! > > > > Users currently cannot easily know how to enable filters like the > > drawtext filter. Attached patch tries to improve this. > > + --enable-gnutl

Re: [FFmpeg-devel] [PATCH 2/2] fate: add libavcodec/options-test

2014-12-06 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 12:50:50AM +0100, Lukasz Marek wrote: > Signed-off-by: Lukasz Marek > --- > tests/fate/libavcodec.mak | 4 + > tests/ref/fate/libavcodec-options | 161 > ++ > 2 files changed, 165 insertions(+) > create mode 100644 tests/ref/

Re: [FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-06 Thread Michael Niedermayer
On Sat, Dec 06, 2014 at 05:52:21PM +0100, Lukasz Marek wrote: > On 06.12.2014 15:59, Michael Niedermayer wrote: > >On Fri, Dec 05, 2014 at 12:21:21AM +0100, Michael Niedermayer wrote: > >>On Thu, Dec 04, 2014 at 12:50:49AM +0100, Lukasz Marek wrote: > >>[...] > >> > >>>index 49c8b40..4148752 100644

Re: [FFmpeg-devel] [PATCH] avcodec/libxavs: remove global header code

2014-12-06 Thread Michael Niedermayer
On Fri, Jan 17, 2014 at 07:53:49PM +0100, Michael Niedermayer wrote: > The code does not work, also the resulting files cannot be played when > one fixes the code so it doesnt crash > > Signed-off-by: Michael Niedermayer > --- > libavcodec/libxavs.c | 30 -- > 1 fil

[FFmpeg-devel] [PATCH] avformat/dashenc: make durations 64bit

2014-12-06 Thread Michael Niedermayer
Fix potential integer overflow Fix CID1254944 Signed-off-by: Michael Niedermayer --- libavformat/dashenc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 149e7d9..fa497fd 100644 --- a/libavformat/dashenc.c ++

Re: [FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-06 Thread Lukasz Marek
On 06.12.2014 15:59, Michael Niedermayer wrote: On Fri, Dec 05, 2014 at 12:21:21AM +0100, Michael Niedermayer wrote: On Thu, Dec 04, 2014 at 12:50:49AM +0100, Lukasz Marek wrote: [...] index 49c8b40..4148752 100644 --- a/libavcodec/options.c +++ b/libavcodec/options.c @@ -303,3 +303,176 @@ con

Re: [FFmpeg-devel] [PATCH] lavfi/zoomoan: fix frame count logic

2014-12-06 Thread Paul B Mahol
On 12/1/14, Nicolas George wrote: > I prefer first patch, as it makes the code simpler and starting at 0 > simplifies all arithmetic operations, but it makes a slight behaviour > change, so I also propose the second version. Note how the 1-indexed logic > makes the documentation for "on" awkward.

[FFmpeg-devel] [PATCH] avformat/matroskadec: fix handling of recursive SeekHead elements

2014-12-06 Thread wm4
When matroska_execute_seekhead() is called, it goes through the list of seekhead entries and attempts to read elements not read yet. When doing this, the parser can find further SeekHead elements, and will extend the matroska->seekhead list. This can lead to a (practically) infinite loop with certa

Re: [FFmpeg-devel] [PATCH] lavfi/zoomoan: fix frame count logic

2014-12-06 Thread Clément Bœsch
On Mon, Dec 01, 2014 at 04:25:42PM +0100, Nicolas George wrote: > I prefer first patch, as it makes the code simpler and starting at 0 > simplifies all arithmetic operations, but it makes a slight behaviour > change, so I also propose the second version. Note how the 1-indexed logic > makes the doc

Re: [FFmpeg-devel] [PATCH] lavfi/zoomoan: fix frame count logic

2014-12-06 Thread Michael Niedermayer
On Mon, Dec 01, 2014 at 04:25:42PM +0100, Nicolas George wrote: > I prefer first patch, as it makes the code simpler and starting at 0 > simplifies all arithmetic operations, but it makes a slight behaviour > change, so I also propose the second version. Note how the 1-indexed logic > makes the doc

Re: [FFmpeg-devel] [PATCH]Fixed av_fast_realloc calling error when compiled to the Dynamic library

2014-12-06 Thread Michael Niedermayer
On Fri, Dec 05, 2014 at 06:05:30PM +0800, 杨超 wrote: > Hello,All: > source base: ffmpeg-2.2.10env:androidso/a: so > When calling the av_fast_realloc or av_fast_malloc. There will be a error ( > the processing will feign death ). > Fixed patch: > ---a/libavcodec/utils.c+++b/libavcodec/utils.c@@ -12

Re: [FFmpeg-devel] [PATCH 1/2] lavc/options: add test for avcodec_copy_context

2014-12-06 Thread Michael Niedermayer
On Fri, Dec 05, 2014 at 12:21:21AM +0100, Michael Niedermayer wrote: > On Thu, Dec 04, 2014 at 12:50:49AM +0100, Lukasz Marek wrote: > [...] > > > index 49c8b40..4148752 100644 > > --- a/libavcodec/options.c > > +++ b/libavcodec/options.c > > @@ -303,3 +303,176 @@ const AVClass *avcodec_get_subtit

Re: [FFmpeg-devel] [PATCH 2/3] avformat/hlsenc: added segment file deletion

2014-12-06 Thread Michael Niedermayer
On Thu, Dec 04, 2014 at 05:05:25PM -0600, Christian Suloway wrote: > Option removes segments no longer in playlist when older than playlist + > segment duration > > Signed-off-by: Christian Suloway > --- > libavformat/hlsenc.c | 72 > > 1 fi

Re: [FFmpeg-devel] [PATCH]Slightly improve configure --help output

2014-12-06 Thread Michael Niedermayer
On Sat, Dec 06, 2014 at 01:52:26PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Users currently cannot easily know how to enable filters like the drawtext > filter. Attached patch tries to improve this. > > Please comment, Carl Eugen > configure | 19 ++- > 1 file changed, 10 ins

[FFmpeg-devel] [PATCH]Slightly improve configure --help output

2014-12-06 Thread Carl Eugen Hoyos
Hi! Users currently cannot easily know how to enable filters like the drawtext filter. Attached patch tries to improve this. Please comment, Carl Eugen diff --git a/configure b/configure index c046e34..d5c73a0 100755 --- a/configure +++ b/configure @@ -194,25 +194,26 @@ Individual component opti

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2014-12-06 Thread Anshul
On 12/06/2014 04:22 PM, Anshul wrote: > On 12/05/2014 07:56 PM, Nicolas George wrote: >> Hi. I had time to look at the code with some more details. Comments are >> below. >> >>> >From 31f69ccfb45247a7cc203084a931b8523284aa13 Mon Sep 17 00:00:00 2001 >>> From: Anshul Maheshwari >>> Date: Wed, 3 Dec

Re: [FFmpeg-devel] [PATCH 2/2] Adding closed caption decoder

2014-12-06 Thread Anshul
On 12/05/2014 07:56 PM, Nicolas George wrote: > Hi. I had time to look at the code with some more details. Comments are > below. > >> >From 31f69ccfb45247a7cc203084a931b8523284aa13 Mon Sep 17 00:00:00 2001 >> From: Anshul Maheshwari >> Date: Wed, 3 Dec 2014 23:37:22 +0530 >> Subject: [PATCH 2/2] A

Re: [FFmpeg-devel] [PATCH] doc: add loglevel numerical values

2014-12-06 Thread Clément Bœsch
On Fri, Dec 05, 2014 at 12:36:18PM -0900, Lou Logan wrote: > Signed-off-by: Lou Logan > --- > doc/fftools-common-opts.texi | 10 +- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/doc/fftools-common-opts.texi b/doc/fftools-common-opts.texi > index 0e8f849..c496419 10064