[FFmpeg-cvslog] r11848 - trunk/libavformat/matroskadec.c

Rich Felker dalias
Mon Feb 4 04:39:49 CET 2008


On Mon, Feb 04, 2008 at 04:20:03AM +0100, Michael Niedermayer wrote:
> On Mon, Feb 04, 2008 at 03:36:35AM +0100, Luca Barbato wrote:
> > michael wrote:
> > > Author: michael
> > > Date: Mon Feb  4 02:04:27 2008
> > > New Revision: 11848
> > > 
> > > Log:
> > > Remove incorrect cast found by -Wwrite-strings.
> > 
> > > -                put_buffer(&b, (uint8_t *) "TTA1", 4);
> > > +                put_buffer(&b, "TTA1", 4);
> > 
> > "TTA1" isn't uint8_t, put_buffer accepts const unsigned char
> >  (const uint8_t*) would be more correct.
> 
> no no no, you dont do it correctly
> its:
> 
> (void)put_buffer((ByteIOContext *)&b, (const unsigned char*)"TTA1", (int)4);

LMAO this should go in the quotes! :-)

Rich




More information about the ffmpeg-cvslog mailing list