Print this page
3742 zfs comments need cleaner, more consistent style
Submitted by:   Will Andrews <willa@spectralogic.com>
Submitted by:   Alan Somers <alans@spectralogic.com>
Reviewed by:    Matthew Ahrens <mahrens@delphix.com>
Reviewed by:    George Wilson <george.wilson@delphix.com>
Reviewed by:    Eric Schrock <eric.schrock@delphix.com>

@@ -44,11 +44,12 @@
 #define ZFS_ACL_VERSION_INITIAL 0ULL
 #define ZFS_ACL_VERSION_FUID    1ULL
 #define ZFS_ACL_VERSION         ZFS_ACL_VERSION_FUID
 
 /*
- * ZFS ACLs are store in various forms.
+ * ZFS ACLs (Access Control Lists) are stored in various forms.
+ *
  * Files created with ACL version ZFS_ACL_VERSION_INITIAL
  * will all be created with fixed length ACEs of type
  * zfs_oldace_t.
  *
  * Files with ACL version ZFS_ACL_VERSION_FUID will be created

@@ -134,12 +135,12 @@
         void            (*ace_who_set)(void *acep,
                             uint64_t who); /* set who/fuid */
         size_t          (*ace_size)(void *acep); /* how big is this ace */
         size_t          (*ace_abstract_size)(void); /* sizeof abstract entry */
         int             (*ace_mask_off)(void); /* off of access mask in ace */
-        int             (*ace_data)(void *acep, void **datap);
                             /* ptr to data if any */
+        int             (*ace_data)(void *acep, void **datap);
 } acl_ops_t;
 
 /*
  * A zfs_acl_t structure is composed of a list of zfs_acl_node_t's.
  * Each node will have one or more ACEs associated with it.  You will