Justin Ruggles <[email protected]> writes:

> From: Franz Brauße <[email protected]>
>
> Fixes Bug #265
>
> Signed-off-by: Justin Ruggles <[email protected]>
> ---
>  libavcodec/smacker.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libavcodec/smacker.c b/libavcodec/smacker.c
> index d425387..4602730 100644
> --- a/libavcodec/smacker.c
> +++ b/libavcodec/smacker.c
> @@ -655,7 +655,7 @@ static int smka_decode_frame(AVCodecContext *avctx, void 
> *data,
>      }
>      if(bits) { //decode 16-bit data
>          for(i = stereo; i >= 0; i--)
> -            pred[i] = av_bswap16(get_bits(&gb, 16));
> +            pred[i] = sign_extend(av_bswap16(get_bits(&gb, 16)), 16);
>          for(i = 0; i <= stereo; i++)
>              *samples++ = pred[i];
>          for(; i < unp_size / 2; i++) {
> -- 

LGTM

BTW, ARM has an instruction for that.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to