Hi, On Sat, Mar 17, 2012 at 5:50 PM, Paul B Mahol <[email protected]> wrote: > On 3/18/12, Ronald S. Bultje <[email protected]> wrote: >> --- >> libavcodec/adpcm.c | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c >> index 6409e15..2f6b8a1 100644 >> --- a/libavcodec/adpcm.c >> +++ b/libavcodec/adpcm.c >> @@ -1091,8 +1091,8 @@ static int adpcm_decode_frame(AVCodecContext *avctx, >> void *data, >> } >> break; >> case CODEC_ID_ADPCM_CT: >> - for (n = nb_samples >> (1 - st); n > 0; n--, src++) { >> - uint8_t v = *src; >> + for (n = nb_samples >> (1 - st); n > 0; n--) { >> + int v = bytestream2_get_byteu(&gb); > > Why v was changed from uint8_t to int?
Native sizes generally compute faster. There is no reason why it is uint8_t in the first place, so I changed it. It may generate slightly better code. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
