Re: [libav-devel] mov, mxfdec return value refactoring

2012-01-14 Thread Daniel Huang
problem fixed. Daniel On Sat, Jan 14, 2012 at 10:52 AM, Diego Biurrun wrote: > On Sat, Jan 14, 2012 at 04:21:05PM +0100, Diego Biurrun wrote: > > On Fri, Jan 13, 2012 at 02:25:15PM -0600, Daniel Huang wrote: > > > > > > --- a/libavformat/mov.c > > > +++ b/libavformat/mov.c > > > @@ -731,9 +733,

Re: [libav-devel] [PATCH] Escape 130 decoder

2012-01-14 Thread Paul B Mahol
On 1/14/12, Kostya Shishkov wrote: > I've picked Diego's version and changed some things I didn't like and > updated > to the current HEAD. Samples from our repository look Good Enough^TM. > Artifacts are still there. ___ libav-devel mailing list libav-

Re: [libav-devel] [PATCH] tta: cast output data pointer to the correct type

2012-01-14 Thread Paul B Mahol
On 1/7/12, Justin Ruggles wrote: > fixes "warning: assignment from incompatible pointer type" > --- > libavcodec/tta.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > Bump. ___ libav-devel mailing list libav-devel@libav.org https://list

Re: [libav-devel] [PATCH] Escape 130 decoder

2012-01-14 Thread Justin Ruggles
On 01/14/2012 03:35 PM, Kostya Shishkov wrote: > +/** > + * Decode a single frame. > + * @param avctx decoder context > + * @param data decoded frame > + * @param data_size size of the decoded frame > + * @param buf input buffer > + * @param buf_size input buffer size > + * @return 0 success, -1 o

[libav-devel] [PATCH] Escape 130 decoder

2012-01-14 Thread Kostya Shishkov
I've picked Diego's version and changed some things I didn't like and updated to the current HEAD. Samples from our repository look Good Enough^TM. >From 1690804a42ef37fb6c6cb883430af1287d84e76e Mon Sep 17 00:00:00 2001 From: Eli Friedman Date: Fri, 16 Dec 2011 21:30:27 +0100 Subject: [PATCH] Esca

Re: [libav-devel] [PATCH 2/2] avformat: add SMJPEG muxer

2012-01-14 Thread Paul B Mahol
--- Changelog|1 + doc/general.texi |2 +- libavformat/Makefile |1 + libavformat/allformats.c |2 +- libavformat/smjpegenc.c | 149 ++ libavformat/version.h|2 +- 6 files changed, 154 insertions(+),

Re: [libav-devel] [PATCH] vb: Use bytestream2 functions to prevent buffer overreads.

2012-01-14 Thread Justin Ruggles
On 01/14/2012 02:04 PM, Laurentiu Ion wrote: > --- > libavcodec/vb.c | 75 +++--- > 1 files changed, 27 insertions(+), 48 deletions(-) lgtm. will test/push later today. Thanks, Justin ___ libav-devel

[libav-devel] [PATCH] vb: Use bytestream2 functions to prevent buffer overreads.

2012-01-14 Thread Laurentiu Ion
--- libavcodec/vb.c | 75 +++--- 1 files changed, 27 insertions(+), 48 deletions(-) diff --git a/libavcodec/vb.c b/libavcodec/vb.c index 96387b6..c52ba12 100644 --- a/libavcodec/vb.c +++ b/libavcodec/vb.c @@ -44,7 +44,7 @@ typedef struct VBDecCont

Re: [libav-devel] LibAV-FATE on Solaris x86

2012-01-14 Thread Sean McGovern
On 1/14/12, Måns Rullgård wrote: > Sean McGovern writes: > >> On Thursday, December 29, 2011, Måns Rullgård wrote: >>> Sean McGovern writes: >>> On Thursday, December 29, 2011, Måns Rullgård wrote: > > What happens if you remove the () around the check_exec call? > N

[libav-devel] [PATCH] vb: Use bytestream2 functions to prevent buffer overreads.

2012-01-14 Thread Laurentiu Ion
--- libavcodec/vb.c | 72 ++ 1 files changed, 24 insertions(+), 48 deletions(-) diff --git a/libavcodec/vb.c b/libavcodec/vb.c index 96387b6..4405137 100644 --- a/libavcodec/vb.c +++ b/libavcodec/vb.c @@ -44,7 +44,7 @@ typedef struct VBDecCont

Re: [libav-devel] [PATCH 1/3] fate: Give some tests more sensible names.

2012-01-14 Thread Justin Ruggles
On 01/14/2012 12:32 PM, Diego Biurrun wrote: > --- [...] > rename tests/ref/fate/{truemotion1-15 => duck-tm1-15} (100%) > rename tests/ref/fate/{truemotion1-24 => duck-tm1-24} (100%) the AVCodec name is "truemotion1" so i would prefer to have that in the test name. [...] > rename tests/ref/f

Re: [libav-devel] LibAV-FATE on Solaris x86

2012-01-14 Thread Måns Rullgård
Sean McGovern writes: > On Thursday, December 29, 2011, Måns Rullgård wrote: >> Sean McGovern writes: >> >>> On Thursday, December 29, 2011, Måns Rullgård wrote: What happens if you remove the () around the check_exec call? >>> >>> No difference other than the Terminated line en

Re: [libav-devel] LibAV-FATE on Solaris x86

2012-01-14 Thread Sean McGovern
On Thursday, December 29, 2011, Måns Rullgård wrote: > Sean McGovern writes: > >> On Thursday, December 29, 2011, Måns Rullgård wrote: >>> >>> What happens if you remove the () around the check_exec call? >>> >> >> No difference other than the Terminated line ending up in config.log >> instead o

Re: [libav-devel] [PATCH 3/3] fate: add some shorthands to run groups of tests

2012-01-14 Thread Diego Biurrun
On Sat, Jan 14, 2012 at 06:23:19PM +0100, Diego Biurrun wrote: > --- > tests/fate/ac3.mak| 15 ++ > tests/fate/adpcm.mak | 31 +++- > tests/fate/amrnb.mak | 19 +++- > tests/fate/amrwb.mak | 23 -- > tests/f

Re: [libav-devel] [PATCH 2/3] fate: split ADPCM and DPCM test references into separate files.

2012-01-14 Thread Diego Biurrun
On Sat, Jan 14, 2012 at 06:23:18PM +0100, Diego Biurrun wrote: > --- > tests/Makefile |1 + > tests/fate/adpcm.mak | 40 > tests/fate/dpcm.mak | 42 -- > 3 files changed, 41 insertions(+), 42 deletions(

[libav-devel] [PATCH 3/3] fate: add some shorthands to run groups of tests

2012-01-14 Thread Diego Biurrun
--- tests/fate/ac3.mak| 15 ++ tests/fate/adpcm.mak | 31 +++- tests/fate/amrnb.mak | 19 +++- tests/fate/amrwb.mak | 23 -- tests/fate/atrac.mak | 11 -- tests/fate/audio.mak |7

[libav-devel] [PATCH 1/3] fate: Give some tests more sensible names.

2012-01-14 Thread Diego Biurrun
--- tests/fate/demux.mak |6 -- tests/fate/dpcm.mak| 52 +++ tests/fate/lossless-audio.mak | 12 ++-- tests/fate/microsoft.mak |4 +- tests/fate/pcm.mak

[libav-devel] [PATCH 2/3] fate: split ADPCM and DPCM test references into separate files.

2012-01-14 Thread Diego Biurrun
--- tests/Makefile |1 + tests/fate/adpcm.mak | 40 tests/fate/dpcm.mak | 42 -- 3 files changed, 41 insertions(+), 42 deletions(-) create mode 100644 tests/fate/adpcm.mak diff --git a/tests/Makefile

Re: [libav-devel] [PATCH 2/3] fate: split ADPCM and DPCM test references into separate files.

2012-01-14 Thread Justin Ruggles
On 01/14/2012 12:23 PM, Diego Biurrun wrote: > --- > tests/Makefile |1 + > tests/fate/adpcm.mak | 40 > tests/fate/dpcm.mak | 42 -- > 3 files changed, 41 insertions(+), 42 deletions(-) > create mode

[libav-devel] [PATCH 3/3] fate: add some shorthands to run groups of tests

2012-01-14 Thread Diego Biurrun
--- tests/fate/ac3.mak| 15 ++ tests/fate/adpcm.mak | 31 +++- tests/fate/amrnb.mak | 19 +++- tests/fate/amrwb.mak | 23 -- tests/fate/atrac.mak | 11 -- tests/fate/audio.mak |7

[libav-devel] [PATCH 2/3] fate: split ADPCM and DPCM test references into separate files.

2012-01-14 Thread Diego Biurrun
--- tests/Makefile |1 + tests/fate/adpcm.mak | 40 tests/fate/dpcm.mak | 42 -- 3 files changed, 41 insertions(+), 42 deletions(-) create mode 100644 tests/fate/adpcm.mak diff --git a/tests/Makefile

Re: [libav-devel] [PATCH 2/2] avformat: add SMJPEG muxer

2012-01-14 Thread Justin Ruggles
On 01/14/2012 11:47 AM, Paul B Mahol wrote: > --- > Changelog|1 + > doc/general.texi |2 +- > libavformat/Makefile |1 + > libavformat/allformats.c |2 +- > libavformat/smjpegenc.c | 149 > ++ > libavforma

Re: [libav-devel] [PATCH] swscale-test: Drop unused argc and argv arguments fro main().

2012-01-14 Thread Diego Biurrun
On Fri, Jan 13, 2012 at 09:05:37AM -0800, Ronald S. Bultje wrote: > On Fri, Jan 13, 2012 at 8:01 AM, Diego Biurrun wrote: > > --- > >  libswscale/colorspace-test.c |    2 +- > >  1 files changed, 1 insertions(+), 1 deletions(-) > > Why? I was trawling -Wextra output for sensible issues, this was

Re: [libav-devel] [libav-commits] FATE: add tests for targa

2012-01-14 Thread Diego Biurrun
On Fri, Jan 13, 2012 at 09:28:35PM +, Paul B Mahol wrote: > On 1/13/12, Maans Rullgaard wrote: > > Paul B Mahol writes: > > > >> On 1/13/12, Maans Rullgaard wrote: > >>> "Justin Ruggles " writes: > >>> > +# fate-targa-conformance-CCM8: CMD = framecrc -i > $(SAMPLES)/targa-conform

[libav-devel] [PATCH] vb: Use bytestream2 functions to prevent buffer overreads.

2012-01-14 Thread Laurentiu Ion
--- libavcodec/vb.c | 69 +++--- 1 files changed, 24 insertions(+), 45 deletions(-) diff --git a/libavcodec/vb.c b/libavcodec/vb.c index 96387b6..0d69ca6 100644 --- a/libavcodec/vb.c +++ b/libavcodec/vb.c @@ -44,7 +44,7 @@ typedef struct VBDecCont

Re: [libav-devel] mov, mxfdec return value refactoring

2012-01-14 Thread Diego Biurrun
On Sat, Jan 14, 2012 at 04:21:05PM +0100, Diego Biurrun wrote: > On Fri, Jan 13, 2012 at 02:25:15PM -0600, Daniel Huang wrote: > > > > --- a/libavformat/mov.c > > +++ b/libavformat/mov.c > > @@ -731,9 +733,11 @@ static int mov_read_mdhd(MOVContext *c, AVIOContext > > *pb, MOVAtom atom) > > v

[libav-devel] [PATCH 2/2] avformat: add SMJPEG muxer

2012-01-14 Thread Paul B Mahol
--- Changelog|1 + doc/general.texi |2 +- libavformat/Makefile |1 + libavformat/allformats.c |2 +- libavformat/smjpegenc.c | 149 ++ libavformat/version.h|2 +- 6 files changed, 154 insertions(+),

[libav-devel] [PATCH 1/2] avformat: split out common SMJPEG code

2012-01-14 Thread Paul B Mahol
This is a preparation for adding SMJPEG muxer. --- libavformat/Makefile|2 +- libavformat/smjpeg.c| 159 ++ libavformat/smjpeg.h| 45 libavformat/smjpegdec.c | 177 +++ 4 files chan

Re: [libav-devel] mov, mxfdec return value refactoring

2012-01-14 Thread Diego Biurrun
On Fri, Jan 13, 2012 at 02:25:15PM -0600, Daniel Huang wrote: > > --- a/libavformat/mov.c > +++ b/libavformat/mov.c > @@ -731,9 +733,11 @@ static int mov_read_mdhd(MOVContext *c, AVIOContext *pb, > MOVAtom atom) > version = avio_r8(pb); > -if (version > 1) > -return -1; /* unsupp

Re: [libav-devel] Patch error_resilience:cosmetics

2012-01-14 Thread Diego Biurrun
On Fri, Jan 13, 2012 at 07:18:39PM +0200, доналд овчаров wrote: > My patch is ready to be reviewed. You are still using your broken mailer and breaking threads, which I very specifically told you not to do. Let me make this very clear, your task has two parts: 1) Creating a patch fit for inclus

Re: [libav-devel] [PATCH] sgidec: Use bytestream2 functions to prevent buffer overreads.

2012-01-14 Thread Justin Ruggles
On 01/13/2012 12:35 PM, Aneesh Dogra wrote: > The patch also adds several bytestream macros to deal with native endian. > --- > libavcodec/bytestream.h | 37 + > libavcodec/sgidec.c | 103 > +++ > 2 files changed, 88 insertions(+

Re: [libav-devel] [PATCH 10/10] avserver: fix build after the next bump.

2012-01-14 Thread Justin Ruggles
On 01/12/2012 03:57 AM, Anton Khirnov wrote: > --- > avserver.c | 22 -- > 1 files changed, 12 insertions(+), 10 deletions(-) > > diff --git a/avserver.c b/avserver.c > index 15fa64c..f3ac319 100644 fine with me. we at least need to keep it compiling. -Justin ___

Re: [libav-devel] [PATCH] SMJPEG muxer

2012-01-14 Thread Kostya Shishkov
On Fri, Jan 13, 2012 at 09:22:52PM +, Paul B Mahol wrote: > --- > Changelog|1 + > doc/general.texi |2 +- > libavformat/Makefile |3 +- > libavformat/allformats.c |2 +- > libavformat/smjpeg.c | 159 ++--- >