Hi,

Quoth Tavian Barnes <taviana...@tavianator.com>:
> On Fri, Feb 14, 2025 at 11:25 AM Tavian Barnes
> > +static int
> > +pri_print0(struct arg *arg)
> > +{
> > +       if (fwrite(arg->path, strlen(arg->path) + 1, 1, stdout) != 1)
> > +               eprintf("fwrite failed:");
> > +       return 1;
> > +}
> > +

I noticed that -print0 is part of POSIX now, and since it can be useful
sometimes (I personally don't use it, but I can understand its usage),
I am  going to apply this change unless someone complains. To be actually
useful, this change requires the equivalent change in xargs, otherwise
this can be a bit useless.

> > @@ -678,6 +688,13 @@ get_print_arg(char *argv[], union extra *extra)
> >         return argv;
> >  }
> >
> > +static char **
> > +get_print0_arg(char *argv[], union extra *extra)
> > +{
> > +       gflags.print = 0;
> > +       return argv;
> > +}
> 
> This function is unused, leftover from before I realized it could
> share get_print_arg().

Indeed, removed from the patch.



Reply via email to