[MPlayer-cvslog] r32915 - trunk/gui/skin/font.c

Clément Bœsch ubitux at gmail.com
Thu Feb 17 12:16:09 CET 2011


On Thu, Feb 17, 2011 at 12:06:27PM +0100, Diego Biurrun wrote:
> On Wed, Feb 16, 2011 at 07:21:02PM +0100, ib wrote:
> > 
> > Log:
> > Initialize array in a compiler independent way.
> > 
> > --- trunk/gui/skin/font.c	Wed Feb 16 19:10:04 2011	(r32914)
> > +++ trunk/gui/skin/font.c	Wed Feb 16 19:21:02 2011	(r32915)
> > @@ -27,7 +27,7 @@
> >  
> > -bmpFont * Fonts[MAX_FONTS] = {};
> > +bmpFont * Fonts[MAX_FONTS] = {NULL};
> 
> I think the standard way to write this is '{ 0 }'.
> 

What happens here is Fonts[] = {NULL, 0, 0, 0, ... 0}. Will it does not
matter technically, it makes people thinking it will be {NULL, NULL, NULL,
...} while it's not. It's not a standard issue though.

-- 
Clément B.


More information about the MPlayer-cvslog mailing list