xargs: trivial code shrink
function old new delta
process_stdin 343 336 -7
process0_stdin 124 117 -7
xargs_main 807 787 -20
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
diff --git a/findutils/xargs.c b/findutils/xargs.c
index 857773d..5c26685 100644
--- a/findutils/xargs.c
+++ b/findutils/xargs.c
@@ -150,7 +150,6 @@
* (buf has extra byte at the end to accomodate terminating NUL
* of "tail characters" string).
* Otherwise, the returned pointer points to NUL byte.
- * The args[] vector is NULL-terminated.
* On entry, buf[] may contain some "seed chars" which are to become
* the beginning of the first parameter.
*/
@@ -241,7 +240,7 @@
}
ret:
*p = '\0';
- store_param(NULL);
+ /* store_param(NULL) - caller will do it */
dbg_msg("return:'%s'", s);
return s;
}
@@ -293,7 +292,7 @@
}
ret:
*p = '\0';
- store_param(NULL);
+ /* store_param(NULL) - caller will do it */
dbg_msg("return:'%s'", s);
return s;
}
@@ -334,7 +333,7 @@
}
ret:
*p = '\0';
- store_param(NULL);
+ /* store_param(NULL) - caller will do it */
dbg_msg("return:'%s'", s);
return s;
}
@@ -461,13 +460,10 @@
buf = xzalloc(n_max_chars + 1);
+ n_max_arg = n_max_chars;
if (opt & OPT_UPTO_NUMBER) {
n_max_arg = xatou_range(max_args, 1, INT_MAX);
- if (n_max_arg < n_max_chars)
- goto skip;
}
- n_max_arg = n_max_chars;
- skip:
/* Allocate pointers for execvp */
/* We can statically allocate (argc + n_max_arg + 1) elements
@@ -489,6 +485,7 @@
G.idx = argc;
rem = read_args(n_max_chars, n_max_arg, buf);
+ store_param(NULL);
if (!G.args[argc]) {
if (*rem != '\0')
@@ -499,10 +496,11 @@
opt |= OPT_NO_EMPTY;
if (opt & (OPT_INTERACTIVE | OPT_VERBOSE)) {
- for (i = 0; G.args[i]; i++) {
- if (i)
- bb_putchar_stderr(' ');
- fputs(G.args[i], stderr);
+ const char *fmt = " %s" + 1;
+ char **args = G.args;
+ for (i = 0; args[i]; i++) {
+ fprintf(stderr, fmt, args[i]);
+ fmt = " %s";
}
if (!(opt & OPT_INTERACTIVE))
bb_putchar_stderr('\n');