sbase/tee.c

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

#include "util.h"

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

int
main(int argc, char *argv[])
{
	FILE **fps = NULL;
	size_t i, n, nfps;
	int ret = 0, aflag = 0, iflag = 0;
	char buf[BUFSIZ];

	ARGBEGIN {
	case 'a':
		aflag = 1;
		break;
	case 'i':
		iflag = 1;
		break;
	default:
		usage();
	} ARGEND;

	if (iflag && signal(SIGINT, SIG_IGN) == SIG_ERR)
		eprintf("signal:");
	nfps = argc + 1;
	fps = ecalloc(nfps, sizeof(*fps));

	for (i = 0; i < argc; i++)
		if (!(fps[i] = fopen(argv[i], aflag ? "a" : "w")))
			eprintf("fopen %s:", argv[i]);
	fps[i] = stdout;

	while ((n = fread(buf, 1, sizeof(buf), stdin))) {
		for (i = 0; i < nfps; i++) {
			if (fwrite(buf, 1, n, fps[i]) == n)
				continue;
			eprintf("fwrite %s:", (i != argc) ? argv[i] : "<stdout>");
		}
	}

	ret |= fshut(stdin, "<stdin>") | fshut(stdout, "<stdout>");

	return ret;
}
initial commit 2011-05-23 01:36:34 +00:00			`/* See LICENSE file for copyright and license details. */`
tee: -i option ignores SIGINT 2015-01-22 21:08:25 +00:00			`#include <signal.h>`
initial commit 2011-05-23 01:36:34 +00:00			`#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
initial commit 2011-05-23 01:36:34 +00:00			`#include "util.h"`

Eliminating the getopt disgrace. 2013-06-14 18:20:47 +00:00			`static void`
			`usage(void)`
			`{`
Audit tee(1) 1) "duplicate" implies that you can only specify two outputs, "multiply" is a better word describing the functionality. 2) fix other wording in the manpage 3) fix usage() 4) reorder local variables 5) fix sizeof() style 6) we need argv later, don't increment argv and rather iterate over argc. 7) Improve error messages, print the filename which the write failed to instead of printing the buffer itself (how much sense does that make, printing 1024 Bytes of garbage?). Also, give the name of the function which failed. 2015-03-04 22:05:11 +00:00			`eprintf("usage: %s [-ai] [file ...]\n", argv0);`
Eliminating the getopt disgrace. 2013-06-14 18:20:47 +00:00			`}`

initial commit 2011-05-23 01:36:34 +00:00			`int`
			`main(int argc, char *argv[])`
			`{`
Audit tee(1) 1) "duplicate" implies that you can only specify two outputs, "multiply" is a better word describing the functionality. 2) fix other wording in the manpage 3) fix usage() 4) reorder local variables 5) fix sizeof() style 6) we need argv later, don't increment argv and rather iterate over argc. 7) Improve error messages, print the filename which the write failed to instead of printing the buffer itself (how much sense does that make, printing 1024 Bytes of garbage?). Also, give the name of the function which failed. 2015-03-04 22:05:11 +00:00			`FILE **fps = NULL;`
			`size_t i, n, nfps;`
Simplify return & fshut() logic Get rid of the !!()-constructs and use ret where available (or introduce it). In some cases, there would be an "abort" on the first fshut-error, but we want to close all files and report all warnings and then quit, not just the warning for the first file. 2015-05-24 23:33:19 +00:00			`int ret = 0, aflag = 0, iflag = 0;`
Eliminating the getopt disgrace. 2013-06-14 18:20:47 +00:00			`char buf[BUFSIZ];`
initial commit 2011-05-23 01:36:34 +00:00
Eliminating the getopt disgrace. 2013-06-14 18:20:47 +00:00			`ARGBEGIN {`
			`case 'a':`
Un-boolify sbase It actually makes the binaries smaller, the code easier to read (gems like "val == true", "val == false" are gone) and actually predictable in the sense of that we actually know what we're working with (one bitwise operator was quite adventurous and should now be fixed). This is also more consistent with the other suckless projects around which don't use boolean types. 2014-11-13 20:24:47 +00:00			`aflag = 1;`
Eliminating the getopt disgrace. 2013-06-14 18:20:47 +00:00			`break;`
tee: -i option ignores SIGINT 2015-01-22 21:08:25 +00:00			`case 'i':`
			`iflag = 1;`
			`break;`
Eliminating the getopt disgrace. 2013-06-14 18:20:47 +00:00			`default:`
			`usage();`
			`} ARGEND;`

tee: -i option ignores SIGINT 2015-01-22 21:08:25 +00:00			`if (iflag && signal(SIGINT, SIG_IGN) == SIG_ERR)`
			`eprintf("signal:");`
remove unnecessary calls to realloc from tee 2014-06-02 22:15:52 +00:00			`nfps = argc + 1;`
Audit tee(1) 1) "duplicate" implies that you can only specify two outputs, "multiply" is a better word describing the functionality. 2) fix other wording in the manpage 3) fix usage() 4) reorder local variables 5) fix sizeof() style 6) we need argv later, don't increment argv and rather iterate over argc. 7) Improve error messages, print the filename which the write failed to instead of printing the buffer itself (how much sense does that make, printing 1024 Bytes of garbage?). Also, give the name of the function which failed. 2015-03-04 22:05:11 +00:00			`fps = ecalloc(nfps, sizeof(*fps));`
remove unnecessary calls to realloc from tee 2014-06-02 22:15:52 +00:00
Audit tee(1) 1) "duplicate" implies that you can only specify two outputs, "multiply" is a better word describing the functionality. 2) fix other wording in the manpage 3) fix usage() 4) reorder local variables 5) fix sizeof() style 6) we need argv later, don't increment argv and rather iterate over argc. 7) Improve error messages, print the filename which the write failed to instead of printing the buffer itself (how much sense does that make, printing 1024 Bytes of garbage?). Also, give the name of the function which failed. 2015-03-04 22:05:11 +00:00			`for (i = 0; i < argc; i++)`
			`if (!(fps[i] = fopen(argv[i], aflag ? "a" : "w")))`
			`eprintf("fopen %s:", argv[i]);`
remove unnecessary calls to realloc from tee 2014-06-02 22:15:52 +00:00			`fps[i] = stdout;`
initial commit 2011-05-23 01:36:34 +00:00
Audit tee(1) 1) "duplicate" implies that you can only specify two outputs, "multiply" is a better word describing the functionality. 2) fix other wording in the manpage 3) fix usage() 4) reorder local variables 5) fix sizeof() style 6) we need argv later, don't increment argv and rather iterate over argc. 7) Improve error messages, print the filename which the write failed to instead of printing the buffer itself (how much sense does that make, printing 1024 Bytes of garbage?). Also, give the name of the function which failed. 2015-03-04 22:05:11 +00:00			`while ((n = fread(buf, 1, sizeof(buf), stdin))) {`
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			`for (i = 0; i < nfps; i++) {`
Audit tee(1) 1) "duplicate" implies that you can only specify two outputs, "multiply" is a better word describing the functionality. 2) fix other wording in the manpage 3) fix usage() 4) reorder local variables 5) fix sizeof() style 6) we need argv later, don't increment argv and rather iterate over argc. 7) Improve error messages, print the filename which the write failed to instead of printing the buffer itself (how much sense does that make, printing 1024 Bytes of garbage?). Also, give the name of the function which failed. 2015-03-04 22:05:11 +00:00			`if (fwrite(buf, 1, n, fps[i]) == n)`
			`continue;`
			`eprintf("fwrite %s:", (i != argc) ? argv[i] : "<stdout>");`
Eliminating the getopt disgrace. 2013-06-14 18:20:47 +00:00			`}`
			`}`

Simplify return & fshut() logic Get rid of the !!()-constructs and use ret where available (or introduce it). In some cases, there would be an "abort" on the first fshut-error, but we want to close all files and report all warnings and then quit, not just the warning for the first file. 2015-05-24 23:33:19 +00:00			`ret \|= fshut(stdin, "<stdin>") \| fshut(stdout, "<stdout>");`

			`return ret;`
initial commit 2011-05-23 01:36:34 +00:00			`}`