Re: [FFmpeg-devel] Exact screen size not working: Re: [PATCH] Fix segfault with x11grab when switching windows.

2015-05-06 Thread Nicolas George
Le septidi 17 floréal, an CCXXIII, davidpbrown a écrit : tldr; Using exact screen size doesn't work but less one pixel does. The attached patch should fix this. Please feel anyone free to apply without waiting on me. davidpbrown: in the future, remember that bug reports should go to the users

[FFmpeg-devel] Exact screen size not working: Re: [PATCH] Fix segfault with x11grab when switching windows.

2015-05-06 Thread davidpbrown
@ffmpeg-devel@ffmpeg.org tldr; Using exact screen size doesn't work but less one pixel does. I'm new to ffmpeg but I see a recent message below and wonder there is an error now. The ffmpeg I built from source only yesterday. On a 1366x768 screen this below, sees the error suggested: ffmpeg

Re: [FFmpeg-devel] OS/2:Makedef.cmd cleanup

2015-05-06 Thread Michael Niedermayer
On Tue, May 05, 2015 at 09:07:59PM -0700, Dave Yeo wrote: From 448cf672d7793e0d5166d66c7daffacdc8a3af72 Mon Sep 17 00:00:00 2001 From: Dave Yeo daver...@telus.net Date: Tue, 5 May 2015 20:22:51 -0700 Subject: [PATCH] OS/2:Makedef.cmd cleanup Remove PROTMODE as it doesn't make sense for

[FFmpeg-devel] [PATCH] configure: replace arch loongson with arch extra list loongson

2015-05-06 Thread 周晓勇
From a5031b4c4b97f790a40603cff9a1f45cbb043005 Mon Sep 17 00:00:00 2001 From: ZhouXiaoyong zhouxiaoy...@loongson.cn Date: Wed, 6 May 2015 14:05:21 +0800 Subject: [PATCH] configure: replace arch loongson with arch extra list loongson fate pass when do configure without --cc='ccache gcc' option:

[FFmpeg-devel] [RFC]Fix flac with high lpc precision

2015-05-06 Thread Carl Eugen Hoyos
Hi! Attached is my attempt to fix ticket #4421 based on the analysis by trac user lvqcl. If the patch is ok, I will add the encoder check to the decoder and the version bump. Please review, Carl Eugen TODO: Fix indentation in libavcodec/x86/flacdsp_init.c() diff --git

[FFmpeg-devel] [PATCH] avcodec/libx265: use x265 Multi-library Interface to query the API

2015-05-06 Thread Gopu Govindaswamy
From 24fc78ebee006693f235ea2fd07994f7829b367e Mon Sep 17 00:00:00 2001 From: Gopu Govindaswamy g...@multicorewareinc.com Date: Wed, 6 May 2015 12:47:21 +0530 Subject: [PATCH] avcodec/libx265: use x265 Multi-library Interface to query the API ffmpeg can now use the x265 multi-library interface to

Re: [FFmpeg-devel] [PATCH] configure: replace arch loongson with arch extra list loongson

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 02:38:21PM +0800, 周晓勇 wrote: From a5031b4c4b97f790a40603cff9a1f45cbb043005 Mon Sep 17 00:00:00 2001 From: ZhouXiaoyong zhouxiaoy...@loongson.cn Date: Wed, 6 May 2015 14:05:21 +0800 Subject: [PATCH] configure: replace arch loongson with arch extra list loongson fate

Re: [FFmpeg-devel] Loongson Optimization Patchs and Machine Express Issuse

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 05:18:15PM +0800, 周晓勇 wrote: -原始邮件- 发件人: Michael Niedermayer mich...@niedermayer.cc 发送时间: 2015年5月6日 星期三 收件人: FFmpeg development discussions and patches ffmpeg-devel@ffmpeg.org 抄送: gaoxiang gaoxi...@loongson.cn, 孟小甫 mengxia...@loongson.cn 主题: Re:

Re: [FFmpeg-devel] [PATCH] diracdec: check if reference could not be allocated

2015-05-06 Thread Andreas Cadhalpun
On 06.05.2015 02:26, Michael Niedermayer wrote: On Tue, May 05, 2015 at 11:51:48PM +0200, Andreas Cadhalpun wrote: s-ref_pics[i] is later used as ref argument of interpolate_refplane, where it is dereferenced. If it is NULL, it causes a segmentation fault. Signed-off-by: Andreas Cadhalpun

[FFmpeg-devel] Re: [PATCH] diracdec: check that block length is large enough

2015-05-06 Thread Andreas Cadhalpun
On 06.05.2015 01:49, Michael Niedermayer wrote: On Tue, May 05, 2015 at 10:39:50PM +0200, Andreas Cadhalpun wrote: +if (s-plane[0].xblen s-chroma_x_shift = 0 || s-plane[0].yblen s-chroma_y_shift = 0) { +av_log(s-avctx, AV_LOG_ERROR, Block length too small\n); +return

[FFmpeg-devel] [PATCH] dirac: use and forward error codes

2015-05-06 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun andreas.cadhal...@googlemail.com --- libavcodec/dirac_dwt.c | 2 +- libavcodec/diracdec.c | 75 -- 2 files changed, 43 insertions(+), 34 deletions(-) diff --git a/libavcodec/dirac_dwt.c b/libavcodec/dirac_dwt.c

Re: [FFmpeg-devel] [RFC]Fix flac with high lpc precision

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 09:27:22AM +0200, Carl Eugen Hoyos wrote: Hi! Attached is my attempt to fix ticket #4421 based on the analysis by trac user lvqcl. If the patch is ok, I will add the encoder check to the decoder and the version bump. Please review, Carl Eugen TODO: Fix

Re: [FFmpeg-devel] Exact screen size not working: Re: [PATCH] Fix segfault with x11grab when switching windows.

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 11:55:25AM +0200, Nicolas George wrote: Le septidi 17 floréal, an CCXXIII, davidpbrown a écrit : tldr; Using exact screen size doesn't work but less one pixel does. The attached patch should fix this. Please feel anyone free to apply without waiting on me.

[FFmpeg-devel] [PATCH 5/9] vp9: parse profile 2/3 bitdepth in frame header.

2015-05-06 Thread Ronald S. Bultje
--- libavcodec/vp9.c | 36 1 file changed, 24 insertions(+), 12 deletions(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 4ba46f3..298470a 100644 --- a/libavcodec/vp9.c +++ b/libavcodec/vp9.c @@ -478,36 +478,47 @@ static enum AVPixelFormat

[FFmpeg-devel] [PATCH 7/9] vp9: add inter-frame profile 2/3 suport.

2015-05-06 Thread Ronald S. Bultje
--- libavcodec/vp9.c | 136 --- libavcodec/vp9_mc_template.c | 37 +++- libavcodec/vp9dsp_template.c | 81 ++ 3 files changed, 167 insertions(+), 87 deletions(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c

Re: [FFmpeg-devel] [PATCH] ff_h[yc]scale_fast_mmext always clobbers %rbx

2015-05-06 Thread Michael Niedermayer
Hi On Wed, May 06, 2015 at 11:52:59AM -0700, Nick Lewycky wrote: Newer versions of clang will allocate %rbx as one of the inline asm inputs, Thats great even in PIC. This occurs when building ffmpeg with clang -fsanitize=address -O1 -fPIE. Because the asm does clobber %bx whether PIC is on

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-05-06 Thread Michael Niedermayer
On Thu, May 07, 2015 at 01:29:11AM +0530, Anshul wrote: On 05/06/2015 01:21 AM, Michael Niedermayer wrote: On Tue, May 05, 2015 at 08:33:42PM +0530, Anshul wrote: [...] and here is shorter file then previous one.

Re: [FFmpeg-devel] [PATCH]Also support real-world qdraw samples

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 11:11:53PM +0200, Carl Eugen Hoyos wrote: Hi! Attached patch allows decoding of the qdraw files I found. Please comment, Carl Eugen if that works, i guess its an improvment and should be ok [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] ffserver jpg patch

2015-05-06 Thread ill
Feed cam.ffm File /tmp/cam.ffm #when remarked, no file is beeing created and the stream keeps working!! FileMaxSize 400K # Only allow connections from localhost to the feed. ACL allow 127.0.0.1 ACL allow 192.168.1.2 192.168.1.49 /Feed Stream test.jpg

Re: [FFmpeg-devel] [PATCH] ff_h[yc]scale_fast_mmext always clobbers %rbx

2015-05-06 Thread Nick Lewycky
On 6 May 2015 at 15:06, Michael Niedermayer michae...@gmx.at wrote: Hi On Wed, May 06, 2015 at 11:52:59AM -0700, Nick Lewycky wrote: Newer versions of clang will allocate %rbx as one of the inline asm inputs, Thats great even in PIC. This occurs when building ffmpeg with clang

Re: [FFmpeg-devel] [PATCH] ff_h[yc]scale_fast_mmext always clobbers %rbx

2015-05-06 Thread Carl Eugen Hoyos
Nick Lewycky nlewycky at google.com writes: On 6 May 2015 at 11:58, Carl Eugen Hoyos: Nick Lewycky nlewycky at google.com writes: This occurs when building ffmpeg with clang -fsanitize=address -O1 -fPIE. What is the usecase for -O1? It's very important for asan. ASan

Re: [FFmpeg-devel] ffserver jpg patch

2015-05-06 Thread Reynaldo H. Verdejo Pinochet
Patch looks OK at a first glance. Any chance you can attach the relevant feed and stream sections of the ffserver.conf file you used in your tests? Also, it would be great if you can use git format-patch with your email/name so we can preserve autorship information on the commit. Bests, --

Re: [FFmpeg-devel] [PATCH 6/9] vp9: add keyframe profile 2/3 support.

2015-05-06 Thread Carl Eugen Hoyos
Ronald S. Bultje rsbultje at gmail.com writes: I still wonder if it was a good idea to add the formats 8 and 16... I think the biggest issue with going with 16 and using bits_per_coded_sample, is to enforce that the lowest bits are actually zero. I think this shouldn't be enforced.

[FFmpeg-devel] [PATCH]qdraw autodetection

2015-05-06 Thread Carl Eugen Hoyos
Hi! Attached patch allows the autodetection of qdraw images as decoded by libavcodec. Please comment, Carl Eugen diff --git a/libavformat/Makefile b/libavformat/Makefile index 8d9a770..bca9d5b 100644 --- a/libavformat/Makefile +++ b/libavformat/Makefile @@ -206,6 +206,7 @@

Re: [FFmpeg-devel] [PATCH 6/9] vp9: add keyframe profile 2/3 support.

2015-05-06 Thread Carl Eugen Hoyos
Ronald S. Bultje rsbultje at gmail.com writes: upper bits are zero in libvpx, so it uses formats that are analogous to our p10/p12 pixfmts. It (confusingly) calls the formats p16 in its internal API, and sets bits_per_sample, but this has the exact opposite meaning of what it means in

Re: [FFmpeg-devel] [PATCH]qdraw autodetection

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 10:05:29PM +0200, Carl Eugen Hoyos wrote: Hi! Attached patch allows the autodetection of qdraw images as decoded by libavcodec. Please comment, Carl Eugen Makefile |1 + allformats.c |1 + img2dec.c| 10 ++ version.h|4 ++--

[FFmpeg-devel] [PATCH]Also support real-world qdraw samples

2015-05-06 Thread Carl Eugen Hoyos
Hi! Attached patch allows decoding of the qdraw files I found. Please comment, Carl Eugen diff --git a/libavcodec/qdrw.c b/libavcodec/qdrw.c index b650ade..b7a7cf5 100644 --- a/libavcodec/qdrw.c +++ b/libavcodec/qdrw.c @@ -124,6 +124,10 @@ static int decode_frame(AVCodecContext *avctx, int

Re: [FFmpeg-devel] [PATCH] ff_h[yc]scale_fast_mmext always clobbers %rbx

2015-05-06 Thread Nick Lewycky
On 6 May 2015 at 13:08, Carl Eugen Hoyos ceho...@ag.or.at wrote: Nick Lewycky nlewycky at google.com writes: On 6 May 2015 at 11:58, Carl Eugen Hoyos: Nick Lewycky nlewycky at google.com writes: This occurs when building ffmpeg with clang -fsanitize=address -O1 -fPIE.

Re: [FFmpeg-devel] [PATCH 6/9] vp9: add keyframe profile 2/3 support.

2015-05-06 Thread Ronald S. Bultje
Hi, On Wed, May 6, 2015 at 4:28 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Ronald S. Bultje rsbultje at gmail.com writes: I still wonder if it was a good idea to add the formats 8 and 16... I think the biggest issue with going with 16 and using bits_per_coded_sample, is to

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-05-06 Thread Carl Eugen Hoyos
Michael Niedermayer michael at niedermayer.cc writes: please make a smaller file this is still 16 times larger than the next largest file in fate/sub I uploaded a 1.5M file: http://www.datafilehost.com/d/02ce0ef8 $ ffmpeg -i Closedcaption_rollup.ts -vcodec copy -ss 37 -t 5 out.m2v If

Re: [FFmpeg-devel] [PATCH 6/9] vp9: add keyframe profile 2/3 support.

2015-05-06 Thread Ronald S. Bultje
Hi, On Wed, May 6, 2015 at 2:00 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Ronald S. Bultje rsbultje at gmail.com writes: Overall, the effect would be minor, like in the lower single-digit percents or perhaps even fractional percent, but I would absolutely expect a small performance

[FFmpeg-devel] [PATCH 8/9] vp9: add profile 2/3 fate tests.

2015-05-06 Thread Ronald S. Bultje
--- tests/fate/vpx.mak | 8 +--- tests/ref/fate/vp9p2-20-10bit-yuv420 | 15 +++ tests/ref/fate/vp9p2-20-12bit-yuv420 | 15 +++ tests/ref/fate/vp9p3-20-10bit-yuv422 | 15 +++ tests/ref/fate/vp9p3-20-10bit-yuv440 | 25

[FFmpeg-devel] [PATCH 9/9] libvpxdec: add 440 pixfmts.

2015-05-06 Thread Ronald S. Bultje
--- libavcodec/libvpxdec.c | 13 + 1 file changed, 13 insertions(+) diff --git a/libavcodec/libvpxdec.c b/libavcodec/libvpxdec.c index 75231f8..11f7be1 100644 --- a/libavcodec/libvpxdec.c +++ b/libavcodec/libvpxdec.c @@ -72,6 +72,9 @@ static int set_pix_fmt(AVCodecContext *avctx,

[FFmpeg-devel] [PATCH] ff_h[yc]scale_fast_mmext always clobbers %rbx

2015-05-06 Thread Nick Lewycky
Newer versions of clang will allocate %rbx as one of the inline asm inputs, even in PIC. This occurs when building ffmpeg with clang -fsanitize=address -O1 -fPIE. Because the asm does clobber %bx whether PIC is on or off, just include %bx in the clobber list regardless of whether PIC is on or off.

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-05-06 Thread Anshul
On 05/06/2015 01:21 AM, Michael Niedermayer wrote: On Tue, May 05, 2015 at 08:33:42PM +0530, Anshul wrote: [...] and here is shorter file then previous one. http://gsocdev.ccextractor.org/~anshul/test_video/Closedcaption_atsc_rollup.ts considering that this is created by FFmpeg Metadata:

Re: [FFmpeg-devel] [PATCH] ff_h[yc]scale_fast_mmext always clobbers %rbx

2015-05-06 Thread Carl Eugen Hoyos
Nick Lewycky nlewycky at google.com writes: This occurs when building ffmpeg with clang -fsanitize=address -O1 -fPIE. What is the usecase for -O1? Did you check performance with and without your patch? Carl Eugen ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 6/9] vp9: add keyframe profile 2/3 support.

2015-05-06 Thread Carl Eugen Hoyos
Ronald S. Bultje rsbultje at gmail.com writes: +static void vert_4x4_c(uint8_t *_dst, ptrdiff_t stride, + const uint8_t *left, const uint8_t *_top) Once upon a time, it was claimed that we must not use identifiers starting with _. Would it be slower to decode to

[FFmpeg-devel] [PATCH 1/9] lavu: add yuv440p10/12 pixfmts.

2015-05-06 Thread Ronald S. Bultje
--- libavutil/pixdesc.c | 48 libavutil/pixfmt.h | 6 ++ 2 files changed, 54 insertions(+) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index de1ea7fb..5086edc 100644 --- a/libavutil/pixdesc.c +++ b/libavutil/pixdesc.c @@ -605,6

[FFmpeg-devel] ffserver jpg patch

2015-05-06 Thread ill
Can this patch be applied now? It doesn't have any line breaks anymore. On Mon, May 04, 2015 at 06:35:36PM -0600, ill wrote: I changed it to git and add the .write header to the last part. The first part compiled after I changed ffserver.c, so it is no longer corrupted. Muxer detection

Re: [FFmpeg-devel] [PATCH 6/9] vp9: add keyframe profile 2/3 support.

2015-05-06 Thread Ronald S. Bultje
Hi, On Wed, May 6, 2015 at 1:40 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Ronald S. Bultje rsbultje at gmail.com writes: +static void vert_4x4_c(uint8_t *_dst, ptrdiff_t stride, + const uint8_t *left, const uint8_t *_top) Once upon a time, it was claimed that we

[FFmpeg-devel] [PATCH 4/9] vp9: add profile 2/3 to exported profiles.

2015-05-06 Thread Ronald S. Bultje
--- libavcodec/avcodec.h | 2 ++ libavcodec/vp9.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 3d5e82d..f1a330e 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@ -2914,6 +2914,8 @@ typedef struct AVCodecContext {

Re: [FFmpeg-devel] [PATCH] diracdec: check that block length is large enough

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 03:39:02PM +0200, Andreas Cadhalpun wrote: On 06.05.2015 01:49, Michael Niedermayer wrote: On Tue, May 05, 2015 at 10:39:50PM +0200, Andreas Cadhalpun wrote: +if (s-plane[0].xblen s-chroma_x_shift = 0 || s-plane[0].yblen s-chroma_y_shift = 0) { +

Re: [FFmpeg-devel] [PATCH 6/9] vp9: add keyframe profile 2/3 support.

2015-05-06 Thread Carl Eugen Hoyos
Ronald S. Bultje rsbultje at gmail.com writes: Overall, the effect would be minor, like in the lower single-digit percents or perhaps even fractional percent, but I would absolutely expect a small performance gain from using p10/p12 over p16 w/ bits_per_coded_sample. Also note most of

Re: [FFmpeg-devel] [PATCH] ff_h[yc]scale_fast_mmext always clobbers %rbx

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 04:08:09PM -0700, Nick Lewycky wrote: On 6 May 2015 at 15:06, Michael Niedermayer michae...@gmx.at wrote: Hi On Wed, May 06, 2015 at 11:52:59AM -0700, Nick Lewycky wrote: Newer versions of clang will allocate %rbx as one of the inline asm inputs, Thats

Re: [FFmpeg-devel] [PATCH] configure: replace arch loongson with arch extra list loongson

2015-05-06 Thread 周晓勇
-原始邮件- 发件人: Michael Niedermayer mich...@niedermayer.cc 发送时间: 2015年5月6日 星期三 收件人: FFmpeg development discussions and patches ffmpeg-devel@ffmpeg.org 抄送: gaoxiang gaoxi...@loongson.cn, 孟小甫 mengxia...@loongson.cn 主题: Re: [FFmpeg-devel] [PATCH] configure: replace arch loongson with

Re: [FFmpeg-devel] [PATCH] configure: replace arch loongson with arch extra list loongson

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 02:38:21PM +0800, 周晓勇 wrote: From a5031b4c4b97f790a40603cff9a1f45cbb043005 Mon Sep 17 00:00:00 2001 From: ZhouXiaoyong zhouxiaoy...@loongson.cn Date: Wed, 6 May 2015 14:05:21 +0800 Subject: [PATCH] configure: replace arch loongson with arch extra list loongson fate

[FFmpeg-devel] [PATCH 1/2] avformat/mxf: Simplify PRINT_KEY()

2015-05-06 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/mxf.h | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/libavformat/mxf.h b/libavformat/mxf.h index 71a4084..dd08bdc 100644 --- a/libavformat/mxf.h +++ b/libavformat/mxf.h @@ -102,22 +102,14 @@

[FFmpeg-devel] [PATCH 2/2] avformat/mxf: Always evaluate PRINT_KEY() arguments at compile time

2015-05-06 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/mxf.h | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/libavformat/mxf.h b/libavformat/mxf.h index dd08bdc..1763063 100644 --- a/libavformat/mxf.h +++ b/libavformat/mxf.h @@ -111,7 +111,15 @@ const

[FFmpeg-devel] [PATCH] ripemd: move ripemd{256, 320} into separate functions

2015-05-06 Thread James Almer
This allows the removal of a few branches. Before: lavu RIPEMD-160 size: 1048576 runs: 1024 time:7.052 +- 0.010 After: lavu RIPEMD-160 size: 1048576 runs: 1024 time:6.865 +- 0.015 Signed-off-by: James Almer jamr...@gmail.com --- libavutil/ripemd.c | 354

Re: [FFmpeg-devel] [PATCH] dirac: use and forward error codes

2015-05-06 Thread Michael Niedermayer
On Wed, May 06, 2015 at 04:54:40PM +0200, Andreas Cadhalpun wrote: Signed-off-by: Andreas Cadhalpun andreas.cadhal...@googlemail.com --- libavcodec/dirac_dwt.c | 2 +- libavcodec/diracdec.c | 75 -- 2 files changed, 43 insertions(+), 34