Re: acl, copy-file: First step towards more consistent function names

[Pádraig Brady]

On 25/08/2024 11:48, Bruno Haible wrote:
> Pádraig Brady wrote:
> > I always thought the x...() variants indicate they might eXit().
> >
> > Are there any other x...() variants that just output a diagnostic,
> > and don't exit()?
>
> Indeed, we currently have three categories of x...() variants:
>
>    - Most of them emit diagnostics and call exit (EXIT_FAILURE)
>      in some conditions.
>
>    - xset_acl, xcopy_acl: emit diagnostics but do not call
>      exit (EXIT_FAILURE). I think that's simply because ACL-related
>      errors are generally OK to ignore.
>
>    - These do not emit diagnostics. The 'x' stands for "extended".
>        xnanosleep
>        xsize
>        xstrtod, xstrtold
>        xstrtol, xstrtoll, xstrtoimax, xstrtoumax
>
> Should we leave this as is? If not, what would be a better naming convention?

I've no strong opinion, I was just a little surprised at
introducing the new x...() category.
I'm fine to leave it as it is now.

thanks,
Pádraig