emacs-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] POSIX ACL support


From: Eli Zaretskii
Subject: Re: [PATCH] POSIX ACL support
Date: Mon, 19 Nov 2012 19:47:57 +0200

> From: Glenn Morris <address@hidden>
> Date: Mon, 19 Nov 2012 12:11:38 -0500
> Cc: address@hidden
> 
> > -(defun backup-buffer-copy (from-name to-name modes context)
> > +(defun backup-buffer-copy (from-name to-name modes context acl)
> 
> [and several other such]
> 
> This is really a general comment, not an issue with your patch in
> particular, so don't let it distract. I just wonder if it is possible to
> find a different way to pass this information around, so that we don't
> need to have an ever increasing list of function arguments and return
> values (preserve-this, preserve-that, preserve-the-other, etc...).

Agreed.

> I don't have a good suggestion for how to do that.

How about a cons cell of the form (TYPE . DATA), where TYPE can be
selinux-context, posix-acls, etc., and DATA is whatever value is
appropriate for TYPE?  Or maybe (:TYPE DATA)?



reply via email to

[Prev in Thread] Current Thread [Next in Thread]