Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-27 Thread Timo Rothenpieler
I haven't looked in detail, but this doesn't seem like a good idea to me. For example it can't handle having different encoders using different cards/hardware. I don't know what other options there might be, maybe this is the best, but it seems not particularly good. The only other option

Re: [FFmpeg-devel] [Patch 2/5]Fix h264 on POWER LE: libavcodec/ppc/h264dsp.c

2014-11-27 Thread Michael Niedermayer
On Thu, Nov 27, 2014 at 02:35:24PM +0800, rongyan wrote: Hi, We present 5 patches to fix h264 bugs for POWER8 little endian, which are sent in 5 seperate emails. This is the second, to fix the functions h264_idct8_add_altivec(); h264_idct_dc_add_internal();

[FFmpeg-devel] [PATCH] lavu/imgutils: remove redundant and wrong check in av_image_fill_arrays

2014-11-27 Thread Stefano Sabatini
--- libavutil/imgutils.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/libavutil/imgutils.c b/libavutil/imgutils.c index 00b2031..7f3032b 100644 --- a/libavutil/imgutils.c +++ b/libavutil/imgutils.c @@ -331,9 +331,6 @@ int av_image_fill_arrays(uint8_t *dst_data[4], int dst_linesize[4],

Re: [FFmpeg-devel] [Patch 1/5]Fix h264 on POWER LE: libavutil/ppc/util_altivec.h

2014-11-27 Thread Michael Niedermayer
On Thu, Nov 27, 2014 at 02:32:10PM +0800, rongyan wrote: Hi, We present 5 patches to fix h264 bugs for POWER8 little endian, which are sent in 5 seperate emails. This is the first, to add marcos vcswapi2s(), vcswapc(), VEC_LD2(), VEC_SPLAT16(), and VEC_SLD16(). The fate test result after

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-27 Thread Timo Rothenpieler
Is it necessary to split the _api part in a separate file? The whole code is a bit large, but still manageable, and merging the files would avoid some headers overhead. Done that primarily to keep things cleaned up and easier to read. Can as well put it all in one huge file. I think moving

Re: [FFmpeg-devel] [PATCH] lavu/imgutils: remove redundant and wrong check in av_image_fill_arrays

2014-11-27 Thread Michael Niedermayer
On Thu, Nov 27, 2014 at 12:29:12PM +0100, Stefano Sabatini wrote: --- libavutil/imgutils.c | 3 --- 1 file changed, 3 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically faster algorithms should always be preferred if you

[FFmpeg-devel] [PATCH] msvc: Fix compilation errors due to header include order.

2014-11-27 Thread Matt Oliver
The patch lavf: Use wchar functions for filenames on windows for mkdir/rmdir/rename/unlink that was recently added has broken msvc/icl builds. see: http://fate.ffmpeg.org/report.cgi?time=20141126233318slot=x86_32-msvc12-windows-native This is because winsock2.h must be included before windows.h.

Re: [FFmpeg-devel] [PATCH] lavu/imgutils: remove redundant and wrong check in av_image_fill_arrays

2014-11-27 Thread Michael Niedermayer
On Thu, Nov 27, 2014 at 12:36:42PM +0100, Michael Niedermayer wrote: On Thu, Nov 27, 2014 at 12:29:12PM +0100, Stefano Sabatini wrote: --- libavutil/imgutils.c | 3 --- 1 file changed, 3 deletions(-) LGTM that is assuming its ok if the output arrays change on error, i think the

Re: [FFmpeg-devel] [PATCH] msvc: Fix compilation errors due to header include order.

2014-11-27 Thread Michael Niedermayer
On Thu, Nov 27, 2014 at 10:38:01PM +1100, Matt Oliver wrote: The patch lavf: Use wchar functions for filenames on windows for mkdir/rmdir/rename/unlink that was recently added has broken msvc/icl builds. see:

[FFmpeg-devel] [PATCH] Move extralibs variables using ldl after ldl definition

2014-11-27 Thread timo
From: Timo Rothenpieler t...@rothenpieler.org --- configure | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 38619c4..8ecc7fa 100755 --- a/configure +++ b/configure @@ -2569,16 +2569,13 @@ drawtext_filter_deps=libfreetype

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-27 Thread Jean-Baptiste Kempf
On 26 Nov, Nicolas George wrote : Le sextidi 6 frimaire, an CCXXIII, Hendrik Leppkes a écrit : Thats just wrong. There is not one line of proprietary/non-free code that gets included or linked in libavcodec. (IANAL) I do not think it works that way. You could apply the same reasoning

Re: [FFmpeg-devel] [PATCH] lavu/imgutils: remove redundant and wrong check in av_image_fill_arrays

2014-11-27 Thread Stefano Sabatini
On date Thursday 2014-11-27 12:38:21 +0100, Michael Niedermayer encoded: On Thu, Nov 27, 2014 at 12:36:42PM +0100, Michael Niedermayer wrote: On Thu, Nov 27, 2014 at 12:29:12PM +0100, Stefano Sabatini wrote: --- libavutil/imgutils.c | 3 --- 1 file changed, 3 deletions(-) LGTM

Re: [FFmpeg-devel] lavu/opt: add escaping to av_opt_serialize

2014-11-27 Thread Stefano Sabatini
On date Tuesday 2014-11-25 20:38:59 +0100, Lukasz Marek encoded: Patch attached. From 2a9de161df4e1d0083264dde94ec5b8af59cec35 Mon Sep 17 00:00:00 2001 From: Lukasz Marek lukasz.m.lu...@gmail.com Date: Tue, 25 Nov 2014 20:25:10 +0100 Subject: [PATCH] lavu/opt: add escaping to

[FFmpeg-devel] [PATCH 1/2] avcodec/pngdec: split P frames handling to a separate function.

2014-11-27 Thread Benoit Fouet
--- libavcodec/pngdec.c | 29 + 1 file changed, 17 insertions(+), 12 deletions(-) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index 35dcd76..8529956 100644 --- a/libavcodec/pngdec.c +++ b/libavcodec/pngdec.c @@ -825,6 +825,22 @@ static int

[FFmpeg-devel] [PATCH 2/2] avcodec/pngdec: add support for 'over' blend operation for 'none' dispose operation.

2014-11-27 Thread Benoit Fouet
--- libavcodec/pngdec.c | 160 +++- 1 file changed, 133 insertions(+), 27 deletions(-) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index 8529956..4c9d321 100644 --- a/libavcodec/pngdec.c +++ b/libavcodec/pngdec.c @@ -29,6 +29,7 @@

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-27 Thread Timo Rothenpieler
Refactored the code even more now, new, tested, patch is attached and on github. The patch is ontop of my other patch that fixes the order in configure. commit 2b26ed5562911f39a52ef00942003a52ea06640b Author: Timo Rothenpieler t...@rothenpieler.org Date: Wed Nov 26 11:08:11 2014 +0100

Re: [FFmpeg-devel] [PATCH] Add NVENC encoder

2014-11-27 Thread Hendrik Leppkes
Am 27.11.2014 14:29 schrieb Jean-Baptiste Kempf j...@videolan.org: On 26 Nov, Nicolas George wrote : Le sextidi 6 frimaire, an CCXXIII, Hendrik Leppkes a écrit : Thats just wrong. There is not one line of proprietary/non-free code that gets included or linked in libavcodec. (IANAL)

Re: [FFmpeg-devel] [PATCH 2/2] lavc/options: fix leaks in avcodec_copy_context

2014-11-27 Thread Lukasz Marek
On 27 November 2014 at 04:17, Michael Niedermayer michae...@gmx.at wrote: On Thu, Nov 27, 2014 at 12:43:57AM +0100, Lukasz Marek wrote: On 24.11.2014 05:16, Lukasz Marek wrote: Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavcodec/options.c | 23 +++

Re: [FFmpeg-devel] [PATCH] avformat/mov: strengthen some table allocations

2014-11-27 Thread Clément Bœsch
On Thu, Nov 27, 2014 at 12:20:07AM +0100, Michael Niedermayer wrote: On Wed, Nov 26, 2014 at 10:48:10PM +0100, Clément Bœsch wrote: From: Clément Bœsch clem...@stupeflix.com --- libavformat/mov.c | 48 +++- 1 file changed, 27 insertions(+),

Re: [FFmpeg-devel] lavu/opt: add escaping to av_opt_serialize

2014-11-27 Thread Lukasz Marek
On 27 November 2014 at 15:05, Stefano Sabatini stefa...@gmail.com wrote: On date Tuesday 2014-11-25 20:38:59 +0100, Lukasz Marek encoded: Patch attached. From 2a9de161df4e1d0083264dde94ec5b8af59cec35 Mon Sep 17 00:00:00 2001 From: Lukasz Marek lukasz.m.lu...@gmail.com Date: Tue, 25 Nov

[FFmpeg-devel] DVB subtitles with multiple langage

2014-11-27 Thread Frédéric HUGENELL
hello I just uploaded a few minutes ago a sample of a movie in the incoming folder, as requested by ffmpeg as you can see on the picture in attach I recorded this movie on french sat tv, it contains 2 audio streams and 2 subtitles stream, voir for french langage and the other one for deaf people

Re: [FFmpeg-devel] [PATCH 2/3] lavf/ffmenc: store recommended encoder configuration

2014-11-27 Thread Lukasz Marek
On 27.11.2014 01:10, Michael Niedermayer wrote: On Wed, Nov 26, 2014 at 11:53:41PM +0100, Lukasz Marek wrote: ffmenc will store recommended encoder configuration if present. This will allow the user to base on local defaults and apply only explicitly set options. If recommended encoder

Re: [FFmpeg-devel] [PATCH] lavu/opt: add const to av_opt_copy arg

2014-11-27 Thread Lukasz Marek
On 27.11.2014 04:06, Michael Niedermayer wrote: On Thu, Nov 27, 2014 at 12:37:38AM +0100, Lukasz Marek wrote: On 24.11.2014 02:06, Michael Niedermayer wrote: On Sun, Nov 23, 2014 at 10:25:39PM +0100, Lukasz Marek wrote: On 23.11.2014 02:29, Michael Niedermayer wrote: On Sun, Nov 23, 2014 at

Re: [FFmpeg-devel] [PATCH] Move extralibs variables using ldl after ldl definition

2014-11-27 Thread Michael Niedermayer
On Thu, Nov 27, 2014 at 01:35:48PM +0100, t...@rothenpieler.org wrote: From: Timo Rothenpieler t...@rothenpieler.org --- configure | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/pngdec: split P frames handling to a separate function.

2014-11-27 Thread Michael Niedermayer
On Thu, Nov 27, 2014 at 03:26:26PM +0100, Benoit Fouet wrote: --- libavcodec/pngdec.c | 29 + 1 file changed, 17 insertions(+), 12 deletions(-) applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Freedom in capitalist

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/pngdec: add support for 'over' blend operation for 'none' dispose operation.

2014-11-27 Thread Michael Niedermayer
On Thu, Nov 27, 2014 at 03:26:27PM +0100, Benoit Fouet wrote: --- libavcodec/pngdec.c | 160 +++- 1 file changed, 133 insertions(+), 27 deletions(-) applied thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] DVB subtitles with multiple langage

2014-11-27 Thread Anshul
On November 27, 2014 2:27:59 PM IST, Frédéric HUGENELL fred.hugen...@gmail.com wrote: hello I just uploaded a few minutes ago a sample of a movie in the incoming folder, as requested by ffmpeg as you can see on the picture in attach I recorded this movie on french sat tv, it contains 2 audio

[FFmpeg-devel] [Patch 1/4]Fix h264 on POWER LE: libavcodec/ppc/h264dsp.c

2014-11-27 Thread rongyan
Hi, We present 4 patches to fix h264 bugs for POWER8 little endian, which are sent in 4 seperate emails. This is the first, to fix the functions h264_idct8_add_altivec(), h264_idct_dc_add_internal(), h264_loop_filter_luma_altivec(), write16x4() VEC_1D_DCT(), weight_h264_W_altivec(),

[FFmpeg-devel] [Patch 2/4]Fix h264 on POWER LE: libavcodec/ppc/h264chroma_template.c

2014-11-27 Thread rongyan
Hi, We present 4 patches to fix h264 bugs for POWER8 little endian, which are sent in 4 seperate emails. This is the third, to fix the functions PREFIX_h264_chroma_mc8_altivec(), PREFIX_no_rnd_vc1_chroma_mc8_altivec(), CHROMA_MC8_ALTIVEC_CORE_SIMPLE(), CHROMA_MC8_ALTIVEC_CORE(), add marcos

[FFmpeg-devel] [Patch 3/4]Fix h264 on POWER LE: libavcodec/ppc/h264qpel_template.c

2014-11-27 Thread rongyan
Hi, We present 4 patches to fix h264 bugs for POWER8 little endian, which are sent in 4 seperate emails. This is the first, to fix functions PREFIX_h264_qpel16_h_lowpass_altivec(), PREFIX_h264_qpel16_v_lowpass_altivec(), PREFIX_h264_qpel16_hv_lowpass_altivec(), add marcos load_alignment().

[FFmpeg-devel] [Patch 4/4]Fix h264 on POWER LE: libavcodec/ppc/h264qpel.c

2014-11-27 Thread rongyan
Hi, We present 4 patches to fix h264 bugs for POWER8 little endian, which are sent in 4 seperate emails. This is the fifth, to fix the functions put_pixels16_l2_altivec(), avg_pixels16_l2_altivec(), add marcos put_unligned_store(), avg_unligned_store(). The fate test result after merge these 4