[illumos-Developer] Review: 742 Resurrect the ZFS "aclmode" property and 279, 664, 807

Albert Lee trisk at opensolaris.org
Mon Mar 14 13:14:39 PDT 2011


On Mon, Mar 14, 2011 at 12:32 AM, Gordon Ross <gordon.w.ross at gmail.com> wrote:
> Overall, this looks good to me.  Just a couple minor suggestions/nits.
>
> acl_common.c:1572-1579
>  Suggest replacing the uint32_t locals with a
>  trivial_acl_t local, then struct assign at the end.
>

If that looks cleaner to you, sure.

-Albert



More information about the Developer mailing list