From 9cdbe5b80fcff0cb4697e5ef8bfe2c394a276b8c Mon Sep 17 00:00:00 2001 From: Jim Warner Date: Tue, 26 Apr 2016 00:00:00 -0500 Subject: [PATCH] top: miscellaneous accumulated tweaks of code/comments This commit contains the following changes, which were prompted by a coverity analysis which Craig initiated: . comment typo predicting 'String not null terminated' . eliminate 'Logically dead code' from insp_make_row() And this tweak unrelated to coverity is also included: . use more modern (recommended) approach for time call Signed-off-by: Jim Warner --- top/top.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/top/top.c b/top/top.c index a30228bd..7eac2b23 100644 --- a/top/top.c +++ b/top/top.c @@ -818,7 +818,7 @@ static int ioch (int ech, char *buf, unsigned cnt) { // it may have been the beginning of a lengthy escape sequence tcflush(STDIN_FILENO, TCIFLUSH); - // note: we do NOT produce a vaid 'string' + // note: we do NOT produce a valid 'string' return rc; } // end: ioch @@ -2304,7 +2304,7 @@ static void sysinfo_refresh (int forced) { if (forced) mem_secs = cpu_secs = 0; - time(&cur_secs); + cur_secs = time(NULL); /*** hotplug_acclimated ***/ if (3 <= cur_secs - mem_secs) { @@ -2614,7 +2614,6 @@ static inline void insp_make_row (int col, int row) { int fr, to, ofs; int hicap = 0; - capNO; if (col < INSP_RLEN(row)) memcpy(tline, Insp_p[row] + col, sizeof(tline)); else tline[0] = '\n';