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.