Re: [FFmpeg-devel] [PATCH] qdm2: Allow hard-coding VLC tables.

2014-11-22 Thread Reimar Döffinger
On Sun, Sep 07, 2014 at 02:12:34PM +0200, Reimar Döffinger wrote: > Also adds a lot of infrastructure necessary for it. > Some of it is a bit ugly though. > Increases binary size for hardcoded tables by about 12 kB, > which is about 15 kB from qdm2_table minus data and code > save

Re: [FFmpeg-devel] [PATCH] qdm2: Allow hard-coding VLC tables.

2014-11-22 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 11:59:39AM -0500, compn wrote: > On Sat, 22 Nov 2014 14:17:09 +0100 > Reimar Döffinger wrote: > > > On Sun, Sep 07, 2014 at 02:12:34PM +0200, Reimar Döffinger wrote: > > > Also adds a lot of infrastructure necessary for it. > > >

Re: [FFmpeg-devel] [PATCH 1/1] ffv1dec: Avoid unnecessarily large stack usage and copies.

2014-11-22 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 02:42:25PM +0100, Michael Niedermayer wrote: > On Sat, Nov 22, 2014 at 02:14:13PM +0100, Reimar Döffinger wrote: > > Ideally the compiler could figure this out on its own, > > but it seems it can't. > > An alternative that would avoid the messy ex

[FFmpeg-devel] [PATCH] svq1enc: reduce stack usage of recursively-called function.

2014-11-22 Thread Reimar Döffinger
--- libavcodec/svq1enc.c | 2 +- libavcodec/svq1enc.h | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/svq1enc.c b/libavcodec/svq1enc.c index 288da1f..2a0d780 100644 --- a/libavcodec/svq1enc.c +++ b/libavcodec/svq1enc.c @@ -96,7 +96,7 @@ static int encode_block(SVQ1

[FFmpeg-devel] [PATCH] flacenc: calculate lower sum levels in-place.

2014-11-22 Thread Reimar Döffinger
Should improve cache usage and reduces stack usage. Also reduces number of copies in case many levels have the same number of bits. --- libavcodec/flacenc.c | 34 +++--- 1 file changed, 19 insertions(+), 15 deletions(-) diff --git a/libavcodec/flacenc.c b/libavcodec/fl

Re: [FFmpeg-devel] [PATCH] v210dec: Fix width calculation

2014-11-22 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 07:11:57PM +, Kieran Kunhya wrote: > --- > libavcodec/v210dec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/v210dec.c b/libavcodec/v210dec.c > index ae03952..978dffe 100644 > --- a/libavcodec/v210dec.c > +++ b/libavcodec/v210dec.

[FFmpeg-devel] [PATCH] lpc: Remove stack usage by allocating LLSModel in context.

2014-11-22 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger --- libavcodec/lpc.c | 2 +- libavcodec/lpc.h | 4 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/libavcodec/lpc.c b/libavcodec/lpc.c index f54f6f8..deb02e7 100644 --- a/libavcodec/lpc.c +++ b/libavcodec/lpc.c @@ -208,7 +208,7 @@ int

Re: [FFmpeg-devel] [PATCH] lpc: Remove stack usage by allocating LLSModel in context.

2014-11-22 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 08:49:45PM +0100, Reimar Döffinger wrote: > Signed-off-by: Reimar Döffinger Was of course meant to be "reduce stack usage" in subject, fixed locally. I thought about adding a alloc function for LLSModel while at it, but the code changes would have neede

[FFmpeg-devel] [PATCH] xface: Fix encoder crashes due to too small on-stack array.

2014-11-22 Thread Reimar Döffinger
Also add a FATE test. Signed-off-by: Reimar Döffinger --- libavcodec/xface.h | 9 + libavcodec/xfaceenc.c | 3 +++ libavformat/nut.c | 1 + tests/fate/vcodec.mak | 5 + tests/ref/vsynth/vsynth1-xface | 4 tests/ref/vsynth/vsynth2-xface

[FFmpeg-devel] [PATCH 1/2] xface: reduce table sizes.

2014-11-22 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger --- libavcodec/xface.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/xface.h b/libavcodec/xface.h index 6fbe908..63df5d3 100644 --- a/libavcodec/xface.h +++ b/libavcodec/xface.h @@ -85,8 +85,8 @@ enum XFaceColor

[FFmpeg-devel] [PATCH 2/2] xface: reduce stack usage by directly storing 2 bytes data instead of pointers.

2014-11-22 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger --- libavcodec/xfaceenc.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/xfaceenc.c b/libavcodec/xfaceenc.c index 0ade302..7edef1e 100644 --- a/libavcodec/xfaceenc.c +++ b/libavcodec/xfaceenc.c @@ -74,7 +74,7 @@ static int

Re: [FFmpeg-devel] [PATCH] xface: Fix encoder crashes due to too small on-stack array.

2014-11-22 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 11:14:45PM +0100, Reimar Döffinger wrote: > Also add a FATE test. > > Signed-off-by: Reimar Döffinger > --- > libavcodec/xface.h | 9 + > libavcodec/xfaceenc.c | 3 +++ > libavformat/nut.c | 1 + I forgot

Re: [FFmpeg-devel] [PATCH 1/2] xface: reduce table sizes.

2014-11-22 Thread Reimar Döffinger
On Sun, Nov 23, 2014 at 12:20:00AM +0100, Stefano Sabatini wrote: > On date Saturday 2014-11-22 23:31:58 +0100, Reimar Döffinger encoded: > > Signed-off-by: Reimar Döffinger > > --- > > libavcodec/xface.h | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletio

Re: [FFmpeg-devel] [PATCH] lpc: Remove stack usage by allocating LLSModel in context.

2014-11-22 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 11:26:56PM +0100, Michael Niedermayer wrote: > On Sat, Nov 22, 2014 at 08:49:45PM +0100, Reimar Döffinger wrote: > > Signed-off-by: Reimar Döffinger > > --- > > libavcodec/lpc.c | 2 +- > > libavcodec/lpc.h | 4 > > 2 files cha

Re: [FFmpeg-devel] [PATCH] svq1enc: reduce stack usage of recursively-called function.

2014-11-22 Thread Reimar Döffinger
On Sun, Nov 23, 2014 at 03:57:47AM +0100, Michael Niedermayer wrote: > On Sat, Nov 22, 2014 at 07:50:59PM +0100, Reimar Döffinger wrote: > > --- > > libavcodec/svq1enc.c | 2 +- > > libavcodec/svq1enc.h | 2 ++ > > 2 files changed, 3 insertions(+), 1 deletion(-

Re: [FFmpeg-devel] [PATCH] avutil/bprint: make header work with C++, simplify

2014-11-24 Thread Reimar Döffinger
On 24.11.2014, at 15:27, Nicolas George wrote: > Le quartidi 4 frimaire, an CCXXIII, wm4 a écrit : >> This change should be ABI-compatible. Since the size of the useable >> internal buffer was actually different on 32 bit and 64 bit, so the code >> contains a somewhat ugly kludge to maintain ABI-c

Re: [FFmpeg-devel] pipeline multithreading

2014-11-24 Thread Reimar Döffinger
On Mon, Nov 24, 2014 at 12:35:58PM +0100, Daniel Oberhoff wrote: > inout -> filter1 -> filter2 -> output > > some threads processing frame n in the output (i.e. encoding), other threads > procesing frame n+1 in filter2, others processing frame n+2 in filter1, and > yet others processing frame n+

Re: [FFmpeg-devel] pipeline multithreading

2014-11-25 Thread Reimar Döffinger
On 25.11.2014, at 10:01, Daniel Oberhoff wrote: >> Am 24.11.2014 um 17:16 schrieb Reimar Döffinger : >> >>> On Mon, Nov 24, 2014 at 12:35:58PM +0100, Daniel Oberhoff wrote: >>> inout -> filter1 -> filter2 -> output >>> >>> some thread

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

2014-11-26 Thread Reimar Döffinger
On 26.11.2014, at 14:58, Timo Rothenpieler wrote: >> I cannot read it like this but I am not a native >> speaker. >> Maybe you could mark it as non-free until the FSF >> tells us their interpretation? > > Allright, new patch is attached. > >> ff_nvenc_encoder is missing pix_fmts afaict but >> co

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

2014-11-28 Thread Reimar Döffinger
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: > http://fate.ffmpeg.org/report.cgi?time=20141126233318&slot=x86_32-msvc12-windows

Re: [FFmpeg-devel] [PATCH] h264_i386: Optimize decode_significance_8x8_x86 for 64 bit.

2014-12-03 Thread Reimar Döffinger
On 03.12.2014, at 01:40, Michael Niedermayer wrote: > On Sat, Nov 22, 2014 at 02:09:01PM +0100, Reimar Döffinger wrote: >> On Mon, Nov 17, 2014 at 01:41:13PM +0100, Michael Niedermayer wrote: >>> On Mon, Nov 17, 2014 at 08:19:32AM +0100, Reimar Döffinger wrote: >>&g

Re: [FFmpeg-devel] [PATCH] h264_i386: Optimize decode_significance_8x8_x86 for 64 bit.

2014-12-03 Thread Reimar Döffinger
On Wed, Dec 03, 2014 at 01:19:48PM +0100, Michael Niedermayer wrote: > On Wed, Dec 03, 2014 at 09:00:39AM +0100, Reimar Döffinger wrote: > > On 03.12.2014, at 01:40, Michael Niedermayer wrote: > > > On Sat, Nov 22, 2014 at 02:09:01PM +0100, Reimar Döffinger wrote: > > &g

Re: [FFmpeg-devel] [PATCH] avformat/mov: Disable XMP metadata by default

2014-12-09 Thread Reimar Döffinger
On Tue, Dec 09, 2014 at 12:59:24PM +, Vittorio Giovara wrote: > On Tue, Dec 9, 2014 at 8:04 AM, wm4 wrote: > > On Tue, 9 Dec 2014 02:57:01 +0100 > > Michael Niedermayer wrote: > > > > It seems very wrong to export this info optionally just because > > ffmpeg.c is too stupid to handle it corr

Re: [FFmpeg-devel] [PATCH 1/2] lavd/avdevice: replace av_ with ff_ for caps option table

2014-12-13 Thread Reimar Döffinger
On 13.12.2014, at 20:42, James Almer wrote: > On 13/12/14 4:27 PM, Lukasz Marek wrote: >> This symbol is required for query capabilities callbacks. >> This symbol is only required by libavdevice and should not be exported. > > This need a deprecation and an FF_API scheduled removal. We can't remo

Re: [FFmpeg-devel] [PATCH] avcodec/xiph: mark returned header pointers const from avpriv_split_xiph_headers()

2014-12-14 Thread Reimar Döffinger
On 12.12.2014, at 14:46, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer Should be fine. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] mjpeg2jpeg_filter

2014-12-15 Thread Reimar Döffinger
On Sun, Dec 14, 2014 at 08:10:31AM -0500, Don Moir wrote: > I needed to convert AVI1 to normal jpeg. This happens a lot with camera > streams and the code in mjpeg2jpeg_filter works fine. > > I was looking at the code in mjpeg2jpeg_filter in file mjpeg2jpeg_bsf.c and > noticing that just creates

Re: [FFmpeg-devel] [PATCH] wavdec: RIFX file format support

2014-12-15 Thread Reimar Döffinger
On Mon, Dec 15, 2014 at 10:24:55AM +, Carl Eugen Hoyos wrote: > Thomas Volkert gmx.net> writes: > > > +#include > > This will hopefully be unneeded. > > > codec->sample_rate = avio_rl32(pb); > > codec->bit_rate= avio_rl32(pb) * 8; > > codec->block_align = avio_rl16(pb

Re: [FFmpeg-devel] How to make the link.exe as MSVC's link

2014-12-16 Thread Reimar Döffinger
On Tue, Dec 16, 2014 at 11:14:59AM +0100, Hendrik Leppkes wrote: > On Tue, Dec 16, 2014 at 10:11 AM, Jesse Jiang > wrote: > > > > Yes, it is true. > > But I think this job is not hard, we can use which command to get the cl > > path. Like $ which cl/c/Program Files (x86)/Microsoft Visual Studio >

Re: [FFmpeg-devel] How to make the link.exe as MSVC's link

2014-12-16 Thread Reimar Döffinger
On Tue, Dec 16, 2014 at 09:11:18AM +, Jesse Jiang wrote: > Yes, it is true. > But I think this job is not hard, we can use which command to get the cl > path. Like $ which cl/c/Program Files (x86)/Microsoft Visual Studio > 12.0/VC/BIN/cl.exe > Then we can replace the cl.exe with link.exe, so

Re: [FFmpeg-devel] [PATCH] qdm2: Allow hard-coding VLC tables.

2014-12-16 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 03:43:56PM +0100, wm4 wrote: > On Sat, 22 Nov 2014 14:17:09 +0100 > Reimar Döffinger wrote: > > > On Sun, Sep 07, 2014 at 02:12:34PM +0200, Reimar Döffinger wrote: > > > Also adds a lot of infrastructure necessary for it. > > >

Re: [FFmpeg-devel] [PATCH] flacenc: calculate lower sum levels in-place.

2014-12-16 Thread Reimar Döffinger
On Sun, Nov 23, 2014 at 08:42:29PM +0100, Michael Niedermayer wrote: > On Sat, Nov 22, 2014 at 08:07:47PM +0100, Reimar Döffinger wrote: > > Should improve cache usage and reduces stack usage. > > Also reduces number of copies in case many levels > > have th

[FFmpeg-devel] [PATCH] aacps.c: Move large arrays to context to reduce stack usage.

2014-12-16 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger --- libavcodec/aacps.c | 4 ++-- libavcodec/aacps.h | 2 ++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/aacps.c b/libavcodec/aacps.c index 20012f9..ea5a5d2 100644 --- a/libavcodec/aacps.c +++ b/libavcodec/aacps.c @@ -908,8 +908,8

Re: [FFmpeg-devel] [PATCH] wavdec: RIFX file format support

2014-12-17 Thread Reimar Döffinger
On Wed, Dec 17, 2014 at 11:55:17AM +0100, Thomas Volkert wrote: > On 12/16/2014 08:36 AM, Reimar Döffinger wrote: > >On Mon, Dec 15, 2014 at 10:24:55AM +, Carl Eugen Hoyos wrote: > >>> codec->sample_rate = avio_rl32(pb); > >>> co

Re: [FFmpeg-devel] [PATCH] tools/probetest: support testing a single specified input format

2014-12-21 Thread Reimar Döffinger
On 21.12.2014, at 21:09, Michael Niedermayer wrote: > +int j; > + > +for (j = i = 1; i +if (av_isdigit(argv[i][0])) { > +if (j++ == 1) { > +retry_count = atoi(argv[i]); > +} else > +max_size = atoi(argv[i]); > +} else

Re: [FFmpeg-devel] [PATCH] avcodec.h: document needed buffer size for av_get_codec_tag_string()

2014-12-21 Thread Reimar Döffinger
On 21.12.2014, at 00:10, Nicolas George wrote: > Le primidi 1er nivôse, an CCXXIII, Thomas Volkert a écrit : >> If the code is changed, the documentation should be changed as well. > > That will not update applications written with the documentation for the old > version but built with the new ve

Re: [FFmpeg-devel] [PATCH] lavfi: Port fspp to FFmpeg

2014-12-22 Thread Reimar Döffinger
On 22.12.2014, at 17:23, Stefano Sabatini wrote: > On date Monday 2014-12-22 13:07:03 +0100, Stefano Sabatini encoded: >> On date Friday 2014-12-19 16:46:40 +0530, arwa arif encoded: >> [...] >>> From c1c3255203226663fc382a0994182df3d558afe6 Mon Sep 17 00:00:00 2001 >>> From: Arwa Arif >>> Date:

Re: [FFmpeg-devel] [PATCH] lavfi: Port fspp to FFmpeg

2014-12-24 Thread Reimar Döffinger
On Tue, Dec 23, 2014 at 08:12:50AM +0100, Stefano Sabatini wrote: > On date Monday 2014-12-22 17:57:34 +0100, Reimar Döffinger encoded: > > On 22.12.2014, at 17:23, Stefano Sabatini wrote: > [...] > > > I'm testing with: > > > ffmpeg -i matrixbench_mpeg2.mpg

Re: [FFmpeg-devel] [PATCH] qdm2: Allow hard-coding VLC tables.

2014-12-24 Thread Reimar Döffinger
On Sat, Dec 20, 2014 at 12:38:38AM +, Carl Eugen Hoyos wrote: > Reimar Döffinger gmx.de> writes: > > This patch has broken "make checkheaders". > > Should the header just not be checked or is there > another fix? Sorry, I missed the mail. Assuming tablep

Re: [FFmpeg-devel] Frame threaded encoders

2014-12-24 Thread Reimar Döffinger
On Mon, Dec 22, 2014 at 10:57:21PM +, Kieran Kunhya wrote: > Hi, > > Does the CODEC_CAP_FRAME_THREADS API support frame threaded encoding? It should, at the very least quite a few intra codecs use it to support multithreading, see pngenc.c and mjpegenc.c for example. It is however possible th

Re: [FFmpeg-devel] [PATCH] tools/probetest: support testing a single specified input format

2014-12-24 Thread Reimar Döffinger
On Tue, Dec 23, 2014 at 03:39:43AM +0100, Michael Niedermayer wrote: > On Mon, Dec 22, 2014 at 12:45:43AM +0100, Michael Niedermayer wrote: > > This reduces the time the test takes significantly when only one > > formats needs to be tested > > > > Signed-off-by: Michael Niedermayer > > --- > > t

Re: [FFmpeg-devel] [PATCH]Do not store more than 30 bit for r10k

2014-12-27 Thread Reimar Döffinger
On Sat, Dec 27, 2014 at 12:41:12AM +0100, Carl-Eugen Hoyos wrote: > diff --git a/libavcodec/r210enc.c b/libavcodec/r210enc.c > index d61cd75..4cbebd7 100644 > --- a/libavcodec/r210enc.c > +++ b/libavcodec/r210enc.c > @@ -62,7 +62,7 @@ static int encode_frame(AVCodecContext *avctx, AVPacket > *pkt,

Re: [FFmpeg-devel] [PATCH]Do not store more than 30 bit for r10k

2014-12-27 Thread Reimar Döffinger
On Sat, Dec 27, 2014 at 05:02:39PM +, Carl Eugen Hoyos wrote: > Reimar Döffinger gmx.de> writes: > > > -uint16_t b = *src++ >> 4; > > +uint16_t b = *src++ >> 6; > > if (avctx->codec_id == AV_CODEC_ID_R210) > &g

Re: [FFmpeg-devel] [PATCH]Fix r10kdec output

2014-12-27 Thread Reimar Döffinger
On Sat, Dec 27, 2014 at 10:39:39PM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch allows to use -f framecrc to check r10k -> gbrp10 ffv1 -> > r10k roundtrips: The decoder currently writes incorrect lsb for blue that > are different for the input and output files. This does not affect th

Re: [FFmpeg-devel] [PATCH]Do not store more than 30 bit for r10k

2014-12-27 Thread Reimar Döffinger
On 28.12.2014, at 00:58, Carl Eugen Hoyos wrote: > Reimar Döffinger gmx.de> writes: > >> Though as noted in the commit message I am not sure that >> for better compatibility the extra bits shouldn't be 1 >> instead... > > I am not against, but I wonder

Re: [FFmpeg-devel] ffmpeg improvement for x265

2014-12-30 Thread Reimar Döffinger
On Tue, Dec 30, 2014 at 10:31:14AM +0100, wm4 wrote: > On Tue, 30 Dec 2014 13:38:00 +0800 (CST) > llj wrote: > > > hello, > > My name is Liujun Liu, a developer of x265 group. > > firstly I want to describe what the problem is. There is an option > > HI_BIT_DEPTH in x265 CMake build option, this

Re: [FFmpeg-devel] [PATCH] avfilter/vf_cropdetect: adjust Default threshold based on bit depth

2014-12-30 Thread Reimar Döffinger
On Tue, Dec 30, 2014 at 09:12:13PM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavfilter/vf_cropdetect.c |9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/libavfilter/vf_cropdetect.c b/libavfilter/vf_cropdetect.c > index dd2

Re: [FFmpeg-devel] Parallelization

2014-12-31 Thread Reimar Döffinger
On 31.12.2014, at 04:00, David Ison wrote: > 3. I am planning initial implementations using OpenMP. There is good > support on Gnu, LLVM and other compilers > . That is _not_ true. There is no OpenMP support in LLVM, what they misleadingly list is an u

Re: [FFmpeg-devel] [PATCH]Allow libspeex detection also without pkg-config

2014-12-31 Thread Reimar Döffinger
On Wed, Dec 31, 2014 at 01:25:33PM +0100, Nicolas George wrote: > Le primidi 11 nivôse, an CCXXIII, Carl Eugen Hoyos a écrit : > > Attached patch fixes ticket #4197 for me. > > Installing pkg-config fixes ticket #4197 for you. > > I object to any useless obfuscation to configure. I don't care mu

Re: [FFmpeg-devel] [PATCH]Allow libspeex detection also without pkg-config

2014-12-31 Thread Reimar Döffinger
On Wed, Dec 31, 2014 at 02:09:42PM +0100, Nicolas George wrote: > Le primidi 11 nivôse, an CCXXIII, Reimar Döffinger a écrit : > > I don't care much, but I still find not having to use pkg-config > > somewhat more convenient in a cross-compilation environment where > >

Re: [FFmpeg-devel] [PATCH]Allow libspeex detection also without pkg-config

2014-12-31 Thread Reimar Döffinger
On Wed, Dec 31, 2014 at 02:23:06PM +0100, Clément Bœsch wrote: > On Wed, Dec 31, 2014 at 02:02:20PM +0100, Reimar Döffinger wrote: > [...] > > There is also a (minority) of projects with pkg-config files that > > cause undesirable side-effects which can hide errors. > > Fr

Re: [FFmpeg-devel] [PATCH]Allow libspeex detection also without pkg-config

2014-12-31 Thread Reimar Döffinger
On Wed, Dec 31, 2014 at 02:38:20PM +0100, Clément Bœsch wrote: > On Wed, Dec 31, 2014 at 02:23:54PM +0100, Reimar Döffinger wrote: > > On Wed, Dec 31, 2014 at 02:09:42PM +0100, Nicolas George wrote: > > > Le primidi 11 nivôse, an CCXXIII, Reimar Döffinger a écrit : > > >

Re: [FFmpeg-devel] [PATCH]Allow libspeex detection also without pkg-config

2014-12-31 Thread Reimar Döffinger
On Wed, Dec 31, 2014 at 03:43:32PM +0100, Clément Bœsch wrote: > On Wed, Dec 31, 2014 at 02:57:43PM +0100, Reimar Döffinger wrote: > > On Wed, Dec 31, 2014 at 02:38:20PM +0100, Clément Bœsch wrote: > > > On Wed, Dec 31, 2014 at 02:23:54PM +0100, Reimar Döffinger wrote: > >

Re: [FFmpeg-devel] [PATCH] Add pkg-config for libfdk_aac

2014-12-31 Thread Reimar Döffinger
On Wed, Dec 31, 2014 at 04:03:59PM +0100, Clément Bœsch wrote: > On Wed, Dec 31, 2014 at 03:58:50PM +0100, wm4 wrote: > > On Wed, 31 Dec 2014 14:11:20 +0100 > > Clément Bœsch wrote: > > > > > On Wed, Dec 31, 2014 at 12:14:03PM +, Carl Eugen Hoyos wrote: > > > > Nicolas George nsup.org> write

Re: [FFmpeg-devel] [PATCH]Allow libspeex detection also without pkg-config

2014-12-31 Thread Reimar Döffinger
On 31.12.2014, at 16:52, Nicolas George wrote: > Le primidi 11 nivôse, an CCXXIII, Reimar Döffinger a écrit : >> >> Also I'm not arguing it's impossible, I'm arguing it's inconvenient. >> It's something like 2-3 additional steps for each library.

[FFmpeg-devel] [PATCH] configure: Add toolchain-prefix option.

2015-01-01 Thread Reimar Döffinger
Useful to select e.g. between mingw32, 32-bit mingw64 and 64-bit mingw64 toolchains, which all are not cross-compilation toolchains. Signed-off-by: Reimar Döffinger --- configure | 8 1 file changed, 8 insertions(+) diff --git a/configure b/configure index 9085200..5c1cbca 100755

[FFmpeg-devel] [PATCH] configure: fix --cross-prefix/--toolchain-prefix to work with --toolchain.

2015-01-01 Thread Reimar Döffinger
Before the prefix was silently dropped if --toolchain was specified. Signed-off-by: Reimar Döffinger --- configure | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/configure b/configure index 5c1cbca..1aaadd4 100755 --- a/configure +++ b

Re: [FFmpeg-devel] [PATCH] Changelog: Add cropdetect non 8bpp

2015-01-01 Thread Reimar Döffinger
On Thu, Jan 01, 2015 at 03:34:15AM +0100, Michael Niedermayer wrote: > Suggested-by: Reimar > Signed-off-by: Michael Niedermayer > --- > Changelog |1 + > 1 file changed, 1 insertion(+) > > diff --git a/Changelog b/Changelog > index 37ce0f0..e9d8355 100644 > --- a/Changelog > +++ b/Changelog

[FFmpeg-devel] [PATCH] configure: Factor out some pkg-config hacks.

2015-01-01 Thread Reimar Döffinger
Make require_pkg_config check for extra argument that should not be there. Change use_pkg_config to allow specifying fallback libs and cflags. Signed-off-by: Reimar Döffinger --- configure | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/configure b

[FFmpeg-devel] [PATCH] configure: Factor out some pkg-config hacks.

2015-01-01 Thread Reimar Döffinger
Make require_pkg_config check for extra argument that should not be there. Change use_pkg_config to allow specifying fallback libs and cflags. Signed-off-by: Reimar Döffinger --- configure | 22 +++--- 1 file changed, 15 insertions(+), 7 deletions(-) diff --git a/configure b

Re: [FFmpeg-devel] [PATCH] configure: Factor out some pkg-config hacks.

2015-01-01 Thread Reimar Döffinger
On 01.01.2015, at 12:04, Clément Bœsch wrote: > On Thu, Jan 01, 2015 at 11:40:25AM +0100, Reimar Döffinger wrote: >> >> +enabled libx264 && use_pkg_config x264 "stdint.h x264.h" >> x264_encoder_encode -lx264 && >>

Re: [FFmpeg-devel] [PATCH] configure: Factor out some pkg-config hacks.

2015-01-01 Thread Reimar Döffinger
On 01.01.2015, at 12:49, Clément Bœsch wrote: > On Thu, Jan 01, 2015 at 12:43:52PM +0100, Reimar Döffinger wrote: >> On 01.01.2015, at 12:04, Clément Bœsch wrote: >>> On Thu, Jan 01, 2015 at 11:40:25AM +0100, Reimar Döffinger wrote: >>>> >>>> +enable

Re: [FFmpeg-devel] [PATCH] avformat/aiffdec: only read codec tag when there is space in header

2015-01-06 Thread Reimar Döffinger
On 04.01.2015, at 22:44, Peter Ross wrote: > Signed-off-by: Peter Ross > --- > Revised patch. Now passes FATE. > > libavformat/aiffdec.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavformat/aiffdec.c b/libavformat/aiffdec.c > index 91ef2a4..8dbed32 100644 > ---

Re: [FFmpeg-devel] [PATCH] Force text files to be downloaded in UNIX line ending (LF)

2015-01-11 Thread Reimar Döffinger
On 10.01.2015, at 11:18, Derek Buitenhuis wrote: > On 1/10/2015 8:40 AM, Timotius Margo wrote: >> This causes a problem with autoconf and prevents >> user from building FFmpeg successfully without applying a workaround. > > I forgot to mention this is fixed in newer make.exe's, and I think > it's

Re: [FFmpeg-devel] [PATCH 2/2] add libyami.cpp for h264 decoding by libyami

2015-01-12 Thread Reimar Döffinger
On 09.01.2015, at 09:15, "Zhao, Halley" wrote: > From: "Zhao, Halley" > > - do not support multi-thread decoding, it is unnecessary for hw While it might not be possible to support in a proper way, and the FFmpeg multithreading is probably useless in the context of this library, but I think t

Re: [FFmpeg-devel] [RFC] Support dynamic loading of third-party libs

2015-01-16 Thread Reimar Döffinger
On Wed, Jan 14, 2015 at 06:26:06PM +0100, Michael Niedermayer wrote: > ive no idea how portable it is but one way to avoid most of the > mess would be to use attribute weak / lazy linking > that way as long as lib functions arent called nothing should try to > load the lib and so it should work wit

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-18 Thread Reimar Döffinger
On Sat, Jan 17, 2015 at 11:35:41AM -0800, Philip Langdale wrote: > There is a long sad story behind all this, but it's somewhat ambiguous as to > whether DVD content should be treated as 720 pixels wide or 704 pixels, with > 16 pixels cut off. If you decide is should be 704 pixels wide, you need to

Re: [FFmpeg-devel] [PATCH] configure: Add toolchain-prefix option.

2015-01-18 Thread Reimar Döffinger
On Thu, Jan 01, 2015 at 10:00:25AM +0100, Reimar Döffinger wrote: > Useful to select e.g. between mingw32, 32-bit mingw64 and 64-bit mingw64 > toolchains, which all are not cross-compilation toolchains. Ping? ___ ffmpeg-devel mailing list ffmpeg

Re: [FFmpeg-devel] [PATCH] configure: fix --cross-prefix/--toolchain-prefix to work with --toolchain.

2015-01-18 Thread Reimar Döffinger
On Thu, Jan 01, 2015 at 10:03:52AM +0100, Reimar Döffinger wrote: > Before the prefix was silently dropped if --toolchain was specified. Ping? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] inline asm: remove % from clobber list.

2015-01-18 Thread Reimar Döffinger
This is simpler, consistent with some other asm code and the gcc documentation and in addition also works with e.g. tinycc. Signed-off-by: Reimar Döffinger --- libavcodec/msmpeg4.c | 2 +- libavcodec/x86/ac3dsp_init.c | 4 +-- libavcodec/x86/cabac.h

[FFmpeg-devel] [PATCH] swresample_internal.h: Move struct declaration before first use.

2015-01-18 Thread Reimar Döffinger
It seems more logical and works with more restricted C compilers like tinycc. Signed-off-by: Reimar Döffinger --- libswresample/swresample_internal.h | 42 ++--- 1 file changed, 21 insertions(+), 21 deletions(-) diff --git a/libswresample/swresample_internal.h

Re: [FFmpeg-devel] [PATCH] configure: Add toolchain-prefix option.

2015-01-18 Thread Reimar Döffinger
On 19.01.2015, at 03:05, Michael Niedermayer wrote: > On Thu, Jan 01, 2015 at 10:00:25AM +0100, Reimar Döffinger wrote: >> Useful to select e.g. between mingw32, 32-bit mingw64 and 64-bit mingw64 >> toolchains, which all are not cross-compilation toolchains. >> >> Sign

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-18 Thread Reimar Döffinger
On 19.01.2015, at 01:16, compn wrote: > On Sun, 18 Jan 2015 18:41:09 +0100 > Reimar Döffinger wrote: >> Oh my god, this is so horribly broken I vote for doing a "return >> -EBROKENAPI" and tell them we'll enable this when NVidia fixes their >> stuff. >&g

Re: [FFmpeg-devel] [PATCH] configure: Add toolchain-prefix option.

2015-01-19 Thread Reimar Döffinger
On Mon, Jan 19, 2015 at 01:37:17PM +0100, Hendrik Leppkes wrote: > On Mon, Jan 19, 2015 at 1:32 PM, Michael Niedermayer > wrote: > > > It's even more annoying if you want to compile for 32 bit on a 64 bit > > system (you can use -m32 at least in some cases, but this seems simpler and > > more cons

Re: [FFmpeg-devel] [PATCH] tests: drop bc dependency

2015-01-20 Thread Reimar Döffinger
On Mon, Jan 19, 2015 at 10:58:02PM +0100, Clément Bœsch wrote: > -echo "scale=2; v = $1 - $2; if (v < 0) v = -v; if (v > $3) r = 1; r" | bc > +awk "BEGIN { v=$1-$2; printf \"%d\\n\", ((v<0?v:-v) > $3) ? 1 : 0 }" I'd suggest single quotes for the outer ones, then you do not need to escape \

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-21 Thread Reimar Döffinger
On 21.01.2015, at 07:17, Agatha Hu wrote: > On 2015/1/18 4:01, Philip Langdale wrote: >> There is a long sad story behind all this, but it's somewhat ambiguous as to >> whether DVD content should be treated as 720 pixels wide or 704 pixels, with >> 16 pixels cut off. If you decide is should be 704

Re: [FFmpeg-devel] [PATCH] libx265: Pass through user-set frame type

2015-01-21 Thread Reimar Döffinger
On Wed, Jan 21, 2015 at 04:52:33PM +, Derek Buitenhuis wrote: > On 1/21/2015 4:46 PM, Timo Rothenpieler wrote: > >> No? It has X265_TYPE_AUTO as the fallback if it is NULL. Just like > >> libx264.c: > > > > Is falls back to that if pic->pict_type is NULL, but if pic is NULL, it > > will just

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-21 Thread Reimar Döffinger
On 22.01.2015, at 05:15, Agatha Hu wrote: > On 2015/1/21 16:00, Reimar Döffinger wrote: >> On 21.01.2015, at 07:17, Agatha Hu wrote: >>> On 2015/1/18 4:01, Philip Langdale wrote: > >>>> >>> >>> Here's the reply from NVENC engineers &

Re: [FFmpeg-devel] Question about supported_fps in libavutil/timecode.c::check_fps

2015-01-25 Thread Reimar Döffinger
On 24.01.2015, at 21:09, wm4 wrote: > On Sat, 24 Jan 2015 18:37:01 + > Derek Buitenhuis wrote: > >> On 1/24/2015 4:33 PM, wm4 wrote: >>> Which ones? We even expect C99 support from the compiler. >> >> Doesn't matter. It's the project's policy to have decls at >> block beginnings. Yes some o

Re: [FFmpeg-devel] [PATCH] Move stream_options to avformat

2015-01-25 Thread Reimar Döffinger
On 25.01.2015, at 03:08, Michael Niedermayer wrote: > On Sun, Jan 25, 2015 at 02:31:33AM +0100, wm4 wrote: >> As an experienced API user, I don't have the slightest clue what I'd do with this API, or where to find information about it. >>> >>> the primary goal is to remove duplic

Re: [FFmpeg-devel] [PATCH] configure: support MSYS2

2015-01-25 Thread Reimar Döffinger
On Sun, Jan 25, 2015 at 03:59:12PM +0100, Hendrik Leppkes wrote: > On Sun, Jan 25, 2015 at 3:26 PM, Hendrik Leppkes > wrote: > > > MSYS2 uses a system name of "MSYS_NT-6.3" instead of "MINGW32_NT-6.3" in > > MSYS1. > > > > Apparently this isn't quite correct, and you have to start MSYS2 with a >

Re: [FFmpeg-devel] [PATCH] configure: add --build-date and --build-time options

2015-01-25 Thread Reimar Döffinger
On Thu, Jan 22, 2015 at 03:49:09PM +0100, Andreas Cadhalpun wrote: > Hi, > > On 22.01.2015 15:31, wm4 wrote: > >If the build date is not correct, then what value does the build date > >have at all? > > How do you know it is 'correct' currently? > One can change the system clock (or use faketime).

Re: [FFmpeg-devel] [PATCH] configure: support MSYS2

2015-01-25 Thread Reimar Döffinger
On Sun, Jan 25, 2015 at 06:43:54PM +0100, Michael Niedermayer wrote: > On Sun, Jan 25, 2015 at 05:02:58PM +0100, Reimar Döffinger wrote: > > I don't really see anything wrong with this, however I'd rather have > > a compiler-based detection. > > I.e. if the compiler

[FFmpeg-devel] [PATCH] Support BMP files that do not properly align lines.

2015-01-26 Thread Reimar Döffinger
Quite a few programs missed that detail of the spec (including old versions of FFmpeg I believe) and when we would otherwise fail anyway it seems worth a try to us a simple byte-aligned stride instead. Signed-off-by: Reimar Döffinger --- libavcodec/bmp.c | 10 +++--- 1 file changed, 7

Re: [FFmpeg-devel] [PATCH] Support BMP files that do not properly align lines.

2015-01-26 Thread Reimar Döffinger
On 26.01.2015, at 22:49, Michael Niedermayer wrote: > On Mon, Jan 26, 2015 at 08:21:16PM +0100, Reimar Döffinger wrote: >> Quite a few programs missed that detail of the spec >> (including old versions of FFmpeg I believe) and when >> we would otherwise fail anyway it seems w

Re: [FFmpeg-devel] libavutil: Added twofish block cipher

2015-01-26 Thread Reimar Döffinger
On 25 January 2015 20:00:07 CET, supraja reddy wrote: >Sorry. I forgot to make a small change. Updated the patch. > Not specifically related to this patch, but with this many implementations I think it is high time we warn about the security implications of using them. My original implementation

Re: [FFmpeg-devel] [PATCH] Support BMP files that do not properly align lines.

2015-01-27 Thread Reimar Döffinger
On Tue, Jan 27, 2015 at 09:50:26PM +0100, Michael Niedermayer wrote: > On Tue, Jan 27, 2015 at 06:38:22AM +0100, Reimar Döffinger wrote: > > On 26.01.2015, at 22:49, Michael Niedermayer wrote: > > > On Mon, Jan 26, 2015 at 08:21:16PM +0100, Reimar Döffinger wrote: > >

Re: [FFmpeg-devel] [PATCH 3/3] lavc/libopenjpegenc: add cinema_setup_encoder function to allow creation of dci compliant files

2015-01-29 Thread Reimar Döffinger
On Wed, Jan 28, 2015 at 04:41:26PM +0100, Jean First wrote: > +if (parameters->numresolution > 6) { > +parameters->numresolution = 6; > +} FFMAX > +if (!((image->comps[0].w == 2048) || (image->comps[0].h == > 1080))) { w != 2048 && h != 1080 s

Re: [FFmpeg-devel] [PATCHv2] flac: ignore IDv3 tags if vorbis tags exist

2015-02-02 Thread Reimar Döffinger
On Mon, Feb 02, 2015 at 06:49:11PM -0500, Ben Boeckel wrote: > FLAC doesn't really support IDv3 tags, so warn if they are found at all. > If vorbis tags are found, toss out the IDv3 tags. They are kept if > vorbis tags aren't found to at least have something there. I'm not really convinced this ma

Re: [FFmpeg-devel] [PATCHv2] flac: ignore IDv3 tags if vorbis tags exist

2015-02-02 Thread Reimar Döffinger
On 03.02.2015, at 01:26, Lou Logan wrote: > On Mon, Feb 2, 2015, at 03:10 PM, Reimar Döffinger wrote: >> I'm not really convinced this makes sense. >> There are thousands of programs that can edit IDv3 tags, but >> comparatively few that can handle vorbis tags. >&g

[FFmpeg-devel] [PATCH] avformat_alloc_context: initialize raw_packet_buffer_remaining_size.

2015-05-09 Thread Reimar Döffinger
This currently works for most users because avformat_open_input sets it, but this patch fixes any applications not using that function. Signed-off-by: Reimar Döffinger --- libavformat/options.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/options.c b/libavformat/options.c

Re: [FFmpeg-devel] [PATCH] avformat_alloc_context: initialize raw_packet_buffer_remaining_size.

2015-05-10 Thread Reimar Döffinger
On Sat, May 09, 2015 at 11:41:17PM +0200, Michael Niedermayer wrote: > On Sat, May 09, 2015 at 08:57:26PM +0200, Reimar Döffinger wrote: > > This currently works for most users because > > avformat_open_input sets it, but this patch fixes any > > applications no

Re: [FFmpeg-devel] [RFC] DXVA2 decoding and FFmpeg

2015-05-18 Thread Reimar Döffinger
On 18.05.2015, at 12:37, Stefano Sabatini wrote: > On Thu, May 14, 2015 at 2:52 PM, Stefano Sabatini > wrote: > >> On date Thursday 2015-05-14 13:01:51 +0200, Stefano Sabatini encoded: >>> On date Tuesday 2015-05-12 15:54:17 +0200, Hendrik Leppkes encoded: >> [...] One limitation is as t

Re: [FFmpeg-devel] [PATCH] avcodec/flacdec: Add lpc16m instead of using lpc32 + post analysis and correction

2015-05-18 Thread Reimar Döffinger
On 17.05.2015, at 17:03, Michael Niedermayer wrote: > This is slower on x86 as the lpc16* functions are not optimized while lpc32 is > If someone wants to optimize lpc16m and it would actually be faster then this > could be applied and used until then the current solution in git is better as > its

Re: [FFmpeg-devel] [PATCH] avcodec/codec_desc: rename smv video codec name to smvjpeg to match decoder name

2015-06-29 Thread Reimar Döffinger
On Tue, Jun 16, 2015 at 09:18:13PM +0200, Hendrik Leppkes wrote: > On Tue, Jun 16, 2015 at 8:33 PM, wm4 wrote: > > On Tue, 16 Jun 2015 13:29:55 + > > Paul B Mahol wrote: > > > >> Also fixes clash with smv audio codec. > >> > >> Signed-off-by: Paul B Mahol > >> --- > >> libavcodec/codec_desc

Re: [FFmpeg-devel] [PATCH] Compute individual stream durations in matroska muxer. Write them as binary tags. Parse the binary tags in matroska demuxer, and write them to AVStream

2015-07-28 Thread Reimar Döffinger
On 29.07.2015, at 03:10, Sasi Inguva wrote: > A little context on why I need this. I use individual stream durations to > validate , if the outputs after transcoding have the same stream durations. > For this we need to know the stream durations through a quick parse of the > header of the file, s

Re: [FFmpeg-devel] [PATCH] Compute individual stream durations in matroska muxer. Write them as binary tags. Parse the binary tags in matroska demuxer, and write them to AVStream

2015-07-28 Thread Reimar Döffinger
On 29.07.2015, at 05:00, Reimar Döffinger wrote: > On 29.07.2015, at 03:10, Sasi Inguva wrote: >> A little context on why I need this. I use individual stream durations to >> validate , if the outputs after transcoding have the same stream durations. >> For this we ne

Re: [FFmpeg-devel] [PATCH 1/4] blowfish: add av_blowfish_alloc()

2015-07-31 Thread Reimar Döffinger
On 31.07.2015, at 19:18, James Almer wrote: > Signed-off-by: James Almer > --- > doc/APIchanges | 3 +++ > libavutil/blowfish.c | 15 +++ > libavutil/blowfish.h | 10 ++ > libavutil/version.h | 5 - > 4 files changed, 32 insertions(+), 1 deletion(-) > > diff --git a/

Re: [FFmpeg-devel] [libav-devel] [PATCH 0/20] removal of deprecated features

2015-07-31 Thread Reimar Döffinger
On 31.07.2015, at 17:22, Hendrik Leppkes wrote: > On Fri, Jul 31, 2015 at 5:12 PM, compn wrote: >> On Thu, 30 Jul 2015 17:05:12 +0200 >> Andreas Cadhalpun wrote: >> >>> Hi, >>> >>> On 28.07.2015 15:36, Vittorio Giovara wrote: This set contains the removal of all deprecated features marked

Re: [FFmpeg-devel] how to flush audio delayed frames

2015-08-01 Thread Reimar Döffinger
On 30.07.2015, at 13:49, qw wrote: > avcodec_encode_audio() is used to encode audio frame. How to flush audio > delayed frames in audio encoder? This is the wrong list for that kind of question. But look for "flushing" in the avcodec_encode_audio2 documentation. _

Re: [FFmpeg-devel] [libav-devel] [PATCH 0/20] removal of deprecated features

2015-08-01 Thread Reimar Döffinger
On 30.07.2015, at 17:05, Andreas Cadhalpun wrote: > Hi, > > On 28.07.2015 15:36, Vittorio Giovara wrote: >> This set contains the removal of all deprecated features marked as >> such until 2012/early 2013. This was announced several times in the >> past months and agreed at several meetings (si

<    2   3   4   5   6   7   8   >