On Sat, Mar 19, 2016 at 05:42:33PM -0400, Vittorio Giovara wrote:
> The intrax8 decoding process does not imply any kind of error
> correction, and the only call present is more related to how mpegvideo
> works rather than anything else.
> 
> Therefore have the parent decoders carried out er when actually needed.

carry out error resilience

> Drop two variables that were initialized but never used.
> Update documentation accordingly.

This is somewhat redundant.

> --- a/libavcodec/intrax8.c
> +++ b/libavcodec/intrax8.c
>  error:
> -    ff_er_add_slice(&s->er, s->resync_mb_x, s->resync_mb_y,
> -                        (s->mb_x>>1)-1, (s->mb_y>>1)-1,
> -                        ER_MB_END );
>      return 0;

Since you just return after the label you should just skip the goto and
return from those places directly.

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to