Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 2 Jun 2018 04:20:43 +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: r334519 - head/usr.bin/top
Message-ID:  <201806020420.w524KhMb074558@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: eadler
Date: Sat Jun  2 04:20:42 2018
New Revision: 334519
URL: https://svnweb.freebsd.org/changeset/base/334519

Log:
  top(1): clean up a bit
  
  - remove unused defines
  - use standard defines for STDOUT
  - don't cast for memset
  - avoid using (void) cast

Modified:
  head/usr.bin/top/machine.c
  head/usr.bin/top/screen.c

Modified: head/usr.bin/top/machine.c
==============================================================================
--- head/usr.bin/top/machine.c	Sat Jun  2 04:08:52 2018	(r334518)
+++ head/usr.bin/top/machine.c	Sat Jun  2 04:20:42 2018	(r334519)
@@ -841,7 +841,7 @@ get_process_info(struct system_info *si, struct proces
 	total_inblock = 0;
 	total_oublock = 0;
 	total_majflt = 0;
-	memset((char *)process_states, 0, sizeof(process_states));
+	memset(process_states, 0, sizeof(process_states));
 	prefp = pref;
 	for (pp = pbase, i = 0; i < nproc; pp++, i++) {
 

Modified: head/usr.bin/top/screen.c
==============================================================================
--- head/usr.bin/top/screen.c	Sat Jun  2 04:08:52 2018	(r334518)
+++ head/usr.bin/top/screen.c	Sat Jun  2 04:20:42 2018	(r334519)
@@ -20,15 +20,16 @@
  *  preprocessor variable "TOStop".   --wnl
  */
 
-#include "top.h"
-
 #include <sys/ioctl.h>
 #include <stdlib.h>
 #include <string.h>
 #include <termios.h>
 #include <curses.h>
 #include <termcap.h>
+#include <unistd.h>
+
 #include "screen.h"
+#include "top.h"
 
 int  overstrike;
 int  screen_length;
@@ -54,10 +55,6 @@ static struct termios old_settings;
 static struct termios new_settings;
 static char is_a_terminal = false;
 
-#define	STDIN	0
-#define	STDOUT	1
-#define	STDERR	2
-
 void
 init_termcap(int interactive)
 {
@@ -164,7 +161,7 @@ init_termcap(int interactive)
     PC = (PCptr = tgetstr("pc", &bufptr)) ? *PCptr : 0;
 
     /* set convenience strings */
-    (void) strncpy(home, tgoto(cursor_motion, 0, 0), sizeof(home) - 1);
+    strncpy(home, tgoto(cursor_motion, 0, 0), sizeof(home) - 1);
     home[sizeof(home) - 1] = '\0';
     /* (lower_left is set in get_screensize) */
 
@@ -174,7 +171,7 @@ init_termcap(int interactive)
     get_screensize();
 
     /* if stdout is not a terminal, pretend we are a dumb terminal */
-    if (tcgetattr(STDOUT, &old_settings) == -1)
+    if (tcgetattr(STDOUT_FILENO, &old_settings) == -1)
     {
 	smart_terminal = false;
     }
@@ -184,7 +181,7 @@ void
 init_screen(void)
 {
     /* get the old settings for safe keeping */
-    if (tcgetattr(STDOUT, &old_settings) != -1)
+    if (tcgetattr(STDOUT_FILENO, &old_settings) != -1)
     {
 	/* copy the settings so we can modify them */
 	new_settings = old_settings;
@@ -194,7 +191,7 @@ init_screen(void)
 	new_settings.c_oflag &= ~(TAB3);
 	new_settings.c_cc[VMIN] = 1;
 	new_settings.c_cc[VTIME] = 0;
-	(void) tcsetattr(STDOUT, TCSADRAIN, &new_settings);
+	tcsetattr(STDOUT_FILENO, TCSADRAIN, &new_settings);
 
 	/* remember the erase and kill characters */
 	ch_erase = old_settings.c_cc[VERASE];
@@ -229,7 +226,7 @@ end_screen(void)
     /* if we have settings to reset, then do so */
     if (is_a_terminal)
     {
-	(void) tcsetattr(STDOUT, TCSADRAIN, &old_settings);
+	tcsetattr(STDOUT_FILENO, TCSADRAIN, &old_settings);
     }
 }
 
@@ -239,7 +236,7 @@ reinit_screen(void)
     /* install our settings if it is a terminal */
     if (is_a_terminal)
     {
-	(void) tcsetattr(STDOUT, TCSADRAIN, &new_settings);
+	tcsetattr(STDOUT_FILENO, TCSADRAIN, &new_settings);
     }
 
     /* send init string */



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