[FFmpeg-devel] [PATCH] Modified force_key_frames option to accept frame numbers

2014-11-10 Thread Sylvester Zaluga
Attached a patch which allows for passing in frame numbers at which key frames should be forced. The current modes in which force_key_frames operate are: * force_key_frames 1,2,3,4,5 - parameter is interpreted as timestamps * force_key_frames expr:gte(t,n_forced) - parameter is an expression

[FFmpeg-devel] [PATCH] Modified force_key_frames command line option to accept frame numbers

2014-11-10 Thread Sylvester Zaluga
@item -force_key_frames[:@var{stream_specifier}] @var{time}[,@var{time}...] (@emph{output,per-stream}) +@item -force_key_frames[:@var{stream_specifier}] n:@var{number}[,@var{number}...] (@emph{output,per-stream}) @item -force_key_frames[:@var{stream_specifier}] expr:@var{expr}

[FFmpeg-devel] [PATCH] Modified force_key_frames command line option to accept frame numbers

2014-11-10 Thread Sylvester Zaluga
@item -force_key_frames[:@var{stream_specifier}] @var{time}[,@var{time}...] (@emph{output,per-stream}) +@item -force_key_frames[:@var{stream_specifier}] n:@var{number}[,@var{number}...] (@emph{output,per-stream}) @item -force_key_frames[:@var{stream_specifier}] expr:@var{expr}

[FFmpeg-devel] Modified force_key_frames option to accept frame numbers

2014-11-10 Thread Sylvester Zaluga
***Sending again, this time with diff*** Attached a patch which allows for passing in frame numbers at which key frames should be forced. The current modes in which force_key_frames operate are: * force_key_frames 1,2,3,4,5 - parameter is interpreted as timestamps * force_key_frames

Re: [FFmpeg-devel] Modified force_key_frames option to accept frame numbers

2014-11-10 Thread Lou Logan
Hi, On Sun, Nov 9, 2014, at 02:42 PM, Sylvester Zaluga wrote: ***Sending again, this time with diff*** You sent four similar messages to the mailing list: https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2014-November/165155.html

[FFmpeg-devel] Resubmit patch02 - Fwd: [patch 4/4] Fix bug for POWERLE: libswscale/ppc/swscale_altivec.c

2014-11-10 Thread rongyan
Hi, New patch please find in the attachment. There are two patches to re-submit, this is the second. The fate test result is here: Rong Yan -- -- ??: Tony Lin;linzhaolo...@gmail.com; : 2014??11??10??(??) 11:32 ??:

[FFmpeg-devel] ffmpeg git hooks

2014-11-10 Thread Michael Niedermayer
Hi Are the hooks which are used for ffmpeg git publically visible somewhere ? iam asking as we would like to submit patches to fix some issues in them. For example we would like to add tests/fate/*.mak to the exceptions which allow tabs Thanks -- Michael GnuPG fingerprint:

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

2014-11-10 Thread Stefano Sabatini
On date Sunday 2014-11-09 23:06:17 +0530, arwa arif encoded: [...] From 04b57f43e5904b9016001b86daa96c9a88f64b0e Mon Sep 17 00:00:00 2001 From: Arwa Arif arwaarif1...@gmail.com Date: Thu, 30 Oct 2014 22:06:20 +0530 Subject: [PATCH] lavfi: add xbr filter xBR --- Changelog|

Re: [FFmpeg-devel] [PATCH 1/4] lavu/opt: check for NULL before parsing

2014-11-10 Thread Michael Niedermayer
On Mon, Nov 10, 2014 at 06:22:07AM +0100, Lukasz Marek wrote: On 10.11.2014 03:21, Michael Niedermayer wrote: On Sun, Nov 09, 2014 at 11:22:46PM +0100, Lukasz Marek wrote: set_string_binary crashes with called with val=NULL Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com ---

Re: [FFmpeg-devel] [PATCH 4/4] fate: add opt-test

2014-11-10 Thread Michael Niedermayer
On Sun, Nov 09, 2014 at 11:22:49PM +0100, Lukasz Marek wrote: --- tests/fate/libavutil.mak | 4 +++ tests/ref/fate/opt | 90 2 files changed, 94 insertions(+) LGTM [...] -- Michael GnuPG fingerprint:

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

2014-11-10 Thread Nicolas George
Le decadi 20 brumaire, an CCXXIII, Binathi Bingi a écrit : I use nano editor. If you expect you will work on source codes (not necessarily just FFmpeg) a lot, I believe taking the time of learning a better editor would be a very good investment. I tried to remove trailing whitespace in git

Re: [FFmpeg-devel] Modified force_key_frames option to accept frame numbers

2014-11-10 Thread Nicolas George
Le nonidi 19 brumaire, an CCXXIII, Sylvester Zaluga a écrit : ***Sending again, this time with diff*** Better use git format-patch (or git send-email), so that you have authorship and date information. Attached a patch which allows for passing in frame numbers at which key frames should be

[FFmpeg-devel] [PATCH] Enables parsing DVD color palette directly from IFO file

2014-11-10 Thread TOYAMA Shin-ichi
Attached patch enables parsing DVD color palette directly from IFO file via -palette option using syntax `-palette ifo:filename` This is a straightforward implementation of the procedure described in the following post to ffmpeg-user mailing list.

[FFmpeg-devel] [PATCH] allow tabs in *.mak, they are part of the makefile syntax

2014-11-10 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- update |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/update b/update index 86d51db..92e645d 100644 --- a/update +++ b/update @@ -67,7 +67,7 @@ check_ending_newline () { check_tabs () { - for file in $(git

Re: [FFmpeg-devel] [PATCH] Enables parsing DVD color palette directly from IFO file

2014-11-10 Thread Nicolas George
Le decadi 20 brumaire, an CCXXIII, TOYAMA Shin-ichi a écrit : Attached patch enables parsing DVD color palette directly from IFO file via -palette option using syntax `-palette ifo:filename` This is a straightforward implementation of the procedure described in the following post to

Re: [FFmpeg-devel] [PATCH] Enables parsing DVD color palette directly from IFO file

2014-11-10 Thread James Darnley
On 2014-11-10 15:01, Nicolas George wrote: + fread(c, 1, 1, in); + fread(Y, 1, 1, in); + fread(Cr, 1, 1, in); + fread(Cb, 1, 1, in); You could replace that with a single read into an array; that would not allow to call the variables R, Cr, Cb, but that does not matter

Re: [FFmpeg-devel] [PATCH] Enables parsing DVD color palette directly from IFO file

2014-11-10 Thread Hendrik Leppkes
On Mon, Nov 10, 2014 at 3:01 PM, Nicolas George geo...@nsup.org wrote: + + /* YCrCb - RGB conversion */ + Cr = Cr - 128; + Cb = Cb - 128; + R = Y + Cr + (Cr2) + (Cr3) + (Cr5); + G = Y - ((Cb2) + (Cb4) + (Cb5)) - ((Cr1) + (Cr3) + (Cr4) + (Cr5)); + B =

Re: [FFmpeg-devel] Resubmit patch01 - RE: [patch 2/4] Fix bug for POWERLE: libavcodec/ppc/me_cmp.c

2014-11-10 Thread Michael Niedermayer
On Mon, Nov 10, 2014 at 05:01:56PM +0800, rongyan wrote: Hi, New patch please find in the attachment. There are two patches to re-submit, this is the first. The fate test result is here: Rong Yan [...] -vector unsigned char pix2l = vec_ld(0, pix2); -vector unsigned

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

2014-11-10 Thread Stefano Sabatini
On date Monday 2014-11-10 13:26:14 +0530, Binathi Bingi encoded: [...] From c9d037758693a1522258a64849f7629d7cbd7408 Mon Sep 17 00:00:00 2001 From: Binathi binti...@gmail.com Provide complete name, the patch is used to track copyright. Date: Tue, 4 Nov 2014 21:42:07 +0530 Subject: [PATCH]

Re: [FFmpeg-devel] Resubmit patch02 - Fwd: [patch 4/4] Fix bug for POWERLE: libswscale/ppc/swscale_altivec.c

2014-11-10 Thread Michael Niedermayer
On Mon, Nov 10, 2014 at 05:03:22PM +0800, rongyan wrote: Hi, New patch please find in the attachment. There are two patches to re-submit, this is the second. The fate test result is here: Rong Yan -- 原始邮件 -- 发件人: Tony

Re: [FFmpeg-devel] [PATCH] Enables parsing DVD color palette directly from IFO file

2014-11-10 Thread TOYAMA Shin-ichi
Nicolas George wrote in 20141110140122.ga22...@phare.normalesup.org Thanks for the patch. See comments below. Thank you for your quick reply. I will go through many comments from you and others, and will post revised patch in some time. # I think it will take long for me to update documents ...

[FFmpeg-devel] [PATCH] avformat/mov: strengthen some table allocations

2014-11-10 Thread Clément Bœsch
From: Clément Bœsch clem...@stupeflix.com --- libavformat/mov.c | 35 ++- 1 file changed, 14 insertions(+), 21 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 6ba7b96..4010668 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@

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

2014-11-10 Thread Binathi Bingi
Hi there, Please find the attached patch with modifications. Regards, Binathi On Mon, Nov 10, 2014 at 8:42 PM, Stefano Sabatini stefa...@gmail.com wrote: On date Monday 2014-11-10 13:26:14 +0530, Binathi Bingi encoded: [...] From c9d037758693a1522258a64849f7629d7cbd7408 Mon Sep 17

[FFmpeg-devel] fix for colorspace minimum option value

2014-11-10 Thread jon
From 616d5017d8d2e566db3deb2696cc1672f2019777 Mon Sep 17 00:00:00 2001 From: Jon Morley j...@tweaksoftware.com Date: Mon, 10 Nov 2014 10:43:42 -0800 Subject: [PATCH] options_table.h: min value for colorspace is 0 (AVCOL_SPC_RGB) The min value for colorspace should be zero and not one since the

[FFmpeg-devel] [PATCH] lavd/avfoundation: Use internal av_strtok instead of std lib strtok

2014-11-10 Thread Thilo Borgmann
Hi, proposed on github. I don't see any reason why not to stick to internal variants if we have one. -Thilo From ae4efc96a742ede2fa59ca6cc59ffc1979c75f72 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann thilo.borgm...@mail.de Date: Mon, 10 Nov 2014 20:31:14 +0100 Subject: [PATCH] lavd/avfoundation:

Re: [FFmpeg-devel] [PATCH] lavd/avfoundation: Use internal av_strtok instead of std lib strtok

2014-11-10 Thread Michael Niedermayer
On Mon, Nov 10, 2014 at 08:36:37PM +0100, Thilo Borgmann wrote: Hi, proposed on github. I don't see any reason why not to stick to internal variants if we have one. -Thilo avfoundation.m |8 +--- 1 file changed, 5 insertions(+), 3 deletions(-)

[FFmpeg-devel] [PATCH 2/2] x86/flacdsp: add SSE2 and AVX decorrelate functions

2014-11-10 Thread James Almer
Signed-off-by: James Almer jamr...@gmail.com --- Now also with indep8 (x86_64 only), and I'm pretty much done with these. libavcodec/arm/flacdsp_init_arm.c | 2 +- libavcodec/flacdec.c | 6 +- libavcodec/flacdsp.c | 6 +- libavcodec/flacdsp.h | 6 +-

Re: [FFmpeg-devel] fix for colorspace minimum option value

2014-11-10 Thread Michael Niedermayer
On Mon, Nov 10, 2014 at 10:49:42AM -0800, jon wrote: From 616d5017d8d2e566db3deb2696cc1672f2019777 Mon Sep 17 00:00:00 2001 From: Jon Morley j...@tweaksoftware.com Date: Mon, 10 Nov 2014 10:43:42 -0800 Subject: [PATCH] options_table.h: min value for colorspace is 0 (AVCOL_SPC_RGB) The min

Re: [FFmpeg-devel] [PATCH] vf_interlace: Add SIMD for lowpass filter

2014-11-10 Thread Carl Eugen Hoyos
Kieran Kunhya kierank at obe.tv writes: libavfilter/vf_interlace.c | 55 - patching file libavfilter/vf_interlace.c Hunk #1 FAILED at 30. Hunk #2 FAILED at 70. Hunk #3 FAILED at 116. Hunk #4 succeeded at 152 (offset -1 lines). Hunk #5 succeeded at 202 (offset -2

Re: [FFmpeg-devel] [PATCH] vf_interlace: Add SIMD for lowpass filter

2014-11-10 Thread James Almer
On 10/11/14 6:42 PM, Kieran Kunhya wrote: Can't test since it doesn't apply cleanly, but here are a few comments anyway. diff --git a/libavfilter/x86/vf_interlace.asm b/libavfilter/x86/vf_interlace.asm new file mode 100644 index 000..40b10fc --- /dev/null +++

Re: [FFmpeg-devel] [PATCH] vf_interlace: Add SIMD for lowpass filter

2014-11-10 Thread Kieran Kunhya
Can't pavgw be used here? I don't think so? Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] ffplay: signal the frame queue before closing audio

2014-11-10 Thread Marton Balint
Fixed regression caused by 631ac655c00e978e19d05dab572bc1ffd6078c63 when ffplay does not quit if the audio thread is blocked. Signed-off-by: Marton Balint c...@passwd.hu --- ffplay.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index 490bffa..f79161d

[FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread epirat07
Fixed space vs. tab messup ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 1/3] allow tabs in *.mak, they are part of the makefile syntax

2014-11-10 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- update |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/update b/update index 86d51db..92e645d 100644 --- a/update +++ b/update @@ -67,7 +67,7 @@ check_ending_newline () { check_tabs () { - for file in $(git

Re: [FFmpeg-devel] [PATCH] allow tabs in *.mak, they are part of the makefile syntax

2014-11-10 Thread Michael Niedermayer
On Mon, Nov 10, 2014 at 12:25:20PM -0800, Timothy Gu wrote: What is this? Next time would you mind adding a [TAG] in the summary? thanks! On Mon, Nov 10, 2014 at 5:57 AM, Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- update |2 +-

Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread Timothy Gu
Hi, On Monday, November 10, 2014, epira...@gmail.com wrote: From: ePirat epira...@gmail.com javascript:; Real name please. Send 100-continue header so we detect errors before sending data Use a default content-type (content-type is required since Icecast 2.4.1) These are two unrelated

Re: [FFmpeg-devel] [PATCH 1/3] allow tabs in *.mak, they are part of the makefile syntax

2014-11-10 Thread Timothy Gu
On Monday, November 10, 2014, Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at javascript:; --- update |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM Timothy ___ ffmpeg-devel mailing

[FFmpeg-devel] [PATCH 2/3] remove redundant explicit file.mak cases from the tab test

2014-11-10 Thread Timothy Gu
On Monday, November 10, 2014, Michael Niedermayer michae...@gmx.at javascript:_e(%7B%7D,'cvml','michae...@gmx.at'); wrote: --- update |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM if tested. Timothy ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 3/3] allow fate references to lack the final newline

2014-11-10 Thread Timothy Gu
On Monday, November 10, 2014, Michael Niedermayer michae...@gmx.at wrote: From: Lukasz Marek lukasz.m.lu...@gmail.com javascript:; --- update |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM if tested. Timothy ___ ffmpeg-devel

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

2014-11-10 Thread Lukasz Marek
On 01.11.2014 22:20, Reynaldo H. Verdejo Pinochet wrote: 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

Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread Carl Eugen Hoyos
Timothy Gu timothygu99 at gmail.com writes: From: ePirat epirat07 at gmail.com javascript:; Real name please. Could you elaborate? There are both active developers who don't use real names and copyright holders for different source files who chose to use an alias. Why shouldn't this be

[FFmpeg-devel] [PATCH 2/2] Icecast: Use 100-continue if possible for proper error handling

2014-11-10 Thread epirat07
From: Marvin Scholz epira...@gmail.com Using 100-continue ffmpeg will only send data if the server confirms it, so if there is an error with auth or mounpoint, this allows that it is properly reported to the user. Else ffmpeg sends data and just quits at some point without an error message. ---

[FFmpeg-devel] [PATCH 1/2] Icecast: always send a content-type

2014-11-10 Thread epirat07
From: Marvin Scholz epira...@gmail.com use a default (audio/mpeg for historical reason) if none. Required since Icecast 2.4.1 --- libavformat/icecast.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavformat/icecast.c b/libavformat/icecast.c index 7d60e44..d7a2d18

Re: [FFmpeg-devel] [PATCH 3/4] web/style.less: colors: Reindent and use more obvious functions

2014-11-10 Thread Timothy Gu
On Thu, Nov 6, 2014 at 6:55 PM, Timothy Gu timothyg...@gmail.com wrote: Signed-off-by: Timothy Gu timothyg...@gmail.com --- I feel like using hsl() in this case is clearer that the color is plain gray. Bikeshedding welcome. --- src/less/style.less | 31 +++ 1

[FFmpeg-devel] Resubmit patch02 - [patch 4/4] Fix bug forPOWERLE: libswscale/ppc/swscale_altivec.c

2014-11-10 Thread rongyan
Hi, Thanks for your patience and review. New patch please find in the attachment. This is the second of two. The fate test result is here: Regards, Rong Yan Subject: [FFmpeg-devel] Resubmit

[FFmpeg-devel] [PATCH 3/9] lavf/ffm: store/restore private codec context

2014-11-10 Thread Lukasz Marek
TODO: bump micro Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavformat/ffmdec.c | 41 - libavformat/ffmenc.c | 27 +-- 2 files changed, 61 insertions(+), 7 deletions(-) diff --git a/libavformat/ffmdec.c

[FFmpeg-devel] [PATCH 4/9] ffmpeg_opt: allow to force codec in new_output_stream

2014-11-10 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffmpeg_opt.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index 77ef0c4..1b3f73a 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -1036,11 +1036,12 @@ static int

[FFmpeg-devel] [PATCH 5/9] ffmpeg_opt: use codec private context in ffserver streams

2014-11-10 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffmpeg_opt.c | 48 1 file changed, 32 insertions(+), 16 deletions(-) diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index 1b3f73a..fdbab58 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -1618,6

[FFmpeg-devel] [PATCH 6/9] ffserver_config: handle codec private options

2014-11-10 Thread Lukasz Marek
This commit allows to set codec's private option. As side effect this commit also improves preset support. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- Changelog | 1 + doc/ffserver.texi | 10 ++- ffserver_config.c | 219

[FFmpeg-devel] [PATCH 8/9] bavcodec/options_table: add pixel_format and video_size options

2014-11-10 Thread Lukasz Marek
Adding these options simplify ffm format implementation based on AVOption API. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavcodec/options_table.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h index beb767c..5dd0359

[FFmpeg-devel] [PATCH 7/9] libavcodec/options_table: set min to -1 for timecode_frame_start

2014-11-10 Thread Lukasz Marek
timecode_frame_start is set to -1 in avcodec_get_context_defaults3() AVOptions API complains about it. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavcodec/options_table.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/options_table.h

[FFmpeg-devel] [PATCH 9/9] lavf/ffm: use AVOption API to store/restore stream properties

2014-11-10 Thread Lukasz Marek
This is a generic solution that will not reqiore modifications when new options are added. This also fixes problem with current implementation when qmin or qmax=-1. Only 8 bits was sent and read back as 255. Fixes #1275 Fixes #1461 Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com ---

Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread Lukasz Marek
W dniu wtorek, 11 listopada 2014 Marvin Scholz epira...@gmail.com napisał(a): Please ignore the patches I sent before ([PATCH 1/2] ffserver_config: drop requirement videosize being multiple of 16), git send-email did something unexpected and somehow picked the wrong ones… This mailing list