Commits

Sebastian Freundt committed 25144bd Merge

Merge branch 'next'

* next:
fix to fix, bring back errno errors for duration strings
chore, request ANSI-C code off gperf explicitly
fix, don't print errnos willy-nilly, have the caller of error() pass it
fix, don't print errnos willy-nilly, have the caller of error() pass it

  • Participants
  • Parent commits c1da131, 0b15705

Comments (0)

Files changed (11)

File lib/Makefile.am

 
 ## gperf rule
 %.c: %.gperf
-	$(AM_V_GEN) $(GPERF) $< --output-file $@
+	$(AM_V_GEN) $(GPERF) -L ANSI-C $< --output-file $@
 
 # 
 # Help the developers get nice post-processed source files

File src/Makefile.am

 
 ## gperf rule
 %.c: %.gperf
-	$(AM_V_GEN) $(GPERF) $< --output-file $@
+	$(AM_V_GEN) $(GPERF) -L ANSI-C $< --output-file $@
 
 
 ## just for parallel builds
 	fputs("dadd: ", stderr);
 	vfprintf(stderr, fmt, vap);
 	va_end(vap);
-	if (eno || errno) {
+	if (eno) {
 		fputc(':', stderr);
 		fputc(' ', stderr);
-		fputs(strerror(eno ? eno : errno), stderr);
+		fputs(strerror(eno), stderr);
 	}
 	fputc('\n', stderr);
 	return;
 					/* that's ok, must be a date then */
 					dt_given_p = true;
 				} else {
-					error(0, "Error: \
+					error(errno, "Error: \
 cannot parse duration string `%s'", st.istr);
 					res = 1;
 					goto dur_out;
 
 		/* using the prchunk reader now */
 		if ((pctx = init_prchunk(STDIN_FILENO)) == NULL) {
-			error(0, "could not open stdin");
+			error(errno, "could not open stdin");
 			goto ndl_free;
 		}
 
 
 		/* using the prchunk reader now */
 		if ((pctx = init_prchunk(STDIN_FILENO)) == NULL) {
-			error(0, "could not open stdin");
+			error(errno, "could not open stdin");
 		}
 
 		/* build the clo and then loop */
 	fputs("dconv: ", stderr);
 	vfprintf(stderr, fmt, vap);
 	va_end(vap);
-	if (eno || errno) {
+	if (eno) {
 		fputc(':', stderr);
 		fputc(' ', stderr);
-		fputs(strerror(eno ? eno : errno), stderr);
+		fputs(strerror(eno), stderr);
 	}
 	fputc('\n', stderr);
 	return;
 
 		/* using the prchunk reader now */
 		if ((pctx = init_prchunk(STDIN_FILENO)) == NULL) {
-			error(0, "Error: could not open stdin");
+			error(errno, "Error: could not open stdin");
 			goto ndl_free;
 		}
 		while (prchunk_fill(pctx) >= 0) {
 	fputs("ddiff: ", stderr);
 	vfprintf(stderr, fmt, vap);
 	va_end(vap);
-	if (eno || errno) {
+	if (eno) {
 		fputc(':', stderr);
 		fputc(' ', stderr);
-		fputs(strerror(eno ? eno : errno), stderr);
+		fputs(strerror(eno), stderr);
 	}
 	fputc('\n', stderr);
 	return;
 
 		/* using the prchunk reader now */
 		if ((pctx = init_prchunk(STDIN_FILENO)) == NULL) {
-			error(0, "Error: could not open stdin");
+			error(errno, "Error: could not open stdin");
 			goto out;
 		}
 		while (prchunk_fill(pctx) >= 0) {
 	fputs("dgrep: ", stderr);
 	vfprintf(stderr, fmt, vap);
 	va_end(vap);
-	if (eno || errno) {
+	if (eno) {
 		fputc(':', stderr);
 		fputc(' ', stderr);
-		fputs(strerror(eno ? eno : errno), stderr);
+		fputs(strerror(eno), stderr);
 	}
 	fputc('\n', stderr);
 	return;
 
 		/* using the prchunk reader now */
 		if ((pctx = init_prchunk(STDIN_FILENO)) == NULL) {
-			error(0, "Error: could not open stdin");
+			error(errno, "Error: could not open stdin");
 			goto ndl_free;
 		}
 		while (prchunk_fill(pctx) >= 0) {

File src/dround.c

 	fputs("dround: ", stderr);
 	vfprintf(stderr, fmt, vap);
 	va_end(vap);
-	if (eno || errno) {
+	if (eno) {
 		fputc(':', stderr);
 		fputc(' ', stderr);
-		fputs(strerror(eno ? eno : errno), stderr);
+		fputs(strerror(eno), stderr);
 	}
 	fputc('\n', stderr);
 	return;
 					/* that's ok, must be a date then */
 					dt_given_p = true;
 				} else {
-					error(0, "Error: \
+					error(errno, "Error: \
 cannot parse duration/rounding string `%s'", st.istr);
 				}
 			}
 
 		/* using the prchunk reader now */
 		if ((pctx = init_prchunk(STDIN_FILENO)) == NULL) {
-			error(0, "Error: could not open stdin");
+			error(errno, "Error: could not open stdin");
 			goto ndl_free;
 		}
 		while (prchunk_fill(pctx) >= 0) {
 	fputs("dseq: ", stderr);
 	vfprintf(stderr, fmt, vap);
 	va_end(vap);
-	if (eno || errno) {
+	if (eno) {
 		fputc(':', stderr);
 		fputc(' ', stderr);
-		fputs(strerror(eno ? eno : errno), stderr);
+		fputs(strerror(eno), stderr);
 	}
 	fputc('\n', stderr);
 	return;
 		do {
 			if (dt_io_strpdtdur(&st, argi->alt_inc_arg) < 0) {
 				if (!argi->quiet_given) {
-					error(0, "Error: \
+					error(errno, "Error: \
 cannot parse duration string `%s'", argi->alt_inc_arg);
 				}
 				res = 1;
 	fputs("dtest: ", stderr);
 	vfprintf(stderr, fmt, vap);
 	va_end(vap);
-	if (eno || errno) {
+	if (eno) {
 		fputc(':', stderr);
 		fputc(' ', stderr);
-		fputs(strerror(eno ? eno : errno), stderr);
+		fputs(strerror(eno), stderr);
 	}
 	fputc('\n', stderr);
 	return;

File src/prchunk.c

 	fputs("prchunk: ", stderr);
 	vfprintf(stderr, fmt, vap);
 	va_end(vap);
-	if (eno || errno) {
+	if (eno) {
 		fputc(':', stderr);
 		fputc(' ', stderr);
-		fputs(strerror(eno ? eno : errno), stderr);
+		fputs(strerror(eno), stderr);
 	}
 	fputc('\n', stderr);
 	return;
 	}
 	/* get our prchunk up n running */
 	if ((ctx = init_prchunk(fd)) == NULL) {
-		error(0, "Error: ctx NULL");
+		error(errno, "Error: ctx NULL");
 		return 1;
 	}
 	/* fill the buffer */

File src/strptime.c

 	fputs("strptime: ", stderr);
 	vfprintf(stderr, fmt, vap);
 	va_end(vap);
-	if (eno || errno) {
+	if (eno) {
 		fputc(':', stderr);
 		fputc(' ', stderr);
-		fputs(strerror(eno ? eno : errno), stderr);
+		fputs(strerror(eno), stderr);
 	}
 	fputc('\n', stderr);
 	return;
 
 	/* using the prchunk reader now */
 	if ((pctx = init_prchunk(STDIN_FILENO)) == NULL) {
-		error(0, "Error: could not open stdin");
+		error(errno, "Error: could not open stdin");
 		return;
 	}
 	while (prchunk_fill(pctx) >= 0) {