[FFmpeg-devel] [PATCH]Fail more gracefully for multichannel wavpack (in another container)

Kostya kostya.shishkov
Fri Jan 7 13:55:53 CET 2011


On Fri, Jan 07, 2011 at 01:47:22PM +0100, Carl Eugen Hoyos wrote:
> Hi!
> 
> For current output, see issue 2504.
> 
> Please comment, Carl Eugen

> Index: libavcodec/wavpack.c
> ===================================================================
> --- libavcodec/wavpack.c	(revision 26251)
> +++ libavcodec/wavpack.c	(working copy)
> @@ -660,6 +660,10 @@
>      WavpackContext *s = avctx->priv_data;
>  
>      s->avctx = avctx;
> +    if (avctx->channels > 2) {
> +        av_log(avctx, AV_LOG_ERROR, "Multichannel WavPack is not supported yet.\n");
> +        return -1;
> +    }
>      s->stereo = (avctx->channels == 2);
>      if(avctx->bits_per_coded_sample <= 16)
>          avctx->sample_fmt = AV_SAMPLE_FMT_S16;

ok, though it's better if you wrote patch for multichannel support
instead



More information about the ffmpeg-devel mailing list