Re: svn commit: r11750 - branches/locking/subversion/include

On Thu, 2004-11-04 at 15:19, sussman@tigris.org wrote:
> +/** An opaque object representing a user. */
> +typedef struct svn_fs_user_t svn_fs_user_t;
> +
> +/**
> + * ### Note: this paves the way for future ACL stuff;

Sorry I tuned out of the IRC conversation when this was proposed.

I don't think we should be paving this much of the way for ACLs. We
can't give this opaque user concept enough review without a complete ACL
design document. Let's just put a username parameter in svn_fs_lock()
and svn_fs_unlock() and remove it later when ACLs show up.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Nov 5 04:53:58 2004