[openbeosstorage] Re: ddm questions

Tyler Dauwalder <tyler@xxxxxxxxxxxxx> wrote:
> On 2003-10-24 at 17:40:52 [-0700], you wrote:
> > > Isn't there one declared in headers/private/kernel/vm.h right 
> > > next to
> > > user_strlcpy() and friends, and defined similarly in
> > > src/kernel/core/arch/x86/arch_cpu.h, etc?
> > Yeah, it's just not implemented anywhere :-)
> What about in src/kernel/core/arch/x86/arch_cpu.c ? Isn't that an 
> implementation?

Indeed...

> > I don't think we should have it either, because I almost can't 
> > think of
> > a good reason to use it.
> Fair enough.

(that's why I think we should get rid of it anyway)

> > I wouldn't have seen it if you
> > hadn't used user_strcpy() :-)
> Now I'm confused. Is/was something not building for you because of 
> the 
> user_strcpy() calls? I seem to have no problem with it over here. And 
> further, I can't see how it's any more unimplemented than 
> user_strlcpy() or 
> user_strncpy(); there seems to be corresponding code for 
> user_strcpy() 
> everywhere there's code for those two functions (unless I'm missing 
> something, which wouldn't be a first ;-). 

... okay, now I am confused as well. I could not build the ddm because 
of user_strcpy() - it said "undefined reference", so I assumed it's not 
part of the kernel.
But a closer look could have told me that it couldn't build the 
libdisk_device_manager.so only (I just tried again). Maybe our build 
misses a dependency to rebuild the lib that would contain 
user_strcpy()? (it builds now that user_strcpy() is not used anymore, 
though - and libkernelland_emu.so was rebuild as well here - maybe I 
simply forgot to update it [although I think I almost do a cvs update 
twice a day in the root directory...])

Well, so let's just ignore that, and I will remove user_strcpy() from 
the kernel - thanks for the hint ;-)

Bye,
   Axel.

Other related posts: