Module Name: src
Committed By: jschauma
Date: Tue Feb 14 18:58:55 UTC 2023
Modified Files:
src/usr.bin/split: split.c
Log Message:
consistently use EXIT_FAILURE / EXIT_SUCCESS instead of 1 / 0
To generate a diff of this commit:
cvs rdiff -u -r1.31 -r1.32 src/usr.bin/split/split.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/usr.bin/split/split.c
diff -u src/usr.bin/split/split.c:1.31 src/usr.bin/split/split.c:1.32
--- src/usr.bin/split/split.c:1.31 Tue Feb 14 18:26:59 2023
+++ src/usr.bin/split/split.c Tue Feb 14 18:58:55 2023
@@ -1,4 +1,4 @@
-/* $NetBSD: split.c,v 1.31 2023/02/14 18:26:59 jschauma Exp $ */
+/* $NetBSD: split.c,v 1.32 2023/02/14 18:58:55 jschauma Exp $ */
/*
* Copyright (c) 1987, 1993, 1994
@@ -39,7 +39,7 @@ __COPYRIGHT("@(#) Copyright (c) 1987, 19
#if 0
static char sccsid[] = "@(#)split.c 8.3 (Berkeley) 4/25/94";
#endif
-__RCSID("$NetBSD: split.c,v 1.31 2023/02/14 18:26:59 jschauma Exp $");
+__RCSID("$NetBSD: split.c,v 1.32 2023/02/14 18:58:55 jschauma Exp $");
#endif /* not lint */
#include <sys/param.h>
@@ -95,21 +95,21 @@ main(int argc, char *argv[])
p = argv[optind] + 1;
numlines = strtoull(p, &ep, 10);
if (numlines == 0 || *ep != '\0')
- errx(1, "%s: illegal line count.", p);
+ errx(EXIT_FAILURE, "%s: illegal line count.", p);
}
break;
case 'a': /* Suffix length. */
if (!isdigit((unsigned char)optarg[0]) ||
(sfxlen = (size_t)strtoul(optarg, &ep, 10)) == 0 ||
*ep != '\0')
- errx(1, "%s: illegal suffix length.", optarg);
+ errx(EXIT_FAILURE, "%s: illegal suffix length.", optarg);
autosfx = 0;
break;
case 'b': /* Byte count. */
if (!isdigit((unsigned char)optarg[0]) ||
(bytecnt = strtoull(optarg, &ep, 10)) == 0 ||
(*ep != '\0' && *ep != 'k' && *ep != 'm'))
- errx(1, "%s: illegal byte count.", optarg);
+ errx(EXIT_FAILURE, "%s: illegal byte count.", optarg);
if (*ep == 'k')
bytecnt *= 1024;
else if (*ep == 'm')
@@ -121,13 +121,13 @@ main(int argc, char *argv[])
if (!isdigit((unsigned char)optarg[0]) ||
(numlines = strtoull(optarg, &ep, 10)) == 0 ||
*ep != '\0')
- errx(1, "%s: illegal line count.", optarg);
+ errx(EXIT_FAILURE, "%s: illegal line count.", optarg);
break;
case 'n': /* Chunks. */
if (!isdigit((unsigned char)optarg[0]) ||
(chunks = (size_t)strtoul(optarg, &ep, 10)) == 0 ||
*ep != '\0')
- errx(1, "%s: illegal number of chunks.", optarg);
+ errx(EXIT_FAILURE, "%s: illegal number of chunks.", optarg);
break;
default:
usage();
@@ -138,7 +138,7 @@ main(int argc, char *argv[])
if (*argv != NULL) {
if (strcmp(*argv, "-") != 0 &&
(ifd = open(*argv, O_RDONLY, 0)) < 0)
- err(1, "%s", *argv);
+ err(EXIT_FAILURE, "%s", *argv);
++argv;
}
@@ -187,10 +187,10 @@ split1(off_t bytecnt, int maxcnt)
for (bcnt = 0;;)
switch (len = read(ifd, bfr, MAXBSIZE)) {
case 0:
- exit(0);
+ exit(EXIT_SUCCESS);
/* NOTREACHED */
case -1:
- err(1, "read");
+ err(EXIT_FAILURE, "read");
/* NOTREACHED */
default:
if (!file_open) {
@@ -204,7 +204,7 @@ split1(off_t bytecnt, int maxcnt)
/* LINTED: bytecnt - bcnt <= len */
dist = bytecnt - bcnt;
if (bigwrite(ofd, bfr, dist) != (size_t)dist)
- err(1, "write");
+ err(EXIT_FAILURE, "write");
len -= dist;
for (C = bfr + dist; len >= bytecnt;
/* LINTED: bytecnt <= len */
@@ -216,7 +216,7 @@ split1(off_t bytecnt, int maxcnt)
/* LINTED: as above */
if (bigwrite(ofd,
C, bytecnt) != (size_t)bytecnt)
- err(1, "write");
+ err(EXIT_FAILURE, "write");
}
if (len) {
if (!maxcnt || (nfiles < maxcnt)) {
@@ -225,7 +225,7 @@ split1(off_t bytecnt, int maxcnt)
}
/* LINTED: len >= 0 */
if (bigwrite(ofd, C, len) != (size_t)len)
- err(1, "write");
+ err(EXIT_FAILURE, "write");
} else
file_open = 0;
bcnt = len;
@@ -233,7 +233,7 @@ split1(off_t bytecnt, int maxcnt)
bcnt += len;
/* LINTED: len >= 0 */
if (bigwrite(ofd, bfr, len) != (size_t)len)
- err(1, "write");
+ err(EXIT_FAILURE, "write");
}
}
}
@@ -254,10 +254,10 @@ split2(off_t numlines)
for (lcnt = 0;;)
switch (len = read(ifd, bfr, MAXBSIZE)) {
case 0:
- exit(0);
+ exit(EXIT_SUCCESS);
/* NOTREACHED */
case -1:
- err(1, "read");
+ err(EXIT_FAILURE, "read");
/* NOTREACHED */
default:
if (!file_open) {
@@ -268,7 +268,7 @@ split2(off_t numlines)
if (*Ce == '\n' && ++lcnt == numlines) {
bcnt = Ce - Cs + 1;
if (bigwrite(ofd, Cs, bcnt) != (size_t)bcnt)
- err(1, "write");
+ err(EXIT_FAILURE, "write");
lcnt = 0;
Cs = Ce + 1;
if (len)
@@ -279,7 +279,7 @@ split2(off_t numlines)
if (Cs < Ce) {
bcnt = Ce - Cs;
if (bigwrite(ofd, Cs, bcnt) != (size_t)bcnt)
- err(1, "write");
+ err(EXIT_FAILURE, "write");
}
}
}
@@ -294,12 +294,12 @@ split3(off_t chunks)
struct stat sb;
if (fstat(ifd, &sb) == -1) {
- err(1, "stat");
+ err(EXIT_FAILURE, "stat");
/* NOTREACHED */
}
if (chunks > sb.st_size) {
- errx(1, "can't split into more than %d files",
+ errx(EXIT_FAILURE, "can't split into more than %d files",
(int)sb.st_size);
/* NOTREACHED */
}
@@ -322,7 +322,7 @@ newfile(void)
fpnt = fname + strlen(fname);
fpnt[sfxlen] = '\0';
} else if (close(ofd) != 0)
- err(1, "%s", fname);
+ err(EXIT_FAILURE, "%s", fname);
quot = fnum;
@@ -362,10 +362,10 @@ newfile(void)
quot = quot / 26;
}
if (quot > 0)
- errx(1, "too many files.");
+ errx(EXIT_FAILURE, "too many files.");
++fnum;
if ((ofd = open(fname, O_WRONLY | O_CREAT | O_TRUNC, DEFFILEMODE)) < 0)
- err(1, "%s", fname);
+ err(EXIT_FAILURE, "%s", fname);
}
static size_t
@@ -392,5 +392,5 @@ usage(void)
(void)fprintf(stderr,
"usage: %s [-b byte_count] [-l line_count] [-n chunk_count] [-a suffix_length] "
"[file [prefix]]\n", getprogname());
- exit(1);
+ exit(EXIT_FAILURE);
}