Page MenuHomeFreeBSD
Files F167293348

D54457.id172973.diff
No OneTemporary
Actions

D54457.id172973.diff
View Options

diff --git a/bin/df/df.c b/bin/df/df.c
--- a/bin/df/df.c
+++ b/bin/df/df.c
@@ -38,15 +38,18 @@
#include <sys/stat.h>
#include <sys/mount.h>
#include <sys/sysctl.h>
+
#include <getopt.h>
#include <libutil.h>
#include <locale.h>
+#include <stdbool.h>
#include <stdint.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sysexits.h>
#include <unistd.h>
+
#include <libxo/xo.h>
#define UNITS_SI 1
@@ -54,13 +57,13 @@
/* Maximum widths of various fields. */
struct maxwidths {
- int mntfrom;
- int fstype;
- int total;
- int used;
- int avail;
- int iused;
- int ifree;
+ size_t mntfrom;
+ size_t fstype;
+ size_t total;
+ size_t used;
+ size_t avail;
+ size_t iused;
+ size_t ifree;
};
static void addstat(struct statfs *, struct statfs *);
@@ -68,7 +71,7 @@
static const char **makevfslist(char *fslist, int *skip);
static int checkvfsname(const char *vfsname, const char **vfslist, int skip);
static int checkvfsselected(char *);
-static int int64width(int64_t);
+static size_t int64width(int64_t);
static char *makenetvfslist(void);
static void prthuman(const struct statfs *, int64_t);
static void prthumanval(const char *, int64_t);
@@ -78,13 +81,8 @@
static void update_maxwidths(struct maxwidths *, const struct statfs *);
static void usage(void);
-static __inline int
-imax(int a, int b)
-{
- return (a > b ? a : b);
-}
-
-static int aflag = 0, cflag, hflag, iflag, kflag, lflag = 0, nflag, Tflag;
+static bool aflag, cflag, iflag, kflag, lflag, nflag, Tflag;
+static int hflag;
static int thousands;
static int skipvfs_l, skipvfs_t;
static const char **vfslist_l, **vfslist_t;
@@ -120,7 +118,7 @@
NULL)) != -1)
switch (ch) {
case 'a':
- aflag = 1;
+ aflag = true;
break;
case 'b':
/* FALLTHROUGH */
@@ -138,7 +136,7 @@
hflag = 0;
break;
case 'c':
- cflag = 1;
+ cflag = true;
break;
case 'g':
setenv("BLOCKSIZE", "1g", 1);
@@ -151,10 +149,10 @@
hflag = UNITS_2;
break;
case 'i':
- iflag = 1;
+ iflag = true;
break;
case 'k':
- kflag++;
+ kflag = true;
setenv("BLOCKSIZE", "1k", 1);
hflag = 0;
break;
@@ -163,14 +161,14 @@
if (lflag)
break;
vfslist_l = makevfslist(makenetvfslist(), &skipvfs_l);
- lflag = 1;
+ lflag = true;
break;
case 'm':
setenv("BLOCKSIZE", "1m", 1);
hflag = 0;
break;
case 'n':
- nflag = 1;
+ nflag = true;
break;
case 't':
if (vfslist_t != NULL)
@@ -178,7 +176,7 @@
vfslist_t = makevfslist(optarg, &skipvfs_t);
break;
case 'T':
- Tflag = 1;
+ Tflag = true;
break;
case ',':
thousands = 1;
@@ -285,7 +283,7 @@
mntsize = getmntinfo(&mntbuf, MNT_NOWAIT);
for (i = 0; i < mntsize; i++) {
- if (!strcmp(mntbuf[i].f_mntfromname, name))
+ if (strcmp(mntbuf[i].f_mntfromname, name) == 0)
return (mntbuf[i].f_mntonname);
}
return (NULL);
@@ -305,7 +303,7 @@
fslist += 2;
*skip = 1;
}
- for (i = 0, nextcp = fslist; *nextcp; nextcp++)
+ for (i = 0, nextcp = fslist; *nextcp != '\0'; nextcp++)
if (*nextcp == ',')
i++;
if ((av = malloc((size_t)(i + 2) * sizeof(char *))) == NULL) {
@@ -466,8 +464,8 @@
const char *format;
if (++timesthrough == 1) {
- mwp->mntfrom = imax(mwp->mntfrom, (int)strlen("Filesystem"));
- mwp->fstype = imax(mwp->fstype, (int)strlen("Type"));
+ mwp->mntfrom = MAX(mwp->mntfrom, strlen("Filesystem"));
+ mwp->fstype = MAX(mwp->fstype, strlen("Type"));
if (thousands) { /* make space for commas */
mwp->total += (mwp->total - 1) / 3;
mwp->used += (mwp->used - 1) / 3;
@@ -481,10 +479,10 @@
(int)strlen(header);
} else {
header = getbsize(&headerlen, &blocksize);
- mwp->total = imax(mwp->total, headerlen);
+ mwp->total = MAX(mwp->total, (size_t)headerlen);
}
- mwp->used = imax(mwp->used, (int)strlen("Used"));
- mwp->avail = imax(mwp->avail, (int)strlen("Avail"));
+ mwp->used = MAX(mwp->used, strlen("Used"));
+ mwp->avail = MAX(mwp->avail, strlen("Avail"));
xo_emit("{T:/%-*s}", mwp->mntfrom, "Filesystem");
if (Tflag)
@@ -493,9 +491,9 @@
mwp->total, header,
mwp->used, "Used", mwp->avail, "Avail");
if (iflag) {
- mwp->iused = imax(hflag ? 0 : mwp->iused,
+ mwp->iused = MAX(hflag ? 0 : mwp->iused,
(int)strlen(" iused"));
- mwp->ifree = imax(hflag ? 0 : mwp->ifree,
+ mwp->ifree = MAX(hflag ? 0 : mwp->ifree,
(int)strlen("ifree"));
xo_emit(" {T:/%*s} {T:/%*s} {T:\%iused}",
mwp->iused - 2, "iused", mwp->ifree, "ifree");
@@ -588,36 +586,35 @@
if (blocksize == 0)
getbsize(&dummy, &blocksize);
- mwp->mntfrom = imax(mwp->mntfrom, (int)strlen(sfsp->f_mntfromname));
- mwp->fstype = imax(mwp->fstype, (int)strlen(sfsp->f_fstypename));
- mwp->total = imax(mwp->total, int64width(
+ mwp->mntfrom = MAX(mwp->mntfrom, strlen(sfsp->f_mntfromname));
+ mwp->fstype = MAX(mwp->fstype, strlen(sfsp->f_fstypename));
+ mwp->total = MAX(mwp->total, int64width(
fsbtoblk((int64_t)sfsp->f_blocks, sfsp->f_bsize, blocksize)));
- mwp->used = imax(mwp->used,
+ mwp->used = MAX(mwp->used,
int64width(fsbtoblk((int64_t)sfsp->f_blocks -
(int64_t)sfsp->f_bfree, sfsp->f_bsize, blocksize)));
- mwp->avail = imax(mwp->avail, int64width(fsbtoblk(sfsp->f_bavail,
+ mwp->avail = MAX(mwp->avail, int64width(fsbtoblk(sfsp->f_bavail,
sfsp->f_bsize, blocksize)));
- mwp->iused = imax(mwp->iused, int64width((int64_t)sfsp->f_files -
+ mwp->iused = MAX(mwp->iused, int64width((int64_t)sfsp->f_files -
sfsp->f_ffree));
- mwp->ifree = imax(mwp->ifree, int64width(sfsp->f_ffree));
+ mwp->ifree = MAX(mwp->ifree, int64width(sfsp->f_ffree));
}
/* Return the width in characters of the specified value. */
-static int
+static size_t
int64width(int64_t val)
{
- int len;
+ size_t len = 0;
- len = 0;
/* Negative or zero values require one extra digit. */
if (val <= 0) {
val = -val;
len++;
}
- while (val > 0) {
+ do {
len++;
val /= 10;
- }
+ } while (val > 0);
return (len);
}

File Metadata

Mime Type
text/plain
Expires
Fri, Aug 21, 5:37 PM (16 h, 10 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
37043787
Default Alt Text
D54457.id172973.diff (5 KB)

Event Timeline