Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/lib/libkvm



On Wed, Feb 07, 2007 at 03:42:23AM +0200, Elad Efrat wrote:

> Andrew Doran wrote:
> 
> > Thanks for fixing this. Redefining kauth_cred_t in userland isn't the right
> > way to go about it though. Please change it so that kauth_cred_t is defined
> > in a non-installed header file under sys/sys.
> 
> I'd rather not. Having it exposed in sys/sys is just asking for people
> to access its members directly.

If others do access it directly, that's not your problem - it's theirs. All
you need to do is document it as implementation private, and protect it with
a define.

Will you be fixing this?

Cheers,
Andrew



Home | Main Index | Thread Index | Old Index