Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Michael Niedermayer
On Mon, Nov 23, 2015 at 01:57:24PM -0500, Ganesh Ajjanagadde wrote: > On Mon, Nov 23, 2015 at 1:02 PM, Michael Niedermayer wrote: > > On Mon, Nov 23, 2015 at 12:43:52PM -0500, Ganesh Ajjanagadde wrote: > >> On Sun, Nov 22, 2015 at 3:56 PM, Ganesh Ajjanagadde

[FFmpeg-devel] [PATCH] Map description to the (c)inf tag

2015-11-23 Thread Brian Wheeler
Signed-off-by: Brian Wheeler --- libavformat/mov.c| 2 +- libavformat/movenc.c | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 0eb7272..960667f 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Michael Niedermayer
On Mon, Nov 23, 2015 at 12:43:52PM -0500, Ganesh Ajjanagadde wrote: > On Sun, Nov 22, 2015 at 3:56 PM, Ganesh Ajjanagadde wrote: > > On Sun, Nov 22, 2015 at 3:07 PM, Michael Niedermayer > > wrote: > >> On Sun, Nov 22, 2015 at 12:05:49PM -0500, Ganesh

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Nicolas George
Le tridi 3 frimaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > I do recall that, but mandelbrot has issues with such benchmarks due > to variable number of iterations as I observed while playing with it > and as pointed out by Michael. It was hence that I proposed the inner > loop benchmark which

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Ganesh Ajjanagadde
On Mon, Nov 23, 2015 at 1:15 PM, Nicolas George wrote: > Le tridi 3 frimaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >> I do recall that, but mandelbrot has issues with such benchmarks due >> to variable number of iterations as I observed while playing with it >> and as pointed

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Ganesh Ajjanagadde
On Mon, Nov 23, 2015 at 12:54 PM, Nicolas George wrote: > Le tridi 3 frimaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >> It seems like getting rid of the sqrt altogether has a very slight >> positive impact (if any at all). I can post the patch, but would like >> to know what to

Re: [FFmpeg-devel] [PATCH] avformat/mp3dec: prefer "fast_seek" to TOC seek for CBR files.

2015-11-23 Thread Chris Cunningham
Adding everyone back on the cc list for a convo with wm4 (meant for everyone). Expect a new patch shortly to go with this discussion. Also, just a note for posterity and to make sure I haven't missed anything: It seems the mp3 TOC is really not a precise tool for seeking, particularly in large

[FFmpeg-devel] [PATCH] Fix sample_aspect_ratio computation for stereo matroska content.

2015-11-23 Thread Aaron Colwell
matroskaenc.c applies divisors to the display width/height when generating stereo content. This patch adds the corresponding multipliers to matroskadec.c so that the original sample aspect ratio can be recovered. --- libavformat/matroskadec.c | 34 -- 1 file

Re: [FFmpeg-devel] acrossfade duration accuracy

2015-11-23 Thread Paul B Mahol
On 11/23/15, Ryan Williams wrote: > I am getting an unexpected number of samples in the output of the > 'acrossfade' filter. > In the following example I would expect acrossfade.wav to contain 72,000 > samples but it only contains 70,976 samples. > Am I doing something

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-23 Thread Clément Bœsch
On Sun, Nov 22, 2015 at 04:27:02PM +0100, Michael Niedermayer wrote: [...] > > > IIUC src_x/y are only an approximation, as the exact motion vector > > > isnt an integer in fullpel units so it would be mostly cosmetic > > > if x + 4 >> 8, or /8 is used > > > > yes > > > > mpeg used x >> n, i

Re: [FFmpeg-devel] [PATCH] configure: Add option to use gmp

2015-11-23 Thread Matt Oliver
configure: Add user options to enable gcrypt/gmp for rtmp(t)e support. Signed-off-by: Matt Oliver --- configure | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/configure b/configure index 4a28836..fd439ec 100755 --- a/configure +++

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo: use constants for rc_strategy

2015-11-23 Thread Clément Bœsch
On Sat, Nov 21, 2015 at 10:26:17PM +0100, Michael Niedermayer wrote: > On Sat, Nov 21, 2015 at 09:28:31PM +0100, Clément Bœsch wrote: > > --- > > libavcodec/mpegvideo.h | 10 +- > > libavcodec/ratecontrol.c | 10 +++--- > > 2 files changed, 16 insertions(+), 4 deletions(-) > > > >

Re: [FFmpeg-devel] [PATCH] swscale/arm: add ff_nv{12, 21}_to_{argb, rgba, abgr, bgra}_neon

2015-11-23 Thread Clément Bœsch
On Thu, Nov 19, 2015 at 11:48:53AM +0100, Clément Bœsch wrote: > From: Matthieu Bouron > > Signed-off-by: Matthieu Bouron > Signed-off-by: Clément Bœsch > > --- > The function takes about 29ms with a 1080p

[FFmpeg-devel] [PATCH] avfilter/vsrc_mandelbrot: avoid sqrt for epsilon calculation

2015-11-23 Thread Ganesh Ajjanagadde
This rewrites into the mathematically equivalent expression avoiding sqrt, and results in a very minor speedup. Tested on x86-64, Haswell, GNU/Linux. Command: ffmpeg -v error -f lavfi -i mandelbrot -f null - old (draw_mandelbrot): 3982389425 decicycles in draw_mandelbrot, 256 runs, 0

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo_enc: Remove slice structured mode from H.263 as well as the code automatically enabing it

2015-11-23 Thread Michael Niedermayer
On Sun, Nov 22, 2015 at 08:35:12PM +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > There is no such thing as a slice structured mode in the original version 1 > H.263, > that mode was added in H.263+ in 1998. Also the headers for slice structured >

[FFmpeg-devel] [PATCH] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header.

2015-11-23 Thread Alex Agranovsky
This addresses ticket 5023From a2a0b9e0da14b6e82aa783535ec1878c8ffbede0 Mon Sep 17 00:00:00 2001 From: Alex Agranovsky Date: Tue, 24 Nov 2015 00:06:14 -0500 Subject: [PATCH 1/2] Allow mpjpeg demuxer to process MIME parts which do not include Content-Length header. Fixes

Re: [FFmpeg-devel] [PATCH] Fix sample_aspect_ratio computation for stereo matroska content.

2015-11-23 Thread Michael Niedermayer
On Mon, Nov 23, 2015 at 08:37:40PM +, Aaron Colwell wrote: > matroskaenc.c applies divisors to the display width/height when generating > stereo content. This patch adds the corresponding multipliers to > matroskadec.c > so that the original sample aspect ratio can be recovered. > --- >

Re: [FFmpeg-devel] [PATCH]Recognize more extensions to encode raw hevc.

2015-11-23 Thread Michael Niedermayer
On Tue, Nov 24, 2015 at 12:44:34AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch allows more extensions for hevc, as requested on the user > mailing list. > > Please comment, Carl Eugen should be ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH] Map description to the (c)inf tag

2015-11-23 Thread Michael Niedermayer
On Sun, Nov 22, 2015 at 11:51:15PM -0500, Brian Wheeler wrote: > Signed-off-by: Brian Wheeler > --- > libavformat/mov.c| 2 +- > libavformat/movenc.c | 1 + > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index

[FFmpeg-devel] [PATCH] avutil/lls: speed up performance of solve_lls

2015-11-23 Thread Ganesh Ajjanagadde
This is a trivial rewrite of the loops that results in better cache efficiency. Essentially, iterating backwards over an array is a bad idea, since it leads to many cache misses: forward iteration fetches a new cache line that gets subsequently used, while backwards iteration fetches the cache

[FFmpeg-devel] [PATCH]Recognize more extensions to encode raw hevc.

2015-11-23 Thread Carl Eugen Hoyos
Hi! Attached patch allows more extensions for hevc, as requested on the user mailing list. Please comment, Carl Eugen diff --git a/libavformat/rawenc.c b/libavformat/rawenc.c index 7a1a3d0..9474ed1 100644 --- a/libavformat/rawenc.c +++ b/libavformat/rawenc.c @@ -238,7 +238,7 @@ AVOutputFormat

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-23 Thread Michael Niedermayer
On Mon, Nov 23, 2015 at 05:07:30PM +0100, Clément Bœsch wrote: > On Mon, Nov 23, 2015 at 03:54:19PM +0100, Michael Niedermayer wrote: > > On Mon, Nov 23, 2015 at 11:01:02AM +0100, Clément Bœsch wrote: > > > On Sun, Nov 22, 2015 at 04:27:02PM +0100, Michael Niedermayer wrote: > > > [...] > > > > >

[FFmpeg-devel] Test avformat_open_input by command line

2015-11-23 Thread Lucas Andrade
When Kodi tries to play a specific live stream it returns error 403 after some time due to the update of a cookie. I've done a patch to ffmpeg and version 2.8.2 works like a charm with the command line like: ffmpeg -i "fullurl.m3u8" -c copy "teste.ts" But with the new Kodi 12.0 beta that has the

Re: [FFmpeg-devel] [PATCH] avfilter/vsrc_mandelbrot: avoid sqrt for epsilon calculation

2015-11-23 Thread Michael Niedermayer
On Mon, Nov 23, 2015 at 05:19:52PM -0500, Ganesh Ajjanagadde wrote: > This rewrites into the mathematically equivalent expression avoiding sqrt, > and results in a very minor speedup. > > Tested on x86-64, Haswell, GNU/Linux. > Command: > ffmpeg -v error -f lavfi -i mandelbrot -f null - > > old

Re: [FFmpeg-devel] fps filter anomaly

2015-11-23 Thread James Almer
On 11/23/2015 11:36 PM, Ryan Williams wrote: > I am attempting to convert a 0.2fps Slideshow.mov to 25fps using the fps > filter. > The problem is the first output frame pauses for 5 seconds. > When viewing the output video I expect the frame counter to increment at a > rate of 25fps but it

Re: [FFmpeg-devel] [PATCH] ffserver: Fix memory leak and uncheked av_strdup return

2015-11-23 Thread Reynaldo H. Verdejo Pinochet
On 11/22/2015 11:10 AM, wm4 wrote: > [..] > > The function that's called here is generated by: > > MAKE_ACCESSORS(AVStream, stream, char *, recommended_encoder_configuration) > > This just copies the pointer. So I guess freeing it would generate a > dangling pointer? And if the field was

Re: [FFmpeg-devel] fps filter anomaly

2015-11-23 Thread Ryan Williams
Created Ticket #5035. I apologize for not following the correct procedure. It seems the fps filter does not duplicate input frame #0 in order to acheive the desired frame rate. -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of James Almer Sent:

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Michael Niedermayer
On Mon, Nov 23, 2015 at 03:48:40PM -0500, Ganesh Ajjanagadde wrote: > On Mon, Nov 23, 2015 at 2:13 PM, Michael Niedermayer wrote: > > On Mon, Nov 23, 2015 at 01:57:24PM -0500, Ganesh Ajjanagadde wrote: > >> On Mon, Nov 23, 2015 at 1:02 PM, Michael Niedermayer

Re: [FFmpeg-devel] [PATCH 2/3] avformat/rawvideodec: Rework packet size calculation

2015-11-23 Thread Timothy Gu
On Sat, Nov 21, 2015 at 07:23:08PM +0100, Michael Niedermayer wrote: > width and height are unused after this > patch should be fine otherwise Fixed and pushed. Thanks! Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

[FFmpeg-devel] fps filter anomaly

2015-11-23 Thread Ryan Williams
I am attempting to convert a 0.2fps Slideshow.mov to 25fps using the fps filter. The problem is the first output frame pauses for 5 seconds. When viewing the output video I expect the frame counter to increment at a rate of 25fps but it remains on 0 for the first 5 seconds. [root@ryan tmp]# wget

Re: [FFmpeg-devel] [PATCH 3/3] avformat/v210: Check width and height

2015-11-23 Thread Timothy Gu
On Sat, Nov 21, 2015 at 07:20:48PM +0100, Michael Niedermayer wrote: > width and height are unused after this patch Fixed and pushed. Thanks! Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avutil/lls: speed up performance of solve_lls

2015-11-23 Thread Ganesh Ajjanagadde
On Mon, Nov 23, 2015 at 8:00 PM, Ganesh Ajjanagadde wrote: > This is a trivial rewrite of the loops that results in better cache > efficiency. Essentially, iterating backwards over an array is a bad > idea, since it leads to many cache misses: forward iteration fetches a >

Re: [FFmpeg-devel] Get rid of magic number in dsicinvideo.c

2015-11-23 Thread Ganesh Ajjanagadde
On Mon, Nov 23, 2015 at 10:34 AM, Mohammad G wrote: > Greetings, > > File of interest is: libavcodec/dsicinvideo.c > > Line 103: > memcpy(huff_code_table, src, 15); > > Magic number 15 triggers some source code analysis tools to detect it as a > potential issue. > > I'm

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-23 Thread Clément Bœsch
On Mon, Nov 23, 2015 at 03:54:19PM +0100, Michael Niedermayer wrote: > On Mon, Nov 23, 2015 at 11:01:02AM +0100, Clément Bœsch wrote: > > On Sun, Nov 22, 2015 at 04:27:02PM +0100, Michael Niedermayer wrote: > > [...] > > > > > IIUC src_x/y are only an approximation, as the exact motion vector > >

Re: [FFmpeg-devel] [PATCH] avutil/motion_vector: export subpel motion information

2015-11-23 Thread Michael Niedermayer
On Mon, Nov 23, 2015 at 11:01:02AM +0100, Clément Bœsch wrote: > On Sun, Nov 22, 2015 at 04:27:02PM +0100, Michael Niedermayer wrote: > [...] > > > > IIUC src_x/y are only an approximation, as the exact motion vector > > > > isnt an integer in fullpel units so it would be mostly cosmetic > > > >

[FFmpeg-devel] [PATCH] replace magic number with size of array

2015-11-23 Thread webmaster
From: Mohammad Ghasembeigi --- libavcodec/dsicinvideo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/dsicinvideo.c b/libavcodec/dsicinvideo.c index f95cbc7..cb5baf8 100644 --- a/libavcodec/dsicinvideo.c +++ b/libavcodec/dsicinvideo.c @@

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Ganesh Ajjanagadde
On Mon, Nov 23, 2015 at 1:02 PM, Michael Niedermayer wrote: > On Mon, Nov 23, 2015 at 12:43:52PM -0500, Ganesh Ajjanagadde wrote: >> On Sun, Nov 22, 2015 at 3:56 PM, Ganesh Ajjanagadde wrote: >> > On Sun, Nov 22, 2015 at 3:07 PM, Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] replace magic number with size of array

2015-11-23 Thread Ganesh Ajjanagadde
On Mon, Nov 23, 2015 at 12:03 PM, wrote: > From: Mohammad Ghasembeigi > > --- > libavcodec/dsicinvideo.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/dsicinvideo.c b/libavcodec/dsicinvideo.c > index

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Ganesh Ajjanagadde
On Sun, Nov 22, 2015 at 3:56 PM, Ganesh Ajjanagadde wrote: > On Sun, Nov 22, 2015 at 3:07 PM, Michael Niedermayer wrote: >> On Sun, Nov 22, 2015 at 12:05:49PM -0500, Ganesh Ajjanagadde wrote: >>> Signed-off-by: Ganesh Ajjanagadde >>>

Re: [FFmpeg-devel] [PATCH] replace magic number with size of array

2015-11-23 Thread Mohammad G
On Tue, Nov 24, 2015 at 4:10 AM, Ganesh Ajjanagadde wrote: > On Mon, Nov 23, 2015 at 12:03 PM, wrote: > > From: Mohammad Ghasembeigi > > > > --- > > libavcodec/dsicinvideo.c | 2 +- > > 1 file changed, 1 insertion(+), 1

Re: [FFmpeg-devel] FFmpeg 2.7.3 and 2.6.5, 2.5.9 and 2.4.12

2015-11-23 Thread Michael Niedermayer
On Thu, Nov 12, 2015 at 02:55:50AM +0100, Michael Niedermayer wrote: > Hi > > I intend to make 2.7.3 and 2.6.5 soon (day(s)), if you want something > backported do it before i make the release > (or if its something critical tell me and i can wait longer of course) releases made Next is 2.4.12

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Nicolas George
Le tridi 3 frimaire, an CCXXIV, Ganesh Ajjanagadde a écrit : > It seems like getting rid of the sqrt altogether has a very slight > positive impact (if any at all). I can post the patch, but would like > to know what to benchmark. There are numerous choices, e.g > draw_mandelbrot as a whole, the

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Ganesh Ajjanagadde
On Mon, Nov 23, 2015 at 2:13 PM, Michael Niedermayer wrote: > On Mon, Nov 23, 2015 at 01:57:24PM -0500, Ganesh Ajjanagadde wrote: >> On Mon, Nov 23, 2015 at 1:02 PM, Michael Niedermayer >> wrote: >> > On Mon, Nov 23, 2015 at 12:43:52PM -0500, Ganesh

Re: [FFmpeg-devel] [PATCH 09/10] avfilter/vsrc_mandelbrot: use hypot()

2015-11-23 Thread Michael Niedermayer
On Mon, Nov 23, 2015 at 03:48:40PM -0500, Ganesh Ajjanagadde wrote: > On Mon, Nov 23, 2015 at 2:13 PM, Michael Niedermayer wrote: > > On Mon, Nov 23, 2015 at 01:57:24PM -0500, Ganesh Ajjanagadde wrote: > >> On Mon, Nov 23, 2015 at 1:02 PM, Michael Niedermayer