Re: [FFmpeg-devel] [PATCH 01/16] web: Modernize with Bower, Bootstrap, LESS

2014-07-14 Thread James Darnley
On 2014-07-14 10:31, Clément Bœsch wrote: src/about | 2 - src/about_js | 0 src/about_title | 2 +- src/archive | 2 - src/archive_js| 0 src/archive_title | 2 +- src/bugreports

Re: [FFmpeg-devel] [PATCH 01/16] web: Modernize with Bower, Bootstrap, LESS

2014-07-20 Thread James Darnley
On 2014-07-20 15:07, Clément Bœsch wrote: On Sun, Jul 20, 2014 at 12:59:58PM +0200, James Darnley wrote: On 2014-07-14 12:34, db0 company wrote: On Mon, Jul 14, 2014 at 12:32 PM, James Darnley james.darn...@gmail.com wrote: On 2014-07-14 12:23, db0 company wrote: On Mon, Jul 14, 2014 at 12

Re: [FFmpeg-devel] [PATCH 1/4] lavc/flacenc: add sse4 version of the 16-bit lpc encoder

2014-07-20 Thread James Darnley
On 2014-03-15 00:01, Michael Niedermayer wrote: On Wed, Mar 12, 2014 at 01:03:03PM +0100, James Darnley wrote: +; Is it worth looping correctly over the first samples? The most that ever need +; to be copied is 32 so we might as well just unroll the loop and do all 32. implementations

Re: [FFmpeg-devel] rectification filter

2014-07-29 Thread James Darnley
On 2014-07-29 07:41, Daniel Oberhoff wrote: 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 + *

Re: [FFmpeg-devel] [PATCH 1/4] lavc/flacenc: add sse4 version of the 16-bit lpc encoder

2014-08-08 Thread James Darnley
On 2014-07-21 01:48, Michael Niedermayer wrote: On Mon, Jul 21, 2014 at 12:32:23AM +0200, James Darnley wrote: On 2014-03-15 00:01, Michael Niedermayer wrote: On Wed, Mar 12, 2014 at 01:03:03PM +0100, James Darnley wrote: +; Is it worth looping correctly over the first samples? The most

Re: [FFmpeg-devel] [PATCH] avcodec: add C xvid IDCT support

2014-08-11 Thread James Darnley
On 2014-08-11 12:39, Michael Niedermayer wrote: On Mon, Aug 11, 2014 at 10:56:11AM +0200, James Darnley wrote: On 2014-08-11 07:44, Clément Bœsch wrote: /** * @file * Walken IDCT * Alternative idct implementations for decoding compatibility * * @author Skal * @note this C version

[FFmpeg-devel] [PATCH 4/6] lavc/flacenc: partially unroll loop in flac_enc_lpc_32

2014-08-12 Thread James Darnley
Now does 6 samples per iteration, up from 2. From 1.6 to 2.1 times faster again. 2.5 to 3.9 times faster overall. Runtime is reduced by a further 4 to 17%. Reduced by 9 to 65% overall. Same conditions as previously. --- libavcodec/x86/flac_dsp_gpl.asm | 32 +++-

[FFmpeg-devel] [PATCH 6/6] lavc/flacdsp: document limitations of the LPC encoder

2014-08-12 Thread James Darnley
--- libavcodec/flacdsp.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/libavcodec/flacdsp.h b/libavcodec/flacdsp.h index 272cf2a..36cd904 100644 --- a/libavcodec/flacdsp.h +++ b/libavcodec/flacdsp.h @@ -27,6 +27,13 @@ typedef struct FLACDSPContext {

Re: [FFmpeg-devel] [PATCH 6/6] lavc/flacdsp: document limitations of the LPC encoder

2014-08-13 Thread James Darnley
On 2014-08-13 04:50, Michael Niedermayer wrote: On Tue, Aug 12, 2014 at 11:22:07PM +0200, James Darnley wrote: --- libavcodec/flacdsp.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/libavcodec/flacdsp.h b/libavcodec/flacdsp.h index 272cf2a..36cd904 100644

Re: [FFmpeg-devel] [PATCH 6/6] lavc/flacdsp: document limitations of the LPC encoder

2014-08-13 Thread James Darnley
On 2014-08-13 14:05, Michael Niedermayer wrote: On Wed, Aug 13, 2014 at 10:46:45AM +0200, James Darnley wrote: On 2014-08-13 04:50, Michael Niedermayer wrote: On Tue, Aug 12, 2014 at 11:22:07PM +0200, James Darnley wrote: --- libavcodec/flacdsp.h |7 +++ 1 files changed, 7

[FFmpeg-devel] [PATCH 1/2] lavc/flacdsp: document limitations of the LPC encoder

2014-08-14 Thread James Darnley
State that the maximum value of order is 32. This limit is used in both C and x86 assebly code. --- libavcodec/flacdsp.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/libavcodec/flacdsp.h b/libavcodec/flacdsp.h index 272cf2a..e8487c8 100644 ---

[FFmpeg-devel] [PATCH] fate: test flac LPC encoder

2014-08-14 Thread James Darnley
The LPC encoder is not used at compression levels below 3 but the basic acodec test uses -compression_level 2. This adds a test using -compression_level 5, which is the current default. --- tests/fate/acodec.mak|4 +++- tests/ref/acodec/flac-comp-5 |4 2 files changed, 7

[FFmpeg-devel] [PATCH 2/2] lavc/flacdsp: change lpc_encoder function pointer prototype

2014-08-14 Thread James Darnley
This should help to clarify the API. --- libavcodec/flacdsp.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavcodec/flacdsp.h b/libavcodec/flacdsp.h index e8487c8..23d1ae4 100644 --- a/libavcodec/flacdsp.h +++ b/libavcodec/flacdsp.h @@ -35,7 +35,7 @@ typedef struct

Re: [FFmpeg-devel] [PATCH] fate: test flac LPC encoder

2014-08-14 Thread James Darnley
After further searching and understanding the makefile I see that the lpc code is tested by the complete fate sample suite. This patch is unnecessary. Anyway to answer the questions... On 2014-08-14 18:14, Christophe Gisquet wrote: Hi, 2014-08-14 18:08 GMT+02:00 James Darnley james.darn

Re: [FFmpeg-devel] [PATCH 2/7] avcodec/dvdsubdec: Dont mix integers with pointers

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/dvdsubdec.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Fine, clearly a pointer. signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] [PATCH 3/7] avcodec/dvbsubdec: dont mix integers with pointers

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/dvbsubdec.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Fine, both pointers. signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] [PATCH 5/7] avformat/udp: dont mix integers with pointers

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/udp.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/udp.c b/libavformat/udp.c index a8baa5b..8412eeb 100644 --- a/libavformat/udp.c

Re: [FFmpeg-devel] [PATCH 6/7] avcodec/hevc_mvs: dont redundantly initialize ref_idx_curr

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/hevc_mvs.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) fine signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] [PATCH 7/7] avcodec/mips/compute_antialias_float: remove unused variable

2014-08-16 Thread James Darnley
On 2014-08-16 01:15, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavcodec/mips/compute_antialias_float.h |3 --- 1 file changed, 3 deletions(-) It is unused so that is fine. signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] [PATCH] doc/filters/apad: extend documentation

2014-08-18 Thread James Darnley
On 2014-08-18 14:53, Stefano Sabatini wrote: +@item whole_len +Set the target number of sample in the audio stream. After the value ^^ samples, plural signature.asc Description: OpenPGP digital signature ___

[FFmpeg-devel] [PATCH] lavu/frame: minor correction of language in comment about linesize

2014-08-27 Thread James Darnley
--- libavutil/frame.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavutil/frame.h b/libavutil/frame.h index 2391d3e..d6d017c 100644 --- a/libavutil/frame.h +++ b/libavutil/frame.h @@ -162,7 +162,7 @@ typedef struct AVFrame { * For audio, only linesize[0] may

Re: [FFmpeg-devel] [PATCH] lavu/frame: minor correction of language in comment about linesize

2014-08-27 Thread James Darnley
On 2014-08-27 22:18, Timothy Gu wrote: On Wed, Aug 27, 2014 at 12:38 PM, James Darnley james.darn...@gmail.com wrote: --- libavutil/frame.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) OK Thanks but don't apply this patch just yet. Similar typos were highlighted on IRC

Re: [FFmpeg-devel] Request for testing

2014-09-02 Thread James Darnley
On 2014-09-02 20:39, Kelly Anderson wrote: TESTacodec-flac-comp-5 TESTacodec-flac-24-comp-5 tests/Makefile:192: recipe for target 'fate-acodec-flac-24-comp-5' failed make: *** [fate-acodec-flac-24-comp-5] Error 1 Thanks. Good to know that the 16-bit version works. I am probably

Re: [FFmpeg-devel] Request for testing

2014-09-02 Thread James Darnley
On 2014-09-02 21:25, James Darnley wrote: On 2014-09-02 20:39, Kelly Anderson wrote: TESTacodec-flac-comp-5 TESTacodec-flac-24-comp-5 tests/Makefile:192: recipe for target 'fate-acodec-flac-24-comp-5' failed make: *** [fate-acodec-flac-24-comp-5] Error 1 Thanks. Good to know

Re: [FFmpeg-devel] [PATCH 1/4] libavutil/error: fix build with musl toolchain

2014-09-02 Thread James Darnley
On 2014-09-03 02:13, Michael Niedermayer wrote: On Tue, Sep 02, 2014 at 11:49:47PM +0200, Reimar Döffinger wrote: [...] reason that configure is already too slow. ccache + --tempprefix + dash instead of bash should help a bit Tell that to Windows! Okay trying dash did reduce the 103

[FFmpeg-devel] [PATCH 0/6] Miscellaneous x86inc patches

2014-09-04 Thread James Darnley
(2): x86inc: Make ym# behave the same way as xm# x86inc: Make INIT_CPUFLAGS support an arbitrary number of cpuflags James Darnley (2): lavc/x86/v210: give cpuflag to INIT macro lavu/x86/x86inc: deprecate INIT_AVX Loren Merritt (1): x86inc: free up variable name n in global namespace

[FFmpeg-devel] [PATCH 1/6] x264asm: warn when inappropriate instruction used in function with specified cpuflags

2014-09-04 Thread James Darnley
From: Anton Mitrofanov bugmas...@narod.ru --- libavutil/x86/x86inc.asm | 581 +++--- 1 files changed, 295 insertions(+), 286 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index 1cfb055..87f3af7 100644 ---

[FFmpeg-devel] [PATCH 4/6] lavu/x86/x86inc: deprecate INIT_AVX

2014-09-04 Thread James Darnley
The same can be done with INIT_XMM avx --- libavutil/x86/x86inc.asm |8 1 files changed, 0 insertions(+), 8 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index 65a23d8..c028c0b 100644 --- a/libavutil/x86/x86inc.asm +++ b/libavutil/x86/x86inc.asm @@

[FFmpeg-devel] [PATCH 6/6] x86inc: Make INIT_CPUFLAGS support an arbitrary number of cpuflags

2014-09-04 Thread James Darnley
From: Henrik Gramner hen...@gramner.com Previously there was a limit of two cpuflags. --- libavutil/x86/x86inc.asm | 41 ++--- 1 files changed, 22 insertions(+), 19 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index

Re: [FFmpeg-devel] [PATCH] avutil/md5: workaround clang 3.5 #20849

2014-09-04 Thread James Darnley
On 2014-09-05 00:31, Michael Niedermayer wrote: This avoids several failures on fate.ffmpeg.org, and thus makes real bugs easier to spot A good reason. +volatile uint8_t in[1000]; // volatile to workaround http://llvm.org/bugs/show_bug.cgi?id=20849 Perhaps add a FIXME to the comment so

Re: [FFmpeg-devel] MXF : default fied dominance is TFF

2014-09-08 Thread James Darnley
On 2014-09-08 14:17, Gaullier Nicolas wrote: [FFmpeg-devel] MXF : default fied dominance is TFF That should be *field* dominance. If you're going to submit another patch please correct the commit message. signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread James Darnley
On 2014-09-11 23:00, Timothy Gu wrote: On Thu, Sep 11, 2014 at 1:57 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Lou Logan lou at lrcd.com writes: - a href=http://www.mplayerhq.hu/DOCS/tech/patches.txt; MPlayer patch guidelines/a I understand that this is not ideal but ... + a

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-11 Thread James Darnley
On 2014-09-11 23:10, James Darnley wrote: I support the change and I am willing to look at adding and adapting some relevant points from the mplayer text. Point 0, diffs please, is covered by our request for git format-patch in the patch submission checklist[1] point 2 and the git how-to page[2

[FFmpeg-devel] [PATCH 0/6] mplayer pacthes text missing points

2014-09-11 Thread James Darnley
This adds some text covering points from that mplayer text which are missing in our docs. I welcome your comments, particularly on the language used. The first two patches are really just cosmetic changes but I noticed them while reading the file. James Darnley (6): docs: add missing code

[FFmpeg-devel] [PATCH 1/6] docs: add missing code formatting around commands

2014-09-11 Thread James Darnley
--- doc/developer.texi | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/doc/developer.texi b/doc/developer.texi index 1e1d3b8..e51e97a 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -328,7 +328,7 @@ area changed: Short 1 line description details describing

[FFmpeg-devel] [PATCH 5/6] docs: mention when one might be offered write access to the git repository

2014-09-11 Thread James Darnley
This addresses point 11 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link --- doc/developer.texi | 5 + 1 file changed, 5 insertions(+) diff --git

[FFmpeg-devel] [PATCH 6/6] docs: mention when and how one should be added as an author

2014-09-11 Thread James Darnley
This addresses point 13 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web/contact: replace MPlayer guidelines link --- doc/developer.texi | 5 + 1 file changed, 5 insertions(+) diff --git

Re: [FFmpeg-devel] [PATCH 1/6] docs: add missing code formatting around commands

2014-09-12 Thread James Darnley
On 2014-09-12 01:33, Timothy Gu wrote: On Thu, Sep 11, 2014 at 4:31 PM, James Darnley james.darn...@gmail.com wrote: --- doc/developer.texi | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) I would use @command{}. I will change it to command. Should I go through the file(s

Re: [FFmpeg-devel] [PATCH 3/6] docs: mention the preferred character to use in options

2014-09-12 Thread James Darnley
On 2014-09-12 01:34, Timothy Gu wrote: On Thu, Sep 11, 2014 at 4:31 PM, James Darnley james.darn...@gmail.com wrote: This addresses point 12 from the mplayer patches text file. Discussion about removing a link to this document can be found in the thread on ffmpeg-devel titled: [PATCH] web

Re: [FFmpeg-devel] [PATCH] web/contact: replace MPlayer guidelines link

2014-09-12 Thread James Darnley
On 2014-09-12 00:15, Carl Eugen Hoyos wrote: James Darnley james.darnley at gmail.com writes: Point 2, unified diffs, is covered by out request for patches. Are we doing to allow git diff dumps? While I always send unified diffs, some people (still) send other diffs, and one regular

Re: [FFmpeg-devel] [WIP] lavfi: port mp/eq to a regular filter

2014-09-15 Thread James Darnley
On 2014-09-15 13:17, Clément Bœsch wrote: On Mon, Sep 15, 2014 at 12:46:14AM +0200, James Darnley wrote: --- Still a couple of things on the todo list (see the top of the new file) but I thought I would send the basics along for people to nit pick. configure| 1

Re: [FFmpeg-devel] [WIP] lavfi: port mp/eq to a regular filter

2014-09-15 Thread James Darnley
On 2014-09-15 02:03, Timothy Gu wrote: On Sep 14, 2014 3:46 PM, James Darnley james.darn...@gmail.com wrote: --- Still a couple of things on the todo list (see the top of the new file) but I thought I would send the basics along for people to nit pick. configure| 1

Re: [FFmpeg-devel] [WIP] lavfi: port mp/eq to a regular filter

2014-09-15 Thread James Darnley
On 2014-09-15 00:46, James Darnley wrote: ... Update: I've got eq2 merged into the one file and working. I would send another WIP patch but I've made several commits, all of which should be merged together. One question I do have though is: should the user options for contrast in eq and eq2

Re: [FFmpeg-devel] [PATCH] x86/mpegvideoencdsp: improve ff_pix_sum16_sse2

2014-10-01 Thread James Darnley
On 2014-10-01 03:24, James Almer wrote: ~15 faster. 15 what faster? 15 cycles? 15%? 15 times? signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 2/2] x86/mlpdec: add ff_mlp_rematrix_channel_{sse4, avx2}

2014-10-02 Thread James Darnley
On 2014-09-30 20:50, James Almer wrote: +pshufdm2, m0, 0xb1 +pshufdm3, m1, 0xb1 +pshufdm3, m1, 0xb1 +pshufdm4, m2, 0xb1 +pshufd xm1, xm0, 0x4e +pshufd xm2, xm0, 0xb1 +pshufd xm3, xm1, 0xb1 +pshufd

[FFmpeg-devel] Use the git hooks feature on your PC

2014-10-12 Thread James Darnley
Since we have a few newbies on the list with the start of OPW and I've seen a few other cases recently, I thought this would be a good time to remind people of the feature of Git called Hooks. Simply put: a git hook is an appropriately named executable file in the .git/hook directory in your

Re: [FFmpeg-devel] [PATCH 2/7] fate: fix some subtitles tests dependencies

2014-10-14 Thread James Darnley
On 2014-10-11 19:59, Clément Bœsch wrote: --- tests/fate/subtitles.mak | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Looks okay. signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 3/7] avformat/assenc: support remuxing old V4 ASS form (SSA)

2014-10-14 Thread James Darnley
On 2014-10-11 19:59, Clément Bœsch wrote: --- libavformat/assenc.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) The diff looks fine. So if the code does what you want it to do, push. Just one comment below. +ass-ssa_mode = !strstr(avctx-extradata, \n[V4+

Re: [FFmpeg-devel] [PATCH 4/7] avcodec/ass: output missing fields in AVSubtitles and output files

2014-10-14 Thread James Darnley
On 2014-10-11 19:59, Clément Bœsch wrote: Fixes Ticket #3207 --- libavcodec/ass.c| 29 +-- libavcodec/version.h| 2 +- tests/ref/fate/sub-aqtitle | 72 +-- tests/ref/fate/sub-charenc | 106

Re: [FFmpeg-devel] [PATCH 5/7] avcodec/ass_split: extend recognized fields in ASS splitter

2014-10-14 Thread James Darnley
On 2014-10-11 19:59, Clément Bœsch wrote: diff --git a/libavcodec/ass_split.h b/libavcodec/ass_split.h index 06c1ce3..c912252 100644 --- a/libavcodec/ass_split.h +++ b/libavcodec/ass_split.h @@ -41,13 +41,28 @@ typedef struct { char *font_name; /** font face (case sensitive) */

Re: [FFmpeg-devel] Facebook page

2014-10-15 Thread James Darnley
On 2014-10-15 10:49, Thomas Volkert wrote: Please, vote for a Facebook url here: http://goo.gl/forms/AHVFvPxp8X This will show us the community opinion about this. I can't. It requires a google account. signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] [FFmpeg-user] How to create trapezoid videos with ffmpeg?

2014-10-17 Thread James Darnley
On 2014-10-17 23:09, Nicholas Robbins wrote: So is it possible to make a filter that is basically just a wrapper around another filter? This hypothetical filter (call it vf_keystone) would just call vf_perspective with crazy options. Clearly I could just copy the vf_perspective filter

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 does not matter

Re: [FFmpeg-devel] [PATCH] avfilter: add removegrain

2015-07-06 Thread James Darnley
On 2015-07-05 20:14, Paul B Mahol wrote: + * TODO: add SIMD If you and nobody else is working assembly then I am volunteering (and have literally just started). signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] avfilter: add removegrain

2015-07-06 Thread James Darnley
On 2015-07-05 20:14, Paul B Mahol wrote: +static int mode01(int c, int a1, int a2, int a3, int a4, int a5, int a6, int a7, int a8) +{ +const int mi = FFMIN(FFMIN(FFMIN(a1, a2), FFMIN(a3, a4)), FFMIN(FFMIN(a5, a6), FFMIN(a7, a8))); +const int ma = FFMAX(FFMAX(FFMAX(a1, a2),

Re: [FFmpeg-devel] How to open OpenMP in ffmpeg compile?

2015-07-30 Thread James Darnley
On 2015-07-30 15:36, Nemo wrote: I need -fopenmp to tell gcc to use OpenMP now. But where should I insert -fopenmp to? Add it to the compiler and linker flags. The easiest way to do this with ffmpeg is probably to configure with --extra-cflags=-fopenmp and --extra-ldflags=-fopenmp

Re: [FFmpeg-devel] [PATCH] avfilter/internal: Improve docs about ff_request_frame()

2015-07-14 Thread James Darnley
On 2015-07-14 21:47, Michael Niedermayer wrote: On Tue, Jul 14, 2015 at 08:42:40PM +0200, James Darnley wrote: On 2015-07-14 19:50, Michael Niedermayer wrote: + * One example of a source returning AVERROR(EAGAIN) is a buffer into which the + * user application pushes new data, it cannot

Re: [FFmpeg-devel] [PATCH] lavfi: add erosion, dilation, deflate inflate filter

2015-07-14 Thread James Darnley
On 2015-07-15 00:15, Paul B Mahol wrote: diff --git a/doc/filters.texi b/doc/filters.texi index 9f7b976..d4d8bad 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -3718,6 +3718,24 @@ Set whether or not chroma is considered in the metric calculations. Default is @code{1}. @end

Re: [FFmpeg-devel] [PATCH] lavfi/removegrain: add x86 and x86_64 SSE2 functions

2015-07-14 Thread James Darnley
On 2015-07-14 23:23, James Almer wrote: On 14/07/15 3:54 PM, James Darnley wrote: On 2015-07-11 18:34, James Almer wrote: On 11/07/15 10:40 AM, James Darnley wrote: new file mode 100644 index 000..5e1feea --- /dev/null +++ b/libavfilter/x86/vf_removegrain.asm @@ -0,0 +1,1215

[FFmpeg-devel] [PATCH] lavfi/removegrain: add x86 and x86_64 SSE2 functions

2015-07-14 Thread James Darnley
..60401fb --- /dev/null +++ b/libavfilter/removegrain.h @@ -0,0 +1,40 @@ +/* + * Copyright (c) 2015 Paul B Mahol + * Copyright (c) 2015 James Darnley + * + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser

Re: [FFmpeg-devel] normalize-audio and video files

2015-07-16 Thread James Darnley
On 2015-07-17 02:01, Emanuel Berg wrote: This zsh, using ffmpeg and normalize-audio, seems to do it. But it is slow. Remember normalize-audio is lightning fast so time is spent getting and putting together the audio with the movie. Perhaps it can be optimized somehow - do tell, if you know...

Re: [FFmpeg-devel] [PATCH] avfilter: add convolution filter

2015-07-17 Thread James Darnley
On 2015-07-15 21:48, Paul B Mahol wrote: Signed-off-by: Paul B Mahol one...@gmail.com --- libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_convolution.c | 331 +++ 3 files changed, 333 insertions(+) create

Re: [FFmpeg-devel] [PATCH] avfilter: add Dynamic Audio Normalizer filter

2015-07-17 Thread James Darnley
On 2015-07-09 18:55, Paul B Mahol wrote: +{ n, enable channels coupling, OFFSET(channels_coupled), AV_OPT_TYPE_INT, {.i64 = 1}, 0, 1, FLAGS }, This should be enable channel coupling +static const double s_pi =

Re: [FFmpeg-devel] [PATCH] avfilter: add Dynamic Audio Normalizer filter

2015-07-17 Thread James Darnley
On 2015-07-09 18:55, Paul B Mahol wrote: diff --git a/doc/filters.texi b/doc/filters.texi index 3fce874..74c408a 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -1520,6 +1520,164 @@ Optional. It should have a value much less than 1 (e.g. 0.05 or 0.02) and is used to prevent

Re: [FFmpeg-devel] [PATCH] avfilter: add Dynamic Audio Normalizer filter

2015-07-17 Thread James Darnley
On 2015-07-17 12:55, Paul B Mahol wrote: On 7/17/15, James Darnley james.darn...@gmail.com wrote: On 2015-07-09 18:55, Paul B Mahol wrote: +{ n, enable channels coupling, OFFSET(channels_coupled), AV_OPT_TYPE_INT, {.i64 = 1}, 0, 1, FLAGS }, This should be enable channel coupling Fixed

Re: [FFmpeg-devel] [PATCH] vf_psnr: sse2 optimizations for sum-squared-error.

2015-07-13 Thread James Darnley
On 2015-07-13 01:34, Ronald S. Bultje wrote: Hi, On Sun, Jul 12, 2015 at 5:54 PM, Paul B Mahol one...@gmail.com wrote: On 7/12/15, Ronald S. Bultje rsbul...@gmail.com wrote: +typedef struct PSNRDSPContext { +uint64_t (*sse_line)(const uint8_t *buf, const uint8_t *ref, int w);

Re: [FFmpeg-devel] [PATCH] avfilter: add deband filter

2015-07-15 Thread James Darnley
On 2015-07-15 00:06, Paul B Mahol wrote: Signed-off-by: Paul B Mahol one...@gmail.com --- doc/filters.texi | 35 ++ libavfilter/Makefile | 1 + libavfilter/allfilters.c | 1 + libavfilter/vf_deband.c | 312 +++ 4 files

Re: [FFmpeg-devel] [PATCH] avfilter/internal: Improve docs about ff_request_frame()

2015-07-14 Thread James Darnley
On 2015-07-14 19:50, Michael Niedermayer wrote: From: Michael Niedermayer mich...@niedermayer.cc Signed-off-by: Michael Niedermayer mich...@niedermayer.cc --- libavfilter/internal.h | 17 + 1 file changed, 17 insertions(+) diff --git a/libavfilter/internal.h

Re: [FFmpeg-devel] [PATCH] lavfi/removegrain: add x86 and x86_64 SSE2 functions

2015-07-14 Thread James Darnley
On 2015-07-11 18:34, James Almer wrote: On 11/07/15 10:40 AM, James Darnley wrote: @@ -566,7 +554,19 @@ static int filter_slice(AVFilterContext *ctx, void *arg, int jobnr, int nb_jobs) } *dst++ = *src++; -for (x = 1; x s-planewidth[i] - 1; x

Re: [FFmpeg-devel] [PATCH] avcodec: disallow hwaccel with frame threads

2015-10-23 Thread James Darnley
On 2015-10-23 13:54, Hendrik Leppkes wrote: > The only reason the combination of frame threads and HWAccel was > considered useful is to allow a seamless fallback to multi-threaded > software decoding if the HWAccel is not available, however the issues > outlined above far outweight this.

Re: [FFmpeg-devel] forcing ints to be 64 bits, possible new FATE client idea

2015-10-21 Thread James Darnley
On 2015-10-21 12:18, wm4 wrote: > with size_t/ptrdiff_t > being 128 bit, and a new "long long long int" type (I swear, they will > do it, even if that type name looks horrible). Please no! Just require a C99 style uint128_t/int128_t type. signature.asc Description: OpenPGP digital signature

Re: [FFmpeg-devel] forcing ints to be 64 bits, possible new FATE client idea

2015-10-21 Thread James Darnley
On 2015-10-21 14:44, Clément Bœsch wrote: > On Wed, Oct 21, 2015 at 06:00:21AM -0400, Ganesh Ajjanagadde wrote: > [...] >> why don't you spend 5 minutes trying to outline to beginners like me >> what is "actually important" in your view? >> > > According to the first 100 answers of the survey,

Re: [FFmpeg-devel] [PATCH] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-10 Thread James Darnley
On 2015-10-10 23:06, Ganesh Ajjanagadde wrote: > ... Is the greatest common denominator (yes, I had to look that up) actually used anywhere that is slow and needs to be fast? All the uses of 'av_gcd' found by grep appear be dealing with timing. I see framerate, timebase, scale. I do see uses

Re: [FFmpeg-devel] [PATCH] gitignore: ignore object file temporaries

2015-10-10 Thread James Darnley
On 2015-10-10 00:43, Ganesh Ajjanagadde wrote: > During a build, a lot of *.o-hash files are created - had not noticed > this as they are usually dumped in tmpfs on Linux. However, they > sometimes are present during a long build in the project directory, making it > annoying to commit while the

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-10-10 Thread James Darnley
On 2015-10-09 14:46, Nicolas George wrote: > Le quartidi 4 vendémiaire, an CCXXIV, James Darnley a écrit : >> I can. You should find it attached to this email. I cleaned it up and >> put two test cases of data into the file. You will need Lua and the >> Lua-iconv mod

Re: [FFmpeg-devel] [PATCH] lavfi/removegrain: add x86 and x86_64 SSE2 functions

2015-07-11 Thread James Darnley
On 2015-07-11 15:45, Carl Eugen Hoyos wrote: James Darnley james.darnley at gmail.com writes: +;* TODO: gpl text goes here. Yes, please copy it from another asm file. And please make sure that the file does not get compiled if --enable-gpl was not specified. Dammit! Of course I

[FFmpeg-devel] [PATCH] lavfi/removegrain: add x86 and x86_64 SSE2 functions

2015-07-11 Thread James Darnley
/libavfilter/removegrain.h new file mode 100644 index 000..60401fb --- /dev/null +++ b/libavfilter/removegrain.h @@ -0,0 +1,40 @@ +/* + * Copyright (c) 2015 Paul B Mahol + * Copyright (c) 2015 James Darnley + * + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute

[FFmpeg-devel] [PATCH] lavfi/removegrain: add x86 and x86_64 SSE2 functions

2015-07-11 Thread James Darnley
James Darnley + * + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any

Re: [FFmpeg-devel] [PATCHv3] avutil/common: add av_rint64_clip

2015-11-13 Thread James Darnley
On 2015-11-13 15:23, Ganesh Ajjanagadde wrote: > diff --git a/libavutil/version.h b/libavutil/version.h > index 909f9a6..ea10ff0 100644 > --- a/libavutil/version.h > +++ b/libavutil/version.h > @@ -56,8 +56,8 @@ > */ > > #define LIBAVUTIL_VERSION_MAJOR 55 > -#define LIBAVUTIL_VERSION_MINOR

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread James Darnley
On 2015-09-27 14:37, Stefano Sabatini wrote: > --- > src/index | 234 > ++ > 1 file changed, 234 insertions(+) > > diff --git a/src/index b/src/index > index 9d12a7b..8937172 100644 > --- a/src/index > +++ b/src/index > @@ -37,6

Re: [FFmpeg-devel] [PATCH 2/2] tests/fate/source-check.sh: Check for common typos

2015-09-29 Thread James Darnley
On 2015-09-29 21:56, Clément Bœsch wrote: > On Tue, Sep 29, 2015 at 09:21:53PM +0200, Hendrik Leppkes wrote: >> I agree, we have patchcheck for typo checking. > > A lot of people do not run patchcheck (I personally never do, and given > that we fix typo on a regular basis I'm probably not the

Re: [FFmpeg-devel] [PATCH] avfilter/vf_maskedmerge: add SIMD for maskedmerge with 8 bit depth input

2015-10-01 Thread James Darnley
On 2015-10-01 19:25, Paul B Mahol wrote: > +cglobal maskedmerge8, 10, 11, 3, 0, bsrc, blinesize, osrc, olinesize, msrc, > mlinesize, dst, dlinesize, w, h You need a guard to prevent this being compiled on x86. > +lea bsrcq, [bsrcq+blinesizeq] > +lea osrcq, [osrcq+olinesizeq] > +lea

Re: [FFmpeg-devel] [PATCH 0/7] [RFC] x86 assembly constants

2015-10-03 Thread James Darnley
On 2015-10-03 04:08, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 2, 2015 at 4:58 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote: > >> On Fri, Oct 2, 2015 at 7:16 PM, Timothy Gu <timothyg...@gmail.com> wrote: >>> On Fri, Oct 2, 2015 at 10:08 AM James Darn

[FFmpeg-devel] [PATCH 4/7] avcodec: use new constants in assembly

2015-10-02 Thread James Darnley
--- libavcodec/x86/ac3dsp.asm | 2 +- libavcodec/x86/bswapdsp.asm | 3 +-- libavcodec/x86/diracdsp_yasm.asm| 6 +- libavcodec/x86/dwt_yasm.asm | 6 +- libavcodec/x86/h263_loopfilter.asm | 2 +- libavcodec/x86/h264_chromamc.asm

[FFmpeg-devel] [PATCH 6/7] swscale: use new constants in assembly

2015-10-02 Thread James Darnley
--- libswscale/x86/Makefile | 1 + libswscale/x86/constants.asm | 1 + libswscale/x86/output.asm| 5 + tests/ref/fate/source| 1 + 4 files changed, 4 insertions(+), 4 deletions(-) create mode 100644 libswscale/x86/constants.asm diff --git a/libswscale/x86/Makefile

[FFmpeg-devel] [PATCH 7/7] fixup! avfilter: use new constants in assembly

2015-10-02 Thread James Darnley
--- tests/ref/fate/source | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/ref/fate/source b/tests/ref/fate/source index 781f4cd..c1383dd 100644 --- a/tests/ref/fate/source +++ b/tests/ref/fate/source @@ -9,6 +9,7 @@ libavcodec/reverse.c libavcodec/x86/constants.asm

Re: [FFmpeg-devel] [PATCH 0/7] [RFC] x86 assembly constants

2015-10-02 Thread James Darnley
On 2015-10-02 19:16, Timothy Gu wrote: > On Fri, Oct 2, 2015 at 10:08 AM James Darnley <james.darn...@gmail.com> > wrote: > >> The third patch uses them in the remaining inline assembly. >> > > That's the crux of the problem: inline asm uses these constants, but

[FFmpeg-devel] [PATCH 1/7] avutil: add shared assembly constants

2015-10-02 Thread James Darnley
--- So here is the test file I was working on with the thoughts I had. --- ; This section is intended to possibly be included in x86inc.asm ; Align all constant to 32 bytes whether they are used in AVX code or not. %assign constant_align 32 ; Value to be used as padding to achieve alignment.

[FFmpeg-devel] [PATCH 3/7] avcodec: use new constants in C inline assembly

2015-10-02 Thread James Darnley
--- libavcodec/x86/cavsdsp.c| 2 +- libavcodec/x86/constants.h | 66 - libavcodec/x86/inline_asm.h | 2 +- libavcodec/x86/vc1dsp_mmx.c | 2 +- 4 files changed, 3 insertions(+), 69 deletions(-) delete mode 100644 libavcodec/x86/constants.h diff

[FFmpeg-devel] [PATCH 5/7] avfilter: use new constants in assembly

2015-10-02 Thread James Darnley
--- libavfilter/x86/Makefile | 2 ++ libavfilter/x86/af_volume.asm | 3 +-- libavfilter/x86/constants.asm | 1 + libavfilter/x86/vf_fspp.asm| 3 +-- libavfilter/x86/vf_removegrain.asm | 3 +-- libavfilter/x86/vf_ssim.asm| 2 +- libavfilter/x86/vf_yadif.asm

[FFmpeg-devel] [PATCH 2/7] avcodec: replace old C file with new assembly constants

2015-10-02 Thread James Darnley
--- libavcodec/x86/Makefile | 2 +- libavcodec/x86/constants.asm | 1 + libavcodec/x86/constants.c | 81 tests/ref/fate/source| 1 + 4 files changed, 3 insertions(+), 82 deletions(-) create mode 100644 libavcodec/x86/constants.asm

[FFmpeg-devel] [PATCH 0/7] [RFC] x86 assembly constants

2015-10-02 Thread James Darnley
that it would eliminate those almost pointless files. -- James Darnley (7): avutil: add shared assembly constants avcodec: replace old C file with new assembly constants avcodec: use new constants in C inline assembly avcodec: use new constants in assembly avfilter: use new constants in assembly

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread James Darnley
On 2015-09-27 17:43, Stefano Sabatini wrote: > On date Sunday 2015-09-27 15:28:50 +0200, James Darnley encoded: >> On 2015-09-27 14:37, Stefano Sabatini wrote: >>> + >>> +TNS still needs some polishing, but has the potential to reduce coding >>> +

Re: [FFmpeg-devel] [PATCH] ffmpeg/web/index: add news describing GSoC 2015 program outcome

2015-09-27 Thread James Darnley
On 2015-09-27 23:06, Ganesh Ajjanagadde wrote: > On Sun, Sep 27, 2015 at 5:00 PM, James Darnley <james.darn...@gmail.com> > wrote: >> On 2015-09-27 17:43, Stefano Sabatini wrote: >>> On date Sunday 2015-09-27 15:28:50 +0200, James Darnley encoded: >>>> On 2

Re: [FFmpeg-devel] [FFmpeg-cvslog] configure: Reenable colorized warnings and check for tput's existence

2015-09-20 Thread James Darnley
On 2015-09-08 05:42, Timothy Gu wrote: > ffmpeg | branch: master | Timothy Gu | Mon Sep 7 > 19:11:16 2015 -0700| [617d53f4c7e43b5df6f99b363b550ff7b0007c6e] | committer: > Timothy Gu > > configure: Reenable colorized warnings and check for tput's existence > > Untested.

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-23 Thread James Darnley
On 2015-09-22 20:22, Nicolas George wrote: > Le primidi 1er vendémiaire, an CCXXIV, James Darnley a écrit : >> +@item -metadata_iconv_code_page_list @var{code_page_list} >> +Force the metadata from input files to be converted through the given >> codepages >> +using

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-24 Thread James Darnley
On 2015-09-24 12:36, Nicolas George wrote: > Le duodi 2 vendémiaire, an CCXXIV, James Darnley a écrit : >> It is not supposed to replace any invalid bytes with a "random" >> character. That sounds like it will only make the problem worse with >> that lossy removal

[FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-22 Thread James Darnley
At present it only converts global metadata as that is what I wanted to do. It should be possible to extend it so that the conversion can be different for different files or streams. --- doc/ffmpeg.texi | 6 +++ ffmpeg.c| 15 ++ ffmpeg.h| 1 + ffmpeg_opt.c| 149

Re: [FFmpeg-devel] [RFC][PATCH] ffmpeg: add option to transform metadata using iconv

2015-09-25 Thread James Darnley
On 2015-09-24 20:37, Nicolas George wrote: > Le tridi 3 vendémiaire, an CCXXIV, James Darnley a écrit : >> I don't know what to say here. I know the encodings needed for iconv >> because I arrived at them by brute force. I wrote a short Lua script to >> iterate over a list o

Re: [FFmpeg-devel] lavu: add text encoding conversion API

2015-09-23 Thread James Darnley
On 2015-09-23 19:28, Nicolas George wrote: > Hi. > > This is a proposal for an API in libavutil to perform text encoding > conversions. I had it in mind to clean up the subtitles code initially, and > now it could be useful for James' proposal about metadata. The issues are > relying on the

  1   2   3   4   5   >