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

2014-11-10 Thread Marvin Scholz
On 11 Nov 2014, at 0:56, Timothy Gu wrote: > Real name please. Changed the patches to used my realname now. > These are two unrelated changes and should therefore be split into two > patches. Splitted them up into two different patches >> -if (NOT_EMPTY(s->content_type)) >> -av_d

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

2014-11-10 Thread Lukasz Marek
W dniu wtorek, 11 listopada 2014 Marvin Scholz 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 > stuff drives

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

2014-11-10 Thread Lukasz Marek
W dniu wtorek, 11 listopada 2014 napisał(a): > From: Lukasz Marek > > > Such strong requirement constraint doesn't allow to use 360p. > Changed to warning when video size is not being multiple of 2. > > Signed-off-by: Lukasz Marek > > --- > ffserver_config.c | 4 ++-- > 1 file changed, 2 inserti

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

2014-11-10 Thread Marvin Scholz
Please ignore the patches I sent before ([PATCH 1/2] ffserver_config: drop requirement video size being multiple of 16), git send-email did something unexpected and somehow picked the wrong ones… This mailing list stuff drives me crazy. ___ ffmpeg-dev

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

2014-11-10 Thread epirat07
From: Marvin Scholz use a default (audio/mpeg for historical reason) if none. Required since Icecast 2.4.1 Not using AVOption default because this breaks content-type warnings (needs to detect if no type was set by the user) --- libavformat/icecast.c | 2 ++ 1 file changed, 2 insertions(+) dif

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

2014-11-10 Thread epirat07
From: Lukasz Marek Such strong requirement constraint doesn't allow to use 360p. Changed to warning when video size is not being multiple of 2. Signed-off-by: Lukasz Marek --- ffserver_config.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ffserver_config.c b/ffserver

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

2014-11-10 Thread Lukasz Marek
On 11.11.2014 08:31, Lukasz Marek wrote: 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

[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 --- libavformat/ffmdec.c | 20

[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 --- 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 100644 --- a/libavcodec/o

[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 --- libavcodec/options_table.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h index b72f

[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 --- 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 +1618,35 @@ static int cop

[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 --- Changelog | 1 + doc/ffserver.texi | 10 ++- ffserver_config.c | 219 ++ ffserver_config.h |

[FFmpeg-devel] [PATCH 1/9] lavu/opt: introduce av_opt_is_set_to_default()

2014-11-10 Thread Lukasz Marek
TODO: bump minor version, update doc/APIchanges New function allows to check if option is set to its default Signed-off-by: Lukasz Marek --- libavutil/opt.c| 127 + libavutil/opt.h| 26 +++ tests/ref/fate/opt | 40 +++

[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 --- 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 get_preset_file_2(const cha

[FFmpeg-devel] [PATCH 2/9] lavu/opt: introduce av_opt_serialize()

2014-11-10 Thread Lukasz Marek
TODO: bump minor version, update doc/APIchanges Function allows to create string containing object's serialized options. Such string may be passed back to av_set_options_string() in order to restore options. Signed-off-by: Lukasz Marek --- libavutil/opt.c| 135 +

[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 --- libavformat/ffmdec.c | 41 - libavformat/ffmenc.c | 27 +-- 2 files changed, 61 insertions(+), 7 deletions(-) diff --git a/libavformat/ffmdec.c b/libavformat/ffmdec.c index 448762b..

[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 pa

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

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

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 wrote: > Signed-off-by: Timothy Gu > --- > I feel like using hsl() in this case is clearer that the color is plain > gray. Bikeshedding welcome. > --- > src/less/style.less | 31 +++ > 1 file changed, 19 insertions(+), 12 del

Re: [FFmpeg-devel] patch for hevc decoder

2014-11-10 Thread Timothy Gu
Hi, On Monday, November 10, 2014, WeiChangjiang wrote: > Hi guys, > The attached is a very minor fix for hevc decoder. For big negative > motion vector, namely, mv_y, the hevc_await_progress will use a negative > value for ff_thread_await_progress while it should be 0 because it needs to > wai

[FFmpeg-devel] patch for hevc decoder

2014-11-10 Thread WeiChangjiang
Hi guys, The attached is a very minor fix for hevc decoder. For big negative motion vector, namely, mv_y, the hevc_await_progress will use a negative value for ff_thread_await_progress while it should be 0 because it needs to wait the first line of ref frame to do ege extending. regards,Chang

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

2014-11-10 Thread epirat07
From: Marvin Scholz 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 100644 --- a/libavfo

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

2014-11-10 Thread epirat07
From: Marvin Scholz 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. --- libavformat/icecast

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

2014-11-10 Thread Carl Eugen Hoyos
Timothy Gu gmail.com> writes: > > From: ePirat gmail.com > > > 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 ok in this case? Carl Eugen ___

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

2014-11-10 Thread Lukasz Marek
On 11.11.2014 00:59, Timothy Gu wrote: On Monday, November 10, 2014, Michael Niedermayer wrote: From: Lukasz Marek > --- update |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM if tested. tested on my local repo and it is working as expected.

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 ffserver_parse_config_stre

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 wrote: > From: Lukasz Marek > > > --- > update |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > LGTM if tested. Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg

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

2014-11-10 Thread Timothy Gu
On Monday, November 10, 2014, Michael Niedermayer > wrote: > --- > update |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > LGTM if tested. Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffm

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 wrote: > Signed-off-by: Michael Niedermayer > > --- > update |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) LGTM Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpe

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

2014-11-10 Thread Timothy Gu
Hi, On Monday, November 10, 2014, wrote: > From: ePirat > 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 changes and should therefore be split into tw

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 wrote: > > Signed-off-by: Michael Niedermayer > > --- > > update |2 +- > > 1 file changed, 1

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

2014-11-10 Thread Michael Niedermayer
From: Lukasz Marek --- update |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/update b/update index 69b3b38..2612365 100644 --- a/update +++ b/update @@ -53,7 +53,7 @@ check_trailing_whitespace () { } check_ending_newline () { - for file in $(git diff-tree --name-only

[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 --- 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 diff-tree --name-only

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

2014-11-10 Thread Michael Niedermayer
--- update |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/update b/update index 92e645d..69b3b38 100644 --- a/update +++ b/update @@ -67,7 +67,7 @@ check_ending_newline () { check_tabs () { - for file in $(git diff-tree --name-only --diff-filter="A|C|M|R" -r $REVISION

[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] Fixes for the Icecast protocol

2014-11-10 Thread epirat07
From: ePirat 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) --- libavformat/icecast.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/icecast.c b/libavformat/icecast.

[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 --- ffplay.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index 490bffa..f79161d 100644 --- a/ff

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

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 > +++ b/libavfilter/x86/vf

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

2014-11-10 Thread Carl Eugen Hoyos
Kieran Kunhya 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 lines).

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

2014-11-10 Thread Carl Eugen Hoyos
TOYAMA Shin-ichi wmail.plala.or.jp> writes: > # I think it will take long for me to update documents ... Then please send your next patch with a note that you are unable to update the documentation. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-de

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

2014-11-10 Thread Kieran Kunhya
--- libavfilter/vf_interlace.c | 55 - libavfilter/x86/Makefile| 2 + libavfilter/x86/vf_interlace.asm| 80 + libavfilter/x86/vf_interlace_init.c | 42 +++ 4 files changed, 150 insertions(+), 29 d

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

2014-11-10 Thread Michael Niedermayer
On Mon, Nov 10, 2014 at 06:21:28PM +0100, Clément Bœsch wrote: > From: Clément Bœsch > > --- > 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 >

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

[FFmpeg-devel] [PATCH 1/2] x86/flacdsp: separate decoder and encoder dsp initialization

2014-11-10 Thread James Almer
Signed-off-by: James Almer --- libavcodec/x86/flacdsp_init.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/libavcodec/x86/flacdsp_init.c b/libavcodec/x86/flacdsp_init.c index ad88e5b..33b31f1 100644 --- a/libavcodec/x86/flacdsp_init.c +++ b/libavcodec/x86/fl

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

2014-11-10 Thread James Almer
Signed-off-by: James Almer --- 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 +- libavcodec/flace

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

2014-11-10 Thread Timothy Gu
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 wrote: > Signed-off-by: Michael Niedermayer > --- > update |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/update b/update > index 86d51

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(-) > d51346e5fd9e1b4c142c0f

[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 Date: Mon, 10 Nov 2014 20:31:14 +0100 Subject: [PATCH] lavd/avfoundation: Use internal av_strto

[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 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 first valid index into

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 wrote: > On date Monday 2014-11-10 13:26:14 +0530, Binathi Bingi encoded: > [...] > > From c9d037758693a1522258a64849f7629d7cbd7408 Mon Sep 17 00:00:00 2001 > > From

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

2014-11-10 Thread Clément Bœsch
From: Clément Bœsch --- 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 @@ -1272,10 +1272,9 @@ static in

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 .

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 Lin";; >

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 Provide complete name, the patch is used to track copyright. > Date: Tue, 4 Nov 2014 21:42:07 +0530 > Subject: [PATCH] Restore Daemon mo

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 u

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 wrote: > > > + > > + /* YCrCb - RGB conversion */ > > + Cr = Cr - 128; > > + Cb = Cb - 128; > > > + R = Y + Cr + (Cr>>2) + (Cr>>3) + (Cr>>5); > > + G = Y - ((Cb>>2) + (Cb>>4) + (Cb>>5)) - ((Cr>>1) + (Cr>>3) + > (Cr>>4) + (Cr

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 d

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 ff

[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 --- 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 diff-tree --name-only

[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. http://ffmpeg.org/pipermail/ffmpeg-user/2013-Se

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

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 pa

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: 9FF2128B147EF6730BADF133

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 > >>--- > >> libavutil/opt.

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 > Date: Thu, 30 Oct 2014 22:06:20 +0530 > Subject: [PATCH] lavfi: add xbr filter xBR > > --- > Changelog|1 + > LICENS

[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: 9FF2128B147EF6730B

[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";; : 2014??11??10??(??) 11:32 ??: "rongyan"; : Fwd

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

2014-11-10 Thread rongyan
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 -- -- ??: "Tony Lin";; : 2014??11??10??(??) 11:33 ??: "rongyan"; : Fwd:

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:

[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 expr:gte(t

[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} (@emph{output,p

[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} (@emph{output,p

[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 T