Staticize, constize, de-registerize; don't shadow global definitions;

set WARNS=2.

Submitted by:	Mike Barcroft <mike@q9media.com>
This commit is contained in:
Dima Dorfman 2001-06-26 22:22:19 +00:00
parent a5cef3b45a
commit fd8879d3f5
2 changed files with 31 additions and 30 deletions

View File

@ -1,5 +1,7 @@
# @(#)Makefile 8.1 (Berkeley) 6/6/93 # From: @(#)Makefile 8.1 (Berkeley) 6/6/93
# $FreeBSD$
PROG= lastcomm PROG= lastcomm
WARNS?= 2
.include <bsd.prog.mk> .include <bsd.prog.mk>

View File

@ -32,7 +32,7 @@
*/ */
#ifndef lint #ifndef lint
static char copyright[] = static const char copyright[] =
"@(#) Copyright (c) 1980, 1993\n\ "@(#) Copyright (c) 1980, 1993\n\
The Regents of the University of California. All rights reserved.\n"; The Regents of the University of California. All rights reserved.\n";
#endif /* not lint */ #endif /* not lint */
@ -61,10 +61,9 @@ static const char rcsid[] =
time_t expand __P((u_int)); time_t expand __P((u_int));
char *flagbits __P((int)); char *flagbits __P((int));
char *getdev __P((dev_t)); const char *getdev __P((dev_t));
int requested __P((char *[], struct acct *)); int requested __P((char *[], struct acct *));
static void usage __P((void)); static void usage __P((void));
char *user_from_uid();
#define AC_UTIME 1 /* user */ #define AC_UTIME 1 /* user */
#define AC_STIME 2 /* system */ #define AC_STIME 2 /* system */
@ -81,15 +80,15 @@ main(argc, argv)
int argc; int argc;
char *argv[]; char *argv[];
{ {
register char *p; char *p;
struct acct ab; struct acct ab;
struct stat sb; struct stat sb;
FILE *fp; FILE *fp;
off_t size; off_t size;
time_t t; time_t t;
int ch; int ch;
char *acctfile; const char *acctfile;
int time = 0; int flags = 0;
acctfile = _PATH_ACCT; acctfile = _PATH_ACCT;
while ((ch = getopt(argc, argv, "f:usecSE")) != -1) while ((ch = getopt(argc, argv, "f:usecSE")) != -1)
@ -99,24 +98,24 @@ main(argc, argv)
break; break;
case 'u': case 'u':
time |= AC_UTIME; /* user time */ flags |= AC_UTIME; /* user time */
break; break;
case 's': case 's':
time |= AC_STIME; /* system time */ flags |= AC_STIME; /* system time */
break; break;
case 'e': case 'e':
time |= AC_ETIME; /* elapsed time */ flags |= AC_ETIME; /* elapsed time */
break; break;
case 'c': case 'c':
time |= AC_CTIME; /* user + system time */ flags |= AC_CTIME; /* user + system time */
break; break;
case 'S': case 'S':
time |= AC_BTIME; /* starting time */ flags |= AC_BTIME; /* starting time */
break; break;
case 'E': case 'E':
/* exit time (starting time + elapsed time )*/ /* exit time (starting time + elapsed time )*/
time |= AC_FTIME; flags |= AC_FTIME;
break; break;
case '?': case '?':
@ -125,8 +124,8 @@ main(argc, argv)
} }
/* default user + system time and starting time */ /* default user + system time and starting time */
if (!time) { if (!flags) {
time = AC_CTIME | AC_BTIME; flags = AC_CTIME | AC_BTIME;
} }
argc -= optind; argc -= optind;
@ -143,7 +142,7 @@ main(argc, argv)
size = sb.st_size - sb.st_size % sizeof(struct acct); size = sb.st_size - sb.st_size % sizeof(struct acct);
/* Check if any records to display. */ /* Check if any records to display. */
if (size < sizeof(struct acct)) if ((unsigned)size < sizeof(struct acct))
exit(0); exit(0);
/* /*
@ -184,34 +183,34 @@ main(argc, argv)
/* user + system time */ /* user + system time */
if (time & AC_CTIME) { if (flags & AC_CTIME) {
(void)printf(" %6.2f secs", (void)printf(" %6.2f secs",
(expand(ab.ac_utime) + (expand(ab.ac_utime) +
expand(ab.ac_stime))/AC_HZ); expand(ab.ac_stime))/AC_HZ);
} }
/* usr time */ /* usr time */
if (time & AC_UTIME) { if (flags & AC_UTIME) {
(void)printf(" %6.2f us", expand(ab.ac_utime)/AC_HZ); (void)printf(" %6.2f us", expand(ab.ac_utime)/AC_HZ);
} }
/* system time */ /* system time */
if (time & AC_STIME) { if (flags & AC_STIME) {
(void)printf(" %6.2f sy", expand(ab.ac_stime)/AC_HZ); (void)printf(" %6.2f sy", expand(ab.ac_stime)/AC_HZ);
} }
/* elapsed time */ /* elapsed time */
if (time & AC_ETIME) { if (flags & AC_ETIME) {
(void)printf(" %8.2f es", expand(ab.ac_etime)/AC_HZ); (void)printf(" %8.2f es", expand(ab.ac_etime)/AC_HZ);
} }
/* starting time */ /* starting time */
if (time & AC_BTIME) { if (flags & AC_BTIME) {
(void)printf(" %.16s", ctime(&ab.ac_btime)); (void)printf(" %.16s", ctime(&ab.ac_btime));
} }
/* exit time (starting time + elapsed time )*/ /* exit time (starting time + elapsed time )*/
if (time & AC_FTIME) { if (flags & AC_FTIME) {
t = ab.ac_btime; t = ab.ac_btime;
t += (time_t)(expand(ab.ac_etime)/AC_HZ); t += (time_t)(expand(ab.ac_etime)/AC_HZ);
(void)printf(" %.16s", ctime(&t)); (void)printf(" %.16s", ctime(&t));
@ -225,7 +224,7 @@ time_t
expand(t) expand(t)
u_int t; u_int t;
{ {
register time_t nt; time_t nt;
nt = t & 017777; nt = t & 017777;
t >>= 13; t >>= 13;
@ -238,7 +237,7 @@ expand(t)
char * char *
flagbits(f) flagbits(f)
register int f; int f;
{ {
static char flags[20] = "-"; static char flags[20] = "-";
char *p; char *p;
@ -257,10 +256,10 @@ flagbits(f)
int int
requested(argv, acp) requested(argv, acp)
register char *argv[]; char *argv[];
register struct acct *acp; struct acct *acp;
{ {
register char *p; const char *p;
do { do {
p = user_from_uid(acp->ac_uid, 0); p = user_from_uid(acp->ac_uid, 0);
@ -274,12 +273,12 @@ requested(argv, acp)
return (0); return (0);
} }
char * const char *
getdev(dev) getdev(dev)
dev_t dev; dev_t dev;
{ {
static dev_t lastdev = (dev_t)-1; static dev_t lastdev = (dev_t)-1;
static char *lastname; static const char *lastname;
if (dev == NODEV) /* Special case. */ if (dev == NODEV) /* Special case. */
return ("__"); return ("__");