[BusyBox-cvs] busybox/libbb interface.c,1.11,1.12

Glenn McGrath bug1 at busybox.net
Thu Nov 28 10:20:49 UTC 2002


Update of /var/cvs/busybox/libbb
In directory winder:/tmp/cvs-serv22245/libbb

Modified Files:
	interface.c 
Log Message:
Use error_msg instead of fprintf(stderr, other minor cleanups


Index: interface.c
===================================================================
RCS file: /var/cvs/busybox/libbb/interface.c,v
retrieving revision 1.11
retrieving revision 1.12
diff -u -d -r1.11 -r1.12
--- interface.c	26 Nov 2002 09:02:04 -0000	1.11
+++ interface.c	28 Nov 2002 10:20:45 -0000	1.12
@@ -619,7 +619,7 @@
 			if (strcmp(tmp1, paft->alias))
 				continue;
 			if (strlen(paft->name) + strlen(afname) + 1 >= sizeof(afname)) {
-				fprintf(stderr, _("Too much address family arguments.\n"));
+				error_msg(_("Too many address family arguments."));
 				return (0);
 			}
 			if (paft->flag)
@@ -630,7 +630,7 @@
 			break;
 		}
 		if (!paft->alias) {
-			fprintf(stderr, _("Unknown address family `%s'.\n"), tmp1);
+			error_msg(_("Unknown address family `%s'."), tmp1);
 			return (1);
 		}
 		tmp1 = tmp2;
@@ -693,8 +693,7 @@
 		afp++;
 	}
 	if (strchr(name, ','))
-		fprintf(stderr,
-				_("Please don't supply more than one address family.\n"));
+		error_msg(_("Please don't supply more than one address family."));
 	return (NULL);
 }
 #endif							/* KEEP_UNUSED */
@@ -888,8 +887,9 @@
 		if (af->fd >= 0)
 			sfd = af->fd;
 	}
-	if (sfd < 0)
-		fprintf(stderr, _("No usable address families found.\n"));
+	if (sfd < 0) {
+		error_msg(_("No usable address families found."));
+	}
 	return sfd;
 }
 
@@ -960,8 +960,7 @@
 	   (as of 2.1.128) */
 	skfd2 = get_socket_for_af(AF_INET);
 	if (skfd2 < 0) {
-		fprintf(stderr, _("warning: no inet socket available: %s\n"),
-				strerror(errno));
+		perror_msg(("warning: no inet socket available: %s\n"));
 		/* Try to soldier on with whatever socket we can get hold of.  */
 		skfd2 = sockets_open(0);
 		if (skfd2 < 0)
@@ -1107,8 +1106,7 @@
 
 	fh = fopen(_PATH_PROCNET_DEV, "r");
 	if (!fh) {
-		fprintf(stderr, _("Warning: cannot open %s (%s). Limited output.\n"),
-				_PATH_PROCNET_DEV, strerror(errno));
+		perror_msg(_("Warning: cannot open %s (%s). Limited output.\n"), _PATH_PROCNET_DEV);
 		return if_readconf();
 	}
 	fgets(buf, sizeof buf, fh);	/* eat line */
@@ -1368,7 +1366,7 @@
 		} else {
 			errmsg = strerror(errno);
 		}
-		fprintf(stderr, _("%s: error fetching interface information: %s\n"),
+		error_msg(_("%s: error fetching interface information: %s\n"),
 				ife->name, errmsg);
 		return -1;
 	}
@@ -1438,8 +1436,7 @@
 			val = c - 'A' + 10;
 		else {
 #ifdef DEBUG
-			fprintf(stderr, _("in_ether(%s): invalid ether address!\n"),
-					orig);
+			error_msg(_("in_ether(%s): invalid ether address!\n"), orig);
 #endif
 			errno = EINVAL;
 			return (-1);
@@ -1456,8 +1453,7 @@
 			val >>= 4;
 		else {
 #ifdef DEBUG
-			fprintf(stderr, _("in_ether(%s): invalid ether address!\n"),
-					orig);
+			error_msg(_("in_ether(%s): invalid ether address!"), orig);
 #endif
 			errno = EINVAL;
 			return (-1);
@@ -1469,27 +1465,23 @@
 
 		/* We might get a semicolon here - not required. */
 		if (*bufp == ':') {
-			if (i == ETH_ALEN) {
 #ifdef DEBUG
-				fprintf(stderr, _("in_ether(%s): trailing : ignored!\n"),
-						orig)
-#endif
-					;	/* nothing */
+			if (i == ETH_ALEN) {
+				error_msg(_("in_ether(%s): trailing : ignored!"), orig);
 			}
+#endif
 			bufp++;
 		}
 	}
 
+#ifdef DEBUG
 	/* That's it.  Any trailing junk? */
 	if ((i == ETH_ALEN) && (*bufp != '\0')) {
-#ifdef DEBUG
-		fprintf(stderr, _("in_ether(%s): trailing junk!\n"), orig);
+		error_msg(_("in_ether(%s): trailing junk!"), orig);
 		errno = EINVAL;
 		return (-1);
-#endif
 	}
-#ifdef DEBUG
-	fprintf(stderr, "in_ether(%s): %s\n", orig, pr_ether(sap->sa_data));
+	error_msg("in_ether(%s): %s", orig, pr_ether(sap->sa_data));
 #endif
 
 	return (0);
@@ -1514,7 +1506,7 @@
 /* Start the PPP encapsulation on the file descriptor. */
 static int do_ppp(int fd)
 {
-	fprintf(stderr, _("You cannot start PPP with this program.\n"));
+	error_msg(_("You cannot start PPP with this program."));
 	return -1;
 }
 #endif							/* KEEP_UNUSED */




More information about the busybox-cvs mailing list