Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-28 Thread Nicolas George
Le decadi 10 thermidor, an CCXXII, Eli Kara a écrit : It would solve the problem for DVDs with structure, which is probably what most people have when they rip a DVD. Certainly. But the other scenarios where the user only have a big MPEG-PS file and nothing else, are likely too. Just out of

Re: [FFmpeg-devel] [PATCH] hevc_mc: reduce stride for bidir temp buffers

2014-07-28 Thread Christophe Gisquet
Hi, 2014-07-27 19:18 GMT+02:00 Michael Niedermayer michae...@gmx.at: It is unconditionally set to 64, which is quite higher than the actual block size. is this faster? No, which surprises me, but so be it. On the other hand, if this stride is PB_MAX_SIZE throughout the code, then maybe its

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-28 Thread Nicolas George
Le decadi 10 thermidor, an CCXXII, Oliver Fromme a écrit : Ah! Thank you very much for pointing me to the concat demuxer. I wasn't aware that it can be (ab)used to declare the presence of streams in an input file. I will definitely give that a try. Please let me know of any issue. Well, my

[FFmpeg-devel] [PATCH 1/4] hevc: move MAX_PB_SIZE declaration

2014-07-28 Thread Christophe Gisquet
--- libavcodec/hevc.h| 1 - libavcodec/hevcdsp.h | 2 ++ 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/hevc.h b/libavcodec/hevc.h index 8420f38..0369d8f 100644 --- a/libavcodec/hevc.h +++ b/libavcodec/hevc.h @@ -56,7 +56,6 @@ #define MAX_TRANSFORM_DEPTH 5

[FFmpeg-devel] [PATCH 3/4] hevcdsp: remove more instances of compile-time-fixed parameters

2014-07-28 Thread Christophe Gisquet
--- libavcodec/hevc.c | 8 +++ libavcodec/hevcdsp.h | 8 +++ libavcodec/hevcdsp_template.c | 56 +-- libavcodec/x86/hevc_mc.asm| 18 +++--- libavcodec/x86/hevcdsp.h | 6 ++--- libavcodec/x86/hevcdsp_init.c |

[FFmpeg-devel] [PATCH 4/4] x86: hevcdsp: use compilation-time-fixed constant

2014-07-28 Thread Christophe Gisquet
The stride for some buffers is known. --- libavcodec/x86/hevc_mc.asm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/x86/hevc_mc.asm b/libavcodec/x86/hevc_mc.asm index ea4447b..2c34927 100644 --- a/libavcodec/x86/hevc_mc.asm +++ b/libavcodec/x86/hevc_mc.asm @@

Re: [FFmpeg-devel] [PATCH 1/4] libavcodec: new API for frame threading by step

2014-07-28 Thread Christophe Gisquet
2014-07-23 21:20 GMT+02:00 Christophe Gisquet christophe.gisq...@gmail.com: Didn't refresh the patch with update documentation. Here it is. It didn't contain dummy functions in libavcodec/utils.c. The attached patch should fix failures to build with pthread disabled. -- Christophe From

Re: [FFmpeg-devel] Recommendation for ffmpeg.org background color change

2014-07-28 Thread Lou Logan
On Mon, 28 Jul 2014 07:00:12 -0700, Frantisek Korbel wrote: For this reasons I strongly recommend to set the white background please try it from the 1st August. If you are interested you could supply a patch to ffmpeg-web to somehow implement a user selectable CSS to enable a light background

Re: [FFmpeg-devel] Recommendation for ffmpeg.org background color change

2014-07-28 Thread Clément Bœsch
On Mon, Jul 28, 2014 at 09:46:31AM -0800, Lou Logan wrote: On Mon, 28 Jul 2014 07:00:12 -0700, Frantisek Korbel wrote: For this reasons I strongly recommend to set the white background please try it from the 1st August. If you are interested you could supply a patch to ffmpeg-web to

Re: [FFmpeg-devel] [PATCH] x86: hevc_mc: fix register count usage

2014-07-28 Thread Christophe Gisquet
With patch. From 0b5d56a563df7efdfcf206013680510c62e2569c Mon Sep 17 00:00:00 2001 From: Christophe Gisquet christophe.gisq...@gmail.com Date: Mon, 28 Jul 2014 11:48:21 +0200 Subject: [PATCH 10/15] x86: hevc_mc: fix register count usage A macro was using a fixed register, causing too many GPRs to

Re: [FFmpeg-devel] [PATCH] Delay FF_API_AVFRAME_COLORSPACE until the next libavutil soname bump

2014-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote: On Mon, Jul 28, 2014 at 04:30:02AM +0200, Michael Niedermayer wrote: This works around ABI issues with applications which depend on libavutil internal values like sizeof(AVFrame) One such application is VLC 2.1.4 as well as

Re: [FFmpeg-devel] [PATCH] Delay FF_API_AVFRAME_COLORSPACE until the next libavutil soname bump

2014-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2014 at 08:47:07PM +0200, Michael Niedermayer wrote: On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote: On Mon, Jul 28, 2014 at 04:30:02AM +0200, Michael Niedermayer wrote: [...] in absence of further comments i might apply this patch to master and release/2.3

[FFmpeg-devel] [PATCH 2/2] x86/hevc_deblock: load less data in hevc_h_loop_filter_luma_8

2014-07-28 Thread James Almer
Reading 8 bytes is enough. Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/hevc_deblock.asm | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavcodec/x86/hevc_deblock.asm b/libavcodec/x86/hevc_deblock.asm index f7cd031..ecebd36 100644 ---

Re: [FFmpeg-devel] [PATCH 2/2] x86/hevc_deblock: load less data in hevc_h_loop_filter_luma_8

2014-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2014 at 04:17:35PM -0300, James Almer wrote: Reading 8 bytes is enough. Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/hevc_deblock.asm | 16 1 file changed, 8 insertions(+), 8 deletions(-) applied thanks [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 4/4] lavfi/vf_fps: accept EOF timestamp.

2014-07-28 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 03:39:45PM +0200, Nicolas George wrote: This makes the FPS filter duplicate the last frame to take its duration into account, exactly like the other ones. TODO find corresponding trac ticket(s). Signed-off-by: Nicolas George geo...@nsup.org ---

Re: [FFmpeg-devel] [RFC]Add int64_t probesize2 to AVFormatContext

2014-07-28 Thread Carl Eugen Hoyos
Oliver Fromme oliver at fromme.com writes: That's why I appreciate the patch very much that Carl Eugen has created. Did you test the patch? I think it will not get applied without a test. [...] Maybe I'll try to make a patch that can do this If you consider to invest time in DVD reading,

Re: [FFmpeg-devel] [PATCH] Delay FF_API_AVFRAME_COLORSPACE until the next libavutil soname bump

2014-07-28 Thread Andreas Cadhalpun
On 28.07.2014 20:51, Michael Niedermayer wrote: On Mon, Jul 28, 2014 at 08:47:07PM +0200, Michael Niedermayer wrote: On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote: On Mon, Jul 28, 2014 at 04:30:02AM +0200, Michael Niedermayer wrote: [...] in absence of further comments i

Re: [FFmpeg-devel] [PATCH] Delay FF_API_AVFRAME_COLORSPACE until the next libavutil soname bump

2014-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2014 at 10:54:47PM +0200, Andreas Cadhalpun wrote: On 28.07.2014 20:51, Michael Niedermayer wrote: On Mon, Jul 28, 2014 at 08:47:07PM +0200, Michael Niedermayer wrote: On Mon, Jul 28, 2014 at 08:28:26PM +0200, Reimar Döffinger wrote: On Mon, Jul 28, 2014 at 04:30:02AM +0200,

[FFmpeg-devel] rectification filter

2014-07-28 Thread Daniel Oberhoff
Hello, In the last days I effectively rewrote the rectification filter found in the frei0r suite as a libavfilter video filter and would like to submit this to ffmpeg. It probably needs some cleanup, which I will do by checking your commit guidelines, but I would like to get a first opinion

Re: [FFmpeg-devel] rectification filter

2014-07-28 Thread Daniel Oberhoff
Hi again, Since at least in the web view of the list attachements seem to be stripped, I send it here in the clear: thanks for reviewing! Daniel From 7b5b3b42804714ddc56d2b8cd2c5709b4bb91f70 Mon Sep 17 00:00:00 2001 From: Daniel Oberhoff dan...@danieloberhoff.de Date: Mon, 28 Jul 2014

[FFmpeg-devel] [PATCH] Deprecate AFD field and add AFD as side-data

2014-07-28 Thread Kieran Kunhya
--- doc/APIchanges|4 libavcodec/avcodec.h |5 - libavcodec/mpeg12dec.c| 21 - libavcodec/version.h |5 - libavfilter/vf_showinfo.c |3 +++ libavutil/frame.h | 12 libavutil/version.h |2

Re: [FFmpeg-devel] rectification filter

2014-07-28 Thread Carl Eugen Hoyos
Daniel Oberhoff danieloberhoff at gmail.com writes: + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License Your patch is missing a configure hunk that makes sure that the filter is only compiled if --enable-gpl was

Re: [FFmpeg-devel] [PATCH 3/3] web/download: Rename builds anchors so old links posted elsewhere still work

2014-07-28 Thread Alexander Strasser
On 2014-07-23 13:42 -0800, Lou Logan wrote: On Tue, 22 Jul 2014 10:36:31 -0800, Lou Logan wrote: [...] Actually I'd like to wait on this anchor patch since I contacted Alexander to add some redirects instead, so the new, cleaner names can be used. If that's not possible then I'll push this

[FFmpeg-devel] [PATCH] x86/hevc_deblock: use psignw instead of pmullw where possible

2014-07-28 Thread James Almer
It's slightly faster Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/hevc_deblock.asm | 8 1 file changed, 8 insertions(+) diff --git a/libavcodec/x86/hevc_deblock.asm b/libavcodec/x86/hevc_deblock.asm index f7cd031..7d67a07 100644 --- a/libavcodec/x86/hevc_deblock.asm

Re: [FFmpeg-devel] [PATCH] hevc_filter: run vertical and horizontal together

2014-07-28 Thread Michael Niedermayer
On Sun, Jul 27, 2014 at 10:27:35AM +0200, Christophe Gisquet wrote: One may interleave even more vertical and horizontal passes, but this is less straightforward (more code changes than just moves). -- Christophe hevc_filter.c | 90

Re: [FFmpeg-devel] [PATCH 1/4] lavfi: add filter messages infrastructure.

2014-07-28 Thread Michael Niedermayer
On Tue, Jul 29, 2014 at 01:29:16AM +0200, Nicolas George wrote: Le decadi 10 thermidor, an CCXXII, Michael Niedermayer a écrit : Breaks ABI, AVFrame size is depending on libavutil and may be non constant for a libavfilter build That was intentional (that makes the code simpler with less

Re: [FFmpeg-devel] [PATCH] x86/hevc_deblock: add ff_hevc_[hv]_loop_filter_luma_{8, 10}_sse2

2014-07-28 Thread James Almer
On 13/07/14 8:39 AM, Kieran Kunhya wrote: On 13 July 2014 07:00, James Almer jamr...@gmail.com wrote: Signed-off-by: James Almer jamr...@gmail.com LGTM. There's the odd use of 3-operand in some of the macros. Have you benched AVX? Yes. Tested luma with an 8bit sample and it's ~%5 faster.

Re: [FFmpeg-devel] rectification filter

2014-07-28 Thread Daniel Oberhoff
First off thanks for taking the time to review. Am 29.07.2014 um 01:16 schrieb Carl Eugen Hoyos ceho...@ag.or.at: Daniel Oberhoff danieloberhoff at gmail.com writes: + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public