[FFmpeg-cvslog] r10666 - trunk/libavcodec/adpcm.c

voroshil subversion
Thu Oct 4 17:13:42 CEST 2007


Author: voroshil
Date: Thu Oct  4 17:13:42 2007
New Revision: 10666

Log:
Replace "signed short" typecast with "int16_t" as suggested by
Mans Rullgard. Value at the right side is 16 bit length signed.
We can not know for sure that short is in fact 16 bits, even if
this virtually always is the case.



Modified:
   trunk/libavcodec/adpcm.c

Modified: trunk/libavcodec/adpcm.c
==============================================================================
--- trunk/libavcodec/adpcm.c	(original)
+++ trunk/libavcodec/adpcm.c	Thu Oct  4 17:13:42 2007
@@ -1184,7 +1184,7 @@ static int adpcm_decode_frame(AVCodecCon
         break;
     case CODEC_ID_ADPCM_IMA_AMV:
     case CODEC_ID_ADPCM_IMA_SMJPEG:
-        c->status[0].predictor = (signed short)bytestream_get_le16(&src);
+        c->status[0].predictor = (int16_t)bytestream_get_le16(&src);
         c->status[0].step_index = bytestream_get_le16(&src);
 
         if (avctx->codec->id == CODEC_ID_ADPCM_IMA_AMV)




More information about the ffmpeg-cvslog mailing list