Re: CVS commit: src/sys/rump

[Antti Kantee <pooka%cs.hut.fi@localhost>]

On Wed Apr 14 2010 at 19:04:27 +0000, Christos Zoulas wrote:
> In article <20100414141248.B2272175DD%cvs.netbsd.org@localhost>,
> Antti Kantee  <source-changes-d%NetBSD.org@localhost> wrote:
> >-=-=-=-=-=-
> >
> >Module Name: src
> >Committed By:        pooka
> >Date:                Wed Apr 14 14:12:48 UTC 2010
> >
> >Modified Files:
> >     src/sys/rump/include/rump: rump.h
> >     src/sys/rump/librump/rumpkern: rumpkern.ifspec
> >     src/sys/rump/librump/rumpvfs: rumpvfs.ifspec
> >
> >Log Message:
> >Use "struct kauth_cred *" instead of kauth_cred_t in all exported
> >interfaces.  Allows to remove hairbrained _t typedef dance.
> 
> Not that I have a problem with it, but the whole point of kauth_cred_t
> was to provide opacity on struct kauth_cred, and people were really
> pushing to never expose struct kauth_cred.

I'm pretty sure that assumes that kernel interfaces are not exposed to
userspace for non-NetBSD operating systems.  different rules, different
game ;)

Think of them (exported struct kauth * and internal kauth_cred_t) as
different types which just happen to have, at least now, a very simple
conversion function... and actually they are.  it's just that in C
typedefs don't mean anything.