Re: [libav-devel] [PATCH 4/5] lavr: Add "resample_cutoff" option as a duplicate of "cutoff"

2012-12-19 Thread Luca Barbato
On 12/19/12 10:13 PM, Justin Ruggles wrote: Avoids an option name conflict with libavcodec, which is needed in order to work properly with avconv. --- libavresample/options.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) Ok. ___ liba

[libav-devel] [PATCH 4/5] lavr: Add "resample_cutoff" option as a duplicate of "cutoff"

2012-12-19 Thread Justin Ruggles
Avoids an option name conflict with libavcodec, which is needed in order to work properly with avconv. --- libavresample/options.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/libavresample/options.c b/libavresample/options.c index 824f5e3..85c4677 100644 --- a/libavre

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

2012-12-19 Thread Anton Khirnov
On Wed, 19 Dec 2012 10:58:28 -0500, Justin Ruggles wrote: > --- > Made modifications based on Anton's suggestions. > Also removed unneeded AVFloatDSPContext and corresponding init call. > > libavresample/Makefile|1 + > libavresample/audio_convert.c | 33 +++- > libavresample/aud

Re: [libav-devel] [PATCH] flac: only set channel layout if not previously set or on channel count change

2012-12-19 Thread Anton Khirnov
On Sun, 16 Dec 2012 11:13:21 -0500, Justin Ruggles wrote: > Fixes Bug 402 > --- > Forgot to set avctx->channels before calling ff_flac_set_channel_layout() > in the parser. > > libavcodec/flac_parser.c |6 +- > libavcodec/flacdec.c |3 ++- > 2 files changed, 7 insertions(+), 2

Re: [libav-devel] [PATCH] cmdutils: pass number of groups to split_commandline().

2012-12-19 Thread Martin Storsjö
On Wed, 19 Dec 2012, Anton Khirnov wrote: This makes the code simpler and avoids mixing designated and non-designated initializers in a potentially unsafe way in avconv. --- avconv_opt.c |4 ++-- cmdutils.c | 24 +++- cmdutils.h |2 +- 3 files changed, 14 insertion

[libav-devel] [PATCH] cmdutils: pass number of groups to split_commandline().

2012-12-19 Thread Anton Khirnov
This makes the code simpler and avoids mixing designated and non-designated initializers in a potentially unsafe way in avconv. --- avconv_opt.c |4 ++-- cmdutils.c | 24 +++- cmdutils.h |2 +- 3 files changed, 14 insertions(+), 16 deletions(-) diff --git a/avcon

[libav-devel] [PATCH] x86inc: activate REP_RET automatically

2012-12-19 Thread Loren Merritt
Now RET checks whether it immediately follows a branch, so the programmer dosen't have to keep track of that condition. REP_RET is still needed manually when it's a branch target, but that's much rarer. The implementation involves lots of spurious labels, so get rid of them with `strip`. --- c

[libav-devel] [PATCH 1/1] avconv: use designated initializers consistently in OptionGroupDef

2012-12-19 Thread Janne Grunau
Prevents triggering an assert in c99conv. --- Anton, I know you didn't like this but fixing the converter seems to be non-trivial. avconv_opt.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/avconv_opt.c b/avconv_opt.c index 6c70175..d73181e 100644 --- a/avconv_opt.c +++ b/

Re: [libav-devel] [PATCH] flac: only set channel layout if not previously set or on channel count change

2012-12-19 Thread Justin Ruggles
On 12/16/2012 11:13 AM, Justin Ruggles wrote: > Fixes Bug 402 > --- > Forgot to set avctx->channels before calling ff_flac_set_channel_layout() > in the parser. > > libavcodec/flac_parser.c |6 +- > libavcodec/flacdec.c |3 ++- > 2 files changed, 7 insertions(+), 2 deletions(-) >

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

2012-12-19 Thread Justin Ruggles
--- Made modifications based on Anton's suggestions. Also removed unneeded AVFloatDSPContext and corresponding init call. libavresample/Makefile|1 + libavresample/audio_convert.c | 33 +++- libavresample/audio_convert.h | 22 ++- libavresample/avresample.h|9 + libavresam

Re: [libav-devel] [PATCH 1/1] h264: fix sps parsing for SVC and CAVLC 4:4:4 Intra profiles

2012-12-19 Thread Ronald S. Bultje
Hi, On Dec 19, 2012 4:41 AM, "Janne Grunau" wrote: > > On 2012-12-19 13:16:14 +0100, Hendrik Leppkes wrote: > > On Wed, Dec 19, 2012 at 9:38 AM, Janne Grunau wrote: > > > From: Galland > > > > > > Fixes bug 396. > > > > > > CC: libav-sta...@libav.org > > > --- > > > libavcodec/h264_ps.c | 6 ++

[libav-devel] [PATCH 4/5] lavr: Add "resample_cutoff" option as a duplicate of "cutoff"

2012-12-19 Thread Justin Ruggles
Avoids an option name conflict with libavcodec, which is needed in order to work properly with avconv. --- doc/APIchanges |4 libavresample/options.c |2 ++ libavresample/version.h |2 +- 3 files changed, 7 insertions(+), 1 deletions(-) diff --git a/doc/APIchanges b/doc/

Re: [libav-devel] [PATCH 2/2] flvenc: Check whether seeking back to the header succeeded

2012-12-19 Thread Benjamin Larsson
OK ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 1/2] sapenc: Pass the title on to the chained muxers

2012-12-19 Thread Benjamin Larsson
OK ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 1/2] sapenc: Pass the title on to the chained muxers

2012-12-19 Thread Martin Storsjö
From: Jernej Virag This makes sure it ends up in the SDP, providing a proper session name in the SAP announcements. --- libavformat/sapenc.c |5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/sapenc.c b/libavformat/sapenc.c index fd9be4f..7e2bba7 100644 --- a/libavformat/sap

[libav-devel] [PATCH 2/2] flvenc: Check whether seeking back to the header succeeded

2012-12-19 Thread Martin Storsjö
From: Björn Axelsson The FLV muxer tries to update the header in write_trailer, which is impossible if writing to a pipe or network stream. Don't write header data if seeking to the header fails. --- libavformat/flvenc.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff

Re: [libav-devel] [PATCH 1/1] h264: fix sps parsing for SVC and CAVLC 4:4:4 Intra profiles

2012-12-19 Thread Janne Grunau
On 2012-12-19 13:16:14 +0100, Hendrik Leppkes wrote: > On Wed, Dec 19, 2012 at 9:38 AM, Janne Grunau wrote: > > From: Galland > > > > Fixes bug 396. > > > > CC: libav-sta...@libav.org > > --- > > libavcodec/h264_ps.c | 6 +- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --g

Re: [libav-devel] [PATCH 1/1] h264: fix sps parsing for SVC and CAVLC 4:4:4 Intra profiles

2012-12-19 Thread Hendrik Leppkes
On Wed, Dec 19, 2012 at 9:38 AM, Janne Grunau wrote: > From: Galland > > Fixes bug 396. > > CC: libav-sta...@libav.org > --- > libavcodec/h264_ps.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c > index 7eb8726..ff86f64 1

Re: [libav-devel] [PATCH 06/22] 4xm: operate with pointers to AVFrames instead of whole structs.

2012-12-19 Thread Janne Grunau
On 2012-12-19 13:03:34 +0100, Anton Khirnov wrote: > > On Wed, 19 Dec 2012 11:11:16 +0100, Janne Grunau > wrote: > > On 2012-12-19 10:17:13 +0100, Anton Khirnov wrote: > > > This is more correct and avoids the whole extended_data mess. > > > > extended_data mess in libavcodec/utils.c? Just aski

Re: [libav-devel] [PATCH 1/1] h264: fix sps parsing for SVC and CAVLC 4:4:4 Intra profiles

2012-12-19 Thread Janne Grunau
On 2012-12-19 09:59:52 +0100, Luca Barbato wrote: > On 12/19/12 9:38 AM, Janne Grunau wrote: > > From: Galland > > > > Would be nice having some defines instead of bare numbers, the patch is > ok though. Not sure if defines would be nicer. This is the exact same statement as the in the SPS defi

Re: [libav-devel] [PATCH 1/1] h264: fix sps parsing for SVC and CAVLC 4:4:4 Intra profiles

2012-12-19 Thread Janne Grunau
On 2012-12-19 12:30:15 +0100, Diego Biurrun wrote: > On Wed, Dec 19, 2012 at 09:38:02AM +0100, Janne Grunau wrote: > > From: Galland > > Do we have a realname? yes, commit already changed. Forgot to mention that I asked for the name. Janne ___ libav

Re: [libav-devel] [PATCH 06/22] 4xm: operate with pointers to AVFrames instead of whole structs.

2012-12-19 Thread Anton Khirnov
On Wed, 19 Dec 2012 11:11:16 +0100, Janne Grunau wrote: > On 2012-12-19 10:17:13 +0100, Anton Khirnov wrote: > > This is more correct and avoids the whole extended_data mess. > > extended_data mess in libavcodec/utils.c? Just asking because the patch > doesn't change anything with extended_data.

Re: [libav-devel] [PATCH 1/1] h264: fix sps parsing for SVC and CAVLC 4:4:4 Intra profiles

2012-12-19 Thread Diego Biurrun
On Wed, Dec 19, 2012 at 09:38:02AM +0100, Janne Grunau wrote: > From: Galland Do we have a realname? Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

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

2012-12-19 Thread Diego Biurrun
On Tue, Dec 18, 2012 at 10:50:13AM -0800, Alex Converse wrote: > --- /dev/null > +++ b/libavfilter/af_alength.c > @@ -0,0 +1,210 @@ > +static int alength_config_output(AVFilterLink *outlink) { Please move the { to the next line before pushing. Diego ___

Re: [libav-devel] [PATCH 11/22] anm: return meaningful error codes

2012-12-19 Thread Janne Grunau
On 2012-12-19 10:17:18 +0100, Anton Khirnov wrote: > --- > libavcodec/anm.c |8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/anm.c b/libavcodec/anm.c > index fc328bb..49e925d 100644 > --- a/libavcodec/anm.c > +++ b/libavcodec/anm.c > @@ -44,7 +44,7 @@

Re: [libav-devel] [PATCH 20/22] bfi: return meaningful error codes.

2012-12-19 Thread Diego Biurrun
On Wed, Dec 19, 2012 at 10:17:27AM +0100, Anton Khirnov wrote: > --- > libavcodec/bfi.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) LGTM Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman

Re: [libav-devel] [PATCH 19/22] bethsoftvideo: return meaningful error codes.

2012-12-19 Thread Diego Biurrun
On Wed, Dec 19, 2012 at 10:17:26AM +0100, Anton Khirnov wrote: > --- > libavcodec/bethsoftvideo.c |6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) LGTM Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mai

Re: [libav-devel] [PATCH 14/22] asvdec: cosmetics, reformat

2012-12-19 Thread Diego Biurrun
On Wed, Dec 19, 2012 at 10:17:21AM +0100, Anton Khirnov wrote: > --- a/libavcodec/asvdec.c > +++ b/libavcodec/asvdec.c > @@ -46,124 +46,148 @@ static VLC dc_ccp_vlc; > > -block[0]= 8*get_bits(&a->gb, 8); > +block[0] = 8*get_bits(&a->gb, 8); spaces around * > -if(ccp&8) block

Re: [libav-devel] [PATCH 15/22] asvdec: return meaningful error codes.

2012-12-19 Thread Diego Biurrun
On Wed, Dec 19, 2012 at 10:17:22AM +0100, Anton Khirnov wrote: > --- > libavcodec/asvdec.c | 20 ++-- > 1 file changed, 10 insertions(+), 10 deletions(-) LGTM Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.liba

Re: [libav-devel] [PATCH 13/22] ansi: return a meaningful error code

2012-12-19 Thread Diego Biurrun
On Wed, Dec 19, 2012 at 10:17:20AM +0100, Anton Khirnov wrote: > --- > libavcodec/ansi.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) OK Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo

Re: [libav-devel] [PATCH 12/22] ansi: remove an extra return

2012-12-19 Thread Diego Biurrun
On Wed, Dec 19, 2012 at 10:17:19AM +0100, Anton Khirnov wrote: > --- > libavcodec/ansi.c |1 - > 1 file changed, 1 deletion(-) probably OK Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

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

2012-12-19 Thread Diego Biurrun
On Tue, Dec 18, 2012 at 05:02:31PM -0800, Ronald S. Bultje wrote: > From: "Ronald S. Bultje" > > --- > libavutil/x86/x86inc.asm | 33 - > 1 file changed, 20 insertions(+), 13 deletions(-) This is a pretty non-descriptive log message. Diego __

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

2012-12-19 Thread Diego Biurrun
On Tue, Dec 18, 2012 at 11:06:29PM +0100, 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 bytes > pixel si

Re: [libav-devel] [PATCH 3/6] cmdutils: add a commandline pre-parser.

2012-12-19 Thread Diego Biurrun
On Mon, Dec 17, 2012 at 10:52:33PM +0100, Anton Khirnov wrote: > --- a/cmdutils.c > +++ b/cmdutils.c > @@ -416,6 +439,224 @@ int opt_default(void *optctx, const char *opt, const > char *arg) > > +/* > + * Check whether given option is a group separator. > + * > + * @return index of the group def

Re: [libav-devel] [PATCH 1/1] mov: handle h263 and flv1 for codec_tag 'H', '2', '6', '3'

2012-12-19 Thread Martin Storsjö
On Wed, 19 Dec 2012, Janne Grunau wrote: The sample in https://bugzilla.libav.org/show_bug.cgi?id=393 and samples/F4V/H263_NM_f.mp4 both have codec_tag H263 for different codecs. H263 is apparently used by Flash Media Server for Sorensen Spark videos. Patch based on commit 5442083b1c541dd166b3a

Re: [libav-devel] [PATCH 08/22] 8bps: return meaningful error codes.

2012-12-19 Thread Janne Grunau
On 2012-12-19 10:17:15 +0100, Anton Khirnov wrote: > --- > libavcodec/8bps.c | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/libavcodec/8bps.c b/libavcodec/8bps.c > index 8f0692c..ceed13f 100644 > --- a/libavcodec/8bps.c > +++ b/libavcodec/8bps.c > @@ -69,15

Re: [libav-devel] [PATCH 07/22] 8bps: cosmetics

2012-12-19 Thread Janne Grunau
On 2012-12-19 10:17:14 +0100, Anton Khirnov wrote: > Remove some pointless comments and empty lines. > --- > libavcodec/8bps.c | 25 - > 1 file changed, 25 deletions(-) > > diff --git a/libavcodec/8bps.c b/libavcodec/8bps.c > index 306fd52..8f0692c 100644 > --- a/libavco

Re: [libav-devel] [PATCH 06/22] 4xm: operate with pointers to AVFrames instead of whole structs.

2012-12-19 Thread Janne Grunau
On 2012-12-19 10:17:13 +0100, Anton Khirnov wrote: > This is more correct and avoids the whole extended_data mess. extended_data mess in libavcodec/utils.c? Just asking because the patch doesn't change anything with extended_data. > --- > libavcodec/4xm.c | 52 +++--

Re: [libav-devel] [PATCH 05/22] 4xm: eliminate a pointless indirection

2012-12-19 Thread Janne Grunau
On 2012-12-19 10:17:12 +0100, Anton Khirnov wrote: > --- > libavcodec/4xm.c | 13 ++--- > 1 file changed, 2 insertions(+), 11 deletions(-) > > diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c > index 1084f04..07fb155 100644 > --- a/libavcodec/4xm.c > +++ b/libavcodec/4xm.c > @@ -877,16

Re: [libav-devel] [PATCH 18/22] avs: return meaningful error codes.

2012-12-19 Thread Janne Grunau
On 2012-12-19 10:17:25 +0100, Anton Khirnov wrote: > --- > libavcodec/avs.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/libavcodec/avs.c b/libavcodec/avs.c > index 98a53f2..71b8b0c 100644 > --- a/libavcodec/avs.c > +++ b/libavcodec/avs.c > @@ -54,14 +54,1

Re: [libav-devel] [PATCH 03/22] 4xm: simplify code with FFSWAP

2012-12-19 Thread Janne Grunau
On 2012-12-19 10:17:10 +0100, Anton Khirnov wrote: > --- > libavcodec/4xm.c |6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c > index 6f3ec4a..db77c7b 100644 > --- a/libavcodec/4xm.c > +++ b/libavcodec/4xm.c > @@ -760,7 +760,7 @@

[libav-devel] [PATCH 1/1] mov: handle h263 and flv1 for codec_tag 'H', '2', '6', '3'

2012-12-19 Thread Janne Grunau
The sample in https://bugzilla.libav.org/show_bug.cgi?id=393 and samples/F4V/H263_NM_f.mp4 both have codec_tag H263 for different codecs. H263 is apparently used by Flash Media Server for Sorensen Spark videos. Patch based on commit 5442083b1c541dd166b3adf39131259fc846a62b by Carl Eugen Hoyos. Fix

Re: [libav-devel] [PATCH 04/22] 4xm: don't rely on get_buffer() initializing the frame.

2012-12-19 Thread Luca Barbato
On 12/19/12 10:17 AM, Anton Khirnov wrote: --- libavcodec/4xm.c |1 + 1 file changed, 1 insertion(+) Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 02/22] 4xm: return meaningful error codes

2012-12-19 Thread Luca Barbato
On 12/19/12 10:17 AM, Anton Khirnov wrote: --- libavcodec/4xm.c | 39 --- 1 file changed, 20 insertions(+), 19 deletions(-) Looks ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/

Re: [libav-devel] [PATCH 22/22] bink: operate with pointers to AVFrames instead of whole structs.

2012-12-19 Thread Kostya Shishkov
On Wed, Dec 19, 2012 at 10:17:29AM +0100, Anton Khirnov wrote: > This is more correct and avoids the whole extended_data mess. > --- > libavcodec/bink.c | 53 > ++--- > 1 file changed, 26 insertions(+), 27 deletions(-) Fine with me if it works as

Re: [libav-devel] [PATCH 21/22] bink: return meaningful error codes.

2012-12-19 Thread Kostya Shishkov
On Wed, Dec 19, 2012 at 10:17:28AM +0100, Anton Khirnov wrote: > --- > libavcodec/bink.c | 110 > +++-- > 1 file changed, 57 insertions(+), 53 deletions(-) > > diff --git a/libavcodec/bink.c b/libavcodec/bink.c > index 059601d..2625291 100644 > --

Re: [libav-devel] [PATCH 17/22] aura: return meaningful error codes.

2012-12-19 Thread Kostya Shishkov
On Wed, Dec 19, 2012 at 10:17:24AM +0100, Anton Khirnov wrote: > --- > libavcodec/aura.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/libavcodec/aura.c b/libavcodec/aura.c > index 4960bf8..7256dc8 100644 > --- a/libavcodec/aura.c > +++ b/libavcodec/aura.c

Re: [libav-devel] [PATCH 16/22] aura: cosmetics, reformat

2012-12-19 Thread Kostya Shishkov
On Wed, Dec 19, 2012 at 10:17:23AM +0100, Anton Khirnov wrote: > --- > libavcodec/aura.c | 21 ++--- > 1 file changed, 10 insertions(+), 11 deletions(-) lgtm ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/ma

Re: [libav-devel] [PATCH 10/22] aasc: return meaningful error codes.

2012-12-19 Thread Kostya Shishkov
On Wed, Dec 19, 2012 at 10:17:17AM +0100, Anton Khirnov wrote: > --- > libavcodec/aasc.c |8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/aasc.c b/libavcodec/aasc.c > index 47d25d0..7759c71 100644 > --- a/libavcodec/aasc.c > +++ b/libavcodec/aasc.c >

Re: [libav-devel] [PATCH 09/22] aasc: cosmetics, reformat

2012-12-19 Thread Kostya Shishkov
On Wed, Dec 19, 2012 at 10:17:16AM +0100, Anton Khirnov wrote: > --- > libavcodec/aasc.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) LGTM ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/lis

[libav-devel] [PATCH 22/22] bink: operate with pointers to AVFrames instead of whole structs.

2012-12-19 Thread Anton Khirnov
This is more correct and avoids the whole extended_data mess. --- libavcodec/bink.c | 53 ++--- 1 file changed, 26 insertions(+), 27 deletions(-) diff --git a/libavcodec/bink.c b/libavcodec/bink.c index 2625291..22f7495 100644 --- a/libavcodec/bin

[libav-devel] [PATCH 21/22] bink: return meaningful error codes.

2012-12-19 Thread Anton Khirnov
--- libavcodec/bink.c | 110 +++-- 1 file changed, 57 insertions(+), 53 deletions(-) diff --git a/libavcodec/bink.c b/libavcodec/bink.c index 059601d..2625291 100644 --- a/libavcodec/bink.c +++ b/libavcodec/bink.c @@ -313,7 +313,7 @@ static int rea

[libav-devel] [PATCH 18/22] avs: return meaningful error codes.

2012-12-19 Thread Anton Khirnov
--- libavcodec/avs.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavcodec/avs.c b/libavcodec/avs.c index 98a53f2..71b8b0c 100644 --- a/libavcodec/avs.c +++ b/libavcodec/avs.c @@ -54,14 +54,14 @@ avs_decode_frame(AVCodecContext * avctx, AVFrame *const p =

[libav-devel] [PATCH 20/22] bfi: return meaningful error codes.

2012-12-19 Thread Anton Khirnov
--- libavcodec/bfi.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/bfi.c b/libavcodec/bfi.c index 50b20ba..1971c0c 100644 --- a/libavcodec/bfi.c +++ b/libavcodec/bfi.c @@ -55,16 +55,16 @@ static int bfi_decode_frame(AVCodecContext *avctx, void *data

[libav-devel] [PATCH 16/22] aura: cosmetics, reformat

2012-12-19 Thread Anton Khirnov
--- libavcodec/aura.c | 21 ++--- 1 file changed, 10 insertions(+), 11 deletions(-) diff --git a/libavcodec/aura.c b/libavcodec/aura.c index d0fe946..4960bf8 100644 --- a/libavcodec/aura.c +++ b/libavcodec/aura.c @@ -49,8 +49,7 @@ static int aura_decode_frame(AVCodecContext *avc

[libav-devel] [PATCH 19/22] bethsoftvideo: return meaningful error codes.

2012-12-19 Thread Anton Khirnov
--- libavcodec/bethsoftvideo.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/bethsoftvideo.c b/libavcodec/bethsoftvideo.c index 316adb6..77bf49c 100644 --- a/libavcodec/bethsoftvideo.c +++ b/libavcodec/bethsoftvideo.c @@ -75,9 +75,9 @@ static int bethsoftv

[libav-devel] [PATCH 15/22] asvdec: return meaningful error codes.

2012-12-19 Thread Anton Khirnov
--- libavcodec/asvdec.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/libavcodec/asvdec.c b/libavcodec/asvdec.c index fd6afb6..2ad16c2 100644 --- a/libavcodec/asvdec.c +++ b/libavcodec/asvdec.c @@ -111,7 +111,7 @@ static inline int asv1_decode_block(A

[libav-devel] [PATCH 17/22] aura: return meaningful error codes.

2012-12-19 Thread Anton Khirnov
--- libavcodec/aura.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavcodec/aura.c b/libavcodec/aura.c index 4960bf8..7256dc8 100644 --- a/libavcodec/aura.c +++ b/libavcodec/aura.c @@ -39,7 +39,7 @@ static av_cold int aura_decode_init(AVCodecContext *avctx)

[libav-devel] [PATCH 04/22] 4xm: don't rely on get_buffer() initializing the frame.

2012-12-19 Thread Anton Khirnov
--- libavcodec/4xm.c |1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c index db77c7b..1084f04 100644 --- a/libavcodec/4xm.c +++ b/libavcodec/4xm.c @@ -853,6 +853,7 @@ static int decode_frame(AVCodecContext *avctx, void *data, av_log(avctx,

[libav-devel] [PATCH 14/22] asvdec: cosmetics, reformat

2012-12-19 Thread Anton Khirnov
--- libavcodec/asvdec.c | 230 --- 1 file changed, 128 insertions(+), 102 deletions(-) diff --git a/libavcodec/asvdec.c b/libavcodec/asvdec.c index a53ba77..fd6afb6 100644 --- a/libavcodec/asvdec.c +++ b/libavcodec/asvdec.c @@ -46,124 +46,148 @@ st

[libav-devel] [PATCH 13/22] ansi: return a meaningful error code

2012-12-19 Thread Anton Khirnov
--- libavcodec/ansi.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/ansi.c b/libavcodec/ansi.c index c2a1d83..bcdd362 100644 --- a/libavcodec/ansi.c +++ b/libavcodec/ansi.c @@ -396,8 +396,8 @@ static int decode_frame(AVCodecContext *avctx,

[libav-devel] [PATCH 12/22] ansi: remove an extra return

2012-12-19 Thread Anton Khirnov
--- libavcodec/ansi.c |1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/ansi.c b/libavcodec/ansi.c index edd373d..c2a1d83 100644 --- a/libavcodec/ansi.c +++ b/libavcodec/ansi.c @@ -370,7 +370,6 @@ static int decode_frame(AVCodecContext *avctx, } else {

[libav-devel] [PATCH 10/22] aasc: return meaningful error codes.

2012-12-19 Thread Anton Khirnov
--- libavcodec/aasc.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/aasc.c b/libavcodec/aasc.c index 47d25d0..7759c71 100644 --- a/libavcodec/aasc.c +++ b/libavcodec/aasc.c @@ -56,13 +56,13 @@ static int aasc_decode_frame(AVCodecContext *avctx, cons

[libav-devel] [PATCH 11/22] anm: return meaningful error codes

2012-12-19 Thread Anton Khirnov
--- libavcodec/anm.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/anm.c b/libavcodec/anm.c index fc328bb..49e925d 100644 --- a/libavcodec/anm.c +++ b/libavcodec/anm.c @@ -44,7 +44,7 @@ static av_cold int decode_init(AVCodecContext *avctx) s->frame.

[libav-devel] [PATCH 09/22] aasc: cosmetics, reformat

2012-12-19 Thread Anton Khirnov
--- libavcodec/aasc.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/libavcodec/aasc.c b/libavcodec/aasc.c index cd1df36..47d25d0 100644 --- a/libavcodec/aasc.c +++ b/libavcodec/aasc.c @@ -54,8 +54,8 @@ static int aasc_decode_frame(AVCodecContext *avctx,

[libav-devel] [PATCH 08/22] 8bps: return meaningful error codes.

2012-12-19 Thread Anton Khirnov
--- libavcodec/8bps.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/libavcodec/8bps.c b/libavcodec/8bps.c index 8f0692c..ceed13f 100644 --- a/libavcodec/8bps.c +++ b/libavcodec/8bps.c @@ -69,15 +69,16 @@ static int decode_frame(AVCodecContext *avctx, void *data,

[libav-devel] [PATCH 06/22] 4xm: operate with pointers to AVFrames instead of whole structs.

2012-12-19 Thread Anton Khirnov
This is more correct and avoids the whole extended_data mess. --- libavcodec/4xm.c | 52 +++- 1 file changed, 31 insertions(+), 21 deletions(-) diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c index 07fb155..446cb83 100644 --- a/libavcodec/4xm.c +

[libav-devel] [PATCH 07/22] 8bps: cosmetics

2012-12-19 Thread Anton Khirnov
Remove some pointless comments and empty lines. --- libavcodec/8bps.c | 25 - 1 file changed, 25 deletions(-) diff --git a/libavcodec/8bps.c b/libavcodec/8bps.c index 306fd52..8f0692c 100644 --- a/libavcodec/8bps.c +++ b/libavcodec/8bps.c @@ -44,9 +44,6 @@ static const

[libav-devel] [PATCH 05/22] 4xm: eliminate a pointless indirection

2012-12-19 Thread Anton Khirnov
--- libavcodec/4xm.c | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c index 1084f04..07fb155 100644 --- a/libavcodec/4xm.c +++ b/libavcodec/4xm.c @@ -877,16 +877,6 @@ static int decode_frame(AVCodecContext *avctx, void *data,

[libav-devel] [PATCH 03/22] 4xm: simplify code with FFSWAP

2012-12-19 Thread Anton Khirnov
--- libavcodec/4xm.c |6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c index 6f3ec4a..db77c7b 100644 --- a/libavcodec/4xm.c +++ b/libavcodec/4xm.c @@ -760,7 +760,7 @@ static int decode_frame(AVCodecContext *avctx, void *data, int

[libav-devel] [PATCH 02/22] 4xm: return meaningful error codes

2012-12-19 Thread Anton Khirnov
--- libavcodec/4xm.c | 39 --- 1 file changed, 20 insertions(+), 19 deletions(-) diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c index f78a0a2..6f3ec4a 100644 --- a/libavcodec/4xm.c +++ b/libavcodec/4xm.c @@ -415,7 +415,7 @@ static int decode_p_frame(FourXCo

[libav-devel] [PATCH] Prelude to The Evil Plan, part I

2012-12-19 Thread Anton Khirnov
Hi, this is the first batch of the various random fixes/cosmetics that were accumulated during the creation of the evil plan. There is about 100 such patches in total right now, so I guess it's about time to start sending them in small non-threatening batches. I'd like to ask the do-x-while-you're

Re: [libav-devel] [PATCH 1/1] h264: fix sps parsing for SVC and CAVLC 4:4:4 Intra profiles

2012-12-19 Thread Luca Barbato
On 12/19/12 9:38 AM, Janne Grunau wrote: From: Galland Would be nice having some defines instead of bare numbers, the patch is ok though. lu ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 1/1] h264: fix sps parsing for SVC and CAVLC 4:4:4 Intra profiles

2012-12-19 Thread Janne Grunau
From: Galland Fixes bug 396. CC: libav-sta...@libav.org --- libavcodec/h264_ps.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c index 7eb8726..ff86f64 100644 --- a/libavcodec/h264_ps.c +++ b/libavcodec/h264_ps.c @@ -333,7 +33