[FFmpeg-cvslog] r18541 - trunk/libavcodec/eatgq.c
reimar
subversion
Fri Apr 17 10:42:08 CEST 2009
Author: reimar
Date: Fri Apr 17 10:42:08 2009
New Revision: 18541
Log:
Change tgq_decode_mb function arguments from using int8_t to uint8_t, this does
not really change the code (x86_64/gcc-4.3.3 results stay unchanged) but avoids
several casts and an aliasing violation that broke decoding at least on
PPC64/gcc-4.1.2.
Modified:
trunk/libavcodec/eatgq.c
Modified: trunk/libavcodec/eatgq.c
==============================================================================
--- trunk/libavcodec/eatgq.c Fri Apr 17 10:39:11 2009 (r18540)
+++ trunk/libavcodec/eatgq.c Fri Apr 17 10:42:08 2009 (r18541)
@@ -140,13 +140,13 @@ static void tgq_idct_put_mb_dconly(TgqCo
}
}
-static void tgq_decode_mb(TgqContext *s, int mb_y, int mb_x, const int8_t **bs, const int8_t *buf_end){
+static void tgq_decode_mb(TgqContext *s, int mb_y, int mb_x, const uint8_t **bs, const uint8_t *buf_end){
int mode;
int i;
int8_t dc[6];
DCTELEM block[6][64];
- mode = bytestream_get_byte((const uint8_t**)bs);
+ mode = bytestream_get_byte(bs);
if (mode>buf_end-*bs) {
av_log(s->avctx, AV_LOG_ERROR, "truncated macroblock\n");
return;
@@ -228,7 +228,7 @@ static int tgq_decode_frame(AVCodecConte
for (y=0; y<(avctx->height+15)/16; y++)
for (x=0; x<(avctx->width+15)/16; x++)
- tgq_decode_mb(s, y, x, (const int8_t**)&buf, (const int8_t*)buf_end);
+ tgq_decode_mb(s, y, x, &buf, buf_end);
*data_size = sizeof(AVFrame);
*(AVFrame*)data = s->frame;
More information about the ffmpeg-cvslog
mailing list