[FFmpeg-cvslog] r12148 - trunk/libavcodec/fraps.c

Reimar Döffinger Reimar.Doeffinger
Wed Feb 20 00:07:20 CET 2008


Hello,
On Tue, Feb 19, 2008 at 10:43:05PM +0000, M?ns Rullg?rd wrote:
> reimar <subversion at mplayerhq.hu> writes:
> 
> > Author: reimar
> > Date: Tue Feb 19 22:42:10 2008
> > New Revision: 12148
> >
> > Log:
> > Add casts to avoid warnings:
> > fraps.c:102: warning: passing argument 1 of ?s->dsp.bswap_buf? from incompatible pointer type
> > fraps.c:102: warning: passing argument 2 of ?s->dsp.bswap_buf? from incompatible pointer type
> >
> > Modified:
> >    trunk/libavcodec/fraps.c
> >
> > Modified: trunk/libavcodec/fraps.c
> > ==============================================================================
> > --- trunk/libavcodec/fraps.c	(original)
> > +++ trunk/libavcodec/fraps.c	Tue Feb 19 22:42:10 2008
> > @@ -99,7 +99,7 @@ static int fraps2_decode_plane(FrapsCont
> >      /* we have built Huffman table and are ready to decode plane */
> >
> >      /* convert bits so they may be used by standard bitreader */
> > -    s->dsp.bswap_buf(s->tmpbuf, src, size >> 2);
> > +    s->dsp.bswap_buf((uint32_t *)s->tmpbuf, (const uint32_t *)src, size >> 2);
> 
> Did you verify that those buffers are adequately aligned?

tmpbuf yes. Not sure about src, but unless the specification says
something special about it, it probably is not necessarily aligned...

Greetings,
Reimar D?ffinger




More information about the ffmpeg-cvslog mailing list