Re: [FFmpeg-devel] [PATCH 2/4] dpxenc: enforce alignment requirement

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 4:30 GMT+02:00 Michael Niedermayer michae...@gmx.at: probably ok applied Now that I think of it, this change causes older versions of ffmpeg to be unable to decode the output of the fixed encoder. So: - Should we bump eg a minor version number to let any decoder have the

Re: [FFmpeg-devel] [PATCH 1/4] dpx: use aligned line starts

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 4:17 GMT+02:00 Michael Niedermayer michae...@gmx.at: see http://samples.ffmpeg.org/image-samples/dpx_samples.zip Note that the date metadata in that file does not have a valid format either. -- Christophe ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH 4/4] dpx: fix endianess for RGB 8bits

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 5:01 GMT+02:00 Michael Niedermayer michae...@gmx.at: On Wed, Aug 13, 2014 at 10:21:54AM +, Christophe Gisquet wrote: case 50081: +avctx-pix_fmt = AV_PIX_FMT_BGR24; +break; this possibly breaks decoding of

Re: [FFmpeg-devel] [PATCH 3/4] proresenc_kostya: realloc if buffer too small

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-12 12:44 GMT+02:00 Christophe Gisquet christophe.gisq...@gmail.com: Forgot a parameter to the call to avpriv_request_sample, will be added later. If there's no further comment on this option for the reallocation, here's an updated patch. -- Christophe From

Re: [FFmpeg-devel] Invitation to VDD and registration

2014-08-14 Thread Thilo Borgmann
Am 14.08.14 01:56, schrieb compn: On Wed, 13 Aug 2014 18:48:38 +0200 Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 13.08.2014, at 18:42, Stefano Sabatini stefa...@gmail.com wrote: On date Sunday 2014-07-20 18:16:52 +0200, Jean-Baptiste Kempf encoded: My dear friends of the FFmpeg

Re: [FFmpeg-devel] [PATCH] doc/filters: correct confusing statements about *showinfo shown values

2014-08-14 Thread Stefano Sabatini
On date Wednesday 2014-08-13 08:59:58 -0700, Timothy Gu encoded: On Aug 13, 2014 8:58 AM, Stefano Sabatini stefa...@gmail.com wrote: Fix trac issue #3850. --- OK. Pushed (yesterday), thanks. -- FFmpeg = Fantastic Frightening Merciless Perfectionist Epic Gymnast

Re: [FFmpeg-devel] [PATCH] ffserver: deprecate Port and BindAddress options in favor of HTTPPort and HTTPBindAddress

2014-08-14 Thread Stefano Sabatini
On date Saturday 2014-08-02 16:40:53 +0200, Stefano Sabatini encoded: The new option names are more explicit. --- doc/ffserver.conf | 4 ++-- doc/ffserver.texi | 19 ++- ffserver.c| 10 +++--- 3 files changed, 23 insertions(+), 10 deletions(-) Ping (Reynaldo?)

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-14 Thread Thomas Weber
On Wed, Aug 13, 2014 at 12:53:41AM +0100, Kieran Kunhya wrote: Also ive offered my resignation in the past. I do still offer to resign from the FFmpeg leader position, if it resolves this split between FFmpeg and Libav and make everyone work together again. I never understood why people

Re: [FFmpeg-devel] [PATCH] ffserver: deprecate Port and BindAddress options in favor of HTTPPort and HTTPBindAddress

2014-08-14 Thread Reynaldo H. Verdejo Pinochet
On 08/02/2014 10:40 AM, Stefano Sabatini wrote: The new option names are more explicit. --- [..] Looks OK Bests, -- Reynaldo H. Verdejo Pinochet Sr. Multimedia Engineer, Open Source Group Samsung Research America - Silicon Valley ___ ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 1/2] lavfi/avf_showspectrum: set output frame rate.

2014-08-14 Thread Nicolas George
Le septidi 27 thermidor, an CCXXII, Clément Bœsch a écrit : contet? Thanks for spotting it. The series to merge is now: a3aaaec lavfi/avf_showspectrum: set output frame rate. 65b284a lavfi/avf_showspectrum: fix output pts computation. d4de6d4 lavfi/avf_showspectrum: do not push the frame at

Re: [FFmpeg-devel] [PATCH 4/4] proresenc_kostya: properly account for alpha

2014-08-14 Thread Michael Niedermayer
On Tue, Aug 12, 2014 at 09:08:37AM +0200, Christophe Gisquet wrote: Hi, 2014-08-12 8:10 GMT+02:00 Christophe Gisquet christophe.gisq...@gmail.com: Yeah, I'll drop it. Done. Kostya suggested another rounding (the slice bitstreams are byte-aligned), integrated in this patch. --

Re: [FFmpeg-devel] [PATCH] libavcodec: bump micro to reflect dpx changes

2014-08-14 Thread Michael Niedermayer
On Thu, Aug 14, 2014 at 03:08:39PM +0200, Christophe Gisquet wrote: Hi, 2014-08-14 14:32 GMT+02:00 Michael Niedermayer michae...@gmx.at: is this the patch that you wanted to send ? it changes dpx.c instead of version.h Woops, attached the wrong 0001 one... -- Christophe

[FFmpeg-devel] [PATCH 1/2] lavc/flacdsp: document limitations of the LPC encoder

2014-08-14 Thread James Darnley
State that the maximum value of order is 32. This limit is used in both C and x86 assebly code. --- libavcodec/flacdsp.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/libavcodec/flacdsp.h b/libavcodec/flacdsp.h index 272cf2a..e8487c8 100644 ---

[FFmpeg-devel] [PATCH] fate: test flac LPC encoder

2014-08-14 Thread James Darnley
The LPC encoder is not used at compression levels below 3 but the basic acodec test uses -compression_level 2. This adds a test using -compression_level 5, which is the current default. --- tests/fate/acodec.mak|4 +++- tests/ref/acodec/flac-comp-5 |4 2 files changed, 7

[FFmpeg-devel] [PATCH 2/2] lavc/flacdsp: change lpc_encoder function pointer prototype

2014-08-14 Thread James Darnley
This should help to clarify the API. --- libavcodec/flacdsp.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavcodec/flacdsp.h b/libavcodec/flacdsp.h index e8487c8..23d1ae4 100644 --- a/libavcodec/flacdsp.h +++ b/libavcodec/flacdsp.h @@ -35,7 +35,7 @@ typedef struct

Re: [FFmpeg-devel] [PATCH 1/2] lavc/flacdsp: document limitations of the LPC encoder

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 17:14 GMT+02:00 James Darnley james.darn...@gmail.com: + * This limit is used: + * - when CONFIG_SMALL is 0 to unroll a loop in the C template. + * - when SSE4 (or newer) is available on x86 to unroll a copy loop. Does the implementer of a new dsp version really has

Re: [FFmpeg-devel] [PATCH 2/2] lavc/flacdsp: change lpc_encoder function pointer prototype

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 17:14 GMT+02:00 James Darnley james.darn...@gmail.com: This should help to clarify the API. OK. -- Christophe ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] fate: test flac LPC encoder

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 17:15 GMT+02:00 James Darnley james.darn...@gmail.com: The LPC encoder is not used at compression levels below 3 but the basic acodec test uses -compression_level 2. This adds a test using -compression_level 5, which is the current default. What order is used in that case? It

Re: [FFmpeg-devel] [PATCH] fate: test flac LPC encoder

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 18:08 GMT+02:00 James Darnley james.darn...@gmail.com: Minimum 1, maximum 8. Should I find out what order(s) are actually chosen? The encoder algorithm might change and make that information useless, but that could be useful. I fear that only checking which orders it *tests* is

Re: [FFmpeg-devel] [PATCH 2/3] build: rely on pkg-config for libx264 probing

2014-08-14 Thread Carl Eugen Hoyos
Clément Bœsch u at pkh.me writes: Carl, do you want the fallback to work just when pkg-config is not available I believe this would still leave some use-cases unsolved. or do you actually want a real fallback when it is present but didn't succeed? This is the preferred solution imo.

Re: [FFmpeg-devel] [PATCH 2/2] lavc/flacdsp: change lpc_encoder function pointer prototype

2014-08-14 Thread Michael Niedermayer
On Thu, Aug 14, 2014 at 06:01:49PM +0200, Christophe Gisquet wrote: Hi, 2014-08-14 17:14 GMT+02:00 James Darnley james.darn...@gmail.com: This should help to clarify the API. OK. applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB DNS

Re: [FFmpeg-devel] [PATCH] fate: test flac LPC encoder

2014-08-14 Thread James Darnley
After further searching and understanding the makefile I see that the lpc code is tested by the complete fate sample suite. This patch is unnecessary. Anyway to answer the questions... On 2014-08-14 18:14, Christophe Gisquet wrote: Hi, 2014-08-14 18:08 GMT+02:00 James Darnley

[FFmpeg-devel] [PATCH] libavformat: Add poster_time and time_scale to quicktime metadata output

2014-08-14 Thread Graham Torn
From: gt-sdi gt...@straylightdigital.com Extract poster_time and time_scale from quicktime mdhd atom and add to metadata for output by ffprobe. Signed-off-by: gt-sdi gt...@straylightdigital.com --- libavformat/mov.c | 27 +-- 1 file changed, 25 insertions(+), 2

Re: [FFmpeg-devel] [PATCH v2] added ULs for demuxing avid media composer mxf files

2014-08-14 Thread Tomas Härdin
On Tue, 2014-08-12 at 14:58 -0700, Mark Reid wrote: UL values copied from FMbc version of mxf.c Fixes Ticket#1554, Ticket#3100 and Ticket#3450 --- libavformat/mxf.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/mxf.c b/libavformat/mxf.c index d20ed94..8ebfdf2 100644

[FFmpeg-devel] [PATCH 1/5] avutil/pixelutils: add av_pixelutils_bdiff()

2014-08-14 Thread Clément Bœsch
--- doc/APIchanges | 3 +++ libavutil/pixelutils.c | 21 + libavutil/pixelutils.h | 23 +++ libavutil/version.h| 2 +- 4 files changed, 48 insertions(+), 1 deletion(-) diff --git a/doc/APIchanges b/doc/APIchanges index 067f60f..72cfe96 100644

[FFmpeg-devel] [PATCH 2/5] avfilter/select: use av_pixelutils_bdiff()

2014-08-14 Thread Clément Bœsch
--- libavfilter/f_select.c | 20 +--- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git a/libavfilter/f_select.c b/libavfilter/f_select.c index 546a940..e1837db 100644 --- a/libavfilter/f_select.c +++ b/libavfilter/f_select.c @@ -251,22 +251,12 @@ static double

[FFmpeg-devel] [PATCH 3/5] avutil/pixelutils: faster pixelutils_sad_[au]_16x16

2014-08-14 Thread Clément Bœsch
~560 → ~500 decicycles This is following the comments from Michael in https://ffmpeg.org/pipermail/ffmpeg-devel/2014-August/160599.html Using 2 registers for accumulator didn't help. On the other hand, some re-ordering between the movs and psadbw allowed going ~538 to ~500. ---

[FFmpeg-devel] [PATCH 4/5] avfilter/select: use SAD 16x16 when w/h are multiples of 16

2014-08-14 Thread Clément Bœsch
Note: this isn't actually faster. pixelutils_sad_a_16x16_sse2() is ~500 decicyles. pixelutils_sad_8x8_mmxext() is ~240 decicyles. Still, the overall decode is bigger when using the former. I don't get it. --- libavfilter/f_select.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-)

[FFmpeg-devel] [PATCH 5/5] avfilter/fieldmatch: use av_pixelutils_bdiff()

2014-08-14 Thread Clément Bœsch
Not that much relevant from a performance PoV because not called often. --- libavfilter/vf_fieldmatch.c | 42 ++ 1 file changed, 22 insertions(+), 20 deletions(-) diff --git a/libavfilter/vf_fieldmatch.c b/libavfilter/vf_fieldmatch.c index e2aa60b..4935a30

Re: [FFmpeg-devel] [PATCH v2] added ULs for demuxing avid media composer mxf files

2014-08-14 Thread Michael Niedermayer
On Tue, Aug 12, 2014 at 02:58:04PM -0700, Mark Reid wrote: UL values copied from FMbc version of mxf.c Fixes Ticket#1554, Ticket#3100 and Ticket#3450 --- libavformat/mxf.c | 3 +++ 1 file changed, 3 insertions(+) missing signed-off [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 1/5] avutil/pixelutils: add av_pixelutils_bdiff()

2014-08-14 Thread Michael Niedermayer
On Thu, Aug 14, 2014 at 11:05:11PM +0200, Clément Bœsch wrote: --- doc/APIchanges | 3 +++ libavutil/pixelutils.c | 21 + libavutil/pixelutils.h | 23 +++ libavutil/version.h| 2 +- 4 files changed, 48 insertions(+), 1 deletion(-)

Re: [FFmpeg-devel] [RFC] [PATCH] ffmpeg: fix streamcopy with side data

2014-08-14 Thread Michael Niedermayer
On Thu, Aug 14, 2014 at 12:24:47PM +0200, Christophe Gisquet wrote: Hi, the attached patch seems to fix ticket #3773, but streamcopy is such a core feature of ffmpeg that I assume I'm missing a use case or incorrectly handle the buffers referencing. Another note is that this patch was