Re: [FFmpeg-devel] [PATCH 3/3] ffmpeg: try to guess a good value for the thread message queue.

2015-02-19 Thread Nicolas George
Le nonidi 29 pluviôse, an CCXXIII, Michael Niedermayer a écrit : iam a bit concerned about the possibility of this unneccesarily allocating a million packets i think IIUC this amount will actually be alloated no matter if its needed or not Yes, that will allocate all the FIFO, and with 120

Re: [FFmpeg-devel] [PATCH] avformat: Add simple ACLR atom reading to set the color range

2015-02-19 Thread Kevin Wheatley
V3 of the patch, I've attempted to include the general comments from the thread. New to this version, I've reworked the function that reads the atom into the extradata into one which calls 2 helper functions (one to realloc, one to read), I've then reused these functions to read the ACLR atom

Re: [FFmpeg-devel] [PATCH] doc/filters: simplify lenscorrection option descriptions

2015-02-19 Thread Michael Niedermayer
On Mon, Feb 09, 2015 at 11:04:47AM -0900, Lou Logan wrote: Also mention option defaults. Also fix incorrect k1 and k2 no correction values. Signed-off-by: Lou Logan l...@lrcd.com --- doc/filters.texi | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH 2/2] doc/faq: explain DAR/SAR preserving.

2015-02-19 Thread Nicolas George
Le decadi 30 pluviôse, an CCXXIII, Michael Niedermayer a écrit : this x looks strange, and we had problems with these in the past and removed them: 1afbb4c226203c1a0853c9ad469cd9ef337080e4 The first related commit stated: # Non-ascii chars are not correctly rendered in both MAN and HTML output

Re: [FFmpeg-devel] [PATCH 1/3] lavd/alsa: set frame_size field.

2015-02-19 Thread Nicolas George
Le primidi 1er ventôse, an CCXXIII, Michael Niedermayer a écrit : doesnt this also set it for the muxer ? otherwise it is probably ok but you know this code better than i do ... True, I had not thought long enough to see that it could cause problems. I have moved that to the demuxer, I will

Re: [FFmpeg-devel] [PATCH] lavfi: Add support to process_command in vf_eq.c

2015-02-19 Thread arwa arif
Updated the patch. From 66a8c9d03995c9e7c6ccc05fb9b20756f51c17f4 Mon Sep 17 00:00:00 2001 From: Arwa Arif arwaarif1...@gmail.com Date: Thu, 19 Feb 2015 01:26:44 +0530 Subject: [PATCH] Add process_command to eq. --- doc/filters.texi| 35 +++ libavfilter/vf_eq.c | 171

Re: [FFmpeg-devel] trac sending emails?

2015-02-19 Thread Roger Pack
On 2/9/15, wm4 nfx...@googlemail.com wrote: On Mon, 9 Feb 2015 14:30:27 -0700 Roger Pack rogerdpa...@gmail.com wrote: Hello. I seem to have noticed that on ffmpeg trac's that I have commented on, if there's a later updated, I don't get any email notification of such. Is this expected [ I

[FFmpeg-devel] [PATCH] Add coverity model file

2015-02-19 Thread Michael Niedermayer
The comments/header of the file are taken from qemu, they provide some basic documentation The code from the examples Ive no means to test this except uploading to coverity for FFmpeg, so each commit should stay simple, making it easy to revert. Also please help making this a useful and

Re: [FFmpeg-devel] [PATCH] Add coverity model file

2015-02-19 Thread Timothy Gu
On Thu Feb 19 2015 at 4:43:30 AM Michael Niedermayer michae...@gmx.at wrote: tools/model.c | 42 ++ 1 file changed, 42 insertions(+) create mode 100644 tools/model.c tools/coverity.c sounds better to me. Timothy

Re: [FFmpeg-devel] [PATCH 2/2] doc/faq: explain DAR/SAR preserving.

2015-02-19 Thread Michael Niedermayer
On Thu, Feb 19, 2015 at 11:43:02AM +0100, Nicolas George wrote: Le decadi 30 pluviôse, an CCXXIII, Michael Niedermayer a écrit : this x looks strange, and we had problems with these in the past and removed them: 1afbb4c226203c1a0853c9ad469cd9ef337080e4 The first related commit stated: #

Re: [FFmpeg-devel] [PATCH]Simplify mlp_filter_channel_x86()

2015-02-19 Thread Michael Niedermayer
On Thu, Feb 19, 2015 at 05:13:19PM +, Carl Eugen Hoyos wrote: Hendrik Leppkes h.leppkes at gmail.com writes: Attached patch by Francisco Blas Izquierdo Riera fixes a compilation error in mlpdsp_init.c with -fstack-check and some gcc compilers (I reproduced the issue with gcc

Re: [FFmpeg-devel] [PATCH 2/2] doc/faq: explain DAR/SAR preserving.

2015-02-19 Thread Nicolas George
Le primidi 1er ventôse, an CCXXIII, Michael Niedermayer a écrit : merged Thanks. The web page was regenerated a moment ago and the × looks fine. OTOH, FATE shows: # MAN doc/ffprobe.1 # doc/ffmpeg.pod:1: Unknown command paragraph =encoding utf8 # doc/ffprobe.pod:1: Unknown command paragraph

[FFmpeg-devel] Gaining access to a encoder context in avformat?

2015-02-19 Thread Kevin Wheatley
Hi, I realise this potentially breaks through lots of API boundaries/good taste, but I've noticed a case where the movenc.c code should really ask the specific encoder context (DNXHDEncContext) for some of it's fields to ensure that the atoms written have a consistent setting with what the codec

[FFmpeg-devel] [PATCH] lavf/ffmenc: do not fail on missing codec

2015-02-19 Thread Lukasz Marek
ffm encoder fails when codec is not found. It may happen when stream is being copied. This commit allows to store such stream and provides backward compatibility with version prior 2.5 release. fixes #4266 Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavformat/ffmenc.c | 13

Re: [FFmpeg-devel] [PATCH] lavf/ffmenc: do not fail on missing codec

2015-02-19 Thread Michael Niedermayer
On Thu, Feb 19, 2015 at 11:52:50PM +0100, Lukasz Marek wrote: ffm encoder fails when codec is not found. It may happen when stream is being copied. This commit allows to store such stream and provides backward compatibility with version prior 2.5 release. fixes #4266 Signed-off-by:

Re: [FFmpeg-devel] [PATCH]Simplify mlp_filter_channel_x86()

2015-02-19 Thread Hendrik Leppkes
On Thu, Feb 19, 2015 at 4:25 PM, Carl Eugen Hoyos ceho...@ag.or.at wrote: Hi! Attached patch by Francisco Blas Izquierdo Riera fixes a compilation error in mlpdsp_init.c with -fstack-check and some gcc compilers (I reproduced the issue with gcc 4.7.3) by simplifying the code. I ran some

Re: [FFmpeg-devel] [PATCH]Simplify mlp_filter_channel_x86()

2015-02-19 Thread Carl Eugen Hoyos
Hendrik Leppkes h.leppkes at gmail.com writes: Attached patch by Francisco Blas Izquierdo Riera fixes a compilation error in mlpdsp_init.c with -fstack-check and some gcc compilers (I reproduced the issue with gcc 4.7.3) by simplifying the code. I ran some benchmarks with a very

Re: [FFmpeg-devel] trac sending emails?

2015-02-19 Thread Michael Niedermayer
On Thu, Feb 19, 2015 at 06:36:17AM -0700, Roger Pack wrote: On 2/9/15, wm4 nfx...@googlemail.com wrote: On Mon, 9 Feb 2015 14:30:27 -0700 Roger Pack rogerdpa...@gmail.com wrote: Hello. I seem to have noticed that on ffmpeg trac's that I have commented on, if there's a later updated, I

Re: [FFmpeg-devel] [PATCH 3/3] ffmpeg: try to guess a good value for the thread message queue.

2015-02-19 Thread Michael Niedermayer
On Thu, Feb 19, 2015 at 12:25:44PM +0100, Nicolas George wrote: Le nonidi 29 pluviôse, an CCXXIII, Michael Niedermayer a écrit : iam a bit concerned about the possibility of this unneccesarily allocating a million packets i think IIUC this amount will actually be alloated no matter if its

Re: [FFmpeg-devel] [PATCH] lavfi: Add support to process_command in vf_eq.c

2015-02-19 Thread Michael Niedermayer
On Thu, Feb 19, 2015 at 05:13:15PM +0530, arwa arif wrote: Updated the patch. doc/filters.texi| 35 ++ libavfilter/vf_eq.c | 171 ++-- libavfilter/vf_eq.h | 56 +++-- 3 files changed, 210 insertions(+), 52

Re: [FFmpeg-devel] [PATCH] lavfi: Add support to process_command in vf_eq.c

2015-02-19 Thread Stefano Sabatini
On date Thursday 2015-02-19 17:13:15 +0530, Arwa Arif encoded: Updated the patch. From 66a8c9d03995c9e7c6ccc05fb9b20756f51c17f4 Mon Sep 17 00:00:00 2001 From: Arwa Arif arwaarif1...@gmail.com Date: Thu, 19 Feb 2015 01:26:44 +0530 Subject: [PATCH] Add process_command to eq. ---