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 l...@lrcd.com --- 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

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 anshul.ffm...@gmail.com Date: Wed, 3 Dec 2014 23:37:22 +0530 Subject:

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 anshul.ffm...@gmail.com Date:

[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

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

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 csulo...@globaleagleent.com --- libavformat/hlsenc.c | 72

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

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

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

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

[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

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

2014-12-06 Thread Paul B Mahol
On 12/1/14, Nicolas George geo...@nsup.org 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

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

[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 michae...@gmx.at --- 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 ---

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 michae...@gmx.at --- libavcodec/libxavs.c | 30

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 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 lukasz.m.lu...@gmail.com --- tests/fate/libavcodec.mak | 4 + tests/ref/fate/libavcodec-options | 161 ++ 2 files changed, 165 insertions(+) create mode

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

[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

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 drawtext

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 lukasz.m.lu...@gmail.com --- tests/fate/libavcodec.mak | 4 + tests/ref/fate/libavcodec-options | 161 ++ 2