Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 22 Aug 2018 10:07:15 +0000 (UTC)
From:      Eitan Adler <eadler@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r338189 - head/usr.bin/top
Message-ID:  <201808221007.w7MA7Fvp069323@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: eadler
Date: Wed Aug 22 10:07:15 2018
New Revision: 338189
URL: https://svnweb.freebsd.org/changeset/base/338189

Log:
  top(1): Use warnx and errx instead of fprintf
  
  This also makes make "-v" exit without error, since it isn't.

Modified:
  head/usr.bin/top/display.c
  head/usr.bin/top/screen.c
  head/usr.bin/top/top.c
  head/usr.bin/top/top.h

Modified: head/usr.bin/top/display.c
==============================================================================
--- head/usr.bin/top/display.c	Wed Aug 22 10:04:42 2018	(r338188)
+++ head/usr.bin/top/display.c	Wed Aug 22 10:07:15 2018	(r338189)
@@ -34,6 +34,7 @@
 
 #include <assert.h>
 #include <ctype.h>
+#include <err.h>
 #include <stdarg.h>
 #include <stdbool.h>
 #include <stdlib.h>
@@ -1351,9 +1352,7 @@ setup_buffer(char *buffer, int addlen)
 	}
 
 	if (NULL == b) {
-		fprintf(stderr, "%s: can't allocate sufficient memory\n",
-				myname);
-		exit(4);
+		errx(4, "can't allocate sufficient memory");
 	}
 
 	return b;

Modified: head/usr.bin/top/screen.c
==============================================================================
--- head/usr.bin/top/screen.c	Wed Aug 22 10:04:42 2018	(r338188)
+++ head/usr.bin/top/screen.c	Wed Aug 22 10:07:15 2018	(r338189)
@@ -21,6 +21,8 @@
  */
 
 #include <sys/ioctl.h>
+
+#include <err.h>
 #include <stdlib.h>
 #include <string.h>
 #include <termios.h>
@@ -94,12 +96,11 @@ init_termcap(bool interactive)
     {
 	if (status == -1)
 	{
-	    fprintf(stderr, "%s: can't open termcap file\n", myname);
+	    warnx("can't open termcap file");
 	}
 	else
 	{
-	    fprintf(stderr, "%s: no termcap entry for a `%s' terminal\n",
-		    myname, term_name);
+	    warnx("no termcap entry for a `%s' terminal", term_name);
 	}
 
 	/* pretend it's dumb and proceed */

Modified: head/usr.bin/top/top.c
==============================================================================
--- head/usr.bin/top/top.c	Wed Aug 22 10:04:42 2018	(r338188)
+++ head/usr.bin/top/top.c	Wed Aug 22 10:07:15 2018	(r338189)
@@ -68,7 +68,6 @@ static int max_topn;		/* maximum displayable processes
 
 /* miscellaneous things */
 struct process_select ps;
-const char * myname = "top";
 pid_t mypid;
 
 /* pointers to display routines */
@@ -265,18 +264,6 @@ main(int argc, char *argv[])
     setbuffer(stdout, stdoutbuf, Buffersize);
 #endif
 
-    if (argc > 0)
-    {
-	if ((myname = strrchr(argv[0], '/')) == 0)
-	{
-	    myname = argv[0];
-	}
-	else
-	{
-	    myname++;
-	}
-    }
-
     mypid = getpid();
 
     /* get our name */
@@ -323,9 +310,8 @@ main(int argc, char *argv[])
 	    switch(i)
 	    {
 	      case 'v':			/* show version number */
-		fprintf(stderr, "%s: version FreeBSD\n", myname);
-		exit(1);
-		break;
+			  errx(0, "version FreeBSD");
+			  break;
 
 	      case 'u':			/* toggle uid/username display */
 		do_unames = !do_unames;
@@ -334,8 +320,7 @@ main(int argc, char *argv[])
 	      case 'U':			/* display only username's processes */
 		if ((ps.uid[0] = userid(optarg)) == -1)
 		{
-		    fprintf(stderr, "%s: unknown user\n", optarg);
-		    exit(1);
+		    errx(1, "%s: unknown user\n", optarg);
 		}
 		break;
 
@@ -363,9 +348,7 @@ main(int argc, char *argv[])
 	      case 'd':			/* number of displays to show */
 		if ((i = atoiwi(optarg)) == Invalid || i == 0)
 		{
-		    fprintf(stderr,
-			"%s: warning: display count should be positive -- option ignored\n",
-			myname);
+		    warnx("warning: display count should be positive -- option ignored");
 		    warnings++;
 		}
 		else
@@ -395,9 +378,7 @@ main(int argc, char *argv[])
 				warnings++;
 			}
 			if (delay < 0) {
-				fprintf(stderr,
-						"%s: warning: seconds delay should be positive -- using default\n",
-						myname);
+				warnx("warning: seconds delay should be positive -- using default");
 				delay = 2;
 				warnings++;
 			}
@@ -408,8 +389,7 @@ main(int argc, char *argv[])
 			errno = 0;
 			i = setpriority(PRIO_PROCESS, 0, PRIO_MIN);
 			if (i == -1 && errno != 0) {
-				fprintf(stderr,
-						"%s: warning: `-q' option failed (%m)\n", myname);
+				warnx("warning: `-q' option failed (%m)");
 				warnings++;
 			}
 		break;
@@ -420,11 +400,7 @@ main(int argc, char *argv[])
 		} else if (strcmp(optarg, "cpu") == 0) {
 			displaymode = DISP_CPU;
 		} else {
-			fprintf(stderr,
-			"%s: warning: `-m' option can only take args "
-			"'io' or 'cpu'\n",
-			myname);
-			exit(1);
+			errx(1, "warning: `-m' option can only take args 'io' or 'cpu'");
 		}
 		break;
 
@@ -474,11 +450,9 @@ main(int argc, char *argv[])
 		break;
 
 	      default:
-		fprintf(stderr,
-"Usage: %s [-abCHIijnPqStuvwz] [-d count] [-m io | cpu] [-o field] [-p pid]\n"
-"       [-s time] [-J jail] [-U username] [number]\n",
-			myname);
-		exit(1);
+		errx(1, 
+"[-abCHIijnPqStuvwz] [-d count] [-m io | cpu] [-o field] [-p pid]\n"
+"       [-s time] [-J jail] [-U username] [number]");
 	    }
 	}
 
@@ -487,10 +461,8 @@ main(int argc, char *argv[])
 	{
 	    if ((topn = atoiwi(av[optind])) == Invalid)
 	    {
-		fprintf(stderr,
-			"%s: warning: process display count should be non-negative -- using default\n",
-			myname);
-		warnings++;
+			warnx("warning: process display count should be non-negative -- using default");
+			warnings++;
 	    }
             else
 	    {
@@ -525,8 +497,7 @@ main(int argc, char *argv[])
 	{
 	    const char * const *pp;
 
-	    fprintf(stderr, "%s: '%s' is not a recognized sorting order.\n",
-		    myname, order_name);
+	    warnx("'%s' is not a recognized sorting order.", order_name);
 	    fprintf(stderr, "\tTry one of these:");
 	    pp = statics.order_names;
 	    while (*pp != NULL)
@@ -547,17 +518,14 @@ main(int argc, char *argv[])
     /* initialize display interface */
     if ((max_topn = display_init(&statics)) == -1)
     {
-	fprintf(stderr, "%s: can't allocate sufficient memory\n", myname);
-	exit(4);
+		errx(4, "can't allocate sufficient memory");
     }
 
     /* print warning if user requested more processes than we can display */
     if (topn > max_topn)
     {
-	fprintf(stderr,
-		"%s: warning: this terminal can only display %d processes.\n",
-		myname, max_topn);
-	warnings++;
+		warnx("warning: this terminal can only display %d processes.", max_topn);
+		warnings++;
     }
 
     /* adjust for topn == Infinity */

Modified: head/usr.bin/top/top.h
==============================================================================
--- head/usr.bin/top/top.h	Wed Aug 22 10:04:42 2018	(r338188)
+++ head/usr.bin/top/top.h	Wed Aug 22 10:07:15 2018	(r338189)
@@ -35,8 +35,6 @@ extern int pcpu_stats;
 extern int overstrike;
 extern pid_t mypid;
 
-extern const char * myname;
-
 extern int (*compares[])(const void*, const void*);
 
 const char* kill_procs(char *);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201808221007.w7MA7Fvp069323>