On Mon, Nov 11, 2002 at 05:37:10PM -0600, Peter Samuelson wrote:
 
> [Tom Rini]
> > First, does anyone see any problems with the patch itself?
> 
> Well,
> 
> > -   str_config += sizeof("CONFIG_") - 1;
> > +   str_config += strlen(str_split_token);
> 
> it does seem a bit inefficient to call strlen() for every single line
> of every single source file.  Perhaps today's compilers know that
> strlen() is invariant and has no side effects, but I vote you go ahead
> and optimise it explicitly.

Sounds good, thanks.

-- 
Tom Rini (TR1265)
http://gate.crashing.org/~trini/


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
kbuild-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/kbuild-devel

Reply via email to