Re: [FFmpeg-devel] [PATCH] lavf/oggparsevp8: use ff_vorbis_stream_comment()

2014-08-13 Thread Michael Niedermayer
On Thu, Aug 14, 2014 at 12:31:00AM -0300, James Almer wrote: > commit db68ef89 did not update the vp8 parser > > Signed-off-by: James Almer > --- > libavformat/oggparsevp8.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128

[FFmpeg-devel] [PATCH] avcodec/fic: Check if a frame is available before using it

2014-08-13 Thread Michael Niedermayer
Fixes null pointer dereference Fixes: ficvf.avi Found-by: Piotr Bandurski Signed-off-by: Michael Niedermayer --- libavcodec/fic.c |7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/fic.c b/libavcodec/fic.c index d08d240..5615e69 100644 --- a/libavcodec/fic.c

[FFmpeg-devel] [PATCH] lavf/oggparsevp8: use ff_vorbis_stream_comment()

2014-08-13 Thread James Almer
commit db68ef89 did not update the vp8 parser Signed-off-by: James Almer --- libavformat/oggparsevp8.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/oggparsevp8.c b/libavformat/oggparsevp8.c index b3dd7ab..5959d32 100644 --- a/libavformat/oggparsevp8.c +++ b/lib

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 > +++ b

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 the

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

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 10:27:22PM +0200, Christophe Gisquet wrote: > 2014-08-13 21:35 GMT+02:00 Reimar Döffinger : > > 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. > dpx.c | 10 ++

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 wrote: > > S268M-2003 specifies that each line start is aligned on a 4-byte boundary. > > --- > > libavcodec/dpxenc.c | 44 +--- > > 1 file changed, 37 insertions

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, whic

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

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 08:13:31PM -0400, Justin Jacobs wrote: > 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 t

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 wrote: > On 12.08.2014, at 02:21, Ivan Kalvachev 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/avcode

[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, key=

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 wrote: > On 13.08.2014, at 18:42, Stefano Sabatini 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 20

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 : > 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: Christophe Gisquet Date: We

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

2014-08-13 Thread Clément Bœsch
On Thu, Jun 19, 2014 at 06:59:48PM +0200, Clément Bœsch wrote: > On Thu, Jun 19, 2014 at 04:23:55PM +, Carl Eugen Hoyos wrote: > > Clément Bœsch pkh.me> writes: > > > > > > If your patch is really what everybody wants, please > > > > add a fallback that allows to try the current > > > > way

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 >

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] 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

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

2014-08-13 Thread Clément Bœsch
--- RELEASE_NOTES | 162 +++--- 1 file changed, 19 insertions(+), 143 deletions(-) diff --git a/RELEASE_NOTES b/RELEASE_NOTES index 1e6289b..eb73e88 100644 --- a/RELEASE_NOTES +++ b/RELEASE_NOTES @@ -1,175 +1,51 @@ ┌───

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 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 probably goi

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 wrote: > Hi, > > On 12.08.2014 02:21, Ivan Kalvachev wrote: >> On 8/11/14, Andreas Cadhalpun wrote: >>> Assuming it would be possible to install development packages for both >>> at the same time, which one should be used, when building a program? >> >> It would be d

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

2014-08-13 Thread Mark Reid
On Wed, Aug 13, 2014 at 12:37 AM, Carl Eugen Hoyos wrote: > Mark Reid gmail.com> writes: > > > UL values from libMXF > > No! > > I mean: Those values are not copyrightable > but since you found them in ffmbc it is (imo!) > a matter of courtesy to mention the source. > Please mention both if you

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] [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 : > > 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 > > single-bit corrupt

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" 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 fil

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" 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 add it

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

2014-08-13 Thread Christophe Gisquet
Hi, 2014-08-13 18:46 GMT+02:00 Reimar Döffinger : > 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 > single-bit corruption!) seems worse to me than trying to play an encrypted > file. > I.e. shouldn

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

2014-08-13 Thread Reimar Döffinger
On 13.08.2014, at 18:42, Stefano Sabatini 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 is providi

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 wrote: > On 8/13/14, Christophe Gisquet 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/d

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 ope

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

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 05:58:34PM +0200, Stefano Sabatini wrote: > Fix trac issue #3850. > --- > doc/filters.texi | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB It is dangerous to be right in ma

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" wrote: > > Fix trac issue #3850. > --- OK. [...] Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[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 co

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

2014-08-13 Thread Dmitry Volyntsev
From: Dmitry Volyntsev 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 deletions(-) dif

Re: [FFmpeg-devel] Reintroducing FFmpeg to Debian

2014-08-13 Thread Attila Kinali
On Wed, 13 Aug 2014 00:30:05 +0200 Michael Niedermayer 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 my enemies, let

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] [PATCH 3/3] opts: add list device sources/sinks options

2014-08-13 Thread Michael Niedermayer
On Thu, Aug 07, 2014 at 01:58:57AM +0200, Lukasz Marek wrote: > Allows to list sources/sinks of the devices that implement > that functionality. > [...] > +int show_sinks(void *optctx, const char *opt, const char *arg) > +{ > +#if CONFIG_AVDEVICE > +AVDeviceInfoList *device_list = NULL; > +

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 02:30:20PM +0200, James Darnley wrote: > 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 wrot

Re: [FFmpeg-devel] [PATCH] avutil: turn arm setend into a cpuflag

2014-08-13 Thread Michael Niedermayer
On Thu, Aug 07, 2014 at 06:49:55PM +0200, Michael Niedermayer wrote: > this allows disabling and enabling it > it also prevents crashes if vfpv3 and neon are disabled which previously > would have enabled the flag > > Signed-off-by: Michael Niedermayer > --- > configure |3 +++ > l

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

2014-08-13 Thread James Darnley
On 2014-08-13 09:21, Christophe Gisquet wrote: > Hi, > > 2014-08-12 23:22 GMT+02:00 James Darnley : >> --- >> 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 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 cha

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(-) > >> > >> dif

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 > --- > libavutil/dict.c | 4 > 1 file changed, 4 insertions(+) LGTM [...] -- Michael Gnu

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 fing

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

2014-08-13 Thread Paul B Mahol
On 8/13/14, Christophe Gisquet 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] dpx: abort if encrypted

2014-08-13 Thread Paul B Mahol
On 8/13/14, Christophe Gisquet 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 int decode_frame(AVC

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

2014-08-13 Thread Paul B Mahol
On 8/13/14, 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 > +++ b/libavcodec/dpx.c > @@ -232,6

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

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 08:23:17AM +0200, Hendrik Leppkes wrote: > On Wed, Aug 13, 2014 at 5:34 AM, James Almer wrote: > > Signed-off-by: James Almer > > --- > > libavcodec/dnxhddata.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavcodec/dnxhddata.h b/libavco

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 wrote: > > > On Aug 12, 2014 6:29 PM, "Mark Reid" wrote: > > > > > I didn't realize FFmbc was licensed differently :s. I don't know if it > > > helps, (IANAL either) > > > but the same ULs ca

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 wrote: > On Wed, Aug 13, 2014 at 12:11 AM, Ronald S. Bultje > wrote: > > Fixes ffvp9_fails_where_libvpx.succeeds.webm from ticket 3849. > > --- > > libavcodec/vp9.c | 26 +++--- > > 1 file changed, 15 insertions(+), 11 d

[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, AV_LOG_ERROR,

[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 --- a/libavco

[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 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 0/4] Various fixes for DPX

2014-08-13 Thread Christophe Gisquet
Those fixes are intended around ticket #3692. The core of the issues is that line starts must be aligned on a 4-byte boundary. Christophe Gisquet (4): dpx: use aligned line starts dpxenc: enforce alignment requirement dpx: abort if encrypted dpx: fix endianess for RGB 8bits libavcodec/dp

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 wrote: > > On Tue, Aug 12, 2014 at 11:32:20AM +, Paul B Mahol wrote: > >> Signed-off-by: Paul B Mahol > >> --- > >> libavfilter/vf_perspective.c | 69 > >> +--- >

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..36cd9

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 : > 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 not a multiple of 4... Let's

[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 v3] added ULs for demuxing avid media composer mxf files

2014-08-13 Thread Carl Eugen Hoyos
Mark Reid gmail.com> writes: > UL values from libMXF No! I mean: Those values are not copyrightable but since you found them in ffmbc it is (imo!) a matter of courtesy to mention the source. Please mention both if you want. Sorry, I am on a journey and didn't sleep, Carl Eugen ___

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

2014-08-13 Thread Christophe Gisquet
Hi, 2014-08-12 23:22 GMT+02:00 James Darnley : > --- > 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 > --- a/libavcodec/flacdsp.h > +++ b/libavcodec/flacdsp.h > @