]> Savannah Git Hosting - gnulib.git/commitdiff
file-has-acl: improve acl.h comment
authorPaul Eggert <eggert@cs.ucla.edu>
Mon, 30 Sep 2024 16:24:09 +0000 (09:24 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Mon, 30 Sep 2024 22:29:52 +0000 (15:29 -0700)
* lib/acl.h: Improve comment about ‘152’.

lib/acl.h
lib/file-has-acl.c

index 07ab04250d45cdf395d21e74d414ddfa1290cd09..1a627323ec91d96a0e88bf50a3e0471f78d4c6f5 100644 (file)
--- a/lib/acl.h
+++ b/lib/acl.h
@@ -65,7 +65,10 @@ struct aclinfo
     int err;
 
     /* A small array of char, big enough for most listxattr results.
-       The size is somewhat arbitrary.  For internal use only.  */
+       The size is somewhat arbitrary; it equals the max length of a
+       trivial NFSv4 ACL (a size used by file-has-acl.c in 2023-2024
+       but no longer relevant now), and a different value might be
+       better once experience is gained.  For internal use only.  */
     char __gl_acl_ch[152];
   } u;
 };
index 89faba565f459525eb35691abfb0664d7c0df37b..6924a616bd4e24580f121114f9b21884a8756dd4 100644 (file)
@@ -349,8 +349,7 @@ file_has_aclinfo (char const *name, struct stat const *sb,
               || (S_ISDIR (sb->st_mode)
                   && aclinfo_has_xattr (ai, XATTR_NAME_POSIX_ACL_DEFAULT))));
 
-      /* If there is an NFSv4 ACL, follow up with a getxattr syscall
-         to see whether the NFSv4 ACL is nontrivial.  */
+      /* If there is an NFSv4 ACL, check whether it is nontrivial.  */
       if (nfsv4_acl)
         {
           /* A buffer large enough to hold any trivial NFSv4 ACL.