Re: [FFmpeg-devel] [PATCH] lavc/dnxhd: ff_dnxhd_cid_table is not exported

2014-08-13 Thread Hendrik Leppkes
On Wed, Aug 13, 2014 at 5:34 AM, James Almer jamr...@gmail.com wrote: Signed-off-by: James Almer jamr...@gmail.com --- libavcodec/dnxhddata.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/dnxhddata.h b/libavcodec/dnxhddata.h index 8166ee8..d5629e2 100644

Re: [FFmpeg-devel] [PATCH] vp9: ignore reference segmentation map if error_resilience flag is set.

2014-08-13 Thread Hendrik Leppkes
On Wed, Aug 13, 2014 at 12:11 AM, Ronald S. Bultje rsbul...@gmail.com wrote: Fixes ffvp9_fails_where_libvpx.succeeds.webm from ticket 3849. --- libavcodec/vp9.c | 26 +++--- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/libavcodec/vp9.c

[FFmpeg-devel] [PATCH] dpx: use aligned line starts

2014-08-13 Thread Christophe Gisquet
Hi, the attached patch improves the decoding of ticket #3692 by following S268M-2003. Unfortunately, that probably means ffmpeg's dpx encoder is not compliant to these specifications. -- Christophe From 4ba2ec15c9a111fb4e20523d5a8fa337bedee92d Mon Sep 17 00:00:00 2001 From: Christophe Gisquet

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

2014-08-13 Thread Christophe Gisquet
2014-08-13 9:48 GMT+02:00 Christophe Gisquet christophe.gisq...@gmail.com: Unfortunately, that probably means ffmpeg's dpx encoder is not compliant to these specifications. And indeed the decoder is then no longer able to decode a file produced by the encoder when number of bytes per line is

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

2014-08-13 Thread James Darnley
On 2014-08-13 04:50, Michael Niedermayer wrote: On Tue, Aug 12, 2014 at 11:22:07PM +0200, James Darnley wrote: --- libavcodec/flacdsp.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/libavcodec/flacdsp.h b/libavcodec/flacdsp.h index 272cf2a..36cd904 100644

Re: [FFmpeg-devel] [PATCH] avfitler/vf_perspective: support slice threading

2014-08-13 Thread Clément Bœsch
On Tue, Aug 12, 2014 at 05:12:43PM +0200, Paul B Mahol wrote: On 8/12/14, Clement Boesch u...@pkh.me wrote: On Tue, Aug 12, 2014 at 11:32:20AM +, Paul B Mahol wrote: Signed-off-by: Paul B Mahol one...@gmail.com --- libavfilter/vf_perspective.c | 69

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

2014-08-13 Thread Christophe Gisquet
SMPTE 268M-2003 specifies that each line starts at a 4-bytes boundary. Therefore, modify correspondingly the input buffer strides and size. Partially fixes ticket #3692: DLAD_8b_3c_big.dpx still has inverted colors, which might be related to endianness. --- libavcodec/dpx.c | 34

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

2014-08-13 Thread Christophe Gisquet
S268M-2003 specifies that each line start is aligned on a 4-byte boundary. --- libavcodec/dpxenc.c | 44 +--- 1 file changed, 37 insertions(+), 7 deletions(-) diff --git a/libavcodec/dpxenc.c b/libavcodec/dpxenc.c index 0eb1297..059d8c6 100644 ---

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

2014-08-13 Thread Christophe Gisquet
Fixes DLAD_8b_3c_big.dpx from ticket #3692 --- libavcodec/dpx.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index 2ad7527..d4d6833 100644 --- a/libavcodec/dpx.c +++ b/libavcodec/dpx.c @@ -232,6 +232,8 @@ static int decode_frame(AVCodecContext *avctx,

[FFmpeg-devel] [PATCH 3/4] dpx: abort if encrypted

2014-08-13 Thread Christophe Gisquet
--- libavcodec/dpx.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index 8cd7d73..2ad7527 100644 --- a/libavcodec/dpx.c +++ b/libavcodec/dpx.c @@ -108,6 +108,15 @@ static int decode_frame(AVCodecContext *avctx, av_log(avctx,

Re: [FFmpeg-devel] [PATCH] vp9: ignore reference segmentation map if error_resilience flag is set.

2014-08-13 Thread Ronald S. Bultje
Hi, On Wed, Aug 13, 2014 at 2:25 AM, Hendrik Leppkes h.lepp...@gmail.com wrote: On Wed, Aug 13, 2014 at 12:11 AM, Ronald S. Bultje rsbul...@gmail.com wrote: Fixes ffvp9_fails_where_libvpx.succeeds.webm from ticket 3849. --- libavcodec/vp9.c | 26 +++--- 1 file

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

2014-08-13 Thread Michael Niedermayer
On Tue, Aug 12, 2014 at 07:15:11PM -0700, Mark Reid wrote: On Tue, Aug 12, 2014 at 6:44 PM, Timothy Gu timothyg...@gmail.com wrote: On Aug 12, 2014 6:29 PM, Mark Reid mindm...@gmail.com wrote: I didn't realize FFmbc was licensed differently :s. I don't know if it helps, (IANAL either)

Re: [FFmpeg-devel] [PATCH 3/4] dpx: abort if encrypted

2014-08-13 Thread Paul B Mahol
On 8/13/14, Christophe Gisquet christophe.gisq...@gmail.com wrote: --- libavcodec/dpx.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index 8cd7d73..2ad7527 100644 --- a/libavcodec/dpx.c +++ b/libavcodec/dpx.c @@ -108,6 +108,15 @@ static

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

2014-08-13 Thread Paul B Mahol
On 8/13/14, Christophe Gisquet christophe.gisq...@gmail.com wrote: S268M-2003 specifies that each line start is aligned on a 4-byte boundary. --- libavcodec/dpxenc.c | 44 +--- 1 file changed, 37 insertions(+), 7 deletions(-) probably ok

Re: [FFmpeg-devel] [PATCH 3/4] dict.c: Free non-strduped av_dict_set arguments on error.

2014-08-13 Thread Michael Niedermayer
On Mon, Aug 11, 2014 at 09:17:18PM +0200, Reimar Döffinger wrote: Unfortunately this was not explicitly documented and thus might be very risky. yes maybe a AV_DICT_FREE_ARGS_ON_ERROR could be used, but its not as convenient as getting the behavior by default [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 4/4] dict.c: empty dictionaries should be a NULL pointer.

2014-08-13 Thread Michael Niedermayer
On Mon, Aug 11, 2014 at 09:17:19PM +0200, Reimar Döffinger wrote: Ensure this is even the case if they are empty because we failed adding the first entry. Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- libavutil/dict.c | 4 1 file changed, 4 insertions(+) LGTM [...]

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

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 10:46:45AM +0200, James Darnley wrote: On 2014-08-13 04:50, Michael Niedermayer wrote: On Tue, Aug 12, 2014 at 11:22:07PM +0200, James Darnley wrote: --- libavcodec/flacdsp.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git

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

2014-08-13 Thread James Darnley
On 2014-08-13 14:05, Michael Niedermayer wrote: On Wed, Aug 13, 2014 at 10:46:45AM +0200, James Darnley wrote: On 2014-08-13 04:50, Michael Niedermayer wrote: On Tue, Aug 12, 2014 at 11:22:07PM +0200, James Darnley wrote: --- libavcodec/flacdsp.h |7 +++ 1 files changed, 7

Re: [FFmpeg-devel] [PATCH] avfilter/dctdnoiz: add slice threading

2014-08-13 Thread Michael Niedermayer
On Sun, Aug 10, 2014 at 10:09:02PM +0200, Clément Bœsch wrote: --- I have one or two little things to cleanup after this but I think I'm mostly done. The filter is now usable from a performance point of view (not for real-time, but it doesn't decades to process a normal sized image anymore):

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-13 Thread Attila Kinali
On Wed, 13 Aug 2014 00:30:05 +0200 Michael Niedermayer michae...@gmx.at wrote: I never understood why people who once where friends became mutually so hostile You should know that better than anyone else! You still claim to be my friend, yet you said and did things that i have not seen from

[FFmpeg-devel] [PATCH] libavdevice/v4l2: fix of crash caused by assert

2014-08-13 Thread Dmitry Volyntsev
From: Dmitry Volyntsev xeioexcept...@gmail.com s-buffers_queued constantly decremented and not incremented in case of (s-frame_size 0 buf.bytesused != s-frame_size) condition (caught on long run capture of Logitech C310) --- libavdevice/v4l2.c |7 --- 1 file changed, 4 insertions(+), 3

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

2014-08-13 Thread Stefano Sabatini
Fix trac issue #3850. --- doc/filters.texi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index c5caa77..0e3dfc7 100644 --- a/doc/filters.texi +++ b/doc/filters.texi @@ -862,7 +862,7 @@ The input audio is not modified. The shown line

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

2014-08-13 Thread Timothy Gu
On Aug 13, 2014 8:58 AM, Stefano Sabatini stefa...@gmail.com wrote: Fix trac issue #3850. --- OK. [...] Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2014-08-13 Thread Stefano Sabatini
On date Sunday 2014-07-20 18:16:52 +0200, Jean-Baptiste Kempf encoded: My dear friends of the FFmpeg community, I'd like to invite you to the VideoLAN Dev Days 2014, the 3rd week-end of September, in Dublin, Ireland. Google is providing the hosting. This technical conference about open

Re: [FFmpeg-devel] [PATCH 3/4] dpx: abort if encrypted

2014-08-13 Thread Reimar Döffinger
On 13.08.2014, at 13:48, Paul B Mahol one...@gmail.com wrote: On 8/13/14, Christophe Gisquet christophe.gisq...@gmail.com wrote: --- libavcodec/dpx.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index 8cd7d73..2ad7527 100644 ---

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

2014-08-13 Thread Reimar Döffinger
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 community, I'd like to invite you to the VideoLAN Dev Days 2014, the 3rd week-end of September, in Dublin, Ireland. Google

Re: [FFmpeg-devel] [PATCH] vp9: ignore reference segmentation map if error_resilience flag is set.

2014-08-13 Thread Reimar Döffinger
On 13.08.2014, at 12:48, Ronald S. Bultje rsbul...@gmail.com wrote: Might it be useful to add some of those special samples to fate? Probably. The first 10 frames of the sample in this ticket would show this already, is anyone good with webm file cutting/remuxing? If not, I can try to

Re: [FFmpeg-devel] [PATCH] vp9: ignore reference segmentation map if error_resilience flag is set.

2014-08-13 Thread James Almer
On 13/08/14 1:52 PM, Reimar Döffinger wrote: On 13.08.2014, at 12:48, Ronald S. Bultje rsbul...@gmail.com wrote: Might it be useful to add some of those special samples to fate? Probably. The first 10 frames of the sample in this ticket would show this already, is anyone good with webm

Re: [FFmpeg-devel] [PATCH 3/4] dpx: abort if encrypted

2014-08-13 Thread Reimar Döffinger
On Wed, Aug 13, 2014 at 06:51:00PM +0200, Christophe Gisquet wrote: Hi, 2014-08-13 18:46 GMT+02:00 Reimar Döffinger reimar.doeffin...@gmx.de: More specifically: how certain is it this value is always correct? Failing to play an unencrypted file just because of a wrong flag (or even a

Re: [FFmpeg-devel] [PATCH] avfilter/dctdnoiz: add slice threading

2014-08-13 Thread Clément Bœsch
On Wed, Aug 13, 2014 at 04:02:57PM +0200, Michael Niedermayer wrote: On Sun, Aug 10, 2014 at 10:09:02PM +0200, Clément Bœsch wrote: --- I have one or two little things to cleanup after this but I think I'm mostly done. The filter is now usable from a performance point of view (not for

Re: [FFmpeg-devel] [RFC][PATCH] Allow include files to be installed into subdirectory

2014-08-13 Thread Ivan Kalvachev
On 8/12/14, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Hi, On 12.08.2014 02:21, Ivan Kalvachev wrote: On 8/11/14, Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: Assuming it would be possible to install development packages for both at the same time, which one should

Re: [FFmpeg-devel] [RFC][PATCH] Allow include files to be installed into subdirectory

2014-08-13 Thread Reimar Döffinger
On 12.08.2014, at 02:21, Ivan Kalvachev ikalvac...@gmail.com wrote: It would be decided by the gcc -I inclusion option. It modifies the search paths for header files, so that it checks these paths before the system/default ones (When using `#include libavcodec/avcodec.h `) Libav headers are

Re: [FFmpeg-devel] [PATCH] vp9: ignore reference segmentation map if error_resilience flag is set.

2014-08-13 Thread Michael Niedermayer
On Tue, Aug 12, 2014 at 06:11:05PM -0400, Ronald S. Bultje wrote: Fixes ffvp9_fails_where_libvpx.succeeds.webm from ticket 3849. --- libavcodec/vp9.c | 26 +++--- 1 file changed, 15 insertions(+), 11 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] reset and complete RELEASE_NOTES for the next version

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 08:51:05PM +0200, Clément Bœsch wrote: --- RELEASE_NOTES | 162 +++--- 1 file changed, 19 insertions(+), 143 deletions(-) ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The

Re: [FFmpeg-devel] [PATCH] reset and complete RELEASE_NOTES for the next version

2014-08-13 Thread Clément Bœsch
On Wed, Aug 13, 2014 at 09:00:38PM +0200, Michael Niedermayer wrote: On Wed, Aug 13, 2014 at 08:51:05PM +0200, Clément Bœsch wrote: --- RELEASE_NOTES | 162 +++--- 1 file changed, 19 insertions(+), 143 deletions(-) ok thanks,

Re: [FFmpeg-devel] [PATCH 3/4] dpx: abort if encrypted

2014-08-13 Thread Christophe Gisquet
2014-08-13 21:35 GMT+02:00 Reimar Döffinger reimar.doeffin...@gmx.de: Personally I'd probably go with the missing feature and a warning message on top instead of the return. Same opinion, here's an updated patch. From 47f363c96c3dd15e7e36267ccdb338b102d40078 Mon Sep 17 00:00:00 2001 From:

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

2014-08-13 Thread 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 community, I'd like to invite

[FFmpeg-devel] [PATCH] NULL-check Matroska chapters when reading header

2014-08-13 Thread Justin Jacobs
I have a Matroska file that contains a broken chapter (end time is before start time) that caused any ffmpeg-linked program to crash when accessing it. I've attached a patch that fixes the crash. Also, here is the backtrace of the crash for reference: (gdb) backtrace #0 av_dict_set (pm=0x20,

Re: [FFmpeg-devel] [RFC][PATCH] Allow include files to be installed into subdirectory

2014-08-13 Thread Ivan Kalvachev
On 8/13/14, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 12.08.2014, at 02:21, Ivan Kalvachev ikalvac...@gmail.com wrote: It would be decided by the gcc -I inclusion option. It modifies the search paths for header files, so that it checks these paths before the system/default ones (When

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

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 10:21:51AM +, Christophe Gisquet wrote: SMPTE 268M-2003 specifies that each line starts at a 4-bytes boundary. Therefore, modify correspondingly the input buffer strides and size. Partially fixes ticket #3692: DLAD_8b_3c_big.dpx still has inverted colors, which

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

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 01:51:42PM +0200, Paul B Mahol wrote: On 8/13/14, Christophe Gisquet christophe.gisq...@gmail.com wrote: S268M-2003 specifies that each line start is aligned on a 4-byte boundary. --- libavcodec/dpxenc.c | 44 +--- 1 file

Re: [FFmpeg-devel] [PATCH] NULL-check Matroska chapters when reading header

2014-08-13 Thread Justin Jacobs
Sorry, I didn't realize this had been responded to. I wasn't subbed to the mailing list, so I sent the patch again without realizing. Just disregard that message. The problematic file was the final episode of the anime Nichijou. I think the group that added English subtitles to it auto-created

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

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 10:21:54AM +, Christophe Gisquet wrote: Fixes DLAD_8b_3c_big.dpx from ticket #3692 --- libavcodec/dpx.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c index 2ad7527..d4d6833 100644 --- a/libavcodec/dpx.c +++