On 22 July 2017 at 18:50, foo86 <fooba...@gmail.com> wrote:

> Fixes CID 1409924.
> ---
>  libavcodec/dcaadpcm.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/dcaadpcm.c b/libavcodec/dcaadpcm.c
> index e00f81f011..99cbb40172 100644
> --- a/libavcodec/dcaadpcm.c
> +++ b/libavcodec/dcaadpcm.c
> @@ -80,7 +80,7 @@ static int64_t find_best_filter(const DCAADPCMEncContext
> *s, const int32_t *in,
>  {
>      const premultiplied_coeffs *precalc_data = s->private_data;
>      int i, j, k = 0;
> -    int vq;
> +    int vq = -1;
>      int64_t err;
>      int64_t min_err = 1ll << 62;
>      int64_t corr[15];
> --
> 2.11.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

Patch LGTM, tracing what it would do if it fails is it'll just disable
adpcm encoding.
For small patches like these that you're confident with, just push them
directly, no point in waiting for days when you're a maintainer (though
dcheredn1k wrote that code, you should ping him on IRC for patches to the
encoder).
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to