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