Re: [libav-devel] [PATCH v2] dca: NEON optimised high freq VQ decoding

2011-09-30 Thread Kostya Shishkov
On Fri, Sep 30, 2011 at 08:22:05AM +0200, Luca Barbato wrote: > On 9/29/11 11:21 PM, Mans Rullgard wrote: > >Signed-off-by: Mans Rullgard > >--- > >Slightly improved the neon function using a fixed-point to float > >conversion instruction instead of letting gcc mess up the /16. > > > >Fixed indenta

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Tomas Härdin
On Thu, 2011-09-29 at 13:44 -0700, Alex Converse wrote: > This should result in more idiomatic output that is less confusing to > marginal implementations. > --- > libavformat/movenc.c | 16 ++-- > 1 files changed, 14 insertions(+), 2 deletions(-) > > diff --git a/libavformat/movenc

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Alex Converse
On Fri, Sep 30, 2011 at 12:19 AM, Tomas Härdin wrote: > On Thu, 2011-09-29 at 13:44 -0700, Alex Converse wrote: >> This should result in more idiomatic output that is less confusing to >> marginal implementations. >> --- >>  libavformat/movenc.c |   16 ++-- >>  1 files changed, 14 inse

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Tomas Härdin
On Fri, 2011-09-30 at 01:15 -0700, Alex Converse wrote: > On Fri, Sep 30, 2011 at 12:19 AM, Tomas Härdin > wrote: > > On Thu, 2011-09-29 at 13:44 -0700, Alex Converse wrote: > >> This should result in more idiomatic output that is less confusing to > >> marginal implementations. > >> --- > >> li

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Benjamin Larsson
> > At work we added an "int interlacing;" field to AVCodecContext. It's > only set by a few decoders though (no demuxer sets it), and doesn't > provide field order information. > To support field order, "interlacing" could have the value of 0/1/2 > (progressive, TFF, BFF) or a separate field cou

Re: [libav-devel] [PATCH 1/2] Added ability to enable workaround for dxva2 decoding using older ATI cards

2011-09-30 Thread Janne Grunau
On Sun, Sep 25, 2011 at 12:26:45PM +0200, Janne Grunau wrote: > From: Joakim Plate > > The workaround need to be enabled per pci id which can not > be detected inside ffmpeg. So this adds a flag that enabled > the alternate behavior. > --- > libavcodec/dxva2.h |2 ++ > libavcodec/dxva2_

Re: [libav-devel] [PATCH 1/2] Added ability to enable workaround for dxva2 decoding using older ATI cards

2011-09-30 Thread Janne Grunau
On Mon, Sep 26, 2011 at 11:01:34AM +0200, Diego Biurrun wrote: > On Sun, Sep 25, 2011 at 12:26:45PM +0200, Janne Grunau wrote: > > From: Joakim Plate > > > > The workaround need to be enabled per pci id which can not > > be detected inside ffmpeg. So this adds a flag that enabled > > the alternat

Re: [libav-devel] [PATCH 1/2] Added ability to enable workaround for dxva2 decoding using older ATI cards

2011-09-30 Thread Janne Grunau
On Fri, Sep 30, 2011 at 11:20:18AM +0200, Janne Grunau wrote: > On Sun, Sep 25, 2011 at 12:26:45PM +0200, Janne Grunau wrote: > > From: Joakim Plate > > > > The workaround need to be enabled per pci id which can not > > be detected inside ffmpeg. So this adds a flag that enabled > > the alternate

[libav-devel] [PATCH] prores: add FATE tests

2011-09-30 Thread Diego Biurrun
--- I have remuxed the test samples to only contain one frame and no audio stream, so this should be good to go once the changes propagate through the FATE suite. I could wait with pushing this until the decoder is fixed and the values are more meaningful, but I'm not sure this is really more conv

Re: [libav-devel] [PATCH 15/15] dpcm: return error if packet is too small

2011-09-30 Thread Luca Barbato
On 9/22/11 8:42 PM, Justin Ruggles wrote: --- libavcodec/dpcm.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) Ok ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] dpcm: factor out the stereo flag calculation

2011-09-30 Thread Luca Barbato
On 9/26/11 9:45 PM, Justin Ruggles wrote: --- libavcodec/dpcm.c | 29 +++-- 1 files changed, 15 insertions(+), 14 deletions(-) Ok I guess. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman

Re: [libav-devel] [PATCH 2/2] dxva2_h264: pass the correct 8x8 scaling lists

2011-09-30 Thread Luca Barbato
On 9/25/11 12:26 PM, Janne Grunau wrote: From: Carl Eugen Hoyos Copy the Inter 8x8 scaling list as second 8x8 matrix into DXVA2's quantization matrix data structure instead of a potentially unset Intra chroma scaling matrix. Fix dxva2 decoding for some H264 samples. I assume it works, could y

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Luca Barbato
On 9/30/11 10:59 AM, Benjamin Larsson wrote: At work we added an "int interlacing;" field to AVCodecContext. It's only set by a few decoders though (no demuxer sets it), and doesn't provide field order information. To support field order, "interlacing" could have the value of 0/1/2 (progressiv

Re: [libav-devel] [PATCH 1/3] rtp.h: Correct ff_rtp_get_payload_type() documentation

2011-09-30 Thread Luca Barbato
On 9/26/11 5:56 PM, Rafaël Carré wrote: From: Rafaël Carré Since [0c378ea1f], it can't fail anymore Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 2/3] rtpenc.c: add a payload type private option

2011-09-30 Thread Luca Barbato
On 9/26/11 5:56 PM, Rafaël Carré wrote: From: Rafaël Carré Specifying the payload type is useful when the type number has already been negotiated before creating the stream, for example in SIP protocol. --- libavformat/rtp.c| 11 ++- libavformat/rtp.h|6 -- libavfo

Re: [libav-devel] [PATCH 2/3] rtpenc.c: add a payload type private option

2011-09-30 Thread Martin Storsjö
On Fri, 30 Sep 2011, Luca Barbato wrote: On 9/26/11 5:56 PM, Rafaël Carré wrote: From: Rafaël Carré Specifying the payload type is useful when the type number has already been negotiated before creating the stream, for example in SIP protocol. --- libavformat/rtp.c| 11 ++-

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Tomas Härdin
On Fri, 2011-09-30 at 13:44 +0200, Luca Barbato wrote: > On 9/30/11 10:59 AM, Benjamin Larsson wrote: > > > >> > >> At work we added an "int interlacing;" field to AVCodecContext. It's > >> only set by a few decoders though (no demuxer sets it), and doesn't > >> provide field order information. > >

Re: [libav-devel] [PATCH] XOP/FMA4 CPU detection support

2011-09-30 Thread Diego Biurrun
On Tue, Sep 27, 2011 at 01:57:32PM +0200, Diego Biurrun wrote: > On Tue, Sep 27, 2011 at 11:13:06AM +0200, Reinhard Tartler wrote: > > On Di, Sep 27, 2011 at 00:52:26 (CEST), Diego Biurrun wrote: > > > > > On Mon, Sep 26, 2011 at 03:30:51PM -0700, Jason Garrett-Glaser wrote: > > >> On Mon, Sep 26,

Re: [libav-devel] [PATCH] movenc: Don't wrap whole extradata atoms in glbl.

2011-09-30 Thread Luca Barbato
On 9/30/11 2:11 PM, Tomas Härdin wrote: On Fri, 2011-09-30 at 13:44 +0200, Luca Barbato wrote: On 9/30/11 10:59 AM, Benjamin Larsson wrote: At work we added an "int interlacing;" field to AVCodecContext. It's only set by a few decoders though (no demuxer sets it), and doesn't provide field o

Re: [libav-devel] [PATCH 1/2] Split out yuv2yuv1 luma and chroma in order to make them generic DSP functions

2011-09-30 Thread Kieran Kunhya
Patch updated 0001-Split-out-yuv2yuv1-luma-and-chroma-in-order-to-make-.patch Description: Binary data ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 1/2] Split out yuv2yuv1 luma and chroma in order to make them generic DSP functions

2011-09-30 Thread Ronald S. Bultje
Hi, On Fri, Sep 30, 2011 at 2:57 PM, Kieran Kunhya wrote: > Patch updated Thanks, LGTM. I'll do the simd. Ronald ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 04/15] dpcm: output AV_SAMPLE_FMT_U8 for Sol DPCM subcodecs 1 and 2.

2011-09-30 Thread Justin Ruggles
On 09/24/2011 11:09 AM, Justin Ruggles wrote: > On 09/24/2011 01:51 AM, Luca Barbato wrote: > >> On 9/22/11 8:42 PM, Justin Ruggles wrote: >> >> Sorry to ask, but could you explain why the change in the commit message? > > > How about: > > dpcm: output AV_SAMPLE_FMT_U8 for Sol DPCM subcodecs 1

Re: [libav-devel] [PATCH 07/15] dpcm: remove unnecessary variable by using bytestream functions.

2011-09-30 Thread Justin Ruggles
On 09/22/2011 02:42 PM, Justin Ruggles wrote: > Uses 'buf' directly instead of a separate iterator variable 'in'. > --- > libavcodec/dpcm.c | 66 ++-- > 1 files changed, 28 insertions(+), 38 deletions(-) ping. ___

Re: [libav-devel] [PATCH 10/15] dpcm: check to make sure channels is 1 or 2.

2011-09-30 Thread Justin Ruggles
On 09/22/2011 02:42 PM, Justin Ruggles wrote: > --- > libavcodec/dpcm.c |5 + > 1 files changed, 5 insertions(+), 0 deletions(-) > > diff --git a/libavcodec/dpcm.c b/libavcodec/dpcm.c > index c1cd501..539d4c0 100644 > --- a/libavcodec/dpcm.c > +++ b/libavcodec/dpcm.c > @@ -117,6 +117,11

Re: [libav-devel] [PATCH 11/15] dpcm: replace short with int16_t

2011-09-30 Thread Justin Ruggles
On 09/22/2011 02:42 PM, Justin Ruggles wrote: > --- > libavcodec/dpcm.c |7 +++ > 1 files changed, 3 insertions(+), 4 deletions(-) ping. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 12/15] dpcm: use sol_table_16 directly instead of through the DPCMContext.

2011-09-30 Thread Justin Ruggles
On 09/22/2011 02:42 PM, Justin Ruggles wrote: > --- > libavcodec/dpcm.c |5 ++--- > 1 files changed, 2 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/dpcm.c b/libavcodec/dpcm.c > index d2a291b..b38c6aa 100644 > --- a/libavcodec/dpcm.c > +++ b/libavcodec/dpcm.c > @@ -146,7 +146,6 @

[libav-devel] [PATCH] lavc: add video/audio/encoding flags to global_quality option

2011-09-30 Thread Anton Khirnov
--- libavcodec/options.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavcodec/options.c b/libavcodec/options.c index ef711bf..0bcdf2a 100644 --- a/libavcodec/options.c +++ b/libavcodec/options.c @@ -320,7 +320,7 @@ static const AVOption options[]={ {"pbias", "inter

[libav-devel] [PATCH 2/2] libvpxenc: use libvpx's own defaults for some parameters

2011-09-30 Thread Anton Khirnov
From: Luca Barbato Specifically, qmin/qmax, gop_size and keyint_min. Fixes bug 47. --- libavcodec/libvpxenc.c | 21 - 1 files changed, 16 insertions(+), 5 deletions(-) diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c index 1ff997e..524c53d 100644 --- a/libavco

[libav-devel] [PATCH 1/2] vpxenc: add private options

2011-09-30 Thread Anton Khirnov
From: Luca Barbato Make libvpx support close to the libx264 one. Thanks to Jan Gerber for the support. Signed-off-by: Anton Khirnov --- libavcodec/libvpxenc.c | 63 +-- 1 files changed, 60 insertions(+), 3 deletions(-) diff --git a/libavcodec/lib

[libav-devel] [PATCH] Synchronize various 4CCs and codec tags from FFmpeg.

2011-09-30 Thread Diego Biurrun
--- libavcodec/avcodec.h |3 +++ libavcodec/raw.c |4 +++- libavformat/isom.c |5 + libavformat/riff.c | 10 ++ 4 files changed, 21 insertions(+), 1 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 4d61f44..f985d35 100644 --- a/libavcod

Re: [libav-devel] [PATCH] Synchronize various 4CCs and codec tags from FFmpeg.

2011-09-30 Thread Ronald S. Bultje
Hi, On Fri, Sep 30, 2011 at 6:29 PM, Diego Biurrun wrote: > @@ -210,6 +210,9 @@ enum CodecID { >     CODEC_ID_DFA, >     CODEC_ID_WMV3IMAGE, >     CODEC_ID_VC1IMAGE, > +    CODEC_ID_G2M, No, these are various codecs and we don't want to remix crap files. > + Cosmetics. > +    { CODEC_ID_G2M,

Re: [libav-devel] [PATCH] Synchronize various 4CCs and codec tags from FFmpeg.

2011-09-30 Thread Reinhard Tartler
On Fr, Sep 30, 2011 at 18:32:00 (CEST), Ronald S. Bultje wrote: > Hi, > > On Fri, Sep 30, 2011 at 6:29 PM, Diego Biurrun wrote: >> @@ -210,6 +210,9 @@ enum CodecID { >>     CODEC_ID_DFA, >>     CODEC_ID_WMV3IMAGE, >>     CODEC_ID_VC1IMAGE, >> +    CODEC_ID_G2M, > > No, these are various codecs an

Re: [libav-devel] [PATCH 2/2] Split up yuv2yuvX functions.

2011-09-30 Thread Kieran Kunhya
Patch updated ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] Announce Libav 0.7.2

2011-09-30 Thread Luca Barbato
--- src/download | 24 src/index| 39 +++ 2 files changed, 51 insertions(+), 12 deletions(-) diff --git a/src/download b/src/download index 2d3c63f..5d6d712 100644 --- a/src/download +++ b/src/download @@ -158,25 +158,25 @@ sele

Re: [libav-devel] [PATCH] Synchronize various 4CCs and codec tags from FFmpeg.

2011-09-30 Thread Måns Rullgård
Diego Biurrun writes: > --- > libavcodec/avcodec.h |3 +++ > libavcodec/raw.c |4 +++- > libavformat/isom.c |5 + > libavformat/riff.c | 10 ++ > 4 files changed, 21 insertions(+), 1 deletions(-) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > ind

[libav-devel] [PATCH] lavf: Avoid using av_malloc(0) in av_dump_format

2011-09-30 Thread Martin Storsjö
On OS X, av_malloc(0) returns pointers that cause crashes when freed. --- libavformat/utils.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index d0ad358..0ba6fc3 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -33

Re: [libav-devel] [PATCH] Synchronize various 4CCs and codec tags from FFmpeg.

2011-09-30 Thread Alex Converse
2011/9/30 Måns Rullgård : > Diego Biurrun writes: > >> --- >>  libavcodec/avcodec.h |    3 +++ >>  libavcodec/raw.c     |    4 +++- >>  libavformat/isom.c   |    5 + >>  libavformat/riff.c   |   10 ++ >>  4 files changed, 21 insertions(+), 1 deletions(-) >> >> diff --git a/libavcodec/a

Re: [libav-devel] [PATCH] lavf: Avoid using av_malloc(0) in av_dump_format

2011-09-30 Thread Anton Khirnov
On Fri, 30 Sep 2011 20:33:18 +0300, Martin Storsjö wrote: > On OS X, av_malloc(0) returns pointers that cause crashes when > freed. > --- > libavformat/utils.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/libavformat/utils.c b/libavformat/utils.c > index d0ad35

Re: [libav-devel] [PATCH 1/2] vpxenc: add private options

2011-09-30 Thread Luca Barbato
On 9/30/11 5:30 PM, Anton Khirnov wrote: From: Luca Barbato Make libvpx support close to the libx264 one. Thanks to Jan Gerber for the support. Signed-off-by: Anton Khirnov --- Thank you for the additional polish ___ libav-devel mailing list libav

Re: [libav-devel] [PATCH 2/2] libvpxenc: use libvpx's own defaults for some parameters

2011-09-30 Thread Luca Barbato
On 9/30/11 5:30 PM, Anton Khirnov wrote: From: Luca Barbato Specifically, qmin/qmax, gop_size and keyint_min. Fixes bug 47. --- Thanks for the additional polish =) lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailm

Re: [libav-devel] [PATCH] lavc: add video/audio/encoding flags to global_quality option

2011-09-30 Thread Luca Barbato
On 9/30/11 5:28 PM, Anton Khirnov wrote: --- libavcodec/options.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 04/15] dpcm: output AV_SAMPLE_FMT_U8 for Sol DPCM subcodecs 1 and 2.

2011-09-30 Thread Luca Barbato
On 9/30/11 4:33 PM, Justin Ruggles wrote: On 09/24/2011 11:09 AM, Justin Ruggles wrote: On 09/24/2011 01:51 AM, Luca Barbato wrote: On 9/22/11 8:42 PM, Justin Ruggles wrote: Sorry to ask, but could you explain why the change in the commit message? How about: dpcm: output AV_SAMPLE_FMT_U8

Re: [libav-devel] [PATCH 2/2] Split up yuv2yuvX functions.

2011-09-30 Thread Ronald S. Bultje
Hi, On Fri, Sep 30, 2011 at 7:09 PM, Kieran Kunhya wrote: > Patch updated That's a bold statement to make without attachment. :-). Ronald ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 07/15] dpcm: remove unnecessary variable by using bytestream functions.

2011-09-30 Thread Luca Barbato
On 9/22/11 8:42 PM, Justin Ruggles wrote: Uses 'buf' directly instead of a separate iterator variable 'in'. --- Ok ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 10/15] dpcm: check to make sure channels is 1 or 2.

2011-09-30 Thread Luca Barbato
On 9/22/11 8:42 PM, Justin Ruggles wrote: --- Ok. this format can't support at all more than 2 channels? ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 12/15] dpcm: use sol_table_16 directly instead of through the DPCMContext.

2011-09-30 Thread Luca Barbato
On 9/22/11 8:42 PM, Justin Ruggles wrote: --- libavcodec/dpcm.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) Ok ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 2/2] Split up yuv2yuvX functions.

2011-09-30 Thread Kieran Kunhya
/me blames gmail 0001-Split-up-yuv2yuvX-functions.patch Description: Binary data ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] Announce Libav 0.7.2

2011-09-30 Thread Reinhard Tartler
On Fr, Sep 30, 2011 at 19:31:32 (CEST), Luca Barbato wrote: > --- > src/download | 24 > src/index| 39 +++ > 2 files changed, 51 insertions(+), 12 deletions(-) > > diff --git a/src/download b/src/download > index 2d3c63f..5d6d7

Re: [libav-devel] [PATCH] doc: fix references to obsolete presets directories for avconv/ffmpeg

2011-09-30 Thread Anton Khirnov
On Thu, 29 Sep 2011 22:00:29 +0200, Diego Biurrun wrote: > --- > cmdutils.h |2 +- > doc/ffmpeg.texi |4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/cmdutils.h b/cmdutils.h > index a20b779..1c17433 100644 > --- a/cmdutils.h > +++ b/cmdutils.h > @@ -334,

Re: [libav-devel] [PATCH] lavf: Avoid using av_malloc(0) in av_dump_format

2011-09-30 Thread Martin Storsjö
On Fri, 30 Sep 2011, Anton Khirnov wrote: On Fri, 30 Sep 2011 20:33:18 +0300, Martin Storsjö wrote: On OS X, av_malloc(0) returns pointers that cause crashes when freed. --- libavformat/utils.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavformat/utils.c b/li

Re: [libav-devel] [PATCH 11/15] dpcm: replace short with int16_t

2011-09-30 Thread Anton Khirnov
On Thu, 22 Sep 2011 14:42:33 -0400, Justin Ruggles wrote: > --- > libavcodec/dpcm.c |7 +++ > 1 files changed, 3 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/dpcm.c b/libavcodec/dpcm.c > index 539d4c0..d2a291b 100644 > --- a/libavcodec/dpcm.c > +++ b/libavcodec/dpcm.c > @@

Re: [libav-devel] [PATCH 10/15] dpcm: check to make sure channels is 1 or 2.

2011-09-30 Thread Justin Ruggles
On 09/30/2011 02:03 PM, Luca Barbato wrote: > On 9/22/11 8:42 PM, Justin Ruggles wrote: >> --- > > Ok. this format can't support at all more than 2 channels? They might, I don't know. But the code certainly doesn't. -Justin ___ libav-devel mailing li

Re: [libav-devel] [PATCH] Announce Libav 0.7.2

2011-09-30 Thread Luca Barbato
On 9/30/11 8:41 PM, Reinhard Tartler wrote: I might have gotten it wrong in the release tarball, but please let's the cApitaliZation Right on the website. I'd suggest all bullet point to start with a big letter. Uh, please tell me what to capitalize since I'm afraid I could miss stuff or mess

[libav-devel] [PATCH 1/3] mpc7: check output buffer size before decoding

2011-09-30 Thread Justin Ruggles
--- libavcodec/mpc7.c | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/libavcodec/mpc7.c b/libavcodec/mpc7.c index a98cb0e..02c83fc 100644 --- a/libavcodec/mpc7.c +++ b/libavcodec/mpc7.c @@ -197,7 +197,7 @@ static int mpc7_decode_frame(AVCodecContext * avctx,

[libav-devel] [PATCH 2/3] mpc7: return error if packet is too small.

2011-09-30 Thread Justin Ruggles
--- libavcodec/mpc7.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/libavcodec/mpc7.c b/libavcodec/mpc7.c index 02c83fc..6f79c7b 100644 --- a/libavcodec/mpc7.c +++ b/libavcodec/mpc7.c @@ -203,6 +203,7 @@ static int mpc7_decode_frame(AVCodecContext * avctx, memset(ba

[libav-devel] [PATCH 3/3] mpc8: check output buffer size before decoding

2011-09-30 Thread Justin Ruggles
--- libavcodec/mpc8.c | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/libavcodec/mpc8.c b/libavcodec/mpc8.c index 0e3947b..a126fc8 100644 --- a/libavcodec/mpc8.c +++ b/libavcodec/mpc8.c @@ -241,10 +241,16 @@ static int mpc8_decode_frame(AVCodecContext * avctx,

[libav-devel] [PATCH 0/5] nellymoser patches

2011-09-30 Thread Justin Ruggles
Justin Ruggles (5): nellymoser: check output buffer size before decoding nellymoserdec: allow user to request SAMPLE_FMT_FLT for output samples. nellymoserdec: use NELLY_BLOCK_LEN instead of 64 when appropriate. nellymoserdec: use NELLY_BUF_LEN instead of 128 nellymoserdec: allocate f

[libav-devel] [PATCH 2/5] nellymoserdec: allow user to request SAMPLE_FMT_FLT for output samples.

2011-09-30 Thread Justin Ruggles
--- libavcodec/nellymoserdec.c | 29 ++--- 1 files changed, 22 insertions(+), 7 deletions(-) diff --git a/libavcodec/nellymoserdec.c b/libavcodec/nellymoserdec.c index 2856479..a6748bf 100644 --- a/libavcodec/nellymoserdec.c +++ b/libavcodec/nellymoserdec.c @@ -137,15 +1

[libav-devel] [PATCH 1/5] nellymoser: check output buffer size before decoding

2011-09-30 Thread Justin Ruggles
--- libavcodec/nellymoserdec.c | 17 - 1 files changed, 12 insertions(+), 5 deletions(-) diff --git a/libavcodec/nellymoserdec.c b/libavcodec/nellymoserdec.c index a153dc0..2856479 100644 --- a/libavcodec/nellymoserdec.c +++ b/libavcodec/nellymoserdec.c @@ -156,19 +156,26 @@ sta

[libav-devel] [PATCH 3/5] nellymoserdec: use NELLY_BLOCK_LEN instead of 64 when appropriate.

2011-09-30 Thread Justin Ruggles
--- libavcodec/nellymoserdec.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/nellymoserdec.c b/libavcodec/nellymoserdec.c index a6748bf..bd3ab99 100644 --- a/libavcodec/nellymoserdec.c +++ b/libavcodec/nellymoserdec.c @@ -170,13 +170,13 @@ static int decode

[libav-devel] [PATCH 4/5] nellymoserdec: use NELLY_BUF_LEN instead of 128

2011-09-30 Thread Justin Ruggles
--- libavcodec/nellymoserdec.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavcodec/nellymoserdec.c b/libavcodec/nellymoserdec.c index bd3ab99..d7d1aba 100644 --- a/libavcodec/nellymoserdec.c +++ b/libavcodec/nellymoserdec.c @@ -48,7 +48,7 @@ typedef struct NellyMo

[libav-devel] [PATCH 5/5] nellymoserdec: allocate float_buf only when decoding to int16

2011-09-30 Thread Justin Ruggles
--- libavcodec/nellymoserdec.c |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/libavcodec/nellymoserdec.c b/libavcodec/nellymoserdec.c index d7d1aba..2d59abf 100644 --- a/libavcodec/nellymoserdec.c +++ b/libavcodec/nellymoserdec.c @@ -47,7 +47,7 @@ typedef struct

Re: [libav-devel] [PATCH 1/3] mpc7: check output buffer size before decoding

2011-09-30 Thread Kostya Shishkov
On Fri, Sep 30, 2011 at 06:48:40PM -0400, Justin Ruggles wrote: > --- > libavcodec/mpc7.c | 10 -- > 1 files changed, 8 insertions(+), 2 deletions(-) looks OK ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/l

Re: [libav-devel] [PATCH 2/3] mpc7: return error if packet is too small.

2011-09-30 Thread Kostya Shishkov
On Fri, Sep 30, 2011 at 06:48:41PM -0400, Justin Ruggles wrote: > --- > libavcodec/mpc7.c |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/libavcodec/mpc7.c b/libavcodec/mpc7.c > index 02c83fc..6f79c7b 100644 > --- a/libavcodec/mpc7.c > +++ b/libavcodec/mpc7.c > @@ -2

Re: [libav-devel] [PATCH 3/3] mpc8: check output buffer size before decoding

2011-09-30 Thread Kostya Shishkov
On Fri, Sep 30, 2011 at 06:48:42PM -0400, Justin Ruggles wrote: > --- > libavcodec/mpc8.c | 10 -- > 1 files changed, 8 insertions(+), 2 deletions(-) LGTM ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listi