Re: [FFmpeg-devel] [PATCH] ffplay: log possible error of SDL_EnableKeyRepeat

2015-10-09 Thread wm4
On Thu, 8 Oct 2015 18:46:44 -0400 Ganesh Ajjanagadde wrote: > Note that for the current SDL source code, 0 is always returned. > Nevertheless, this makes the code more robust. > > Signed-off-by: Ganesh Ajjanagadde > --- > ffplay.c | 3 ++- > 1

Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration

2015-10-09 Thread Timo Rothenpieler
Using this sample: https://btbn.de/files/chromakey_sample.mp4 ffmpeg -f lavfi -i color=c=black:s=1280x720 -i chromakey_sample.mp4 -an -c:v libx264 -preset veryfast -crf 18 -shortest -filter_complex "[1:v]chromakey=0x70de77:0.1:0.2:0:1[ckout];[0:v][ckout]overlay[out]" -map "[out]" -y output.mkv

Re: [FFmpeg-devel] [PATCH] build system: fix dependency generation

2015-10-09 Thread Christophe Gisquet
Hi, 2015-10-09 3:34 GMT+02:00 Michael Niedermayer : > should i apply this ? > i cant really say if this is correct or not > though whie testing i noticed that > rm ffmpeg ffprobe ; ./configure --progs-suffix=abc && make -j12 fate > ends with > make: *** No rule to make

[FFmpeg-devel] [PATCH] fix b frame n_quant_offset

2015-10-09 Thread Agatha Hu
--- libavcodec/nvenc.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavcodec/nvenc.c b/libavcodec/nvenc.c index 18bcd96..0e6ef43 100644 --- a/libavcodec/nvenc.c +++ b/libavcodec/nvenc.c @@ -742,8 +742,6 @@ static av_cold int nvenc_encode_init(AVCodecContext

Re: [FFmpeg-devel] [PATCH] build system: fix dependency generation

2015-10-09 Thread Michael Niedermayer
On Fri, Oct 09, 2015 at 10:58:50AM +0200, Christophe Gisquet wrote: > Hi, > > 2015-10-09 3:34 GMT+02:00 Michael Niedermayer : > > should i apply this ? > > i cant really say if this is correct or not > > though whie testing i noticed that > > rm ffmpeg ffprobe ;

Re: [FFmpeg-devel] [PATCH] build system: fix dependency generation

2015-10-09 Thread Christophe Gisquet
2015-10-09 11:49 GMT+02:00 Michael Niedermayer : >> Does that actually work even without the patch? I have the same issue > > no, it doesnt work before either, its unrelated Well I thought it was relevant to the patch and it was causing that regression. So as far as I

Re: [FFmpeg-devel] [PATCH] build: restore videotoolbox compilation on iOS

2015-10-09 Thread Clément Bœsch
On Fri, Oct 02, 2015 at 12:36:54PM +0200, Clément Bœsch wrote: > On Fri, Oct 02, 2015 at 12:20:10PM +0200, wm4 wrote: > > On Fri, 2 Oct 2015 11:23:59 +0200 > > Clément Bœsch wrote: > > > > > On Fri, Oct 02, 2015 at 11:11:37AM +0200, wm4 wrote: > > > [...] > > > > > +#if

Re: [FFmpeg-devel] [PATCH] avcodec/xvmc: apply attribute_deprecated correctly

2015-10-09 Thread Ganesh Ajjanagadde
On Fri, Oct 9, 2015 at 7:34 PM, Ivan Kalvachev wrote: > On 10/10/15, Ganesh Ajjanagadde wrote: >> This fixes a warning observed on Clang 3.7: >> "warning: attribute 'deprecated' is ignored, place it after "struct" to >> apply attribute to type

Re: [FFmpeg-devel] [PATCH] fate/subtitles: Add a new test for WebVTT

2015-10-09 Thread Michael Niedermayer
On Fri, Oct 09, 2015 at 07:21:04PM +0100, Ricardo wrote: > Attached is the sample mentioned. uploaded (note, until its used te first time the sample can still be changed and reuploaded if needed) [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I do not agree

Re: [FFmpeg-devel] [PATCH] fate/subtitles: Add a new test for WebVTT

2015-10-09 Thread Ricardo Constantino
I had git misconfigured to convert CRLF to LF, so I'll probably reupload the patches (if it makes a difference?), but the sample should be OK as is. On 10 October 2015 at 01:26, Michael Niedermayer wrote: > On Fri, Oct 09, 2015 at 07:21:04PM +0100, Ricardo wrote: > >

[FFmpeg-devel] [PATCH 1/3] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-09 Thread Ricardo Constantino
Bare ampersand characters are still accepted, even though out-of-spec. Also fixes adjacent tags not being parsed. Fixes trac #4915 Signed-off-by: Ricardo Constantino --- libavcodec/webvttdec.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git

[FFmpeg-devel] [PATCH 2/3] avformat/webvttdec: Don't stop parsing on comments

2015-10-09 Thread Ricardo Constantino
Signed-off-by: Ricardo Constantino --- libavformat/webvttdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/webvttdec.c b/libavformat/webvttdec.c index 43c2a63..47a3255 100644 --- a/libavformat/webvttdec.c +++ b/libavformat/webvttdec.c @@

Re: [FFmpeg-devel] [PATCH] configure: fix build error with freetype2 during preprocessing

2015-10-09 Thread Ganesh Ajjanagadde
On Sat, Sep 5, 2015 at 11:24 PM, Ganesh Ajjanagadde wrote: > On Sat, Sep 5, 2015 at 8:21 PM, Ganesh Ajjanagadde wrote: >> On Sat, Sep 5, 2015 at 8:08 AM, proud zhu wrote: >>> this patch fix the below error when build with

Re: [FFmpeg-devel] [PATCH] avcodec/xvmc: apply attribute_deprecated correctly

2015-10-09 Thread Ivan Kalvachev
On 10/10/15, Ganesh Ajjanagadde wrote: > This fixes a warning observed on Clang 3.7: > "warning: attribute 'deprecated' is ignored, place it after "struct" to > apply attribute to type declaration [-Wignored-attributes]" > and thus enables deprecation warning for the

Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration

2015-10-09 Thread highgod0401
From: Timo Rothenpieler Date: 2015-10-09 16:41 To: FFmpeg development discussions and patches CC: highgod0401 Subject: Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration > > Using this sample: https://btbn.de/files/chromakey_sample.mp4 > > ffmpeg -f lavfi -i

[FFmpeg-devel] [PATCH] vp9: add itxfm_add eob shortcuts to 10/12bpp functions.

2015-10-09 Thread Ronald S. Bultje
These aren't quite as helpful as the ones in 8bpp, since over there, we can use pmulhrsw, but here the coefficients have too many bits to be able to take advantage of pmulhrsw. However, we can still skip cols for which all coefs are 0, and instead just zero the input data for the row itx. This

Re: [FFmpeg-devel] [PATCH 2/5] lavu/jni: add more JNI helper

2015-10-09 Thread Michael Niedermayer
On Fri, Oct 09, 2015 at 06:26:50PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > libavutil/jni_internal.c | 290 > +++ > libavutil/jni_internal.h | 101 + > 2 files changed, 391

[FFmpeg-devel] [PATCH 3/3] fate/subtitles: Add a new test for WebVTT

2015-10-09 Thread Ricardo Constantino
Includes escapes that should now be supported and a few features not yet fully supported, like comments, regions, classes, ruby, and lang. All were tested with https://quuz.org/webvtt/ for validation, except regions because the validator doesn't support them yet, and I couldn't find any other way

Re: [FFmpeg-devel] [PATCH 1/7] lavc: move bitstream filter args to the bsf ctx

2015-10-09 Thread Michael Niedermayer
On Thu, Oct 08, 2015 at 03:07:35PM -0500, Rodger Combs wrote: > --- > libavcodec/avcodec.h | 1 + > libavcodec/bitstream_filter.c | 5 +++-- > 2 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index ff70d25..ce42e57 100644 >

Re: [FFmpeg-devel] [PATCH] x86/takdsp: use arithmetic shift instructions

2015-10-09 Thread James Almer
On 10/9/2015 11:48 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 9, 2015 at 10:42 PM, James Almer wrote: > >> p1 and p2 are int32_t. >> >> Signed-off-by: James Almer >> --- >> libavcodec/x86/takdsp.asm | 8 >> 1 file changed, 4 insertions(+),

Re: [FFmpeg-devel] [PATCH] doc: fix spelling errors

2015-10-09 Thread Andreas Cadhalpun
On 30.09.2015 03:27, Lou Logan wrote: > On Tue, 29 Sep 2015 20:34:09 +0200, Andreas Cadhalpun wrote: > >> Signed-off-by: Andreas Cadhalpun >> --- >> doc/encoders.texi | 2 +- >> doc/filters.texi | 14 +++--- >>

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideoenc: fix undefined negative left shift

2015-10-09 Thread Michael Niedermayer
On Fri, Oct 09, 2015 at 02:43:04PM -0400, Ganesh Ajjanagadde wrote: > This should fix the first undefined behavior reported in: > https://trac.ffmpeg.org/ticket/4727. > > I can't reproduce the runtime behavior reported in the ticket, hence I > can't confirm that this actually fixes the exact

[FFmpeg-devel] [PATCH 6/9] x86: simple_idct: 12bits versions

2015-10-09 Thread Christophe Gisquet
On 12 frames of a 444p 12 bits DNxHR sequence, _put function: C: 78902 decicycles in idct, 262071 runs, 73 skips avx: 32478 decicycles in idct, 262045 runs, 99 skips Difference between the 2: stddev:0.39 PSNR:104.47 MAXDIFF:2 This is unavoidable and due to the

Re: [FFmpeg-devel] [PATCH] avcodec/xvmc: apply attribute_deprecated correctly

2015-10-09 Thread Ronald S. Bultje
Hi, On Fri, Oct 9, 2015 at 6:44 PM, Ganesh Ajjanagadde wrote: > This fixes a warning observed on Clang 3.7: > "warning: attribute 'deprecated' is ignored, place it after "struct" to > apply attribute to type declaration [-Wignored-attributes]" > and thus enables

Re: [FFmpeg-devel] [PATCH] avfilter/drawtext: allow to format pts with strftime

2015-10-09 Thread Nicolas George
Le septidi 17 vendémiaire, an CCXXIV, Alex Agranovsky a écrit : > From a51f49007381701df18309c73083749413df3cb8 Mon Sep 17 00:00:00 2001 > From: Alex Agranovsky > Date: Thu, 8 Oct 2015 15:54:59 -0400 > Subject: [PATCH] avfilter/drawtext: allow to format pts with strftime > >

[FFmpeg-devel] Trac Outage - Notification of planned works

2015-10-09 Thread Kieran Kunhya
Hello, Please note there will be a trac outage owing to the rerouting of fibre as part of the London Underground extension. Regards, Kieran Kunhya -- Forwarded message - Hello, There is going to be a loss of service at some point during the time frame stated below whilst

Re: [FFmpeg-devel] [PATCH] build system: fix dependency generation

2015-10-09 Thread Christophe Gisquet
Hi, 2015-10-09 3:34 GMT+02:00 Michael Niedermayer : > should i apply this ? > i cant really say if this is correct or not > though whie testing i noticed that > rm ffmpeg ffprobe ; ./configure --progs-suffix=abc && make -j12 fate > ends with > make: *** No rule to make

Re: [FFmpeg-devel] [PATCH] configure: disabling --enable-libmfx and --enable-gpl combination

2015-10-09 Thread Ivan Uskov
Hello Hendrik, Thursday, October 8, 2015, 11:43:37 PM, you wrote: HL> We're not talking about dynamic linking here though, but runtime loading. HL> Can I not use VAAPI because the underlying driver behind it may be closed HL> source? I do not know I'm not expert. But most likely VAAPI

Re: [FFmpeg-devel] [PATCH] build system: fix dependency generation

2015-10-09 Thread Christophe Gisquet
Hi, 2015-10-09 7:59 GMT+02:00 Christophe Gisquet : > But then, why is the stem truncated from dct-test to dct? I have no > idea either. Another rule somewhere? Updating goal targets Considering target file 'libavcodec/dct-test.o'. File 'libavcodec/dct-test.o'

[FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/vf_w3fdif.c | 127 +- libavfilter/w3fdif.h | 47 +++ libavfilter/x86/Makefile | 2 + libavfilter/x86/vf_w3fdif.asm| 270 +++

[FFmpeg-devel] [PATCH 2/9] x86: prores: templatize 10 bits simple_idct

2015-10-09 Thread Christophe Gisquet
This should be reused for a generic simple_idct10 function. Requires a bit of trickery to declare common constants in C. --- libavcodec/x86/constants.c| 28 +++ libavcodec/x86/constants.h| 16 ++ libavcodec/x86/proresdsp.asm | 263

[FFmpeg-devel] [PATCH 4/9] x86: proresdsp: simple_idct: free or use 1 xmm reg

2015-10-09 Thread Christophe Gisquet
m15 is zeroed but never used. If it's not needed, decrease by 1 the number of xmm regs used (prores), otherwise, make use of it, for the rounder in the row pass of simple_idct. --- libavcodec/x86/proresdsp.asm | 8 libavcodec/x86/simple_idct10.asm | 9 +

[FFmpeg-devel] [PATCH] vp9: add 10/12bpp idct_idct_32x32 sse2 SIMD version.

2015-10-09 Thread Ronald S. Bultje
--- libavcodec/x86/vp9dsp_init_16bpp_template.c | 2 + libavcodec/x86/vp9itxfm_16bpp.asm | 531 +--- 2 files changed, 489 insertions(+), 44 deletions(-) diff --git a/libavcodec/x86/vp9dsp_init_16bpp_template.c b/libavcodec/x86/vp9dsp_init_16bpp_template.c

[FFmpeg-devel] [PATCH 1/9] dnxhdenc: fix scan used for bitstream generation

2015-10-09 Thread Christophe Gisquet
The functions related to bitstream reading must use the natural zigzag order, and not the one permuted for use in the iDCT. This currently results in bitstreams with 2 issues: - AC coefficients are encoded in an unexpected order; - Incorrect weights are applied in the (de)quantization. This

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

2015-10-09 Thread Ganesh Ajjanagadde
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 project is being built. Signed-off-by: Ganesh

Re: [FFmpeg-devel] [PATCH] configure: Add FFMPEG_CONFIGURATION and FFMPEG_LICENSE defines to avconfig.h

2015-10-09 Thread Hendrik Leppkes
On Fri, Oct 9, 2015 at 9:58 PM, Nicolas George wrote: > L'octidi 18 vendémiaire, an CCXXIV, Gonzalo a écrit : >> This was sent in another mail, but may be lost since it did not specify a >> PATCH. >> >> The patch is to get the license and compile flags from any application, I >>

[FFmpeg-devel] [PATCH 9/9] x86: dct-test: add 12 bits versions

2015-10-09 Thread Christophe Gisquet
--- libavcodec/dct-test.c | 15 +++ libavcodec/x86/dct-test.c | 13 - 2 files changed, 27 insertions(+), 1 deletion(-) diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c index 9a195ab..1c17dd3 100644 --- a/libavcodec/dct-test.c +++ b/libavcodec/dct-test.c @@

[FFmpeg-devel] [PATCH] avcodec/xvmc: apply attribute_deprecated correctly

2015-10-09 Thread Ganesh Ajjanagadde
This fixes a warning observed on Clang 3.7: "warning: attribute 'deprecated' is ignored, place it after "struct" to apply attribute to type declaration [-Wignored-attributes]" and thus enables deprecation warning for the relevant struct. Signed-off-by: Ganesh Ajjanagadde

Re: [FFmpeg-devel] [PATCH 0/9] Initial support for DNxHR, v2

2015-10-09 Thread Michael Niedermayer
On Thu, Oct 08, 2015 at 07:52:40AM +0200, Christophe Gisquet wrote: > Hi, > > 2015-10-08 1:38 GMT+02:00 Michael Niedermayer : > >> So I have made a fate test on 1 frame for each sequence (total around > >> 1.8MB). > >> > >> Is it ok? > > > > ok > > Here you go. Hendrik

Re: [FFmpeg-devel] [PATCH] doc: fix spelling errors

2015-10-09 Thread Ganesh Ajjanagadde
On Fri, Oct 9, 2015 at 4:13 PM, Andreas Cadhalpun wrote: > On 30.09.2015 03:27, Lou Logan wrote: >> On Tue, 29 Sep 2015 20:34:09 +0200, Andreas Cadhalpun wrote: >> >>> Signed-off-by: Andreas Cadhalpun >>> --- >>>

[FFmpeg-devel] [PATCH 0/9] v2: simple_idct for 10/12 bits and DNxHD

2015-10-09 Thread Christophe Gisquet
Summary of changes since last version: - implemented pure idcts, needed for vf_spp - consequently, modified dct-test as the wrapper is no longer needed - fixed overflow in 12bits version by disabling a trick - freed a xmm reg for prores, and used it for the rounder in simple_idct Christophe

[FFmpeg-devel] [PATCH 5/9] x86: simple_idct: add pure idct functions

2015-10-09 Thread Christophe Gisquet
This is done by not passing clamping regs to the macro, and there, deducing that this is a pure idct. There are not many users (vf_spp is one), and the _add version doesn't look like it will ever be used. --- libavcodec/x86/idctdsp_init.c | 2 + libavcodec/x86/proresdsp.asm

[FFmpeg-devel] [PATCH 7/9] dct-test: add infrastructure for 10 bits

2015-10-09 Thread Christophe Gisquet
--- libavcodec/arm/dct-test.c | 10 -- libavcodec/dct-test.c | 41 +++-- libavcodec/ppc/dct-test.c | 10 -- libavcodec/x86/dct-test.c | 9 +++-- 4 files changed, 58 insertions(+), 12 deletions(-) diff --git a/libavcodec/arm/dct-test.c

[FFmpeg-devel] [PATCH 8/9] x86: dct-test: add 10 bits versions.

2015-10-09 Thread Christophe Gisquet
--- libavcodec/x86/dct-test.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/libavcodec/x86/dct-test.c b/libavcodec/x86/dct-test.c index 692c23b..22a3ecf 100644 --- a/libavcodec/x86/dct-test.c +++ b/libavcodec/x86/dct-test.c @@ -89,7 +89,18 @@ static const

Re: [FFmpeg-devel] [PATCH] configure: Add FFMPEG_CONFIGURATION and FFMPEG_LICENSE defines to avconfig.h

2015-10-09 Thread Gonzalo
El 09/10/15 a las 16:58, Nicolas George escribió: This information is already available through avcodec_configuration() and avcodec_license(). Do you really need a static version? With a shared library, it may not match. Thank you very much. That was exactly the functions I was looking for.

Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration

2015-10-09 Thread highgod0401
From: Timo Rothenpieler Date: 2015-10-09 16:41 To: FFmpeg development discussions and patches CC: highgod0401 Subject: Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration > > Using this sample: https://btbn.de/files/chromakey_sample.mp4 > > ffmpeg -f lavfi -i

Re: [FFmpeg-devel] [PATCH 3/5] lavu/jni: add helpers to manage android application contexts

2015-10-09 Thread Zhang Rui
2015-10-10 0:26 GMT+08:00 Matthieu Bouron : > From: Matthieu Bouron > > --- > libavutil/jni.c | 104 > +++ > libavutil/jni.h | 28 + > libavutil/jni_internal.c |

Re: [FFmpeg-devel] [RFC] Poll for the community

2015-10-09 Thread Ganesh Ajjanagadde
On Fri, Oct 9, 2015 at 9:51 AM, Clément Bœsch wrote: > On Fri, Sep 25, 2015 at 10:41:44AM +0200, Clément Bœsch wrote: >> Hi, >> >> I'd like to suggest a poll for the community in order to define some more >> clear directions for the project. I plan to post the poll on the mailing >>

[FFmpeg-devel] [PATCH] ffserver_config: check for INT_MIN before doing FFABS

2015-10-09 Thread Ganesh Ajjanagadde
FFABS(INT_MIN) is not safe. Alternative of using FFNABS is not as readable. Signed-off-by: Ganesh Ajjanagadde --- ffserver_config.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffserver_config.c b/ffserver_config.c index 691ff7b..9fc1f00 100644 ---

[FFmpeg-devel] [PATCH] h264_mp4toannexb

2015-10-09 Thread 赵宇龙
From 9654bf635c72fd3bd4208afcf4a1b8b8667735db Mon Sep 17 00:00:00 2001 From: zylthinking Date: Fri, 9 Oct 2015 13:46:45 +0800 Subject: [PATCH] fix pps offfset fault when there are more than one sps in avcc the pps offset is used to locate pps in the spspps_buf; however,

[FFmpeg-devel] [PATCH 2/5] lavu/jni: add more JNI helper

2015-10-09 Thread Matthieu Bouron
From: Matthieu Bouron --- libavutil/jni_internal.c | 290 +++ libavutil/jni_internal.h | 101 + 2 files changed, 391 insertions(+) diff --git a/libavutil/jni_internal.c b/libavutil/jni_internal.c index

[FFmpeg-devel] [PATCH 3/5] lavu/jni: add helpers to manage android application contexts

2015-10-09 Thread Matthieu Bouron
From: Matthieu Bouron --- libavutil/jni.c | 104 +++ libavutil/jni.h | 28 + libavutil/jni_internal.c | 30 ++ libavutil/jni_internal.h | 11 + 4 files changed, 173

[FFmpeg-devel] [PATCH 1/5] lavu: add JNI support

2015-10-09 Thread Matthieu Bouron
From: Matthieu Bouron --- configure| 4 +++ libavutil/Makefile | 4 +++ libavutil/jni.c | 37 ++ libavutil/jni.h | 42 + libavutil/jni_internal.c | 69

Re: [FFmpeg-devel] [PATCH] winrt: multithreading support

2015-10-09 Thread Wang Bin
I will update the patch next week ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avcodec/webvttdec: Deal with WebVTT escapes

2015-10-09 Thread Ricardo Constantino
Bare ampersand characters are still accepted, even though out-of-spec. Also fixes adjacent tags not being parsed. Fixes trac #4915 Signed-off-by: Ricardo Constantino --- libavcodec/webvttdec.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git

Re: [FFmpeg-devel] [RFC] Poll for the community

2015-10-09 Thread Clément Bœsch
On Fri, Oct 09, 2015 at 10:45:17AM -0400, Ganesh Ajjanagadde wrote: > On Fri, Oct 9, 2015 at 9:51 AM, Clément Bœsch wrote: > > On Fri, Sep 25, 2015 at 10:41:44AM +0200, Clément Bœsch wrote: > >> Hi, > >> > >> I'd like to suggest a poll for the community in order to define some more >

[FFmpeg-devel] [PATCH 5/5] lavf: add Android content resolver protocol support

2015-10-09 Thread Matthieu Bouron
From: Matthieu Bouron Handles uri starting with content://. --- configure| 5 ++ libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/file.c | 155 +++ libavutil/jni.c

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread Ronald S. Bultje
Hi, looking better already! On Fri, Oct 9, 2015 at 3:58 AM, Paul B Mahol wrote: > +INIT_XMM sse4 > +cglobal w3fdif_scale, 3, 3, 3, 0, out_pixel, work_pixel, linesize > +pxor m1, m1 > +mova m2, [pd_2_23] > +shr

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Unescape HTML entities

2015-10-09 Thread Clément Bœsch
On Thu, Oct 08, 2015 at 10:50:49PM +0100, Ricardo wrote: > That would probably be considered a broken WebVTT file, since "&" need to > be encoded as "". > What about "Clment" or any unsupported escape? [...] -- Clément B. signature.asc Description: PGP signature

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

2015-10-09 Thread Nicolas George
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 module. If your package manager doesn't have that see here: >

Re: [FFmpeg-devel] [PATCH] avformat/srtdec: give WebVTT priority

2015-10-09 Thread Clément Bœsch
On Thu, Oct 08, 2015 at 11:51:32PM +0200, wm4 wrote: > On Thu, 8 Oct 2015 19:40:38 +0200 > wm4 wrote: > > > WevVTT and SRT can look very similar. For some samples, they both > > succeed and return AVPROBE_SCORE_MAX, which leads to probing failure. > > > > This gives

Re: [FFmpeg-devel] [RFC] Poll for the community

2015-10-09 Thread Clément Bœsch
On Fri, Sep 25, 2015 at 10:41:44AM +0200, Clément Bœsch wrote: > Hi, > > I'd like to suggest a poll for the community in order to define some more > clear directions for the project. I plan to post the poll on the mailing > lists and IRC, people are free to forward them elsewhere. > > If anyone

Re: [FFmpeg-devel] [PATCH] avcodec/pthread_frame: update doxygen for update_context_from_thread

2015-10-09 Thread Ronald S. Bultje
Hi, On Fri, Oct 9, 2015 at 9:27 AM, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavcodec/pthread_frame.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c

Re: [FFmpeg-devel] [RFC] Poll for the community

2015-10-09 Thread Clément Bœsch
On Fri, Sep 25, 2015 at 08:39:16AM -0400, Ganesh Ajjanagadde wrote: > On Fri, Sep 25, 2015 at 4:41 AM, Clément Bœsch wrote: > > Hi, > > > > I'd like to suggest a poll for the community in order to define some more > > clear directions for the project. I plan to post the poll on the

Re: [FFmpeg-devel] [PATCH] build system: fix dependency generation

2015-10-09 Thread Michael Niedermayer
On Fri, Oct 09, 2015 at 12:13:05PM +0200, Christophe Gisquet wrote: > 2015-10-09 11:49 GMT+02:00 Michael Niedermayer : > >> Does that actually work even without the patch? I have the same issue > > > > no, it doesnt work before either, its unrelated > > Well I thought it

Re: [FFmpeg-devel] [PATCH] avcodec/webvttdec: Unescape HTML entities

2015-10-09 Thread Ricardo
I think those were needed when UTF-8 wasn't expected, but WebVTT makes it mandatory, according to https://w3c.github.io/webvtt/#file-structure Are you saying that and should be removed (like it was)? On 9 October 2015 at 13:50, Clément Bœsch wrote: > On Thu, Oct 08, 2015 at

[FFmpeg-devel] Tools for MIPS MSA (MIPS SIMD-Arch)

2015-10-09 Thread Shivraj Patil
Hi all, Please find below the toolchain and QEMU installables for the build and test of MIPS p5600 and i6400 cpu targets, MIPS Toolchain: * Download the toolchain for MIPS 32Bit (p5600) from following link (For 64Bit linux host)

[FFmpeg-devel] [PATCH 4/5] lavu/jni: let avpriv_init_jfields load application classes

2015-10-09 Thread Matthieu Bouron
From: Matthieu Bouron --- libavutil/jni_internal.c | 25 + libavutil/jni_internal.h | 1 + 2 files changed, 22 insertions(+), 4 deletions(-) diff --git a/libavutil/jni_internal.c b/libavutil/jni_internal.c index 58eb570..78d13a4 100644

Re: [FFmpeg-devel] [PATCH 1/5] lavu: add JNI support

2015-10-09 Thread Jean-Baptiste Kempf
On 09 Oct, Matthieu Bouron wrote : > +HEADERS-$(CONFIG_JNI) += jni.h You are going to install a jni.h header? With my kindest regards, -- Jean-Baptiste Kempf http://www.jbkempf.com/ - +33 672 704 734 Sent from my Electronic Device

Re: [FFmpeg-devel] Install config.h?

2015-10-09 Thread Ronald S. Bultje
Hi, On Fri, Oct 9, 2015 at 12:49 PM, Gonzalo wrote: > Hi, > > I would like my program to print out its ffmpeg compile options like > ffplay or ffmpeg do. However, that info seems to be part of config.h which > is not installed. I would like to make a request to install it

Re: [FFmpeg-devel] Add JNI and Android content resolver support

2015-10-09 Thread wm4
On Fri, 9 Oct 2015 18:26:48 +0200 Matthieu Bouron wrote: > Hello, > > The following patchset add support for the Java Native Interface in > FFmpeg through different helpers to: > > * manage the JNI environment (attach/detach), > * load classes/methods/fields

Re: [FFmpeg-devel] [PATCH 4/9] x86: simple_idct_put: 12bits versions

2015-10-09 Thread Michael Niedermayer
On Thu, Oct 08, 2015 at 08:22:51AM +0200, Christophe Gisquet wrote: > On 12 frames of a 444p 12 bits DNxHR sequence: > C: 78902 decicycles in idct, 262071 runs, 73 skips > avx: 32478 decicycles in idct, 262045 runs, 99 skips > > Difference between the 2: > stddev:0.39

[FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/vf_w3fdif.c | 132 ++-- libavfilter/w3fdif.h | 47 libavfilter/x86/Makefile | 2 + libavfilter/x86/vf_w3fdif.asm| 252 +++

[FFmpeg-devel] Install config.h?

2015-10-09 Thread Gonzalo
Hi, I would like my program to print out its ffmpeg compile options like ffplay or ffmpeg do. However, that info seems to be part of config.h which is not installed. I would like to make a request to install it unless there's other way to get that info. Thank you very much. Gonzalo

Re: [FFmpeg-devel] [PATCH 5/5] lavf: add Android content resolver protocol support

2015-10-09 Thread Jean-Baptiste Kempf
On 09 Oct, Matthieu Bouron wrote : > Handles uri starting with content://. I don't see what this has to do with libavformat. It's OS specific, not a file format, trivial to do within any calling application and probably going to break easily... With my kindest regards, -- Jean-Baptiste Kempf

Re: [FFmpeg-devel] Add JNI and Android content resolver support

2015-10-09 Thread Matthieu Bouron
On Fri, Oct 9, 2015 at 6:55 PM, Jean-Baptiste Kempf wrote: > On 09 Oct, Matthieu Bouron wrote : > > The following patchset add support for the Java Native Interface in > > FFmpeg through different helpers to: > > > > * manage the JNI environment (attach/detach), > > * load

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread Ronald S. Bultje
Hi, On Fri, Oct 9, 2015 at 12:44 PM, Paul B Mahol wrote: > +packssdw m0, m0 > +packuswb m0, m0 > +packssdw m1, m1 > +packuswb m1, m1 > +SBUTTERFLY dq, 0, 1, 2

Re: [FFmpeg-devel] [PATCH] ffplay: log possible error of SDL_EnableKeyRepeat

2015-10-09 Thread Ganesh Ajjanagadde
On Fri, Oct 9, 2015 at 4:21 AM, wm4 wrote: > On Thu, 8 Oct 2015 18:46:44 -0400 > Ganesh Ajjanagadde wrote: > >> Note that for the current SDL source code, 0 is always returned. >> Nevertheless, this makes the code more robust. >> >> Signed-off-by:

Re: [FFmpeg-devel] [PATCH] avfilter/drawtext: allow to format pts with strftime

2015-10-09 Thread Alex Agranovsky
Thanks for your comments. I’m attaching the amended patch, hopefully it addresses all of them. Please let me know if something else is out of order. - Alex On October 9, 2015 at 2:23:42 AM, Nicolas George (geo...@nsup.org) wrote: Le septidi 17 vendémiaire, an CCXXIV, Alex Agranovsky a écrit 

[FFmpeg-devel] [PATCH] avcodec/pthread_frame: update doxygen for update_context_from_thread

2015-10-09 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavcodec/pthread_frame.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c index c89b4ca..7651211 100644 --- a/libavcodec/pthread_frame.c +++ b/libavcodec/pthread_frame.c @@

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread Michael Niedermayer
On Fri, Oct 09, 2015 at 07:58:45PM +0200, Michael Niedermayer wrote: > On Fri, Oct 09, 2015 at 06:44:03PM +0200, Paul B Mahol wrote: > > Signed-off-by: Paul B Mahol > > --- > > libavfilter/vf_w3fdif.c | 132 ++-- > > libavfilter/w3fdif.h |

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread James Almer
On 10/9/2015 1:44 PM, Paul B Mahol wrote: > +cglobal w3fdif_complex_low, 4, 7, 9, 0, work_line, in_lines_cur0, coef, > linesize > +movq m3, [coefq] > +DEFINE_ARGSwork_line, in_lines_cur0, in_lines_cur1, linesize, > offset, in_lines_cur2, in_lines_cur3 > +SPLATW

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread Michael Niedermayer
On Fri, Oct 09, 2015 at 06:44:03PM +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/vf_w3fdif.c | 132 ++-- > libavfilter/w3fdif.h | 47 > libavfilter/x86/Makefile | 2 + >

Re: [FFmpeg-devel] [PATCH 3/9] x86: simple_idct_put: 10bits versions

2015-10-09 Thread Michael Niedermayer
On Thu, Oct 08, 2015 at 08:22:50AM +0200, Christophe Gisquet wrote: > Modeled from the prores version. Clips to [0;1023] and is bitexact. > Bitexactness requires to add an offset in a different place compared > to prores or C, and makes the function approximately 2% slower. > > For 16 frames of a

[FFmpeg-devel] [PATCH] avcodec/apedec: use int64_t for FFABS

2015-10-09 Thread Ganesh Ajjanagadde
res, absres are currently int's, which on most platforms is 32 bits. Unfortunately, data is untrusted, and on line 1267 res is manipulated with data. Thus, res can take on INT32_MIN/INT_MIN with crafted data, making FFABS on line 1282 unsafe. Once again, using FFNABS will make it less readable:

Re: [FFmpeg-devel] Install config.h?

2015-10-09 Thread Gonzalo
El 09/10/15 a las 14:18, Ronald S. Bultje escribió: avconfig.h is the installed counterpart. If you want something from config.h available in applications, move it to avconfig.h. config.h will never be installed. Thanks. Find attached a patch for configure to have avconfig.h list the

[FFmpeg-devel] [PATCH] fate/subtitles: Add a new test for WebVTT

2015-10-09 Thread Ricardo Constantino
Includes escapes that should now be supported and a few features not yet supported, like comments, regions, classes, ruby, lang. All were tested with https://quuz.org/webvtt/ for validation, except regions because the validator doesn't support them yet. Signed-off-by: Ricardo Constantino

Re: [FFmpeg-devel] [PATCH] fate/subtitles: Add a new test for WebVTT

2015-10-09 Thread Ricardo
Attached is the sample mentioned. WebVTT_extended_tester.vtt Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avcodec/mpegvideoenc: fix undefined negative left shift

2015-10-09 Thread Ganesh Ajjanagadde
This should fix the first undefined behavior reported in: https://trac.ffmpeg.org/ticket/4727. I can't reproduce the runtime behavior reported in the ticket, hence I can't confirm that this actually fixes the exact issue reported in the ticket. Regardless, I can confirm that this is a genuine

[FFmpeg-devel] [PATCH] avformat/webvttdec: Don't stop parsing on comments

2015-10-09 Thread Ricardo Constantino
Signed-off-by: Ricardo Constantino --- libavformat/webvttdec.c| 3 ++- tests/ref/fate/sub-webvtt2 | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/webvttdec.c b/libavformat/webvttdec.c index 43c2a63..47a3255 100644 ---

[FFmpeg-devel] [PATCH] configure: Add FFMPEG_CONFIGURATION and FFMPEG_LICENSE defines to avconfig.h

2015-10-09 Thread Gonzalo
This was sent in another mail, but may be lost since it did not specify a PATCH. The patch is to get the license and compile flags from any application, I add them to the distribution of avconfig.h. Thanks. diff --git a/configure b/configure index b572d59..4196088 100755 --- a/configure +++

[FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/vf_w3fdif.c | 132 +++--- libavfilter/w3fdif.h | 47 libavfilter/x86/Makefile | 2 + libavfilter/x86/vf_w3fdif.asm| 234 +++

Re: [FFmpeg-devel] [PATCH] ffmpeg: avoid possible undefined behavior

2015-10-09 Thread Michael Niedermayer
On Fri, Oct 09, 2015 at 10:39:27AM -0400, Ganesh Ajjanagadde wrote: > On lines 1633,1634 FFABS(pts) is performed. However, if av_stream_get_end_pts > returns AV_NOPTS_VALUE always, pts remains stuck at INT64_MIN, leading > to undefined behavior on FFABS. > > One could conceive of a solution using

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread Ronald S. Bultje
Hi, On Fri, Oct 9, 2015 at 3:19 PM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/vf_w3fdif.c | 132 +++--- > libavfilter/w3fdif.h | 47 > libavfilter/x86/Makefile | 2 + >

Re: [FFmpeg-devel] [PATCH 5/5] lavf: add Android content resolver protocol support

2015-10-09 Thread Clément Bœsch
On Fri, Oct 09, 2015 at 06:54:31PM +0200, Jean-Baptiste Kempf wrote: > On 09 Oct, Matthieu Bouron wrote : > > Handles uri starting with content://. > > I don't see what this has to do with libavformat. > > It's OS specific, not a file format, trivial to do within any calling > application and

Re: [FFmpeg-devel] [PATCH] avfilter/vf_w3fdif: add x86 SIMD

2015-10-09 Thread James Almer
On 10/9/2015 4:19 PM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/vf_w3fdif.c | 132 +++--- > libavfilter/w3fdif.h | 47 > libavfilter/x86/Makefile | 2 + > libavfilter/x86/vf_w3fdif.asm| 234

Re: [FFmpeg-devel] Install config.h?

2015-10-09 Thread Nicolas George
L'octidi 18 vendémiaire, an CCXXIV, Ronald S. Bultje a écrit : > avconfig.h is the installed counterpart. If you want something from > config.h available in applications, move it to avconfig.h. Before that, look at avcodec_configuration() and cousins. ffmpeg.c and the other command line programs

Re: [FFmpeg-devel] [PATCH] configure: Add FFMPEG_CONFIGURATION and FFMPEG_LICENSE defines to avconfig.h

2015-10-09 Thread Nicolas George
L'octidi 18 vendémiaire, an CCXXIV, Gonzalo a écrit : > This was sent in another mail, but may be lost since it did not specify a > PATCH. > > The patch is to get the license and compile flags from any application, I > add them to the distribution of avconfig.h. > diff --git a/configure

[FFmpeg-devel] [PATCH] avcodec/ac3enc: fix undefined negative left shift

2015-10-09 Thread Ganesh Ajjanagadde
This should fix the undefined behavior reported in: https://trac.ffmpeg.org/ticket/4727. I can reproduce this at runtime: simply stick in an abort call in asym_quant to check if c < 0 and run FATE. I don't know ac3 so I can't confirm if negative coefficients are intentional, but at the moment

Re: [FFmpeg-devel] Add JNI and Android content resolver support

2015-10-09 Thread compn
On Fri, 9 Oct 2015 19:16:01 +0200 wm4 wrote: > On Fri, 9 Oct 2015 18:26:48 +0200 > Matthieu Bouron wrote: > > > Hello, > > > > The following patchset add support for the Java Native Interface in > > FFmpeg through different helpers to: > > >

  1   2   >