Re: [libav-devel] [PATCH] mp4: Don't read an empty Decoder Config Descriptor

On Wed, 21 Sep 2011 15:59:23 -0700, Alex Converse wrote: > --- > libavformat/isom.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/libavformat/isom.c b/libavformat/isom.c > index 48fd8e1..896730c 100644 > --- a/libavformat/isom.c > +++ b/libavformat/isom.c > @@

Re: [libav-devel] [PATCH 3/5] Check for invalid slices offsets in RV30/40 decoder.

On Wed, Sep 21, 2011 at 08:46:31PM +0200, fen...@elivagar.org wrote: > From: Laurent Aimar > > --- > libavcodec/rv34.c | 11 ++- > 1 files changed, 10 insertions(+), 1 deletions(-) looks reasonable ___ libav-devel mailing list libav-devel@li

Re: [libav-devel] [PATCH] rtmp: Make the input FLV parser handle data cut at any point

On Thu, Sep 22, 2011 at 12:00:32AM +0300, Martin Storsjö wrote: > This makes the RTMP writing code able to handle FLV data > fed in arbitrarily small or large chunks, with multiple > consecutive packets in one write call, or having the FLV > packet header split over numerous write calls. > > When

Re: [libav-devel] FATE instances

On Mi, Sep 21, 2011 at 06:21:27 (PDT), Diego Biurrun wrote: > I got permission to use a very powerful x86_64 build host where I am now > running three FATE instances. If somebody wants/needs some other config > tested, let me know. Does that machine support AVX? If yes, it could replace my 'sand

Re: [libav-devel] [PATCH] Apple ProRes decoder

Hi, On Wed, Sep 21, 2011 at 1:17 PM, Alex Converse wrote: > On Wed, Sep 21, 2011 at 10:36 AM, Kostya Shishkov > wrote: >> On Wed, Sep 21, 2011 at 02:07:13PM +0200, Tomas Härdin wrote: >>> On Wed, 2011-09-21 at 13:09 +0200, Diego Biurrun wrote: >>> > From: Maxim Poliakovski >>> > +static int dec

[libav-devel] [PATCH] mp4: Don't read an empty Decoder Config Descriptor

--- libavformat/isom.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavformat/isom.c b/libavformat/isom.c index 48fd8e1..896730c 100644 --- a/libavformat/isom.c +++ b/libavformat/isom.c @@ -418,7 +418,7 @@ int ff_mp4_read_dec_config_descr(AVFormatContext *fc, AVStre

[libav-devel] [PATCH] mp4: Don't read an empty Decoder Config Descriptor

--- libavformat/isom.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavformat/isom.c b/libavformat/isom.c index 48fd8e1..d3d 100644 --- a/libavformat/isom.c +++ b/libavformat/isom.c @@ -418,7 +418,7 @@ int ff_mp4_read_dec_config_descr(AVFormatContext *fc, AVStre

[libav-devel] [PATCH] rtmp: Make the input FLV parser handle data cut at any point

This makes the RTMP writing code able to handle FLV data fed in arbitrarily small or large chunks, with multiple consecutive packets in one write call, or having the FLV packet header split over numerous write calls. When used in conjunction with the flv muxer, the AVIO buffer size still needs to

Re: [libav-devel] [PATCH] Apple ProRes decoder

On Wed, Sep 21, 2011 at 10:36 AM, Kostya Shishkov wrote: > On Wed, Sep 21, 2011 at 02:07:13PM +0200, Tomas Härdin wrote: >> On Wed, 2011-09-21 at 13:09 +0200, Diego Biurrun wrote: >> > From: Maxim Poliakovski >> > +static int decode_frame_header(ProresContext *ctx, const uint8_t *buf, >> > +    

Re: [libav-devel] [PATCH] fate.sh: add support for building extra targets

Diego Biurrun writes: > Sometimes it can be useful to build not just the default make target, but some > other target or targets in addition to the default one. Add an extra step in > the FATE client that builds this extra list of targets from the config file. > --- > tests/fate.sh |7 +

Re: [libav-devel] [PATCH] flvdec: Ignore the index if it's from a creator known to be different

On 9/21/11 9:54 AM, Luca Barbato wrote: Not sure if might make sense adding a private option to disable this quirkmode. is this creator widespread enough to be useful to support it? Let's land this patch for now we might add parsing support as a CodeIn task or as small task, I guess. lu ___

Re: [libav-devel] [PATCH] eval: test isnan(sqrt(-1)) instead of just sqrt(-1)

On 9/21/11 11:26 AM, Anton Khirnov wrote: sqrt(-1) returns "some NaN", it's not specified which exactly. Ok ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 1/5] Check output buffer size in nellymoser decoder.

From: Laurent Aimar --- libavcodec/nellymoserdec.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/libavcodec/nellymoserdec.c b/libavcodec/nellymoserdec.c index a153dc0..23fecbf 100644 --- a/libavcodec/nellymoserdec.c +++ b/libavcodec/nellymoserdec.c @@ -156,6 +156,7 @

[libav-devel] [PATCH 5/5] Release old pictures after a resolution change in vp5/6 decoder

From: Laurent Aimar --- libavcodec/vp56.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/libavcodec/vp56.c b/libavcodec/vp56.c index e18c59e..698201b 100644 --- a/libavcodec/vp56.c +++ b/libavcodec/vp56.c @@ -515,6 +515,16 @@ int ff_vp56_decode_frame(AVCodecCo

[libav-devel] [PATCH 3/5] Check for invalid slices offsets in RV30/40 decoder.

From: Laurent Aimar --- libavcodec/rv34.c | 11 ++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/libavcodec/rv34.c b/libavcodec/rv34.c index a760e5e..abdfce5 100644 --- a/libavcodec/rv34.c +++ b/libavcodec/rv34.c @@ -1513,13 +1513,18 @@ int ff_rv34_decode_frame(AVCod

[libav-devel] [PATCH 4/5] Check for huffman tree building error in vp6 decoder.

From: Laurent Aimar --- libavcodec/vp5.c |3 ++- libavcodec/vp56.c |4 +++- libavcodec/vp56.h |2 +- libavcodec/vp6.c | 20 4 files changed, 18 insertions(+), 11 deletions(-) diff --git a/libavcodec/vp5.c b/libavcodec/vp5.c index 7743a48..a1a38b0 100644 ---

[libav-devel] [PATCH 2/5] Check for missing reference in vp5/6 decoder.

From: Laurent Aimar --- libavcodec/vp56.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/libavcodec/vp56.c b/libavcodec/vp56.c index 5c7e93e..530b890 100644 --- a/libavcodec/vp56.c +++ b/libavcodec/vp56.c @@ -401,6 +401,8 @@ static void vp56_decode_mb(VP56Context *s, i

[libav-devel] [PATCH] fate.sh: add support for building extra targets

Sometimes it can be useful to build not just the default make target, but some other target or targets in addition to the default one. Add an extra step in the FATE client that builds this extra list of targets from the config file. --- tests/fate.sh |7 +++ 1 files changed, 7 insertions(

Re: [libav-devel] [PATCH] flvdec: Ignore the index if it's from a creator known to be different

On Wed, 21 Sep 2011, Luca Barbato wrote: On 9/21/11 7:15 AM, Martin Storsjö wrote: --- libavformat/flvdec.c | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c index cafbeb5..569d734 100644 --- a/libavformat/flvdec.

[libav-devel] [PATCH] eval: test isnan(sqrt(-1)) instead of just sqrt(-1)

sqrt(-1) returns "some NaN", it's not specified which exactly. --- libavutil/eval.c|2 +- tests/ref/fate/eval |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavutil/eval.c b/libavutil/eval.c index 4126cd7..9941ed7 100644 --- a/libavutil/eval.c +++ b/libavutil

Re: [libav-devel] [PATCH] Apple ProRes decoder

On Wed, Sep 21, 2011 at 02:07:13PM +0200, Tomas Härdin wrote: > On Wed, 2011-09-21 at 13:09 +0200, Diego Biurrun wrote: > > From: Maxim Poliakovski > > +static int decode_frame_header(ProresContext *ctx, const uint8_t *buf, > > + const int data_size, AVCodecContext *a

Re: [libav-devel] [PATCH 2/5] fate.sh: Run git-pull in quiet mode to avoid console spam.

Diego Biurrun writes: > Since fate.sh can be run from cron, silent commands are preferrable. > --- > tests/fate.sh |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/tests/fate.sh b/tests/fate.sh > index 6015155..df53192 100755 > --- a/tests/fate.sh > +++ b/tests/fate

Re: [libav-devel] [PATCH] flvdec: Ignore the index if it's from a creator known to be different

On 9/21/11 7:15 AM, Martin Storsjö wrote: --- libavformat/flvdec.c | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c index cafbeb5..569d734 100644 --- a/libavformat/flvdec.c +++ b/libavformat/flvdec.c @@ -140,6 +140

Re: [libav-devel] [PATCH 2/5] fate.sh: Run git-pull in quiet mode to avoid console spam.

On Tue, Sep 20, 2011 at 05:09:44PM +0200, Diego Biurrun wrote: > Since fate.sh can be run from cron, silent commands are preferrable. > --- > tests/fate.sh |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) ping Diego ___ libav-devel mailing

[libav-devel] [PATCH 2/2] tests/tools/examples: Replace direct exit() calls by return.

--- libavcodec/fft-test.c|5 + libavcodec/motion-test.c |4 +--- libavformat/output-example.c | 10 +- libavformat/seek-test.c |7 +++ tools/cws2fws.c | 18 +- 5 files changed, 19 insertions(+), 25 deletions(-) diff --g

[libav-devel] [PATCH 1/2] tests/examples: Mark some variables only used within their files as static.

--- libavcodec/dct-test.c|2 +- libavcodec/motion-test.c |4 ++-- libavformat/output-example.c | 16 libavutil/adler32.c |4 +++- 4 files changed, 14 insertions(+), 12 deletions(-) diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c index

Re: [libav-devel] [PATCH 1/2] flvenc: Write the right metadata entry count

On Wed, 21 Sep 2011, Alex Converse wrote: On Wed, Sep 21, 2011 at 7:14 AM, Martin Storsjö wrote: ---  libavformat/flvenc.c |   13 ++---  1 files changed, 10 insertions(+), 3 deletions(-) Based on my understanding that size is just a size hint. Yes, I don't know of any flv demuxer

Re: [libav-devel] [PATCH 1/2] flvenc: Write the right metadata entry count

On Wed, Sep 21, 2011 at 7:14 AM, Martin Storsjö wrote: > > --- >  libavformat/flvenc.c |   13 ++--- >  1 files changed, 10 insertions(+), 3 deletions(-) > Based on my understanding that size is just a size hint. ___ libav-devel mailing list liba

Re: [libav-devel] [PATCH 7/8] Fix js_vlc_bits value validation when joint stereo is used in cook decoder.

> No one? > It's a pretty straightforward issue, when .joint_stereo is true, the > decoder ends up accessing: > ccpl_huffbits[q->subpacket[i].js_vlc_bits-2] > ccpl_huffcodes[q->subpacket[i].js_vlc_bits-2] > Patch OK, sorry for the wait. MvH Benjamin Larson _

Re: [libav-devel] [PATCH v2] dct-test: factor out some common code and do whas was likely intended

On Wed, Sep 21, 2011 at 03:24:38PM +0100, Mans Rullgard wrote: > This fixes some nonsensical code by moving some should-be-shared > code to separate functions and using these. > > Signed-off-by: Mans Rullgard > --- > Bad sizeof expression now fixed. > --- > libavcodec/dct-test.c | 130 > ++

[libav-devel] [PATCH v2] dct-test: factor out some common code and do whas was likely intended

This fixes some nonsensical code by moving some should-be-shared code to separate functions and using these. Signed-off-by: Mans Rullgard --- Bad sizeof expression now fixed. --- libavcodec/dct-test.c | 130 + 1 files changed, 55 insertions(+), 75

[libav-devel] [PATCH] flvdec: Ignore the index if it's from a creator known to be different

--- libavformat/flvdec.c | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c index cafbeb5..569d734 100644 --- a/libavformat/flvdec.c +++ b/libavformat/flvdec.c @@ -140,6 +140,18 @@ static int parse_keyframes_index(AVForm

[libav-devel] [PATCH 2/2] flvenc: Don't copy metadata entries that are written by the muxer itself

This avoids duplicate, stale metadata entries if transcoding from one flv file to another. --- libavformat/flvenc.c | 20 1 files changed, 20 insertions(+), 0 deletions(-) diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c index 23d19cd..d9f692f 100644 --- a/libavform

[libav-devel] [PATCH 1/2] flvenc: Write the right metadata entry count

--- libavformat/flvenc.c | 13 ++--- 1 files changed, 10 insertions(+), 3 deletions(-) diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c index f3017d7..23d19cd 100644 --- a/libavformat/flvenc.c +++ b/libavformat/flvenc.c @@ -177,9 +177,9 @@ static int flv_write_header(AVFormatCon

Re: [libav-devel] [PATCH] dct-test: factor out some common code and do whas was likely intended

On Wed, Sep 21, 2011 at 02:31:00PM +0100, Mans Rullgard wrote: > This fixes some nonsensical code by moving some should-be-shared > code to separate functions and using these. > > Signed-off-by: Mans Rullgard > --- > libavcodec/dct-test.c | 130 > +--

[libav-devel] [PATCH] dct-test: factor out some common code and do whas was likely intended

This fixes some nonsensical code by moving some should-be-shared code to separate functions and using these. Signed-off-by: Mans Rullgard --- libavcodec/dct-test.c | 130 + 1 files changed, 55 insertions(+), 75 deletions(-) diff --git a/libavcode

[libav-devel] FATE instances

I got permission to use a very powerful x86_64 build host where I am now running three FATE instances. If somebody wants/needs some other config tested, let me know. Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailma

Re: [libav-devel] [PATCH] doc: library versions need to be bumped in version.h

On Wed, Sep 21, 2011 at 02:58:00PM +0200, Diego Biurrun wrote: > --- > doc/developer.texi |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/doc/developer.texi b/doc/developer.texi > index 25bfe5a..89e 100644 > --- a/doc/developer.texi > +++ b/doc/developer.texi >

[libav-devel] [PATCH] doc: library versions need to be bumped in version.h

--- doc/developer.texi |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/doc/developer.texi b/doc/developer.texi index 25bfe5a..89e 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -281,7 +281,7 @@ send a reminder by email. Your patch should eventually be de

Re: [libav-devel] [PATCH] Apple ProRes decoder

On Wed, 2011-09-21 at 13:09 +0200, Diego Biurrun wrote: > From: Maxim Poliakovski > +static int decode_frame_header(ProresContext *ctx, const uint8_t *buf, > + const int data_size, AVCodecContext *avctx) > +{ ... > + > +ctx->qmat_changed = 0; > +ptr = buf +

Re: [libav-devel] [PATCH] Revert "ffmpeg: get rid of useless AVInputStream.nb_streams."

On Wed, 21 Sep 2011, Anton Khirnov wrote: This reverts commit 2cf8355f98681bdd726b739008acd5483f82f8d7. AVInputStream.nb_streams tracks number of streams found at the beginning, new streams may appear that ffmpeg doesn't know about. Fixes crash in this case. --- ffmpeg.c |8 +--- 1 files

Re: [libav-devel] [PATCH] Apple ProRes decoder

On Wed, Sep 21, 2011 at 01:09:32PM +0200, Diego Biurrun wrote: > From: Maxim Poliakovski > > --- > Changelog |1 + > doc/general.texi |1 + > libavcodec/Makefile|1 + > libavcodec/allcodecs.c |1 + > libavcodec/proresdec.c | 726 >

[libav-devel] [PATCH] Apple ProRes decoder

From: Maxim Poliakovski --- Changelog |1 + doc/general.texi |1 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/proresdec.c | 726 libavcodec/version.h |2 +- 6 files changed, 731 i

Re: [libav-devel] Concurrent ProRes decoder

Maxim writes: > Hi crews, > > Yes, it's no mistake! What you see is just another code implementing > ProRes decoder. > > Now I'm going to explain that confusion. > Writing a ProRes compatible decoder began as team project in 2010. > During this period, I created an internal documentation on the c

Re: [libav-devel] Concurrent ProRes decoder

On Wed, Sep 21, 2011 at 03:42:38AM +0200, Maxim wrote: > > Yes, it's no mistake! What you see is just another code implementing > ProRes decoder. > > Now I'm going to explain that confusion. > Writing a ProRes compatible decoder began as team project in 2010. > During this period, I created an in

Re: [libav-devel] [PATCH 4/6] lavc: replace some deprecated FF_*_TYPE with AV_PICTURE_TYPE_*

On Thu, Sep 15, 2011 at 09:42:16AM +0200, Anton Khirnov wrote: > --- > libavcodec/dpxenc.c|2 +- > libavcodec/mpeg12.c|2 +- > libavcodec/mpegvideo.c |4 ++-- > 3 files changed, 4 insertions(+), 4 deletions(-) obviously OK ___ libav-

Re: [libav-devel] [PATCH 5/6] lavc/utils: move avcodec_init() higher in the file.

On Thu, Sep 15, 2011 at 09:42:17AM +0200, Anton Khirnov wrote: > Fixes build on next major bump. > --- > libavcodec/utils.c | 28 ++-- > 1 files changed, 14 insertions(+), 14 deletions(-) SureWhyNot ___ libav-devel mailing list

Re: [libav-devel] [PATCH 6/6] Remove some forgotten AVCodecContext.palctrl usage.

On Thu, Sep 15, 2011 at 09:42:18AM +0200, Anton Khirnov wrote: > --- > libavcodec/options.c |2 -- > libavformat/mov.c|2 -- > 2 files changed, 0 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/options.c b/libavcodec/options.c > index 6469ea7..ef711bf 100644 > --- a/libavcod

Re: [libav-devel] [PATCH 2/6] lavc: undeprecate AVPALETTE_SIZE and AVPALETTE_COUNT macros

On Thu, Sep 15, 2011 at 09:42:14AM +0200, Anton Khirnov wrote: > They are still being used and shouldn't be removed. > --- > libavcodec/avcodec.h |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index 7fc0e54..c330b45 1

[libav-devel] [PATCH] Revert "ffmpeg: get rid of useless AVInputStream.nb_streams."

This reverts commit 2cf8355f98681bdd726b739008acd5483f82f8d7. AVInputStream.nb_streams tracks number of streams found at the beginning, new streams may appear that ffmpeg doesn't know about. Fixes crash in this case. --- ffmpeg.c |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) d

Re: [libav-devel] Concurrent ProRes decoder

On Wed, Sep 21, 2011 at 03:42:38AM +0200, Maxim wrote: > Hi crews, > > Yes, it's no mistake! What you see is just another code implementing > ProRes decoder. > > Now I'm going to explain that confusion. > Writing a ProRes compatible decoder began as team project in 2010. > During this period, I c