Re: CVS commit: src/common/lib/libc/string

["Christoph Egger" <Christoph_Egger%gmx.de@localhost>]

> > > Modified Files:
> > >   src/common/lib/libc/string: bcmp.c memset.c
> > >
> > > Log Message:
> > > Add local declarations of bcmp() and bzero() because libkern.h no
> longer
> > > provides those declarations.
> > >
> > > It's possible that these can be removed later when we find
> > > out whether these functions are actually needed in the
> > > kernel, but meanwhile, this allows alpha, hp700 and the
> > > powerpc ports (who all build these as part
> > > of libkern) to complete their builds.
> >
> > sys/systm.h contains macros that should eliminate any need
> > for b* in the kernel -- certainly those functions, if
> > provided, link to nothing.
> 
> Yes, and Izumi Tsutsui committed a change which eliminated the
> actual symbols from the kernel, while Christoph Egger committed
> a lot of changes which changed the kernel source code to use the
> mem*() functions directly instead of indirectly via the b*()
> functions.
> 
> (I was thinking of a change which would have been a part of what
> Izumi committed, but wanted to complete a round of builds with
> those changes, but Izumi beat me to it, and did it better.)
> 
> For now, the b*() symbols are optionally available to standalone
> code (and user-land, of course), but have been evicted from the
> kernel.

I did not touch sys/dist and sys/external at all. There are the
remaining uses in the kernel. I could have changed them, too, but
I don't know how to bring the changes upstream or is the
person in charge for synching changes with upstream.

Christoph