Re: [FFmpeg-devel] [PATCH] lavfi/avf_showspectrum: use automatic framing.

2014-08-04 Thread Paul B Mahol
On Sun, Aug 3, 2014 at 7:51 PM, Nicolas George geo...@nsup.org wrote: The framework can ensure that each input frame has exactly the correct number of samples, except the last one. Signed-off-by: Nicolas George geo...@nsup.org --- libavfilter/avf_showspectrum.c | 48

[FFmpeg-devel] [PATCH] New p2p mode for showwaves filter

2014-08-04 Thread mrskman
New patch attached. Thank you. --- doc/filters.texi|3 +++ libavfilter/avf_showwaves.c | 33 - 2 files changed, 35 insertions(+), 1 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index a7919a3..145acbf 100644 ---

Re: [FFmpeg-devel] [PATCH] New p2p mode for showwaves filter

2014-08-04 Thread Paul B Mahol
On Mon, Aug 4, 2014 at 9:24 AM, mrskman mrsk...@gmail.com wrote: New patch attached. Thank you. --- doc/filters.texi|3 +++ libavfilter/avf_showwaves.c | 33 - 2 files changed, 35 insertions(+), 1 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] New p2p mode for showwaves filter

2014-08-04 Thread Paul B Mahol
On Mon, Aug 4, 2014 at 9:34 AM, Paul B Mahol one...@gmail.com wrote: On Mon, Aug 4, 2014 at 9:24 AM, mrskman mrsk...@gmail.com wrote: New patch attached. Thank you. --- doc/filters.texi|3 +++ libavfilter/avf_showwaves.c | 33 - 2

Re: [FFmpeg-devel] [PATCH 1/3] x86/hevc_mc: remove an unnecessary pxor

2014-08-04 Thread Mickaël Raulet
Patch ok. Mickael Le lundi 4 août 2014, James Almer jamr...@gmail.com a écrit : Signed-off-by: James Almer jamr...@gmail.com javascript:; --- libavcodec/x86/hevc_mc.asm | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/x86/hevc_mc.asm

Re: [FFmpeg-devel] [PATCH 0/4] Exploit compile-time constant

2014-08-04 Thread Christophe Gisquet
Hi, 2014-08-02 14:48 GMT+02:00 Michael Niedermayer michae...@gmx.at: seems to fail with libavcodec/x86/hevc_mc.asm:1258: error: (add:2) cannot reference symbol `MAX_PB_SIZE' in preprocessor I forgot the initial patch when generating the patchset, that you can find here. I expect no changes

[FFmpeg-devel] [PATCH] New p2p mode for showwaves filter

2014-08-04 Thread mrskman
New patch attached. Thank you. --- doc/filters.texi|3 +++ libavfilter/avf_showwaves.c | 30 +- 2 files changed, 32 insertions(+), 1 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index a7919a3..145acbf 100644 --- a/doc/filters.texi

[FFmpeg-devel] [PATCH] x86: hevc_deblock: remove unnecessary masking

2014-08-04 Thread Christophe Gisquet
Hi, the attached patch is a low-hanging fruit. I think the code using the computed values could be improved (eg you probably need half the GPRs to store results and you can probably shuffle more efficiently data), but this requires more effort. I'm mostly submitting it because it still applies,

Re: [FFmpeg-devel] [PATCH 1/3] x86/hevc_mc: remove an unnecessary pxor

2014-08-04 Thread Michael Niedermayer
On Mon, Aug 04, 2014 at 10:27:44AM +0200, Mickaël Raulet wrote: Patch ok. applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those who are too smart to engage in politics are punished by being governed by those who are dumber. -- Plato

Re: [FFmpeg-devel] [PATCH] x86/hevc_mc: use fewer instructions in hevc_put_hevc_{uni, bi}_w[24]_{8, 10, 12}

2014-08-04 Thread Michael Niedermayer
On Mon, Aug 04, 2014 at 12:32:05PM +0200, Christophe Gisquet wrote: Hi, 2014-08-04 6:18 GMT+02:00 James Almer jamr...@gmail.com: Signed-off-by: James Almer jamr...@gmail.com Looks good to me. Tested on top of my patches, too. IIRC, this should be evaluated by the WP_ test sequences.

Re: [FFmpeg-devel] [PATCH 0/4] Exploit compile-time constant

2014-08-04 Thread Michael Niedermayer
On Mon, Aug 04, 2014 at 10:31:52AM +0200, Christophe Gisquet wrote: Hi, 2014-08-02 14:48 GMT+02:00 Michael Niedermayer michae...@gmx.at: seems to fail with libavcodec/x86/hevc_mc.asm:1258: error: (add:2) cannot reference symbol `MAX_PB_SIZE' in preprocessor I forgot the initial patch

Re: [FFmpeg-devel] [PATCH 3/3] x86/vp9lpf: use fewer instructions in SPLATB_MIX

2014-08-04 Thread Ronald S. Bultje
Hi, On Sun, Aug 3, 2014 at 10:53 PM, James Almer jamr...@gmail.com wrote: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/vp9lpf.asm | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavcodec/x86/vp9lpf.asm b/libavcodec/x86/vp9lpf.asm index

Re: [FFmpeg-devel] [PATCH] avfilter/dctdnoiz: rewrite [f/i]dct

2014-08-04 Thread Ronald S. Bultje
Hi, On Sun, Aug 3, 2014 at 4:27 PM, Clément Bœsch u...@pkh.me wrote: This removes the avcodec dependency and make the code almost twice as fast. More to come. The DCT factorization is based on Fast and numerically stable algorithms for discrete cosine transforms from Gerlind Plonkaa

Re: [FFmpeg-devel] [PATCH] avfilter/dctdnoiz: rewrite [f/i]dct

2014-08-04 Thread Clément Bœsch
On Mon, Aug 04, 2014 at 11:44:43AM -0400, Ronald S. Bultje wrote: Hi, On Mon, Aug 4, 2014 at 11:42 AM, Clément Bœsch u...@pkh.me wrote: On Mon, Aug 04, 2014 at 11:17:29AM -0400, Ronald S. Bultje wrote: Hi, On Sun, Aug 3, 2014 at 4:27 PM, Clément Bœsch u...@pkh.me wrote:

Re: [FFmpeg-devel] [PATCH] avfilter/dctdnoiz: rewrite [f/i]dct

2014-08-04 Thread Ronald S. Bultje
Hi, On Mon, Aug 4, 2014 at 11:59 AM, Clément Bœsch u...@pkh.me wrote: On Mon, Aug 04, 2014 at 11:44:43AM -0400, Ronald S. Bultje wrote: Hi, On Mon, Aug 4, 2014 at 11:42 AM, Clément Bœsch u...@pkh.me wrote: On Mon, Aug 04, 2014 at 11:17:29AM -0400, Ronald S. Bultje wrote: Hi,

Re: [FFmpeg-devel] [PATCH] avfilter/dctdnoiz: rewrite [f/i]dct

2014-08-04 Thread Ronald S. Bultje
Hi, On Mon, Aug 4, 2014 at 11:42 AM, Clément Bœsch u...@pkh.me wrote: On Mon, Aug 04, 2014 at 11:17:29AM -0400, Ronald S. Bultje wrote: Hi, On Sun, Aug 3, 2014 at 4:27 PM, Clément Bœsch u...@pkh.me wrote: This removes the avcodec dependency and make the code almost twice as fast.

Re: [FFmpeg-devel] [libav-devel] [PATCHv3] Deprecate AFD field and add AFD as side-data

2014-08-04 Thread Vittorio Giovara
On Sun, Aug 3, 2014 at 7:24 PM, Kieran Kunhya kier...@obe.tv wrote: --- doc/APIchanges|4 libavcodec/avcodec.h |5 - libavcodec/mpeg12dec.c| 20 +++- libavcodec/version.h |5 - libavfilter/vf_showinfo.c |3 +++

Re: [FFmpeg-devel] [PATCH] avformat/util: change av_find_default_stream_index() to use a score based system

2014-08-04 Thread Michael Niedermayer
On Fri, Aug 01, 2014 at 10:29:25PM +0200, Michael Niedermayer wrote: Disfavor video streams with unknown resolution and no packets Fixes seeking in audio-only-speex.flv Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/utils.c | 20 ++-- 1 file changed,

Re: [FFmpeg-devel] [PATCH 2/5] lavfi/buffersrc: add add av_buffersrc_close().

2014-08-04 Thread wm4
On Sun, 3 Aug 2014 15:15:37 +0200 Nicolas George geo...@nsup.org wrote: diff --git a/libavfilter/buffersrc.h b/libavfilter/buffersrc.h index ea34c04..28ca545 100644 --- a/libavfilter/buffersrc.h +++ b/libavfilter/buffersrc.h @@ -145,6 +145,8 @@ int av_buffersrc_add_frame(AVFilterContext

Re: [FFmpeg-devel] [PATCH 3/3] x86/vp9lpf: use fewer instructions in SPLATB_MIX

2014-08-04 Thread Ronald S. Bultje
Hi, On Mon, Aug 4, 2014 at 12:17 PM, James Almer jamr...@gmail.com wrote: On 04/08/14 10:27 AM, Ronald S. Bultje wrote: Hi, On Sun, Aug 3, 2014 at 10:53 PM, James Almer jamr...@gmail.com wrote: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/vp9lpf.asm | 5 ++---

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

2014-08-04 Thread Christophe Gisquet
2014-07-28 19:17 GMT+02:00 Christophe Gisquet christophe.gisq...@gmail.com: --- libavcodec/hevc.c | 8 +++ libavcodec/hevcdsp.h | 8 +++ libavcodec/hevcdsp_template.c | 56 +-- libavcodec/x86/hevc_mc.asm| 18

Re: [FFmpeg-devel] [PATCH 3/3] x86/vp9lpf: use fewer instructions in SPLATB_MIX

2014-08-04 Thread Clément Bœsch
On Sun, Aug 03, 2014 at 11:53:40PM -0300, James Almer wrote: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/vp9lpf.asm | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/libavcodec/x86/vp9lpf.asm b/libavcodec/x86/vp9lpf.asm index c5db0ca..def7d5a

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Hendrik Leppkes
Am 04.08.2014 19:59 schrieb Gisle Sælensminde gi...@snirklasjon.no: I'm trying to add support for closed captions in ffmpeg, namely cea608 and cea708. Unlike normal subtitles, these are embedded in the video frames themselves rather than a separate track or file. As a first step I try to write

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread wm4
On Mon, 4 Aug 2014 20:47:37 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On Mon, Aug 04, 2014 at 08:19:38PM +0200, Hendrik Leppkes wrote: Am 04.08.2014 19:59 schrieb Gisle Sælensminde gi...@snirklasjon.no: I'm trying to add support for closed captions in ffmpeg, namely cea608

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Kieran Kunhya
And as far as I can tell if you want to remux but with separate subtitle stream that would even mean that you have to re-encode the video for no good reason. You can just swap out the caption data. It's guaranteed to be CBR anyway. ___ ffmpeg-devel

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Reimar Döffinger
On Mon, Aug 04, 2014 at 08:04:39PM +0100, Kieran Kunhya wrote: And as far as I can tell if you want to remux but with separate subtitle stream that would even mean that you have to re-encode the video for no good reason. You can just swap out the caption data. It's guaranteed to be CBR

[FFmpeg-devel] [PATCH 1/1] Fix compile error on bfin.

2014-08-04 Thread Bernd Kuhls
After the removal of all Blackfin architecture optimizations in http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=b55d3bbeed375f7b74442c4dd274d116a3e3d2e1 some includes were left behind leading to a compile error: CC libavformat/adtsenc.o In file included from

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Kieran Kunhya
Uh, no. The point is to get the CC data out you need to decode. Now that you have the video decoded, if you want to mux it you have to encode it again. Whether you decode the video or not is unrelated to your ability to replace the SEI and replace the captions. You'd still need to know the

Re: [FFmpeg-devel] [PATCH 1/2] avutil/libm: Replace macro based fminf() by function

2014-08-04 Thread Bernd Kuhls
Michael Niedermayer michae...@gmx.at wrote in news:1402065736-24164-1-git- send-email-michae...@gmx.at: libavutil/libm.h |5 - Hi, the current code in libm.h breaks compilation when the libc does not provide fminf, which is the case for uClibc. Here you will find a patch fixing the

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Gisle Sælensminde
Den 04. aug. 2014 21:11, skrev Reimar Döffinger: On Mon, Aug 04, 2014 at 08:04:39PM +0100, Kieran Kunhya wrote: And as far as I can tell if you want to remux but with separate subtitle stream that would even mean that you have to re-encode the video for no good reason. You can just swap out

Re: [FFmpeg-devel] [PATCH 1/2] avutil/libm: Replace macro based fminf() by function

2014-08-04 Thread wm4
On Mon, 04 Aug 2014 21:25:12 +0200 Bernd Kuhls bernd.ku...@t-online.de wrote: Michael Niedermayer michae...@gmx.at wrote in news:1402065736-24164-1-git- send-email-michae...@gmx.at: libavutil/libm.h |5 - Hi, the current code in libm.h breaks compilation when the libc does not

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Kieran Kunhya
into the decoders. The decoder approch would also not work for insertion of closed captions (for h.264 that would be x264, and I can't see that CC has a place there really). I'm uncertain enough that I would prefere to here the opinion here before submitting a patch. As far as I can see it

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Reimar Döffinger
On Mon, Aug 04, 2014 at 09:35:02PM +0200, Gisle Sælensminde wrote: Bitstream filter: Pro: - Modular, the CC code can be kept separate. - Less complex code Cons: Need for features that I can't see is present in the bitstream filters: - As far as I can see -no support for timestamps

[FFmpeg-devel] [PATCH] lavd/avfoundation: Add device category.

2014-08-04 Thread Thilo Borgmann
Hi, as reported by David Favor, the AVFoundation device is not yet listed by ffmpeg -devices. This patch fixes that. Please Apply. -Thilo From c76b7ce3a2bceb13d273212ec7b5d3142691b24a Mon Sep 17 00:00:00 2001 From: Thilo Borgmann thilo.borgm...@mail.de Date: Mon, 4 Aug 2014 22:06:59 +0200

Re: [FFmpeg-devel] Mac/OSX question about clang + avfoundation + indev support

2014-08-04 Thread Thilo Borgmann
[...] David-Favor-iMac# ffmpeg -devices ffmpeg version 2.3.1-2014-08-01-65165-g229a1e8 Copyright (c) 2000-2014 the FFmpeg developers built on Aug 1 2014 10:13:00 with Apple LLVM version 5.1 (clang-503.0.40) (based on LLVM 3.4svn) configuration: --cc=/usr/bin/clang

Re: [FFmpeg-devel] [PATCH] avfilter/dctdnoiz: rewrite [f/i]dct

2014-08-04 Thread Clément Bœsch
On Sun, Aug 03, 2014 at 01:59:25PM -0700, Timothy Gu wrote: On Aug 3, 2014 1:27 PM, Clément Bœsch u...@pkh.me wrote: This removes the avcodec dependency and make the code almost twice as fast. More to come. The DCT factorization is based on Fast and numerically stable algorithms for

Re: [FFmpeg-devel] [RFC] Implementation of closed caption support

2014-08-04 Thread Reimar Döffinger
On Mon, Aug 04, 2014 at 10:16:25PM +0200, Gisle Sælensminde wrote: Den 04. aug. 2014 21:58, skrev Reimar Döffinger: *scratches head* I wonder now if you misunderstood or if you discarded the variant I pointed out due to effort/complexity? Or because I don't know all the parts of the ffmpeg

Re: [FFmpeg-devel] [PATCH] lavd/avfoundation: Add device category.

2014-08-04 Thread Michael Niedermayer
On Mon, Aug 04, 2014 at 10:10:12PM +0200, Thilo Borgmann wrote: Hi, as reported by David Favor, the AVFoundation device is not yet listed by ffmpeg -devices. This patch fixes that. Please Apply. -Thilo avfoundation.m |1 + 1 file changed, 1 insertion(+)

Re: [FFmpeg-devel] [PATCH 1/1] Fix compile error on bfin.

2014-08-04 Thread Michael Niedermayer
On Mon, Aug 04, 2014 at 09:12:29PM +0200, Bernd Kuhls wrote: After the removal of all Blackfin architecture optimizations in http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=b55d3bbeed375f7b74442c4dd274d116a3e3d2e1 some includes were left behind leading to a compile error: CC

Re: [FFmpeg-devel] [PATCH 3/3] x86/vp9lpf: use fewer instructions in SPLATB_MIX

2014-08-04 Thread Michael Niedermayer
On Sun, Aug 03, 2014 at 11:53:40PM -0300, James Almer wrote: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/x86/vp9lpf.asm | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: