Re: [libav-devel] duplicating frames in avconv

2012-12-09 Thread Anton Khirnov
On Tue, 04 Dec 2012 14:37:33 +0100, Arash Shafiei wrote: > Hi, > > When we use avconv sometimes it duplicates frames to achieve a specified > frame-rate. > > I would like to know at which layer this is implemented. Is this > internal to libavcodec which automatically duplicates frames when it

Re: [libav-devel] [PATCH 0/3] lavr: dithered sample format conversion

2012-12-09 Thread Justin Ruggles
On 12/09/2012 03:16 PM, Justin Ruggles wrote: > av_lfg_get_flt() now takes a scale factor, which makes it more flexible. > > Modified the simple dithering functions to pre-generate the dither noise > in order to use SIMD conversion and addition. Includes SSE2 and FMA4 > implementations. > > Justi

Re: [libav-devel] [PATCH 1/2] rtpdec: Rename a static variable to normal naming conventions

2012-12-09 Thread Luca Barbato
On 12/9/12 8:05 PM, Martin Storsjö wrote: --- libavformat/rtpdec.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) Both patches ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/lib

Re: [libav-devel] duplicating frames in avconv

2012-12-09 Thread Luca Barbato
On 12/4/12 2:37 PM, Arash Shafiei wrote: Hi, When we use avconv sometimes it duplicates frames to achieve a specified frame-rate. I would like to know at which layer this is implemented. Is this internal to libavcodec which automatically duplicates frames when it does not receives enough data o

Re: [libav-devel] [PATCH 3/4] x86: ac3dsp: replace inline asm for in-decoder downmixing with yasm asm

2012-12-09 Thread Måns Rullgård
Diego Biurrun writes: > On Sat, Nov 17, 2012 at 10:33:27AM -0500, Justin Ruggles wrote: >> >> --- a/libavcodec/ac3dsp.c >> +++ b/libavcodec/ac3dsp.c >> @@ -24,6 +24,10 @@ >> #include "ac3.h" >> #include "ac3dsp.h" >> >> +#if ARCH_X86 >> +#include "x86/ac3dsp.h" >> +#endif >> + >> --- /dev/nu

Re: [libav-devel] [PATCH 2/4] ac3dsp: add some special-case handling for the C downmix function

2012-12-09 Thread Måns Rullgård
Justin Ruggles writes: > This is about 200% faster for in-decoder downmixing of 5.0 and 5.1 content. > --- > libavcodec/ac3dsp.c | 39 ++- > 1 files changed, 38 insertions(+), 1 deletions(-) > > diff --git a/libavcodec/ac3dsp.c b/libavcodec/ac3dsp.c > index

Re: [libav-devel] [PATCH 1/4] ac3dsp: reverse matrix in/out order in downmix()

2012-12-09 Thread Måns Rullgård
Justin Ruggles writes: > Also use (float **) instead of (float (*)[2]). This matches the matrix > layout in libavresample so we can reuse some assembly code between the > two. > --- > libavcodec/ac3dec.c | 40 +++- > libavcodec/ac3dec.h |

Re: [libav-devel] [PATCH] sh4: dsputil: remove duplicate of ff_gmc_c()

2012-12-09 Thread Diego Biurrun
On Sun, Dec 09, 2012 at 09:19:14PM +, Mans Rullgard wrote: > This function is an exact duplicate of the generic one. > > Signed-off-by: Mans Rullgard > --- > libavcodec/sh4/dsputil_align.c | 1 - > libavcodec/sh4/qpel.c | 57 > -- > 2 files

[libav-devel] [PATCH] sh4: dsputil: remove duplicate of ff_gmc_c()

2012-12-09 Thread Mans Rullgard
This function is an exact duplicate of the generic one. Signed-off-by: Mans Rullgard --- libavcodec/sh4/dsputil_align.c | 1 - libavcodec/sh4/qpel.c | 57 -- 2 files changed, 58 deletions(-) diff --git a/libavcodec/sh4/dsputil_align.c b/libavcod

[libav-devel] [PATCH 3/3] lavr: add option for dithering during sample format conversion to s16

2012-12-09 Thread Justin Ruggles
--- libavresample/Makefile |1 + libavresample/audio_convert.c | 33 +++- libavresample/audio_convert.h | 22 ++- libavresample/avresample.h |9 + libavresample/dither.c | 475 +++ libavresample/dither.h | 71 ++

[libav-devel] [PATCH 0/3] lavr: dithered sample format conversion

2012-12-09 Thread Justin Ruggles
av_lfg_get_flt() now takes a scale factor, which makes it more flexible. Modified the simple dithering functions to pre-generate the dither noise in order to use SIMD conversion and addition. Includes SSE2 and FMA4 implementations. Justin Ruggles (3): lavu: add av_lfg_get_flt() to get a random

[libav-devel] [PATCH 1/3] lavu: add av_lfg_get_flt() to get a random float

2012-12-09 Thread Justin Ruggles
--- doc/APIchanges |3 +++ libavutil/lfg.h | 19 +++ libavutil/version.h |2 +- 3 files changed, 23 insertions(+), 1 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 1c6247e..4fb94c8 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -13,6 +13,9

[libav-devel] [PATCH 2/3] eac3dec: use av_lfg_get_flt() for SPX noise generation

2012-12-09 Thread Justin Ruggles
--- libavcodec/eac3dec.c |4 ++-- tests/fate/ac3.mak |4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/eac3dec.c b/libavcodec/eac3dec.c index 639e061..ba4d5c6 100644 --- a/libavcodec/eac3dec.c +++ b/libavcodec/eac3dec.c @@ -140,10 +140,10 @@ void ff_eac3_

Re: [libav-devel] [PATCH] ZeroCodec: Flip output

2012-12-09 Thread Kostya Shishkov
On Sun, Dec 09, 2012 at 02:14:09PM -0500, Derek Buitenhuis wrote: > From: Carl Eugen Hoyos > > The initial testing of the VFW binary codec was flawed, > likely due to an AviSynth bug. > > Re-testing using VirtualDub and various professional editing > applications has revealed it should have been

[libav-devel] [PATCH] ZeroCodec: Flip output

2012-12-09 Thread Derek Buitenhuis
From: Carl Eugen Hoyos The initial testing of the VFW binary codec was flawed, likely due to an AviSynth bug. Re-testing using VirtualDub and various professional editing applications has revealed it should have been flipped. Signed-off-by: Derek Buitenhuis --- For the record, I retested these

[libav-devel] [PATCH 2/2] rtpdec: Remove an outdated todo comment

2012-12-09 Thread Martin Storsjö
This comment was added in e309128f, in 2002, and has been brought along since then more or less unmodified. The first point of the todo was implemented in dbf30963 in 2006, the second one is not relevant to rtpdec.c (brought along from rtp.c in 8eb793c4 in 2008) but would be more relevant to the r

[libav-devel] [PATCH 1/2] rtpdec: Rename a static variable to normal naming conventions

2012-12-09 Thread Martin Storsjö
--- libavformat/rtpdec.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/rtpdec.c b/libavformat/rtpdec.c index 38ce7f6..d20c626 100644 --- a/libavformat/rtpdec.c +++ b/libavformat/rtpdec.c @@ -59,12 +59,12 @@ static RTPDynamicProtocolHandler opus_dynami

Re: [libav-devel] [PATCH] configure: fix automatic processing of _extralibs in check_deps

2012-12-09 Thread Diego Biurrun
On Sun, Dec 09, 2012 at 05:34:24PM +, Mans Rullgard wrote: > This fixes the automatic use of $foo_extralibs when feature foo > is enabled indirectly through a _select or _suggest. > > Signed-off-by: Mans Rullgard > --- > configure | 19 +++ > 1 file changed, 15 insertions(+),

[libav-devel] duplicating frames in avconv

2012-12-09 Thread Arash Shafiei
Hi, When we use avconv sometimes it duplicates frames to achieve a specified frame-rate. I would like to know at which layer this is implemented. Is this internal to libavcodec which automatically duplicates frames when it does not receives enough data or it's the user of libavcodec which ta

Re: [libav-devel] [RFC] avio checksum functionality

2012-12-09 Thread Luca Barbato
On 12/9/12 6:26 PM, Måns Rullgård wrote: Luca Barbato writes: On 12/9/12 5:37 PM, Måns Rullgård wrote: Luca Barbato writes: On 12/9/12 1:32 PM, Måns Rullgård wrote: Luca Barbato writes: On 12/09/2012 05:34 AM, Luca Barbato wrote: Hi, I'm wondering if we could completely hide the avio

Re: [libav-devel] [PATCH] configure: fix automatic processing of _extralibs in check_deps

2012-12-09 Thread Sean McGovern
On Sun, Dec 9, 2012 at 12:34 PM, Mans Rullgard wrote: > This fixes the automatic use of $foo_extralibs when feature foo > is enabled indirectly through a _select or _suggest. > > Signed-off-by: Mans Rullgard > --- > configure | 19 +++ > 1 file changed, 15 insertions(+), 4 deleti

[libav-devel] [PATCH] configure: fix automatic processing of _extralibs in check_deps

2012-12-09 Thread Mans Rullgard
This fixes the automatic use of $foo_extralibs when feature foo is enabled indirectly through a _select or _suggest. Signed-off-by: Mans Rullgard --- configure | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/configure b/configure index fcfd5ee..7340360 10

Re: [libav-devel] [RFC] avio checksum functionality

2012-12-09 Thread Måns Rullgård
Luca Barbato writes: > On 12/9/12 5:37 PM, Måns Rullgård wrote: >> Luca Barbato writes: >> >>> On 12/9/12 1:32 PM, Måns Rullgård wrote: Luca Barbato writes: > On 12/09/2012 05:34 AM, Luca Barbato wrote: >> Hi, I'm wondering if we could completely hide the avio checksum fields

[libav-devel] [PATCH] misc typo, style and wording fixes

2012-12-09 Thread Diego Biurrun
--- Log message improved and log.h text fixed according to feedback. libavcodec/arm/fmtconvert_vfp.S |2 +- libavcodec/options_table.h | 106 +++--- libavcodec/sh4/dsputil_align.c |2 +- libavformat/avio_internal.h |2 +- libavformat/bethsof

Re: [libav-devel] [RFC] avio checksum functionality

2012-12-09 Thread Luca Barbato
On 12/9/12 5:37 PM, Måns Rullgård wrote: Luca Barbato writes: On 12/9/12 1:32 PM, Måns Rullgård wrote: Luca Barbato writes: On 12/09/2012 05:34 AM, Luca Barbato wrote: Hi, I'm wondering if we could completely hide the avio checksum fields from the public structure by moving it to an inter

Re: [libav-devel] [RFC] avio checksum functionality

2012-12-09 Thread Måns Rullgård
Luca Barbato writes: > On 12/9/12 1:32 PM, Måns Rullgård wrote: >> Luca Barbato writes: >> >>> On 12/09/2012 05:34 AM, Luca Barbato wrote: Hi, I'm wondering if we could completely hide the avio checksum fields from the public structure by moving it to an internal field or extend it >>>

Re: [libav-devel] [libav-commits] configure: add check_insn function

2012-12-09 Thread Diego Biurrun
On Sun, Dec 09, 2012 at 05:18:21PM +0100, Luca Barbato wrote: > On 12/9/12 3:47 PM, Diego Biurrun wrote: > >On Fri, Dec 07, 2012 at 07:46:09PM +0100, Mans Rullgard wrote: > >>--- a/configure > >>+++ b/configure > >>@@ -698,6 +698,12 @@ void foo(void){ __asm__ volatile($code); } > >> > >>+check_insn

Re: [libav-devel] [RFC] avio checksum functionality

2012-12-09 Thread Luca Barbato
On 12/9/12 1:32 PM, Måns Rullgård wrote: Luca Barbato writes: On 12/09/2012 05:34 AM, Luca Barbato wrote: Hi, I'm wondering if we could completely hide the avio checksum fields from the public structure by moving it to an internal field or extend it to be able to use any hash function from av

Re: [libav-devel] [libav-commits] configure: add check_insn function

2012-12-09 Thread Luca Barbato
On 12/9/12 3:47 PM, Diego Biurrun wrote: On Fri, Dec 07, 2012 at 07:46:09PM +0100, Mans Rullgard wrote: --- a/configure +++ b/configure @@ -698,6 +698,12 @@ void foo(void){ __asm__ volatile($code); } +check_insn(){ +log check_insn "$@" +check_inline_asm ${1}_inline "\"$2\"" +echo "$

Re: [libav-devel] [PATCH 2/2] misc typo fixes

2012-12-09 Thread Måns Rullgård
Diego Biurrun writes: >> > -{"qmin", "min video quantizer scale (VBR)", OFFSET(qmin), >> > AV_OPT_TYPE_INT, {.i64 = 2 }, -1, 69, V|E}, >> > -{"qmax", "max video quantizer scale (VBR)", OFFSET(qmax), >> > AV_OPT_TYPE_INT, {.i64 = 31 }, -1, 69, V|E}, >> > -{"qdiff", "max difference between the qu

Re: [libav-devel] [libav-commits] configure: add check_insn function

2012-12-09 Thread Janne Grunau
On 2012-12-09 15:47:09 +0100, Diego Biurrun wrote: > On Fri, Dec 07, 2012 at 07:46:09PM +0100, Mans Rullgard wrote: > > --- a/configure > > +++ b/configure > > @@ -698,6 +698,12 @@ void foo(void){ __asm__ volatile($code); } > > > > +check_insn(){ > > +log check_insn "$@" > > +check_inline

Re: [libav-devel] [libav-commits] configure: add check_insn function

2012-12-09 Thread Diego Biurrun
On Fri, Dec 07, 2012 at 07:46:09PM +0100, Mans Rullgard wrote: > --- a/configure > +++ b/configure > @@ -698,6 +698,12 @@ void foo(void){ __asm__ volatile($code); } > > +check_insn(){ > +log check_insn "$@" > +check_inline_asm ${1}_inline "\"$2\"" > +echo "$2" | check_as && enable ${1

Re: [libav-devel] [PATCH 2/2] misc typo fixes

2012-12-09 Thread Diego Biurrun
On Sun, Dec 09, 2012 at 02:16:14PM +, Måns Rullgård wrote: > Diego Biurrun writes: > > -{"mv4", "use four motion vector by macroblock (mpeg4)", 0, > > AV_OPT_TYPE_CONST, {.i64 = CODEC_FLAG_4MV }, INT_MIN, INT_MAX, V|E, > > "flags"}, > > +{"mv4", "use four motion vectors per macroblock (MPEG-

Re: [libav-devel] [PATCH 2/2] misc typo fixes

2012-12-09 Thread Måns Rullgård
Diego Biurrun writes: > --- > libavcodec/arm/fmtconvert_vfp.S |2 +- > libavcodec/options_table.h | 98 +++--- > libavcodec/sh4/dsputil_align.c |2 +- > libavformat/avio_internal.h |2 +- > libavformat/bethsoftvid.c |2 +- > libav

[libav-devel] [PATCH 1/2] cosmetics: Use consistent names for multiple inclusion guards.

2012-12-09 Thread Diego Biurrun
--- libavcodec/gsmdec_data.h|6 +++--- libavcodec/options_table.h |6 +++--- libavformat/options_table.h |6 +++--- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/libavcodec/gsmdec_data.h b/libavcodec/gsmdec_data.h index 61715c8..2f3a2d2 100644 --- a/libavcodec/gsm

[libav-devel] [PATCH 2/2] misc typo fixes

2012-12-09 Thread Diego Biurrun
--- libavcodec/arm/fmtconvert_vfp.S |2 +- libavcodec/options_table.h | 98 +++--- libavcodec/sh4/dsputil_align.c |2 +- libavformat/avio_internal.h |2 +- libavformat/bethsoftvid.c |2 +- libavformat/nsvdec.c|2 +-

Re: [libav-devel] [PATCH] dsputil: remove unused macro WRAPPER8_16

2012-12-09 Thread Diego Biurrun
On Sun, Dec 09, 2012 at 01:28:53PM +, Mans Rullgard wrote: > This macro has never been used. > > Signed-off-by: Mans Rullgard > --- > libavcodec/dsputil.h | 6 -- > 1 file changed, 6 deletions(-) OK Diego ___ libav-devel mailing list libav-de

[libav-devel] [PATCH] dsputil: remove unused macro WRAPPER8_16

2012-12-09 Thread Mans Rullgard
This macro has never been used. Signed-off-by: Mans Rullgard --- libavcodec/dsputil.h | 6 -- 1 file changed, 6 deletions(-) diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h index c3ab25e..5d4ab33 100644 --- a/libavcodec/dsputil.h +++ b/libavcodec/dsputil.h @@ -622,12 +622,6 @@ void

Re: [libav-devel] [RFC] avio checksum functionality

2012-12-09 Thread Måns Rullgård
Luca Barbato writes: > On 12/09/2012 05:34 AM, Luca Barbato wrote: >> Hi, I'm wondering if we could completely hide the avio checksum fields >> from the public structure by moving it to an internal field or extend it >> to be able to use any hash function from avutil and fully expose it to >> our

Re: [libav-devel] [RFC] avio checksum functionality

2012-12-09 Thread Luca Barbato
On 12/09/2012 05:34 AM, Luca Barbato wrote: > Hi, I'm wondering if we could completely hide the avio checksum fields > from the public structure by moving it to an internal field or extend it > to be able to use any hash function from avutil and fully expose it to > our users. > > The current situ

Re: [libav-devel] [PATCH] avio: K&R cosmetic formatting

2012-12-09 Thread Anton Khirnov
On Sun, 9 Dec 2012 04:16:09 +0100, Luca Barbato wrote: > --- > libavformat/aviobuf.c | 121 > -- > 1 file changed, 67 insertions(+), 54 deletions(-) > Ok -- Anton Khirnov ___ libav-devel mailing lis