Re: [libav-devel] [RFC] Make Bink IDCT take 32-bit coefficients as input

2011-07-27 Thread Kostya
On Sat, Jul 23, 2011 at 08:21:48PM +0200, Kostya wrote: On Sat, Jul 23, 2011 at 07:11:09PM +0100, Måns Rullgård wrote: Kostya kostya.shish...@gmail.com writes: diff --git a/libavcodec/binkdsp.c b/libavcodec/binkdsp.c new file mode 100644 index 000..456d71d ---

Re: [libav-devel] [RFC] futur of the fftools

2011-07-27 Thread Måns Rullgård
Thierry Foucu tfo...@gmail.com writes: 2011/7/26 Måns Rullgård m...@mansr.com Stefano Sabatini stefano.sabatini-l...@poste.it writes: On date Wednesday 2011-07-27 01:02:53 +0100, Måns Rullgård encoded: Stefano Sabatini stefano.sabatini-l...@poste.it writes: On date Tuesday 2011-07-26

Re: [libav-devel] [RFC] Make Bink IDCT take 32-bit coefficients as input

2011-07-27 Thread Måns Rullgård
Kostya kostya.shish...@gmail.com writes: On Sat, Jul 23, 2011 at 08:21:48PM +0200, Kostya wrote: On Sat, Jul 23, 2011 at 07:11:09PM +0100, Måns Rullgård wrote: Kostya kostya.shish...@gmail.com writes: diff --git a/libavcodec/binkdsp.c b/libavcodec/binkdsp.c new file mode 100644

Re: [libav-devel] [RFC] Make Bink IDCT take 32-bit coefficients as input

2011-07-27 Thread Kostya
On Wed, Jul 27, 2011 at 12:05:56PM +0100, Måns Rullgård wrote: Kostya kostya.shish...@gmail.com writes: On Sat, Jul 23, 2011 at 08:21:48PM +0200, Kostya wrote: On Sat, Jul 23, 2011 at 07:11:09PM +0100, Måns Rullgård wrote: Kostya kostya.shish...@gmail.com writes: diff --git

Re: [libav-devel] [RFC] Make Bink IDCT take 32-bit coefficients as input

2011-07-27 Thread Måns Rullgård
Kostya kostya.shish...@gmail.com writes: On Wed, Jul 27, 2011 at 12:05:56PM +0100, Måns Rullgård wrote: Kostya kostya.shish...@gmail.com writes: On Sat, Jul 23, 2011 at 08:21:48PM +0200, Kostya wrote: On Sat, Jul 23, 2011 at 07:11:09PM +0100, Måns Rullgård wrote: Kostya

Re: [libav-devel] [RFC] Make Bink IDCT take 32-bit coefficients as input

2011-07-27 Thread Kostya
On Wed, Jul 27, 2011 at 12:20:52PM +0100, Måns Rullgård wrote: Kostya kostya.shish...@gmail.com writes: On Wed, Jul 27, 2011 at 12:05:56PM +0100, Måns Rullgård wrote: Kostya kostya.shish...@gmail.com writes: On Sat, Jul 23, 2011 at 08:21:48PM +0200, Kostya wrote: On Sat, Jul 23,

Re: [libav-devel] [RFC] Make Bink IDCT take 32-bit coefficients as input

2011-07-27 Thread Måns Rullgård
Kostya kostya.shish...@gmail.com writes: @@ -1019,11 +1022,11 @@ static int bink_decode_plane(BinkContext *c, GetBitContext *gb, int plane_idx, ublock[*scan++] = get_value(c, BINK_SRC_COLORS); break; case INTRA_BLOCK: -

Re: [libav-devel] [RFC] Make Bink IDCT take 32-bit coefficients as input

2011-07-27 Thread Kostya
On Wed, Jul 27, 2011 at 12:55:00PM +0100, Måns Rullgård wrote: Kostya kostya.shish...@gmail.com writes: @@ -1019,11 +1022,11 @@ static int bink_decode_plane(BinkContext *c, GetBitContext *gb, int plane_idx, ublock[*scan++] = get_value(c, BINK_SRC_COLORS);

Re: [libav-devel] [PATCH 0/5] mpegts: Continuity counter improvements

2011-07-27 Thread aviad rozenhek
On Wed, Jul 20, 2011 at 13:26, aviad rozenhek avia...@gmail.com wrote: On Tue, Jul 19, 2011 at 18:03, Zohar Kelrich lumim...@gmail.com wrote: Improve the continuity counter check. Still unhandled is the case of duplicate TS packets. --- Zohar Kelrich (5): mpegts: Silence can't seek

Re: [libav-devel] [RFC] Make Bink IDCT take 32-bit coefficients as input

2011-07-27 Thread Måns Rullgård
Kostya kostya.shish...@gmail.com writes: On Wed, Jul 27, 2011 at 12:55:00PM +0100, Måns Rullgård wrote: Kostya kostya.shish...@gmail.com writes: @@ -1019,11 +1022,11 @@ static int bink_decode_plane(BinkContext *c, GetBitContext *gb, int plane_idx,

Re: [libav-devel] [RFC] futur of the fftools

2011-07-27 Thread Benjamin Larsson
On 07/27/2011 04:02 PM, Luca Barbato wrote: On 7/27/11 2:02 AM, Måns Rullgård wrote: One packet per line would be infinitely easier to parse than the current mess. A flag to choose between pretty, aligned columns and a single separator would be easy enough to implement. That might work

[libav-devel] [PATCH] dsputil: remove some unused functions

2011-07-27 Thread Mans Rullgard
Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/arm/dsputil_init_neon.c | 14 - libavcodec/arm/dsputil_neon.S | 104 libavcodec/dsputil.c | 72 - libavcodec/dsputil.h | 27 -

Re: [libav-devel] [PATCH] dsputil: remove some unused functions

2011-07-27 Thread Kostya
On Wed, Jul 27, 2011 at 03:33:41PM +0100, Mans Rullgard wrote: Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/arm/dsputil_init_neon.c | 14 - libavcodec/arm/dsputil_neon.S | 104 libavcodec/dsputil.c | 72

[libav-devel] [PATCH 1/2] dsputil: move a bink-only function to binkdsp

2011-07-27 Thread Mans Rullgard
Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/bink.c|2 +- libavcodec/binkdsp.c | 17 + libavcodec/binkdsp.h |1 + libavcodec/dsputil.c | 17 - libavcodec/dsputil.h |1 - 5 files changed, 19 insertions(+), 19 deletions(-) diff

Re: [libav-devel] [PATCH 1/2] dsputil: move a bink-only function to binkdsp

2011-07-27 Thread Kostya
On Wed, Jul 27, 2011 at 03:55:49PM +0100, Mans Rullgard wrote: Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/bink.c|2 +- libavcodec/binkdsp.c | 17 + libavcodec/binkdsp.h |1 + libavcodec/dsputil.c | 17 - libavcodec/dsputil.h |

Re: [libav-devel] [PATCH] dnxhdenc: fix declarations in for loops

2011-07-27 Thread Kostya
On Wed, Jul 27, 2011 at 04:14:43PM +0100, Mans Rullgard wrote: Apparently the gcc warning doesn't trigger on these. Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/dnxhdenc.c |8 +--- 1 files changed, 5 insertions(+), 3 deletions(-) OK

Re: [libav-devel] [PATCH] vp3/theora: flush after seek.

2011-07-27 Thread Luca Barbato
On 7/26/11 7:58 PM, Ronald S. Bultje wrote: From: Ronald S. Bultjersbul...@gmail.com Also error out if we don't have a reference instead of creating a dummy; consider this a RFC, I may remove this from the final patch. Do we have a proper way to signal error concealment? (I'm introducing

Re: [libav-devel] [PATCH 2/2] tiff: [RFC] Add simple parsing for GeoTIFF Keys

2011-07-27 Thread Måns Rullgård
Thomas Kühnel kuehne...@googlemail.com writes: +av_log(s-avctx, AV_LOG_DEBUG, Revision: %d.%d\n, tget_short(buf, s-le), tget_short(buf, s-le)); This line is bad. The order of evaluation of function arguments is undefined, so those tget_short() calls could happen in any order. --

Re: [libav-devel] [PATCH] vp3/theora: flush after seek.

2011-07-27 Thread Ronald S. Bultje
Hi, On Wed, Jul 27, 2011 at 9:21 AM, Luca Barbato lu_z...@gentoo.org wrote: On 7/26/11 7:58 PM, Ronald S. Bultje wrote: From: Ronald S. Bultjersbul...@gmail.com Also error out if we don't have a reference instead of creating a dummy; consider this a RFC, I may remove this from the final

[libav-devel] [PATCH 2/2] tiff: [RFC] Add simple parsing for GeoTIFF Keys

2011-07-27 Thread kuehnelth
Example output for ftp://ftp.remotesensing.org/pub/geotiff/samples/other/erdas_spnad83.tif Fro comparison listgeo output of the GeoTIFF data can be found here ftp://ftp.remotesensing.org/pub/geotiff/samples/other/erdas_spnad83.lgo $ ./ffprobe -loglevel debug

Re: [libav-devel] [RFC] futur of the fftools

2011-07-27 Thread Anton Khirnov
On Tue, 26 Jul 2011 18:35:36 +0200, Luca Barbato lu_z...@gentoo.org wrote: I'd keep a compat layer that triggers if argv[0] == ffmpeg for at least 2 major releases, beside that the plan seems sound. I think it'll be much simpler to keep them as completely separate programs. -- Anton

[libav-devel] [PATCH] h263dec: Propagate AV_LOG_ERRORs from slice decoding through frame decoding with sufficient error reporting

2011-07-27 Thread Dustin Brody
--- doc/ffmpeg.texi |2 ++ libavcodec/avcodec.h |1 + libavcodec/h263dec.c |6 +++--- libavcodec/options.c |1 + 4 files changed, 7 insertions(+), 3 deletions(-) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index 5a783e3..4dca5d8 100644 --- a/doc/ffmpeg.texi +++

Re: [libav-devel] [PATCH] h263dec: Propagate AV_LOG_ERRORs from slice decoding through frame decoding with sufficient error reporting

2011-07-27 Thread Ronald S. Bultje
Hi, On Wed, Jul 27, 2011 at 3:48 PM, Dustin Brody li...@parsoma.net wrote: ---  doc/ffmpeg.texi      |    2 ++  libavcodec/avcodec.h |    1 +  libavcodec/h263dec.c |    6 +++---  libavcodec/options.c |    1 +  4 files changed, 7 insertions(+), 3 deletions(-) diff --git a/doc/ffmpeg.texi

[libav-devel] [PATCH] h263dec: Propagate AV_LOG_ERRORs from slice decoding through frame decoding with sufficient error recognition

2011-07-27 Thread Dustin Brody
--- doc/ffmpeg.texi |2 ++ libavcodec/avcodec.h |1 + libavcodec/h263dec.c |6 +++--- libavcodec/options.c |1 + 4 files changed, 7 insertions(+), 3 deletions(-) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index 5a783e3..4dca5d8 100644 --- a/doc/ffmpeg.texi +++

Re: [libav-devel] [PATCH] vp3/theora: flush after seek.

2011-07-27 Thread Luca Barbato
On 7/27/11 7:50 PM, Ronald S. Bultje wrote: Hi, On Wed, Jul 27, 2011 at 9:21 AM, Luca Barbatolu_z...@gentoo.org wrote: On 7/26/11 7:58 PM, Ronald S. Bultje wrote: From: Ronald S. Bultjersbul...@gmail.com Also error out if we don't have a reference instead of creating a dummy; consider this

Re: [libav-devel] [PATCH] h263dec: Propagate AV_LOG_ERRORs from slice decoding through frame decoding with sufficient error recognition

2011-07-27 Thread Ronald S. Bultje
Hi, On Wed, Jul 27, 2011 at 4:09 PM, Dustin Brody li...@parsoma.net wrote: ---  doc/ffmpeg.texi      |    2 ++  libavcodec/avcodec.h |    1 +  libavcodec/h263dec.c |    6 +++---  libavcodec/options.c |    1 +  4 files changed, 7 insertions(+), 3 deletions(-) LGTM. I'll give this a day or

Re: [libav-devel] [PATCH 2/3] ppc: remove redundant setting of Altivec IDCT

2011-07-27 Thread Luca Barbato
On 7/25/11 5:18 PM, Mans Rullgard wrote: This is already set by dsputil_init_ppc() and is best done in only one place. Ok. I'm thinking about looking for somebody (freescale? IBM) providing us with some speedy PPC access (ssh might be enough), a Power7 or a QorIQ should do fine to test

Re: [libav-devel] [PATCH 3/3] ppc: swscale: disable Altivec scaler for 8-bit input

2011-07-27 Thread Luca Barbato
On 7/25/11 7:21 PM, Måns Rullgård wrote: Ronald S. Bultjersbul...@gmail.com writes: Hi, On Mon, Jul 25, 2011 at 9:58 AM, Ronald S. Bultjersbul...@gmail.com wrote: Hi, 2011/7/25 Måns Rullgårdm...@mansr.com: Ronald S. Bultjersbul...@gmail.com writes: On Mon, Jul 25, 2011 at 8:18 AM, Mans

Re: [libav-devel] [PATCH 1/3] mpegvideo: initialise DSPContext in ff_dct_common_init()

2011-07-27 Thread Luca Barbato
On 7/25/11 5:18 PM, Mans Rullgard wrote: 1 files changed, 2 insertions(+), 1 deletions(-) Seems fine. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH] seeking in asf (and possibly others)

2011-07-27 Thread John Stebbins
First, a disclaimer. I don't know if this is the right way to fix this problem. So I'll describe in detail what is happening and maybe someone can give me more guidance. I have a wmv file with wmv3 video. It flags itself as having b-frames. avformat_seek_file always fails. The sequence of

Re: [libav-devel] [PATCH] seeking in asf (and possibly others)

2011-07-27 Thread Ronald S. Bultje
Hi John, On Wed, Jul 27, 2011 at 5:10 PM, John Stebbins stebb...@jetheaddev.com wrote: First, a disclaimer.  I don't know if this is the right way to fix this problem.  So I'll describe in detail what is happening and maybe someone can give me more guidance. I have a wmv file with wmv3 video.  

[libav-devel] [PATCH] mpeg4videodec: remove dead code that would have detected erroneous encoding

2011-07-27 Thread Dustin Brody
--- libavcodec/mpeg4videodec.c | 22 -- 1 files changed, 0 insertions(+), 22 deletions(-) diff --git a/libavcodec/mpeg4videodec.c b/libavcodec/mpeg4videodec.c index b1abb61..4d5e970 100644 --- a/libavcodec/mpeg4videodec.c +++ b/libavcodec/mpeg4videodec.c @@ -990,28 +990,6

Re: [libav-devel] [PATCH] seeking in asf (and possibly others)

2011-07-27 Thread John Stebbins
On 07/27/2011 05:37 PM, Måns Rullgård wrote: John Stebbinsstebb...@jetheaddev.com writes: So my thought is, if the pkt is a keyframe, it should be ok to set pts = dts (when pts has not been specified) since keyframes are not delayed. That is not true. I was afraid of that. Care to