d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

top(1): clean up a bit

Browse Source 
- remove unused defines
- use standard defines for STDOUT
- don't cast for memset
- avoid using (void) cast
This commit is contained in:
Eitan Adler 2018-06-02 04:20:42 +00:00
parent cffee2bc5b
commit 0059e7102f
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=334519
2 changed files with 10 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
usr.bin/top/machine.c
View File

@ -841,7 +841,7 @@ get_process_info(struct system_info *si, struct process_select *sel,
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++) {

21
usr.bin/top/screen.c
View File

@ -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 */