[LinuxBIOS] r496 - in LinuxBIOSv3: include lib
Uwe Hermann
uwe at hermann-uwe.de
Sun Sep 23 20:04:44 CEST 2007
On Sun, Sep 23, 2007 at 02:55:24PM +0200, Carl-Daniel Hailfinger wrote:
> > +static inline int strncmp(const char *s1, const char *s2, int maxlen)
>
> Do we really want the "inline" keyword? Once we have more than one user,
> it is a waste of text size and with only one user AFAIK gcc inlines it
> automatically.
Yep, I think we can drop 'inline' in almost all cases in the whole code base.
With string.h I also think we should make it string.c and put the
prototypes in a header. I don't like code in header files at all.
Uwe.
--
http://www.hermann-uwe.de | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20070923/32e2cd60/attachment.sig>
More information about the coreboot
mailing list