On 03/17/2012 08:07 PM, Ronald S. Bultje wrote:

> ---
>  libavcodec/adpcm.c |   36 ++++++++++++++++++++++--------------
>  1 files changed, 22 insertions(+), 14 deletions(-)
> 
> diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c
> index 4e67ca1..eed172d 100644
> --- a/libavcodec/adpcm.c
> +++ b/libavcodec/adpcm.c
[...]
> -        c->status[0].idelta = (int16_t)bytestream_get_le16(&src);
> +        c->status[0].idelta = (int16_t)bytestream2_get_le16u(&gb);


Can we go ahead and change these to use sign_extend() instead of using
the implementation-defined casting?

The rest LGTM.

-Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to