On 03/18/2012 01:15 PM, Ronald S. Bultje wrote: > --- > libavcodec/adpcm.c | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c > index 2cb87bb..563faef 100644 > --- a/libavcodec/adpcm.c > +++ b/libavcodec/adpcm.c > @@ -150,8 +150,7 @@ static inline short > adpcm_ima_expand_nibble(ADPCMChannelStatus *c, char nibble, > > step = ff_adpcm_step_table[c->step_index]; > step_index = c->step_index + ff_adpcm_index_table[(unsigned)nibble]; > - if (step_index < 0) step_index = 0; > - else if (step_index > 88) step_index = 88; > + step_index = av_clip(step_index, 0, 88); > > sign = nibble & 8; > delta = nibble & 7;
LGTM -Justin _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
