Re: [PATCH] Use xattr (Linux) in copy-acl.c

[Bruno Haible]

Paul Eggert wrote:
> > +static int
> > +copy_attr_permissions (const char *name, struct error_context *ctx)
> > +{
> > +   int action = attr_copy_action (name, ctx);
> > +   return action == ATTR_ACTION_PERMISSIONS;
> > +}
> 
> 
> This returns bool not int, so it should be declared to return bool.

Nope. This function is used as a callback for attr_copy_file and
attr_copy_fd [1] and therefore needs 'int' as return type.

Bruno

[1] 
https://git.savannah.nongnu.org/gitweb/?p=attr.git;a=blob;f=include/libattr.h;hb=HEAD#l27