[MPlayer-dev-eng] [PATCH] Metadata retrieval from FLAc audio files.

Aurelien Jacobs aurel at gnuage.org
Mon Mar 20 21:07:03 CET 2006


On Mon, 20 Mar 2006 14:34:38 +0100
Diego Biurrun <diego at biurrun.de> wrote:

> On Sat, Mar 18, 2006 at 02:54:23PM +0100, Benjamin Zores wrote:
> > On Sun, 12 Mar 2006 14:48:41 -0500
> > Rich Felker <dalias at aerifal.cx> wrote:
> > 
> > > On Sun, Mar 12, 2006 at 07:38:46PM +0100, Jindrich Makovicka wrote:
> > > > Benjamin Zores wrote:
> > > > > +    case FLAC_VORBIS_COMMENT:
> > > > > +    {
> > > > > +      /* For a description of the format please have a look at */
> > > > > +      /* http://www.xiph.org/vorbis/doc/v-comment.html */
> > > > > +
> > > > > +      uint32_t length, comment_list_len;
> > > > > +      char comments[blk_len];
> > > > 
> > > > This is a c99-ism. Better use either malloc() or constant size & sanity
> > > > check on blk_len.
> > > 
> > > No, allocation on the stack is good. gcc has supported it ever since
> > > ages ago (gcc 2.7 iirc -- way before it was part of c99), as does
> > > every known compiler that could possibly hope to compile MPlayer.
> > 
> > so commit it if it do no more shows any issue.
> 
> Aurel, you're from the GeexBOX team, your duty then ;)

Right. Done.

Aurel




More information about the MPlayer-dev-eng mailing list