Re: [FFmpeg-devel] [PATCH] x86/hevc_deblock: improve 8bit transpose store macros

2014-08-02 Thread Michael Niedermayer
On Sat, Aug 02, 2014 at 11:21:31PM -0300, James Almer wrote: > Up to four instructions less depending on function and instruction set. > > Signed-off-by: James Almer > --- > libavcodec/x86/hevc_deblock.asm | 70 > + > libavutil/x86/x86util.asm | 9

[FFmpeg-devel] [PATCH] x86/hevc_deblock: improve 8bit transpose store macros

2014-08-02 Thread James Almer
Up to four instructions less depending on function and instruction set. Signed-off-by: James Almer --- libavcodec/x86/hevc_deblock.asm | 70 + libavutil/x86/x86util.asm | 9 ++ 2 files changed, 31 insertions(+), 48 deletions(-) diff --git a/lib

Re: [FFmpeg-devel] rectification filter

2014-08-02 Thread Daniel Oberhoff
Am 03.08.2014 um 00:12 schrieb Clément Bœsch : > On Fri, Aug 01, 2014 at 12:56:42PM +0200, Daniel Oberhoff wrote: >> >> Am 01.08.2014 um 12:22 schrieb Clément Bœsch : >> >>> On Fri, Aug 01, 2014 at 12:13:22PM +0200, Daniel Oberhoff wrote: Am 29.07.2014 um 09:54 schrieb Carl Eugen Hoyo

Re: [FFmpeg-devel] [PATCH 3/5] avfilter/deshake: remove avcodec dependency

2014-08-02 Thread Michael Niedermayer
On Sat, Aug 02, 2014 at 11:34:09PM +0200, Clément Bœsch wrote: > --- > configure| 4 +--- > libavfilter/deshake.h| 5 ++--- > libavfilter/vf_deshake.c | 20 > 3 files changed, 11 insertions(+), 18 deletions(-) > > diff --git a/configure b/configure > ind

Re: [FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread Michael Niedermayer
On Sat, Aug 02, 2014 at 11:34:07PM +0200, Clément Bœsch wrote: > --- > configure | 2 + > doc/APIchanges | 3 + > libavutil/Makefile | 3 + > libavutil/pixelutils.c | 142 > libavutil/pixelutils

Re: [FFmpeg-devel] rectification filter

2014-08-02 Thread Clément Bœsch
On Fri, Aug 01, 2014 at 12:56:42PM +0200, Daniel Oberhoff wrote: > > Am 01.08.2014 um 12:22 schrieb Clément Bœsch : > > > On Fri, Aug 01, 2014 at 12:13:22PM +0200, Daniel Oberhoff wrote: > >> > >> Am 29.07.2014 um 09:54 schrieb Carl Eugen Hoyos : > >> > >>> Daniel Oberhoff gmail.com> writes: >

Re: [FFmpeg-devel] rectification filter

2014-08-02 Thread Clément Bœsch
On Fri, Aug 01, 2014 at 01:08:58PM +0200, Daniel Oberhoff wrote: > > Am 01.08.2014 um 13:07 schrieb Carl Eugen Hoyos : > > > Daniel Oberhoff gmail.com> writes: > > > >> all work now, except vuy444p > > > > It's yuv444p > > oops :). > > And how about that test/data/fate/filter-pixfmts-rectifi

Re: [FFmpeg-devel] rectification filter

2014-08-02 Thread Clément Bœsch
On Fri, Aug 01, 2014 at 06:47:38PM +0200, Daniel Oberhoff wrote: [...] > > Did you already explain if / how this filter is > > different from / better than the vignette filter? > > Its completely differen. Vignette modulates brightness, this one geometry, > but I will mention it. > So this is

Re: [FFmpeg-devel] [PATCH] libavformat/icecast.c Add Icecast protocol

2014-08-02 Thread Clément Bœsch
On Mon, Jul 28, 2014 at 10:36:19AM +0200, epira...@gmail.com wrote: > From: ePirat > > Fix header. Note: you should put that under the --- below so it doesn't end up in the commit description. > --- > Add Icecast protocol, a convenience wrapper for the HTTP protocol > --- > Changelog

[FFmpeg-devel] pixelutils API [v2]

2014-08-02 Thread Clément Bœsch
Second attempt. Since previously: - much simpler usage (you basically just call av_pixelutils_get_sad_fn() as a user); filters patches updated accordingly. - 16x16 mmxext added (thanks jamrial) - FATE tests Note for the context since I didn't write anything about it in the previous patchset

[FFmpeg-devel] [PATCH 5/5] avfilter/select: remove optional avcodec dependency

2014-08-02 Thread Clément Bœsch
--- configure | 1 + libavfilter/f_select.c | 50 -- 2 files changed, 13 insertions(+), 38 deletions(-) diff --git a/configure b/configure index 210cafb..b9b1e3b 100755 --- a/configure +++ b/configure @@ -2564,6 +2564,7 @@ removelogo_f

[FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread Clément Bœsch
--- configure | 2 + doc/APIchanges | 3 + libavutil/Makefile | 3 + libavutil/pixelutils.c | 142 libavutil/pixelutils.h | 52 ++ libavutil/version.h | 2 +-

[FFmpeg-devel] [PATCH 4/5] avfilter/mp_decimate: remove avcodec dependency

2014-08-02 Thread Clément Bœsch
--- configure | 4 ++-- libavfilter/vf_mpdecimate.c | 36 2 files changed, 14 insertions(+), 26 deletions(-) diff --git a/configure b/configure index bc187a5..210cafb 100755 --- a/configure +++ b/configure @@ -2549,8 +2549,8 @@ ladspa_filter

[FFmpeg-devel] [PATCH 2/5] build: remove wrong avcodec dep in decimate

2014-08-02 Thread Clément Bœsch
This was probably a mistake with mpdecimate. The decimate filter doesn't depend on avcodec. --- configure | 1 - 1 file changed, 1 deletion(-) diff --git a/configure b/configure index 57edd1d..5159f10 100755 --- a/configure +++ b/configure @@ -5241,7 +5241,6 @@ enabled amovie_filter && prep

[FFmpeg-devel] [PATCH 3/5] avfilter/deshake: remove avcodec dependency

2014-08-02 Thread Clément Bœsch
--- configure| 4 +--- libavfilter/deshake.h| 5 ++--- libavfilter/vf_deshake.c | 20 3 files changed, 11 insertions(+), 18 deletions(-) diff --git a/configure b/configure index 5159f10..bc187a5 100755 --- a/configure +++ b/configure @@ -2531,8 +2531,7 @

Re: [FFmpeg-devel] [PATCH] avformat/tee: flip assigment direction

2014-08-02 Thread Nicolas George
Le quintidi 15 thermidor, an CCXXII, Michael Niedermayer a écrit : > Found-by: CSA > Signed-off-by: Michael Niedermayer > --- > libavformat/tee.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) That looks clearly right. Thanks. Regards, -- Nicolas George signature.asc Descripti

Re: [FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread James Almer
On 02/08/14 6:13 PM, Clément Bœsch wrote: > On Sat, Aug 02, 2014 at 04:29:39PM -0300, James Almer wrote: >> On 02/08/14 3:20 PM, Clément Bœsch wrote: >>> +psrlq m0, m6, 32 >>> +paddw m6, m0 >>> +psrlq m0, m6, 16 >>> +paddw m6, m0 >>> +movdeax, m6

Re: [FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread Clément Bœsch
On Sat, Aug 02, 2014 at 08:35:27PM +0200, Michael Niedermayer wrote: [...] > > +typedef struct AVPixelUtils { > > +/** > > + * Sum of absolute differences block functions. > > + * src1 and src2 addresses need to be aligned to the block size. > > + */ > > +av_pixelutils_sad_fn sa

Re: [FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread Clément Bœsch
On Sat, Aug 02, 2014 at 04:29:39PM -0300, James Almer wrote: > On 02/08/14 3:20 PM, Clément Bœsch wrote: > > +psrlq m0, m6, 32 > > +paddw m6, m0 > > +psrlq m0, m6, 16 > > +paddw m6, m0 > > +movdeax, m6 > > +movzx eax, ax > > You could u

[FFmpeg-devel] [PATCH] avformat/tee: flip assigment direction

2014-08-02 Thread Michael Niedermayer
Found-by: CSA Signed-off-by: Michael Niedermayer --- libavformat/tee.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/tee.c b/libavformat/tee.c index f26e157..681f943 100644 --- a/libavformat/tee.c +++ b/libavformat/tee.c @@ -467,7 +467,7 @@ static int tee_writ

[FFmpeg-devel] [PATCH 2/2] avcodec/takdec: remove unused variable

2014-08-02 Thread Michael Niedermayer
Found-by: CSA Signed-off-by: Michael Niedermayer --- libavcodec/takdec.c |1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/takdec.c b/libavcodec/takdec.c index 83abab6..5810a01 100644 --- a/libavcodec/takdec.c +++ b/libavcodec/takdec.c @@ -448,7 +448,6 @@ static int decode_subframe

[FFmpeg-devel] [PATCH 1/2] avcodec/takdec: move tmp declaration to where its used

2014-08-02 Thread Michael Niedermayer
Makes the code a bit easier to read Signed-off-by: Michael Niedermayer --- libavcodec/takdec.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/takdec.c b/libavcodec/takdec.c index 9bfbfcc..83abab6 100644 --- a/libavcodec/takdec.c +++ b/libavcodec/takdec.

Re: [FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread James Almer
On 02/08/14 3:20 PM, Clément Bœsch wrote: > +psrlq m0, m6, 32 > +paddw m6, m0 > +psrlq m0, m6, 16 > +paddw m6, m0 > +movdeax, m6 > +movzx eax, ax You could use the HADDW macro here. > +;-

[FFmpeg-devel] [PATCH] avcodec/avdct: Add get_pixels()

2014-08-02 Thread Michael Niedermayer
TODO: version bump, update libavfilters to use it Suggested-by: ubitux Signed-off-by: Michael Niedermayer --- libavcodec/avdct.c |9 + libavcodec/avdct.h |4 2 files changed, 13 insertions(+) diff --git a/libavcodec/avdct.c b/libavcodec/avdct.c index 58f4974..2521256 100644

Re: [FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread Clément Bœsch
On Sat, Aug 02, 2014 at 08:30:39PM +0200, Lukasz Marek wrote: > >+static av_always_inline int sad_wxh(const uint8_t *src1, ptrdiff_t stride1, > >+const uint8_t *src2, ptrdiff_t stride2, > >+int w, int h) > >+{ > >+int x, y,

Re: [FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread Michael Niedermayer
On Sat, Aug 02, 2014 at 08:20:17PM +0200, Clément Bœsch wrote: > --- > configure | 2 + > doc/APIchanges | 3 ++ > libavutil/Makefile | 2 + > libavutil/pixelutils.c | 85 ++ > libavutil/pixelutils.h

Re: [FFmpeg-devel] [PATCH 2/5] build: remove wrong avcodec dep in decimate

2014-08-02 Thread Paul B Mahol
On Sat, Aug 2, 2014 at 8:20 PM, Clément Bœsch wrote: > This was probably a mistake with mpdecimate. The decimate filter doesn't > depend on avcodec. > --- > configure | 1 - > 1 file changed, 1 deletion(-) > lgtm ___ ffmpeg-devel mailing list ffmpeg-de

Re: [FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread Lukasz Marek
+static av_always_inline int sad_wxh(const uint8_t *src1, ptrdiff_t stride1, +const uint8_t *src2, ptrdiff_t stride2, +int w, int h) +{ +int x, y, sum = 0; + +for (y = 0; y < h; y++) { +for (x = 0; x < w; x++)

[FFmpeg-devel] [PATCH 5/5] avfilter/select: remove optional avcodec dependency

2014-08-02 Thread Clément Bœsch
--- configure | 1 + libavfilter/f_select.c | 58 +- 2 files changed, 21 insertions(+), 38 deletions(-) diff --git a/configure b/configure index 210cafb..b9b1e3b 100755 --- a/configure +++ b/configure @@ -2564,6 +2564,7 @@ removelogo_f

[FFmpeg-devel] [PATCH 4/5] avfilter/mp_decimate: remove avcodec dependency

2014-08-02 Thread Clément Bœsch
--- configure | 4 ++-- libavfilter/vf_mpdecimate.c | 44 2 files changed, 22 insertions(+), 26 deletions(-) diff --git a/configure b/configure index bc187a5..210cafb 100755 --- a/configure +++ b/configure @@ -2549,8 +2549,8 @@ ladsp

[FFmpeg-devel] [PATCH 3/5] avfilter/deshake: remove avcodec dependency

2014-08-02 Thread Clément Bœsch
--- configure| 4 +--- libavfilter/deshake.h| 5 ++--- libavfilter/vf_deshake.c | 27 +++ 3 files changed, 18 insertions(+), 18 deletions(-) diff --git a/configure b/configure index 5159f10..bc187a5 100755 --- a/configure +++ b/configure @@ -2531,8 +2

[FFmpeg-devel] [PATCH 2/5] build: remove wrong avcodec dep in decimate

2014-08-02 Thread Clément Bœsch
This was probably a mistake with mpdecimate. The decimate filter doesn't depend on avcodec. --- configure | 1 - 1 file changed, 1 deletion(-) diff --git a/configure b/configure index 57edd1d..5159f10 100755 --- a/configure +++ b/configure @@ -5241,7 +5241,6 @@ enabled amovie_filter && prep

[FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

2014-08-02 Thread Clément Bœsch
--- configure | 2 + doc/APIchanges | 3 ++ libavutil/Makefile | 2 + libavutil/pixelutils.c | 85 ++ libavutil/pixelutils.h | 76 +++ libavutil/version.h |

Re: [FFmpeg-devel] HEVC SIMD Optimization on PowerPC

2014-08-02 Thread compn
On Sat, 2 Aug 2014 14:50:52 +0200 Michael Niedermayer wrote: > On Sat, Aug 02, 2014 at 07:18:22AM -0400, compn wrote: > > On Sat, 2 Aug 2014 13:37:10 +0800 > > "Zhenan Lin" wrote: > > > > > Hi, all, > > > > > > > > > > > > Our team plans to contribute SIMD optimization on PowerPC for HEVC >

Re: [FFmpeg-devel] [PATCH] ffserver: initialize pbuffer in prepare_sdp_description()

2014-08-02 Thread Michael Niedermayer
On Sat, Aug 02, 2014 at 04:51:16PM +0200, Stefano Sabatini wrote: > On date Saturday 2014-08-02 14:25:21 +0200, Michael Niedermayer encoded: > > also check pbuffer before use > > > > Found-by: CSA > > Signed-off-by: Michael Niedermayer > > --- > > ffserver.c |4 +++- > > 1 file changed, 3 in

Re: [FFmpeg-devel] mir-swamp.org

2014-08-02 Thread Derek Buitenhuis
On 8/2/2014 5:23 PM, Derek Buitenhuis wrote: > FWIW I got an invite at my gmail account. Never mind, I can't actually register. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] mir-swamp.org

2014-08-02 Thread Derek Buitenhuis
On 8/2/2014 5:14 PM, Michael Niedermayer wrote: > 1. Refusing members from gmail (and i assume other free mail services) > is a major problem, most people use gmail and most invites i did sent > went out to gmail users. Also there was no error or warning when > sending the invites, people just cant

[FFmpeg-devel] mir-swamp.org

2014-08-02 Thread Michael Niedermayer
Hi mir-swamp Ive found out about your free online static code analysis thingy and tried to use it for the FFmpeg project, I thought it cant hurt and maybe could help finding and fixing some bugs ... Ive 2 comments 1. Refusing members from gmail (and i assume other free mail services) is a major

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

2014-08-02 Thread Stefano Sabatini
On date Wednesday 2014-07-30 23:44:46 +0200, Nicolas George encoded: > Metaframes are frames without data, identified by a negative > format code, used to carry special conditions. > They are sent only to filter that declare supporting them. > The only metaframe for now is EOF; this mechanism augme

Re: [FFmpeg-devel] [PATCH] ffserver: initialize pbuffer in prepare_sdp_description()

2014-08-02 Thread Stefano Sabatini
On date Saturday 2014-08-02 14:25:21 +0200, Michael Niedermayer encoded: > also check pbuffer before use > > Found-by: CSA > Signed-off-by: Michael Niedermayer > --- > ffserver.c |4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/ffserver.c b/ffserver.c > index 00153

Re: [FFmpeg-devel] [PATCH] doc/ffserver: merge paragraph starting with "What happens next?" with previous one

2014-08-02 Thread Timothy Gu
On Aug 2, 2014 7:40 AM, "Stefano Sabatini" wrote: > > The name of the paragraph sounded a bit silly, and its text is small so > it's better to merge it with the previous paragraph. > --- > doc/ffserver.texi | 8 +++- > 1 file changed, 3 insertions(+), 5 deletions(-) OK. [...] Timothy _

[FFmpeg-devel] [PATCH] doc/ffserver: merge paragraph starting with "What happens next?" with previous one

2014-08-02 Thread Stefano Sabatini
The name of the paragraph sounded a bit silly, and its text is small so it's better to merge it with the previous paragraph. --- doc/ffserver.texi | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/doc/ffserver.texi b/doc/ffserver.texi index fb90dd6..2bf8152 100644 --- a/d

[FFmpeg-devel] [PATCH] ffserver: deprecate Port and BindAddress options in favor of HTTPPort and HTTPBindAddress

2014-08-02 Thread Stefano Sabatini
The new option names are more explicit. --- doc/ffserver.conf | 4 ++-- doc/ffserver.texi | 19 ++- ffserver.c| 10 +++--- 3 files changed, 23 insertions(+), 10 deletions(-) diff --git a/doc/ffserver.conf b/doc/ffserver.conf index 094c093..b756961 100644 --- a/doc/ffs

Re: [FFmpeg-devel] [PATCH] msvc: fix implicitly declared read/close.

2014-08-02 Thread Michael Niedermayer
On Sat, Aug 02, 2014 at 12:01:37PM +1000, Matt Oliver wrote: > MSVC/ICL does not have unistd.h as standard and instead declares read/close > functions in io.h. This is currently handled in some files but not in a > couple. Adding an include for io.h where needed adds the correct > declarations. >

Re: [FFmpeg-devel] HEVC SIMD Optimization on PowerPC

2014-08-02 Thread Michael Niedermayer
On Sat, Aug 02, 2014 at 07:18:22AM -0400, compn wrote: > On Sat, 2 Aug 2014 13:37:10 +0800 > "Zhenan Lin" wrote: > > > Hi, all, > > > > > > > > Our team plans to contribute SIMD optimization on PowerPC for HEVC > > i thought powerpc was dead? i thought so too, but powerpc optimizations cert

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

2014-08-02 Thread Michael Niedermayer
On Mon, Jul 28, 2014 at 05:17:24PM +, Christophe Gisquet wrote: > MAX_PB_SIZE is used or assumed for various buffer strides. In some cases, > it is used as constant parameter(s) to functions. > > Make use of that knowledge to: > - not pass the parameter > - avoid extra GPR usage > - precompute

[FFmpeg-devel] [PATCH] ffserver: initialize pbuffer in prepare_sdp_description()

2014-08-02 Thread Michael Niedermayer
also check pbuffer before use Found-by: CSA Signed-off-by: Michael Niedermayer --- ffserver.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ffserver.c b/ffserver.c index 00153b4..956d9f5 100644 --- a/ffserver.c +++ b/ffserver.c @@ -2977,6 +2977,8 @@ static int prepare

Re: [FFmpeg-devel] HEVC SIMD Optimization on PowerPC

2014-08-02 Thread compn
On Sat, 2 Aug 2014 13:37:10 +0800 "Zhenan Lin" wrote: > Hi, all, > > > > Our team plans to contribute SIMD optimization on PowerPC for HEVC i thought powerpc was dead? i dont know if anyone is working on it. -compn ___ ffmpeg-devel mailing list ff

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

2014-08-02 Thread Michael Niedermayer
On Wed, Jul 30, 2014 at 11:44:47PM +0200, Nicolas George wrote: > Also deprecate adding a NULL frame to mark EOF. > > TODO APIchanges entry, version bump. > > Signed-off-by: Nicolas George > --- > libavfilter/buffersrc.c | 40 ++-- > libavfilter/buffersrc.h |

Re: [FFmpeg-devel] [PATCH 3/4] ffmpeg: use av_buffersrc_close().

2014-08-02 Thread Michael Niedermayer
On Wed, Jul 30, 2014 at 11:44:48PM +0200, Nicolas George wrote: > Signed-off-by: Nicolas George > --- > ffmpeg.c | 14 -- > 1 file changed, 4 insertions(+), 10 deletions(-) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically faster