sbase/yes.c

/* See LICENSE file for copyright and license details. */
#include <stdio.h>

#include "util.h"

static void
usage(void)
{
	eprintf("usage: %s [string ...]\n", argv0);
}

int
main(int argc, char *argv[])
{
	char **p;

	ARGBEGIN {
	default:
		usage();
	} ARGEND;

	for (p = argv; ; p = (*p && *(p + 1)) ? p + 1 : argv) {
		fputs(*p ? *p : "y", stdout);
		putchar((!*p || !*(p + 1)) ? '\n' : ' ');
	}

	return 1; /* not reached */
}
Adding the yes(1) command. 2012-04-23 14:27:40 +00:00			`/* See LICENSE file for copyright and license details. */`
			`#include <stdio.h>`
Fix coding style It was about damn time. Consistency is very important in such a big codebase. 2014-11-13 17:29:30 +00:00
Adding the yes(1) command. 2012-04-23 14:27:40 +00:00			`#include "util.h"`

use always static for usage and keep usage definition above main 2014-04-22 13:44:16 +00:00			`static void`
			`usage(void)`
			`{`
Finish up yes(1) by adding multiple string support 2015-02-01 01:13:47 +00:00			`eprintf("usage: %s [string ...]\n", argv0);`
use always static for usage and keep usage definition above main 2014-04-22 13:44:16 +00:00			`}`
Adding the yes(1) command. 2012-04-23 14:27:40 +00:00
			`int`
			`main(int argc, char *argv[])`
			`{`
Make already audited tools argv-centric instead of argc-centric This has already been suggested by Evan Gates <evan.gates@gmail.com> and he's totally right about it. So, what's the problem? I wrote a testing program asshole.c with int main(void) { execl("/path/to/sbase/echo", "echo", "test"); return 0; } and checked the results with glibc and musl. Note that the sentinel NULL is missing from the end of the argument list. glibc calculates an argc of 5, musl 4 (instead of 2) and thus mess up things anyway. The powerful arg.h also focuses on argv instead of argc as well, but ignoring argc completely is also the wrong way to go. Instead, a more idiomatic approach is to check *argv only and decrement argc on the go. While at it, I rewrote yes(1) in an argv-centric way as well. All audited tools have been "fixed" and each following audited tool will receive the same treatment. 2015-03-02 13:19:26 +00:00			`char **p;`
Finish up yes(1) by adding multiple string support 2015-02-01 01:13:47 +00:00
Adding the yes(1) command. 2012-04-23 14:27:40 +00:00			`ARGBEGIN {`
			`default:`
revert to per-cmd usage() 2012-05-15 12:32:56 +00:00			`usage();`
Adding the yes(1) command. 2012-04-23 14:27:40 +00:00			`} ARGEND;`

Make already audited tools argv-centric instead of argc-centric This has already been suggested by Evan Gates <evan.gates@gmail.com> and he's totally right about it. So, what's the problem? I wrote a testing program asshole.c with int main(void) { execl("/path/to/sbase/echo", "echo", "test"); return 0; } and checked the results with glibc and musl. Note that the sentinel NULL is missing from the end of the argument list. glibc calculates an argc of 5, musl 4 (instead of 2) and thus mess up things anyway. The powerful arg.h also focuses on argv instead of argc as well, but ignoring argc completely is also the wrong way to go. Instead, a more idiomatic approach is to check *argv only and decrement argc on the go. While at it, I rewrote yes(1) in an argv-centric way as well. All audited tools have been "fixed" and each following audited tool will receive the same treatment. 2015-03-02 13:19:26 +00:00			`for (p = argv; ; p = (p && (p + 1)) ? p + 1 : argv) {`
			`fputs(p ? p : "y", stdout);`
			`putchar((!p \|\| !(p + 1)) ? '\n' : ' ');`
Finish up yes(1) by adding multiple string support 2015-02-01 01:13:47 +00:00			`}`
Audit yes(1) Oh well, time to simplify the loop. Also, change the comment on unreachable code to something more clear. 2015-02-28 20:11:10 +00:00
			`return 1; /* not reached */`
Adding the yes(1) command. 2012-04-23 14:27:40 +00:00			`}`