Re: [FFmpeg-devel] [PATCH] configure: add -Werror=declaration-after-statement for gcc

2014-11-01 Thread Nicolas George
Le primidi 11 brumaire, an CCXXIII, Lukasz Marek a écrit : Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- configure | 1 + 1 file changed, 1 insertion(+) Did something change since 377dfa3d? Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/libwebpenc: support P frames in webp animations

2014-11-01 Thread Clément Bœsch
On Fri, Oct 31, 2014 at 04:48:08PM +0100, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/libwebpenc.c | 80 +-- 1 file changed, 77 insertions(+), 3 deletions(-) diff --git

[FFmpeg-devel] [PATCH]Set bits_per_coded_sample for G.726 in mov

2014-11-01 Thread Carl Eugen Hoyos
Hi! Ticket #4069 indicates that users unfortunately try to put G.726 in mov. Attached patch makes such files decodable. Please comment, Carl Eugen diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 974edea..b9576dc 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@

Re: [FFmpeg-devel] [PATCH 1/2] avformat: add webp muxer

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 10:21:20AM +, Carl Eugen Hoyos wrote: Michael Niedermayer michaelni at gmx.at writes: +{ loop, Number of times to loop the output: 0 - infinite loop, OFFSET(loop), + AV_OPT_TYPE_INT, { .i64 = 1 }, -1, 65535, ENC }, What effect does -1 have? same as

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/libwebpenc: support P frames in webp animations

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 10:54:33AM +0100, Clément Bœsch wrote: On Fri, Oct 31, 2014 at 04:48:08PM +0100, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/libwebpenc.c | 80 +-- 1 file changed,

Re: [FFmpeg-devel] [PATCH] lavfi: add xbr filter

2014-11-01 Thread Stefano Sabatini
On date Saturday 2014-11-01 04:27:24 +0530, arwa arif encoded: [...] On Fri, Oct 31, 2014 at 6:31 PM, Stefano Sabatini stefa...@gmail.com From 9b6b1d546247cf61b2f696835fb8a5e782d353ea Mon Sep 17 00:00:00 2001 From: Arwa Arif arwaarif1...@gmail.com Date: Thu, 30 Oct 2014 22:06:20 +0530

Re: [FFmpeg-devel] [PATCH]Set bits_per_coded_sample for G.726 in mov

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 10:57:18AM +0100, Carl Eugen Hoyos wrote: Hi! Ticket #4069 indicates that users unfortunately try to put G.726 in mov. Attached patch makes such files decodable. Please comment, Carl Eugen should be ok [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH] avformat/rtpdec_h261: code aligned to the HEVC code

2014-11-01 Thread Thomas Volkert
From: Thomas Volkert tho...@homer-conferencing.com --- libavformat/rtpdec_h261.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/libavformat/rtpdec_h261.c b/libavformat/rtpdec_h261.c index b902d2a..43244bb 100644 --- a/libavformat/rtpdec_h261.c +++

[FFmpeg-devel] [PATCH] mpeg4vdpau: Fix priv data size.

2014-11-01 Thread Reimar Döffinger
--- libavcodec/mpeg4videodec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c index 47ac523..846b691 100644 --- a/libavcodec/mpeg4videodec.c +++ b/libavcodec/mpeg4videodec.c @@ -2795,7 +2795,7 @@ AVCodec

[FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.

2014-11-01 Thread Reimar Döffinger
This is a regression, previous code did not require users to set the profile to use VDPAU. In addition, ASP is a superset of SP, so there is not even a good reason to require it. --- libavcodec/vdpau_mpeg4.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/vdpau_mpeg4.c

Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.

2014-11-01 Thread wm4
On Sat, 1 Nov 2014 13:45:46 +0100 Reimar Döffinger reimar.doeffin...@gmx.de wrote: This is a regression, previous code did not require users to set the profile to use VDPAU. It still doesn't...? There are basically 3 ffmpeg vdpau hwaccel APIs now. In addition, ASP is a superset of SP, so

Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.

2014-11-01 Thread Carl Eugen Hoyos
Reimar Döffinger Reimar.Doeffinger at gmx.de writes: +case FF_PROFILE_UNKNOWN: case FF_PROFILE_MPEG4_ADVANCED_SIMPLE: profile = VDP_DECODER_PROFILE_MPEG4_PART2_ASP; Could instead be the default case to set profile to ASP? Or to remove this whole logic? Carl Eugen

Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.

2014-11-01 Thread Jean-Baptiste Kempf
On 01 Nov, Reimar Döffinger wrote : This is a regression, previous code did not require users to set the profile to use VDPAU. If you don't set the profile, how can you know that the hardware supports it? -- Jean-Baptiste Kempf http://www.jbkempf.com/ - +33 672 704 734 Sent from my Electronic

Re: [FFmpeg-devel] [PATCH] mpeg4vdpau: Fix priv data size.

2014-11-01 Thread Carl Eugen Hoyos
Reimar Döffinger Reimar.Doeffinger at gmx.de writes: -.priv_data_size = sizeof(MpegEncContext), +.priv_data_size = sizeof(Mpeg4DecContext), Please commit. Thank you for debugging this, Carl Eugen ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.

2014-11-01 Thread Carl Eugen Hoyos
Jean-Baptiste Kempf jb at videolan.org writes: On 01 Nov, Reimar Döffinger wrote : This is a regression, previous code did not require users to set the profile to use VDPAU. If you don't set the profile, how can you know that the hardware supports it? Does this have any relevance for

[FFmpeg-devel] [PATCH] ffmpeg: init sub2video.last_pts.

2014-11-01 Thread Nicolas George
Get the heartbeat working when the video has negative timestamps. Fix trac ticket #4062. Signed-off-by: Nicolas George geo...@nsup.org --- ffmpeg_filter.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ffmpeg_filter.c b/ffmpeg_filter.c index 2ba2d9b..0555cf6 100644 --- a/ffmpeg_filter.c

Re: [FFmpeg-devel] [PATCH 2/3] idet improvements: add reset_count feature

2014-11-01 Thread Kevin Mitchell
From 2ac0c4b6bab752c92ffa615c028cd4582441f680 Mon Sep 17 00:00:00 2001 From: Kevin Mitchell kevmi...@gmail.com Date: Sat, 1 Nov 2014 05:38:35 -0700 Subject: [PATCH 2/3] avfilter/vf_idet: add a reset_count option like cropdetect This can be useful for videos in which the interlacing pattern

Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.

2014-11-01 Thread Reimar Döffinger
On Sat, Nov 01, 2014 at 02:18:06PM +0100, Jean-Baptiste Kempf wrote: On 01 Nov, Reimar Döffinger wrote : This is a regression, previous code did not require users to set the profile to use VDPAU. If you don't set the profile, how can you know that the hardware supports it? Firstly: As I

Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.

2014-11-01 Thread Reimar Döffinger
On Sat, Nov 01, 2014 at 01:52:17PM +0100, wm4 wrote: On Sat, 1 Nov 2014 13:45:46 +0100 Reimar Döffinger reimar.doeffin...@gmx.de wrote: This is a regression, previous code did not require users to set the profile to use VDPAU. It still doesn't...? It does if you want the hwaccel API.

[FFmpeg-devel] [PATCH 3/3] idet improvements: add repeated field detection

2014-11-01 Thread Kevin Mitchell
From d8141813d2987a2f7f06f738062301254ecb4c28 Mon Sep 17 00:00:00 2001 From: Kevin Mitchell kevmi...@gmail.com Date: Sat, 1 Nov 2014 03:12:48 -0700 Subject: [PATCH 3/3] avfilter/vf_idet: add repeated field detection This is useful for detecting telecine --- doc/filters.texi | 25

Re: [FFmpeg-devel] [PATCH] vdpau_mpeg4: Do not fail on unknown profile.

2014-11-01 Thread wm4
On Sat, 1 Nov 2014 15:13:17 +0100 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Sat, Nov 01, 2014 at 01:52:17PM +0100, wm4 wrote: On Sat, 1 Nov 2014 13:45:46 +0100 Reimar Döffinger reimar.doeffin...@gmx.de wrote: This is a regression, previous code did not require users to

Re: [FFmpeg-devel] [PATCH] configure: add -Werror=declaration-after-statement for gcc

2014-11-01 Thread Lukasz Marek
On 01.11.2014 09:48, Nicolas George wrote: Le primidi 11 brumaire, an CCXXIII, Lukasz Marek a écrit : Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- configure | 1 + 1 file changed, 1 insertion(+) Did something change since 377dfa3d? Hmm, it can be annoying during

Re: [FFmpeg-devel] [PATCH] mpeg4vdpau: Fix priv data size.

2014-11-01 Thread Reimar Döffinger
On Sat, Nov 01, 2014 at 01:15:46PM +, Carl Eugen Hoyos wrote: Reimar Döffinger Reimar.Doeffinger at gmx.de writes: -.priv_data_size = sizeof(MpegEncContext), +.priv_data_size = sizeof(Mpeg4DecContext), Please commit. Pushed. ___

Re: [FFmpeg-devel] [PATCH] avformat/rtpdec_h261: code aligned to the HEVC code

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 01:01:53PM +0100, Thomas Volkert wrote: From: Thomas Volkert tho...@homer-conferencing.com --- libavformat/rtpdec_h261.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] OPW Qualification Task: Enable daemon mode for FFserver

2014-11-01 Thread Binathi Bingi
Hello I tried to incorporate the changes suggested in above mail. Now we have NoDaemon as by default as per the current standard. NoDaemon and Daemon are now treated as two separate options. Code is indented. Reason for fork fail included. Documentation has been changed. From

Re: [FFmpeg-devel] [PATCH] lavfi: add xbr filter

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 07:30:32PM +0530, arwa arif wrote: On Sat, Nov 1, 2014 at 4:40 PM, Stefano Sabatini stefa...@gmail.com wrote: On date Saturday 2014-11-01 04:27:24 +0530, arwa arif encoded: [...] On Fri, Oct 31, 2014 at 6:31 PM, Stefano Sabatini stefa...@gmail.com From

Re: [FFmpeg-devel] OPW Qualification Task: Enable daemon mode for FFserver

2014-11-01 Thread Nicolas George
Le primidi 11 brumaire, an CCXXIII, Binathi Bingi a écrit : I tried to incorporate the changes suggested in above mail. Now we have NoDaemon as by default as per the current standard. NoDaemon and Daemon are now treated as two separate options. Code is indented. Reason for fork fail included.

Re: [FFmpeg-devel] OPW Qualification Task: Enable daemon mode for FFserver

2014-11-01 Thread Lukasz Marek
On 01.11.2014 17:20, Binathi Bingi wrote: +if (config.ffserver_daemon) { +int ffserver_id = 0; You may change int to pid_t too, which is actually returned by fork() +pid_t sid = 0; + +ffserver_id = fork(); ___

Re: [FFmpeg-devel] [PATCH 1/3] idet improvements: add current frame classification to metadata

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 07:06:46AM -0700, Kevin Mitchell wrote: This was requested following my initial patch adding the metadata. Should resolve http://trac.ffmpeg.org/ticket/3832 doc/filters.texi | 13 + libavfilter/version.h |2 +- libavfilter/vf_idet.c | 36

Re: [FFmpeg-devel] [PATCH 2/3] idet improvements: add reset_count feature

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 07:08:53AM -0700, Kevin Mitchell wrote: doc/filters.texi |7 +++ libavfilter/version.h |2 +- libavfilter/vf_idet.c | 45 + libavfilter/vf_idet.h |2 ++ 4 files changed, 43 insertions(+), 13

Re: [FFmpeg-devel] [PATCH] lavfi: add xbr filter

2014-11-01 Thread arwa arif
Scale2x and xBR are very different. But they have implemented xBR also in their source code. Anyways, I found a good link -- http://imgur.com/a/XAwx5; for reference pictures. But, the answers are coming out to be different. Not sure, why this is happening. Maybe, it is because of different

[FFmpeg-devel] [PATCH] ffserver_config: improve AVOption handing

2014-11-01 Thread Lukasz Marek
AVOption are checked right after found in config file. It allows to report exact line in config file. Options provided more than once are threated as errors. It also fixes flag options handing. Flags may occur more than once in config file. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com

Re: [FFmpeg-devel] [PATCH] libavformat/assenc: Add ignore_gaps option

2014-11-01 Thread Clément Bœsch
On Thu, Oct 30, 2014 at 08:16:08PM -0500, Rodger Combs wrote: On Oct 30, 2014, at 07:48, wm4 nfx...@googlemail.com wrote: ... The option name is a bit unspecific - how about ignore_readorder? ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 1/2] avformat: add webp muxer

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 11:40:33AM +0100, Michael Niedermayer wrote: On Sat, Nov 01, 2014 at 10:21:20AM +, Carl Eugen Hoyos wrote: Michael Niedermayer michaelni at gmx.at writes: +{ loop, Number of times to loop the output: 0 - infinite loop, OFFSET(loop), +

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/libwebpenc: support P frames in webp animations

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 11:57:31AM +0100, Michael Niedermayer wrote: On Sat, Nov 01, 2014 at 10:54:33AM +0100, Clément Bœsch wrote: On Fri, Oct 31, 2014 at 04:48:08PM +0100, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/libwebpenc.c |

[FFmpeg-devel] [PATCH] avcodec/libwebpenc: add quality option

2014-11-01 Thread Michael Niedermayer
previously quality could only be set through qscale/global_quality but the scale was inverted. Using a separate option avoids the confusion from qscale working backward. Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/libwebpenc.c |8 libavcodec/version.h|

Re: [FFmpeg-devel] [PATCH 1/4] ffserver_config: drop requirement video size being multiple of 16

2014-11-01 Thread Reynaldo H. Verdejo Pinochet
Hi On 10/31/2014 11:00 PM, Lukasz Marek wrote: [..] diff --git a/ffserver_config.c b/ffserver_config.c index d589ff2..f11db03 100644 --- a/ffserver_config.c +++ b/ffserver_config.c @@ -819,8 +819,6 @@ static int ffserver_parse_config_stream(FFServerConfig *config, const char *cmd,

Re: [FFmpeg-devel] [PATCH 3/4] ffserver_config: improve error handling

2014-11-01 Thread Reynaldo H. Verdejo Pinochet
Hi On 10/31/2014 11:00 PM, Lukasz Marek wrote: [..] @@ -356,9 +356,7 @@ static int ffserver_parse_config_global(FFServerConfig *config, const char *cmd, if (!av_strcasecmp(cmd, Port)) WARNING(Port option is deprecated, use HTTPPort instead\n);

Re: [FFmpeg-devel] [PATCH] lavfi: add xbr filter

2014-11-01 Thread Michael Niedermayer
On Sat, Nov 01, 2014 at 11:35:11PM +0530, arwa arif wrote: Scale2x and xBR are very different. But they have implemented xBR also in their source code. Anyways, I found a good link -- http://imgur.com/a/XAwx5; for reference pictures. But, the answers are coming out to be different. Not sure,

Re: [FFmpeg-devel] [PATCH 4/4] ffserver_config: dont pass pointer to enum as pointer to int

2014-11-01 Thread Reynaldo H. Verdejo Pinochet
Looks OK. Feel free to push. Bests, -- Reynaldo H. Verdejo Pinochet Open Source Group Samsung Research America / Silicon Valley ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/4] ffserver_config: drop presets support

2014-11-01 Thread Lukasz Marek
On 01.11.2014 22:27, Reynaldo H. Verdejo Pinochet wrote: Patch seems OK but I'm missing on the rationalle behind dropping the preset support. Can you elaborate? Patch 1 and 2 are just suggestion from previous reviews by Stefano. Regarding presents itself I have no pressure to remove it, but I

Re: [FFmpeg-devel] [PATCH 3/4] ffserver_config: improve error handling

2014-11-01 Thread Lukasz Marek
On 01.11.2014 23:06, Reynaldo H. Verdejo Pinochet wrote: Hi On 10/31/2014 11:00 PM, Lukasz Marek wrote: [..] @@ -356,9 +356,7 @@ static int ffserver_parse_config_global(FFServerConfig *config, const char *cmd, if (!av_strcasecmp(cmd, Port)) WARNING(Port option is

Re: [FFmpeg-devel] [PATCH 3/4] ffserver_config: improve error handling

2014-11-01 Thread Reynaldo H. Verdejo Pinochet
On 11/01/2014 07:59 PM, Lukasz Marek wrote: [..] I decreased it by 1 automatically rather than for any reason. I didn't want to change logic where it was not needed, and it was not needed here. I guess you ask in general if there should be a limit (not just it is 65535 or 6). I don't

Re: [FFmpeg-devel] [PATCH 4/4] ffserver_config: dont pass pointer to enum as pointer to int

2014-11-01 Thread Lukasz Marek
On 1 November 2014 23:07, Reynaldo H. Verdejo Pinochet reyna...@osg.samsung.com wrote: Looks OK. Feel free to push. Pushed this one ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/4] ffserver_config: drop presets support

2014-11-01 Thread Reynaldo H. Verdejo Pinochet
Hi On 11/01/2014 07:42 PM, Lukasz Marek wrote: [..] Patch 1 and 2 are just suggestion from previous reviews by Stefano. Regarding presents itself I have no pressure to remove it, but I agree with Stefano that it seems to be dead feature. Got it. My take on these matters is usually that

Re: [FFmpeg-devel] [PATCH 2/4] ffserver_config: drop presets support

2014-11-01 Thread Lukasz Marek
On 2 November 2014 00:15, Reynaldo H. Verdejo Pinochet reyna...@osg.samsung.com wrote: Hi On 11/01/2014 07:42 PM, Lukasz Marek wrote: [..] Patch 1 and 2 are just suggestion from previous reviews by Stefano. Regarding presents itself I have no pressure to remove it, but I agree with

Re: [FFmpeg-devel] [PATCH 3/3] idet improvements: add repeated field detection

2014-11-01 Thread Kevin Mitchell
Updated to fit with half life patch On Sat, Nov 1, 2014 at 7:12 AM, Kevin Mitchell kevmi...@gmail.com wrote: From d31cbea4430043ad71f8ad3f099ee8cbbc997875 Mon Sep 17 00:00:00 2001 From: Kevin Mitchell kevmi...@gmail.com Date: Sat, 1 Nov 2014 05:38:35 -0700 Subject: [PATCH 2/2] avfilter/vf_idet:

Re: [FFmpeg-devel] [PATCH] lavfi: add xbr filter

2014-11-01 Thread arwa arif
On Sun, Nov 2, 2014 at 3:36 AM, Michael Niedermayer michae...@gmx.at wrote: On Sat, Nov 01, 2014 at 11:35:11PM +0530, arwa arif wrote: Scale2x and xBR are very different. But they have implemented xBR also in their source code. Anyways, I found a good link -- http://imgur.com/a/XAwx5; for

Re: [FFmpeg-devel] [PATCH] doc/codecs: Better documentation for the libavcodec bitexact flag

2014-11-01 Thread Timothy Gu
On Friday, October 31, 2014, Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at javascript:; --- doc/codecs.texi |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/codecs.texi b/doc/codecs.texi index 2bb6059..5cf2dc0

Re: [FFmpeg-devel] [patch 1/3]Fix bug for POWER LE: /libavcodec/ppc/me_cmp.c

2014-11-01 Thread Timothy Gu
你好: Hi, On Friday, October 31, 2014, Michael Niedermayer michae...@gmx.at wrote: Why do you keep sending patches which duplicate code? Maybe someone can translate this to a language you understand better then english: This is an Chinese translation. Hope this helps. 我把Michael的邮件翻译了一下。

[FFmpeg-devel] New FATE Server Based on Node.js

2014-11-01 Thread Timothy Gu
Hi all, ## Background For the past few weeks I have taken the liberty of learning Node.js. The inefficient godly Perl CGI code of fateserver obviously served as a non-example during my learning process. In the end, I decided to rewrite fateserver in asynchronous Javascript with Express.js and

Re: [FFmpeg-devel] New FATE Server Based on Node.js

2014-11-01 Thread Timothy Gu
Some more information... On Sat, Nov 1, 2014 at 10:33 PM, Timothy Gu timothyg...@gmail.com wrote: ## To-Dos Nevertheless, this is still an WIP. I have written history, results, and log pages, but not yet the index page. There might be bugs I have not discovered. The source is covered with