Re: [libav-devel] [PATCH 4/5] lavr: rename the "cutoff" option to "resample_cutoff"

2012-12-18 Thread Anton Khirnov
On Tue, 18 Dec 2012 21:58:23 -0500, Justin Ruggles wrote: > Avoids an option name conflict with libavcodec. > --- I don't like renaming options just because of avconv deficiencies. We can add the other name as a temporary hack to make it work with avconv, but in the long run avconv should just

Re: [libav-devel] [PATCH] Merge latest x86inc.asm fixes with x264.

2012-12-18 Thread Janne Grunau
On 2012-12-18 17:02:31 -0800, Ronald S. Bultje wrote: > From: "Ronald S. Bultje" > > --- > libavutil/x86/x86inc.asm | 33 - > 1 file changed, 20 insertions(+), 13 deletions(-) > > diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm > index 60d05f4..2

Re: [libav-devel] [PATCH 0/5] lavr: connect options to lavfi and avconv

2012-12-18 Thread Luca Barbato
On 12/19/12 3:58 AM, Justin Ruggles wrote: Justin Ruggles (5): lavu/dict: add av_dict_set_string() Looks fine lavfi: connect libavresample options to af_resample via AVFilterGraph Ok. avconv: pass libavresample options to AVFilterGraph Ok. lavr: rename the "cutoff" option

Re: [libav-devel] [PATCH] Merge latest x86inc.asm fixes with x264.

2012-12-18 Thread Dave Yeo
On 12/18/12 05:02 pm, Ronald S. Bultje wrote: From: "Ronald S. Bultje" --- libavutil/x86/x86inc.asm | 33 - 1 file changed, 20 insertions(+), 13 deletions(-) Fixes 32 bit NASM assembly here, thanks Dave ___ libav-de

[libav-devel] [PATCH 5/5] lavr: add AV_OPT_FLAG_ENCODING_PARAM to options

2012-12-18 Thread Justin Ruggles
--- libavresample/options.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavresample/options.c b/libavresample/options.c index fe07d8d..31d2efb 100644 --- a/libavresample/options.c +++ b/libavresample/options.c @@ -31,7 +31,7 @@ */ #define OFFSET(x) offsetof(AVA

[libav-devel] [PATCH 4/5] lavr: rename the "cutoff" option to "resample_cutoff"

2012-12-18 Thread Justin Ruggles
Avoids an option name conflict with libavcodec. --- doc/APIchanges |4 libavresample/options.c |3 +++ libavresample/version.h |5 - 3 files changed, 11 insertions(+), 1 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 7a700ce..2a2f294 100644 --- a/doc

[libav-devel] [PATCH 3/5] avconv: pass libavresample options to AVFilterGraph

2012-12-18 Thread Justin Ruggles
--- avconv_filter.c | 23 ++- avconv_opt.c|3 +++ cmdutils.c | 12 ++-- cmdutils.h |3 ++- 4 files changed, 37 insertions(+), 4 deletions(-) diff --git a/avconv_filter.c b/avconv_filter.c index 50e1e73..2fef547 100644 --- a/avconv_filter.c +++

[libav-devel] [PATCH 2/5] lavfi: connect libavresample options to af_resample via AVFilterGraph

2012-12-18 Thread Justin Ruggles
--- doc/APIchanges |4 libavfilter/af_resample.c | 30 ++ libavfilter/avfiltergraph.c |7 ++- libavfilter/avfiltergraph.h |1 + libavfilter/version.h |2 +- 5 files changed, 42 insertions(+), 2 deletions(-) diff --git a/d

[libav-devel] [PATCH 1/5] lavu/dict: add av_dict_set_string()

2012-12-18 Thread Justin Ruggles
Can be used to set multiple key/value pairs from a string. --- doc/APIchanges |4 libavutil/dict.c| 49 + libavutil/dict.h| 17 + libavutil/version.h |2 +- 4 files changed, 71 insertions(+), 1 deletions(

[libav-devel] [PATCH 0/5] lavr: connect options to lavfi and avconv

2012-12-18 Thread Justin Ruggles
Justin Ruggles (5): lavu/dict: add av_dict_set_string() lavfi: connect libavresample options to af_resample via AVFilterGraph avconv: pass libavresample options to AVFilterGraph lavr: rename the "cutoff" option to "resample_cutoff" lavr: add AV_OPT_FLAG_ENCODING_PARAM to options avconv_

[libav-devel] [PATCH] Merge latest x86inc.asm fixes with x264.

2012-12-18 Thread Ronald S. Bultje
From: "Ronald S. Bultje" --- libavutil/x86/x86inc.asm | 33 - 1 file changed, 20 insertions(+), 13 deletions(-) diff --git a/libavutil/x86/x86inc.asm b/libavutil/x86/x86inc.asm index 60d05f4..2617cdf 100644 --- a/libavutil/x86/x86inc.asm +++ b/libavutil/x86/x86in

Re: [libav-devel] [PATCH 2/3] h264: clear picture copies when unreferencing them

2012-12-18 Thread Ronald S. Bultje
Hi, On Mon, Dec 17, 2012 at 11:56 PM, Janne Grunau wrote: > On 2012-12-17 18:27:17 -0800, Ronald S. Bultje wrote: >> Hi, >> >> On Dec 17, 2012 1:41 PM, "Janne Grunau" wrote: >> > >> > Copy a neighbouring frame/field from the list as error resilience >> > measure since the decoder assumes frame d

Re: [libav-devel] [PATCH 1/1] mpegvideo: increase edge_emu_buffer size for VC1

2012-12-18 Thread Ronald S. Bultje
Hi, On Tue, Dec 18, 2012 at 2:06 PM, Janne Grunau wrote: > The VC1 decoder uses edge_emu_buffer simultaneously for luma and chroma > and needs more space. That was not a problem before f1d8763a02b5fce > since the size for edge_emu_buffer was always calculated with 2 byte per > pixel since the lin

[libav-devel] [PATCH 1/1] mpegvideo: increase edge_emu_buffer size for VC1

2012-12-18 Thread Janne Grunau
The VC1 decoder uses edge_emu_buffer simultaneously for luma and chroma and needs more space. That was not a problem before f1d8763a02b5fce since the size for edge_emu_buffer was always calculated with 2 byte per pixel since the linesize was not known. Fixes occasionally fate errors in vc1_sa10143

Re: [libav-devel] [PATCH] lavfi: Add the alength audio length filter.

2012-12-18 Thread Anton Khirnov
On Tue, 18 Dec 2012 20:52:05 +0100, Anton Khirnov wrote: > > On Tue, 18 Dec 2012 10:50:13 -0800, Alex Converse > wrote: > > --- > > Changelog|1 + > > doc/filters.texi | 34 > > libavfilter/Makefile |1 + > > libavfilter/af_alength.c | 210 > >

Re: [libav-devel] [PATCH] lavfi: Add the alength audio length filter.

2012-12-18 Thread Anton Khirnov
On Tue, 18 Dec 2012 10:50:13 -0800, Alex Converse wrote: > --- > Changelog|1 + > doc/filters.texi | 34 > libavfilter/Makefile |1 + > libavfilter/af_alength.c | 210 > ++ > libavfilter/allfilters.c |

Re: [libav-devel] [PATCH] lavfi: Add the alength audio length filter.

2012-12-18 Thread Justin Ruggles
On 12/18/2012 01:50 PM, Alex Converse wrote: > +if (!strcmp(s->length, "-1")) > +s->scaled_length = AV_NOPTS_VALUE; > +else > + if ((err = av_parse_time(&s->scaled_length, s->length, 1))) > + return err; Does av_parse_time() allow negative values? If so, it might ma

Re: [libav-devel] [PATCH] mkv: add support for the VP9 tag

2012-12-18 Thread Luca Barbato
On 12/18/2012 04:31 PM, Ronald S. Bultje wrote: > Hi, > > On Mon, Dec 17, 2012 at 8:16 PM, Luca Barbato wrote: >> On 12/16/12 2:09 AM, Luca Barbato wrote: >>> >>> From: Tom Finegan >> >> >>> +AV_CODEC_ID_VP9, >>> >> >> Ping > > I had already said OK - if you add a big phat warning saying th

Re: [libav-devel] [PATCH] lavc: introduce VideoDSPContext

2012-12-18 Thread Luca Barbato
On 12/18/2012 04:40 PM, Ronald S. Bultje wrote: > Hi, > > On Tue, Dec 18, 2012 at 3:19 AM, Diego Biurrun wrote: >> On Mon, Dec 17, 2012 at 09:48:48PM +0100, Luca Barbato wrote: >>> --- a/configure >>> +++ b/configure >>> @@ -1544,6 +1545,7 @@ mpeg4_decoder_select="h263_decoder mpeg4video_parser"

Re: [libav-devel] [PATCH 1/1] avcodec: bump minor for adaptive h264 frame-mt

2012-12-18 Thread Anton Khirnov
On Tue, 18 Dec 2012 20:12:49 +0100, Janne Grunau wrote: > Also adds forgotten Changelog entry. > --- > Changelog| 1 + > libavcodec/version.h | 4 ++-- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/Changelog b/Changelog > index b4a8e60..8dfffcf 100644 > --- a/C

[libav-devel] [PATCH 1/1] avcodec: bump minor for adaptive h264 frame-mt

2012-12-18 Thread Janne Grunau
Also adds forgotten Changelog entry. --- Changelog| 1 + libavcodec/version.h | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/Changelog b/Changelog index b4a8e60..8dfffcf 100644 --- a/Changelog +++ b/Changelog @@ -8,6 +8,7 @@ version : - deprecated the avconv

[libav-devel] [PATCH] lavfi: Add the alength audio length filter.

2012-12-18 Thread Alex Converse
--- Changelog|1 + doc/filters.texi | 34 libavfilter/Makefile |1 + libavfilter/af_alength.c | 210 ++ libavfilter/allfilters.c |1 + libavfilter/version.h|2 +- 6 files changed, 248 insertions

Re: [libav-devel] [PATCH 1/1] h264: increase dist_scale_factor for up to 32 references

2012-12-18 Thread Janne Grunau
On 2012-12-18 10:07:59 -0800, Ronald S. Bultje wrote: > Hi, > > On Tue, Dec 18, 2012 at 8:20 AM, Janne Grunau wrote: > > On 2012-12-18 07:30:03 -0800, Ronald S. Bultje wrote: > >> Hi, > >> > >> On Mon, Dec 17, 2012 at 10:55 PM, Janne Grunau > >> wrote: > >> > On 2012-12-17 18:24:37 -0800, Ronal

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

2012-12-18 Thread Justin Ruggles
On 12/18/2012 01:15 PM, Anton Khirnov wrote: > > On Sun, 16 Dec 2012 16:27:29 -0500, Justin Ruggles > wrote: [...] >> + >> +if (c->method == AV_RESAMPLE_DITHER_TRIANGULAR_NS) { >> +if (sample_rate == 48000) { >> +c->ns_coef_b = ns_48_coef_b; >> +c->ns_coef_a =

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

2012-12-18 Thread Anton Khirnov
On Sun, 16 Dec 2012 16:27:29 -0500, Justin Ruggles wrote: > +static int generate_dither_noise(DitherContext *c, DitherState *state, > + int min_samples) > +{ > +int i; > +int nb_samples = FFALIGN(min_samples, 16) + 16; > +int buf_samples = nb_samples

Re: [libav-devel] [PATCH 1/1] h264: support frame parameter changes during frame-mt

2012-12-18 Thread Ronald S. Bultje
Hi, On Mon, Dec 17, 2012 at 1:34 PM, Janne Grunau wrote: > -if (h != h0 || (HAVE_THREADS && h->s.avctx->active_thread_type & > FF_THREAD_FRAME)) { > -av_log_missing_feature(s->avctx, > - "Width/height changing with threads", 0); > + > +

Re: [libav-devel] [PATCH 1/1] mpegvideo: allocate scratch buffers after linesize is known

2012-12-18 Thread Ronald S. Bultje
Hi, On Mon, Dec 17, 2012 at 11:24 AM, Janne Grunau wrote: > On 2012-12-13 17:58:04 +0100, Janne Grunau wrote: >> Since we can't know which stride a custom get_buffer() implementation is >> going to use we have to allocate this scratch buffers after the linesize >> is known. It was pretty safe for

Re: [libav-devel] [PATCH 1/1] h264: increase dist_scale_factor for up to 32 references

2012-12-18 Thread Ronald S. Bultje
Hi, On Tue, Dec 18, 2012 at 8:20 AM, Janne Grunau wrote: > On 2012-12-18 07:30:03 -0800, Ronald S. Bultje wrote: >> Hi, >> >> On Mon, Dec 17, 2012 at 10:55 PM, Janne Grunau >> wrote: >> > On 2012-12-17 18:24:37 -0800, Ronald S. Bultje wrote: >> >> Hi, >> >> On Dec 17, 2012 1:02 PM, "Janne Gruna

Re: [libav-devel] [PATCH 1/1] h264: increase dist_scale_factor for up to 32 references

2012-12-18 Thread Janne Grunau
On 2012-12-18 07:30:03 -0800, Ronald S. Bultje wrote: > Hi, > > On Mon, Dec 17, 2012 at 10:55 PM, Janne Grunau wrote: > > On 2012-12-17 18:24:37 -0800, Ronald S. Bultje wrote: > >> Hi, > >> On Dec 17, 2012 1:02 PM, "Janne Grunau" wrote: > >> > > >> > Compute dist_scale_factor_field only for MBAF

Re: [libav-devel] [PATCH] lavc: introduce VideoDSPContext

2012-12-18 Thread Diego Biurrun
On Tue, Dec 18, 2012 at 07:40:08AM -0800, Ronald S. Bultje wrote: > On Tue, Dec 18, 2012 at 3:19 AM, Diego Biurrun wrote: > > On Mon, Dec 17, 2012 at 09:48:48PM +0100, Luca Barbato wrote: > >> --- /dev/null > >> +++ b/libavcodec/videodsp_template.c > >> @@ -0,0 +1,92 @@ > >> +/* > >> + * Copyright

Re: [libav-devel] [PATCH] lavc: introduce VideoDSPContext

2012-12-18 Thread Ronald S. Bultje
Hi, On Tue, Dec 18, 2012 at 3:19 AM, Diego Biurrun wrote: > On Mon, Dec 17, 2012 at 09:48:48PM +0100, Luca Barbato wrote: >> --- a/configure >> +++ b/configure >> @@ -1544,6 +1545,7 @@ mpeg4_decoder_select="h263_decoder mpeg4video_parser" >> mpeg4_vdpau_decoder_select="vdpau mpeg4_decoder" >> +m

Re: [libav-devel] [PATCH] mkv: add support for the VP9 tag

2012-12-18 Thread Ronald S. Bultje
Hi, On Mon, Dec 17, 2012 at 8:16 PM, Luca Barbato wrote: > On 12/16/12 2:09 AM, Luca Barbato wrote: >> >> From: Tom Finegan > > >> +AV_CODEC_ID_VP9, >> > > Ping I had already said OK - if you add a big phat warning saying this is experimental and a non-stable bitstream (e.g. under the exper

Re: [libav-devel] [PATCH 1/1] h264: increase dist_scale_factor for up to 32 references

2012-12-18 Thread Ronald S. Bultje
Hi, On Mon, Dec 17, 2012 at 10:55 PM, Janne Grunau wrote: > On 2012-12-17 18:24:37 -0800, Ronald S. Bultje wrote: >> Hi, >> On Dec 17, 2012 1:02 PM, "Janne Grunau" wrote: >> > >> > Compute dist_scale_factor_field only for MBAFF since that is the only >> > case in which it is used. >> >> The patc

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

2012-12-18 Thread Diego Biurrun
On Tue, Dec 18, 2012 at 01:18:22PM +0100, Diego Biurrun wrote: > --- > Updated after feedback from Kostya. > > libavcodec/arm/fmtconvert_vfp.S |2 +- > libavcodec/options_table.h | 106 +++--- > libavcodec/sh4/dsputil_align.c |2 +- > libavformat/avi

Re: [libav-devel] [PATCH] configure: formatting cosmetics

2012-12-18 Thread Diego Biurrun
On Fri, Dec 07, 2012 at 11:34:36AM +0100, Diego Biurrun wrote: > --- > configure | 12 > 1 files changed, 8 insertions(+), 4 deletions(-) OKed by Luca on IRC. Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.o

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

2012-12-18 Thread Diego Biurrun
--- Updated after feedback from Kostya. libavcodec/arm/fmtconvert_vfp.S |2 +- libavcodec/options_table.h | 106 +++--- libavcodec/sh4/dsputil_align.c |2 +- libavformat/avio_internal.h |2 +- libavformat/bethsoftvid.c |2 +- libavf

Re: [libav-devel] [PATCH] lavc: introduce VideoDSPContext

2012-12-18 Thread Luca Barbato
On 12/18/2012 12:19 PM, Diego Biurrun wrote: > On Mon, Dec 17, 2012 at 09:48:48PM +0100, Luca Barbato wrote: >> --- a/configure >> +++ b/configure >> @@ -1544,6 +1545,7 @@ mpeg4_decoder_select="h263_decoder mpeg4video_parser" >> mpeg4_vdpau_decoder_select="vdpau mpeg4_decoder" >> +mpegvideo_select

Re: [libav-devel] [PATCH] lavc: introduce VideoDSPContext

2012-12-18 Thread Diego Biurrun
On Mon, Dec 17, 2012 at 09:48:48PM +0100, Luca Barbato wrote: > --- a/configure > +++ b/configure > @@ -1544,6 +1545,7 @@ mpeg4_decoder_select="h263_decoder mpeg4video_parser" > mpeg4_vdpau_decoder_select="vdpau mpeg4_decoder" > +mpegvideo_select="videodsp" > msmpeg4v1_decoder_select="h263_decode

Re: [libav-devel] [PATCH] Add a VideoDSPContext with core video DSP functions.

2012-12-18 Thread Diego Biurrun
On Sat, Dec 15, 2012 at 02:31:51PM -0800, Ronald S. Bultje wrote: > On Sat, Dec 15, 2012 at 2:20 PM, Diego Biurrun wrote: > > On Sat, Dec 15, 2012 at 09:46:02AM -0800, Ronald S. Bultje wrote: > >> --- /dev/null > >> +++ b/libavcodec/arm/videodsp_arm.S > >> @@ -0,0 +1,33 @@ > >> +@ > >> +@ ARMv4 op

Re: [libav-devel] [PATCH] lavc: introduce VideoDSPContext

2012-12-18 Thread Luca Barbato
On 12/17/2012 09:48 PM, Luca Barbato wrote: > From: "Ronald S. Bultje" > > Move some functions from dsputil. The idea is that videodsp contains > functions that are useful for a large and varied set of video decoders. > Currently, it contains emulated_edge_mc() and prefetch(). > > Signed-off-by: