[hackers] [sbase] untypedef expr, find, test, as is existing style in sbase || Evan Gates

From: <git_AT_suckless.org>
Date: Tue, 17 Mar 2015 21:04:56 +0100 (CET)

commit cab14bf253767fa9f469eef5808f95045ce85ab0
Author: Evan Gates <evan.gates_AT_gmail.com>
Date: Tue Mar 17 10:59:16 2015 -0700

    untypedef expr, find, test, as is existing style in sbase

diff --git a/expr.c b/expr.c
index aa2bea8..ef7c303 100644
--- a/expr.c
+++ b/expr.c
_AT_@ -12,15 +12,15 @@ enum {
         VAL = CHAR_MAX + 1, GE, LE, NE
 };
 
-typedef struct {
- char *s; /* iff s is NULL, Val is an integer */
+struct val {
+ char *s; /* iff s is NULL, val is an integer */
         intmax_t n;
-} Val;
+};
 
 static size_t intlen;
 
 static void
-enan(Val v)
+enan(struct val v)
 {
         if (v.s)
                 enprintf(2, "syntax error: expected integer got `%s'\n", v.s);
_AT_@ -34,7 +34,7 @@ ezero(intmax_t n)
 }
 
 static char *
-valstr(Val val, char *buf, size_t bufsiz)
+valstr(struct val val, char *buf, size_t bufsiz)
 {
         if (val.s)
                 return val.s;
_AT_@ -43,7 +43,7 @@ valstr(Val val, char *buf, size_t bufsiz)
 }
 
 static int
-valcmp(Val a, Val b)
+valcmp(struct val a, struct val b)
 {
         char buf1[intlen], buf2[intlen];
         char *astr = valstr(a, buf1, sizeof(buf1));
_AT_@ -59,8 +59,8 @@ valcmp(Val a, Val b)
  * then return the text matched by it \1 (empty string for no match)
  * else return number of characters matched (0 for no match)
  */
-static Val
-match(Val vstr, Val vregx)
+static struct val
+match(struct val vstr, struct val vregx)
 {
         regex_t re;
         regmatch_t matches[2];
_AT_@ -76,7 +76,7 @@ match(Val vstr, Val vregx)
 
         if (regexec(&re, str, 2, matches, 0)) {
                 regfree(&re);
- return (Val){ (re.re_nsub ? "" : NULL), 0 };
+ return (struct val){ (re.re_nsub ? "" : NULL), 0 };
         }
 
         if (re.re_nsub) {
_AT_@ -87,15 +87,15 @@ match(Val vstr, Val vregx)
                 *p = '\0';
                 d = strtoimax(s, &p, 10);
                 if (*s && !*p) /* string matched by subexpression is an integer */
- return (Val){ NULL, d };
+ return (struct val){ NULL, d };
 
                 /* FIXME? string is never free()d, worth fixing?
                  * need to allocate as it could be in buf1 instead of vstr.s */
- return (Val){ enstrdup(3, s), 0 };
+ return (struct val){ enstrdup(3, s), 0 };
         }
         regfree(&re);
     str += matches[0].rm_so;
- return (Val){ NULL, utfnlen(str, matches[0].rm_eo - matches[0].rm_so) };
+ return (struct val){ NULL, utfnlen(str, matches[0].rm_eo - matches[0].rm_so) };
 }
 
 /* ops points to a stack of operators, opp points to one past the last op
_AT_@ -105,9 +105,9 @@ match(Val vstr, Val vregx)
  * pop operator, pop two values, apply operator, push result
  */
 static void
-doop(int *ops, int **opp, Val *vals, Val **valp)
+doop(int *ops, int **opp, struct val *vals, struct val **valp)
 {
- Val ret, a, b;
+ struct val ret, a, b;
         int op;
 
         /* For an operation, we need a valid operator
_AT_@ -123,30 +123,30 @@ doop(int *ops, int **opp, Val *vals, Val **valp)
 
         switch (op) {
         case '|':
- if ( a.s && *a.s) ret = (Val){ a.s , 0 };
- else if (!a.s && a.n) ret = (Val){ NULL, a.n };
- else if ( b.s && *b.s) ret = (Val){ b.s , 0 };
- else ret = (Val){ NULL, b.n };
+ if ( a.s && *a.s) ret = (struct val){ a.s , 0 };
+ else if (!a.s && a.n) ret = (struct val){ NULL, a.n };
+ else if ( b.s && *b.s) ret = (struct val){ b.s , 0 };
+ else ret = (struct val){ NULL, b.n };
                 break;
         case '&':
                 if (((a.s && *a.s) || a.n) && ((b.s && *b.s) || b.n))
                         ret = a;
                 else
- ret = (Val){ NULL, 0 };
+ ret = (struct val){ NULL, 0 };
                 break;
 
- case '=': ret = (Val){ NULL, valcmp(a, b) == 0 }; break;
- case '>': ret = (Val){ NULL, valcmp(a, b) > 0 }; break;
- case GE : ret = (Val){ NULL, valcmp(a, b) >= 0 }; break;
- case '<': ret = (Val){ NULL, valcmp(a, b) < 0 }; break;
- case LE : ret = (Val){ NULL, valcmp(a, b) <= 0 }; break;
- case NE : ret = (Val){ NULL, valcmp(a, b) != 0 }; break;
+ case '=': ret = (struct val){ NULL, valcmp(a, b) == 0 }; break;
+ case '>': ret = (struct val){ NULL, valcmp(a, b) > 0 }; break;
+ case GE : ret = (struct val){ NULL, valcmp(a, b) >= 0 }; break;
+ case '<': ret = (struct val){ NULL, valcmp(a, b) < 0 }; break;
+ case LE : ret = (struct val){ NULL, valcmp(a, b) <= 0 }; break;
+ case NE : ret = (struct val){ NULL, valcmp(a, b) != 0 }; break;
 
- case '+': enan(a); enan(b); ret = (Val){ NULL, a.n + b.n }; break;
- case '-': enan(a); enan(b); ret = (Val){ NULL, a.n - b.n }; break;
- case '*': enan(a); enan(b); ret = (Val){ NULL, a.n * b.n }; break;
- case '/': enan(a); enan(b); ezero(b.n); ret = (Val){ NULL, a.n / b.n }; break;
- case '%': enan(a); enan(b); ezero(b.n); ret = (Val){ NULL, a.n % b.n }; break;
+ case '+': enan(a); enan(b); ret = (struct val){ NULL, a.n + b.n }; break;
+ case '-': enan(a); enan(b); ret = (struct val){ NULL, a.n - b.n }; break;
+ case '*': enan(a); enan(b); ret = (struct val){ NULL, a.n * b.n }; break;
+ case '/': enan(a); enan(b); ezero(b.n); ret = (struct val){ NULL, a.n / b.n }; break;
+ case '%': enan(a); enan(b); ezero(b.n); ret = (struct val){ NULL, a.n % b.n }; break;
 
         case ':': ret = match(a, b); break;
         }
_AT_@ -160,7 +160,7 @@ doop(int *ops, int **opp, Val *vals, Val **valp)
  * if it is a value, place the value in v for use by parser
  */
 static int
-lex(char *s, Val *v)
+lex(char *s, struct val *v)
 {
         intmax_t d;
         char *p, *ops = "|&=><+-*/%():";
_AT_@ -168,7 +168,7 @@ lex(char *s, Val *v)
         /* clean integer */
         d = strtoimax(s, &p, 10);
         if (*s && !*p) {
- *v = (Val){ NULL, d };
+ *v = (struct val){ NULL, d };
                 return VAL;
         }
 
_AT_@ -182,7 +182,7 @@ lex(char *s, Val *v)
         if (!strcmp(s, "!=")) return NE;
 
         /* nothing matched, treat as string */
- *v = (Val){ s, 0 };
+ *v = (struct val){ s, 0 };
         return VAL;
 }
 
_AT_@ -196,7 +196,7 @@ lex(char *s, Val *v)
 static int
 parse(char *expr[], int exprlen)
 {
- Val vals[exprlen], *valp = vals, v;
+ struct val vals[exprlen], *valp = vals, v;
         int ops[exprlen], *opp = ops;
         int i, type, lasttype = 0;
         char prec[] = { /* precedence of operators */
diff --git a/find.c b/find.c
index 9a427a2..512bd36 100644
--- a/find.c
+++ b/find.c
_AT_@ -17,75 +17,74 @@
 #include "util.h"
 
 /* because putting integers in pointers is undefined by the standard */
-typedef union {
+union extra {
         void *p;
         intptr_t i;
-} Extra;
+};
 
 /* Argument passed into a primary's function */
-typedef struct {
+struct arg {
         char *path;
         struct stat *st;
- Extra extra;
-} Arg;
+ union extra extra;
+};
 
 /* Information about each primary, for lookup table */
-typedef struct {
+struct pri_info {
         char *name;
- int (*func)(Arg *arg);
- char **(*getarg)(char **argv, Extra *extra);
- void (*freearg)(Extra extra);
+ int (*func)(struct arg *arg);
+ char **(*getarg)(char **argv, union extra *extra);
+ void (*freearg)(union extra extra);
         char narg; /* -xdev, -depth, -print don't take args but have getarg() */
-} Pri_info;
+};
 
 /* Information about operators, for lookup table */
-typedef struct {
+struct op_info {
         char *name; /* string representation of op */
- char type; /* from Tok.type */
+ char type; /* from tok.type */
         char prec; /* precedence */
         char nargs; /* number of arguments (unary or binary) */
         char lassoc; /* left associative */
-} Op_info;
+};
 
 /* Token when lexing/parsing
  * (although also used for the expression tree) */
-typedef struct Tok Tok;
-struct Tok {
- Tok *left, *right; /* if (type == NOT) left = NULL */
- Extra extra;
+struct tok {
+ struct tok *left, *right; /* if (type == NOT) left = NULL */
+ union extra extra;
         union {
- Pri_info *pinfo; /* if (type == PRIM) */
- Op_info *oinfo;
+ struct pri_info *pinfo; /* if (type == PRIM) */
+ struct op_info *oinfo;
         } u;
         enum {
                 PRIM = 0, LPAR, RPAR, NOT, AND, OR, END
         } type;
 };
 
-/* structures used for Arg.extra.p and Tok.extra.p */
-typedef struct {
+/* structures used for arg.extra.p and tok.extra.p */
+struct permarg {
         mode_t mode;
         char exact;
-} Permarg;
+};
 
-typedef struct {
+struct okarg {
         char ***braces;
         char **argv;
-} Okarg;
+};
 
 /* for all arguments that take a number
  * +n, n, -n mean > n, == n, < n respectively */
-typedef struct {
+struct narg {
         int (*cmp)(int a, int b);
         int n;
-} Narg;
+};
 
-typedef struct {
- Narg n;
+struct sizearg {
+ struct narg n;
         char bytes; /* size is in bytes, not 512 byte sectors */
-} Sizearg;
+};
 
-typedef struct {
+struct execarg {
         union {
                 struct {
                         char ***braces; /* NULL terminated list of pointers into argv where {} were */
_AT_@ -100,76 +99,75 @@ typedef struct {
         } u;
         char **argv; /* NULL terminated list of arguments (allocated if isplus) */
         char isplus; /* -exec + instead of -exec ; */
-} Execarg;
+};
 
 /* used to find loops while recursing through directory structure */
-typedef struct Findhist Findhist;
-struct Findhist {
- Findhist *next;
- char *path;
- dev_t dev;
- ino_t ino;
+struct findhist {
+ struct findhist *next;
+ char *path;
+ dev_t dev;
+ ino_t ino;
 };
 
 /* Primaries */
-static int pri_name (Arg *arg);
-static int pri_path (Arg *arg);
-static int pri_nouser (Arg *arg);
-static int pri_nogroup(Arg *arg);
-static int pri_xdev (Arg *arg);
-static int pri_prune (Arg *arg);
-static int pri_perm (Arg *arg);
-static int pri_type (Arg *arg);
-static int pri_links (Arg *arg);
-static int pri_user (Arg *arg);
-static int pri_group (Arg *arg);
-static int pri_size (Arg *arg);
-static int pri_atime (Arg *arg);
-static int pri_ctime (Arg *arg);
-static int pri_mtime (Arg *arg);
-static int pri_exec (Arg *arg);
-static int pri_ok (Arg *arg);
-static int pri_print (Arg *arg);
-static int pri_newer (Arg *arg);
-static int pri_depth (Arg *arg);
+static int pri_name (struct arg *arg);
+static int pri_path (struct arg *arg);
+static int pri_nouser (struct arg *arg);
+static int pri_nogroup(struct arg *arg);
+static int pri_xdev (struct arg *arg);
+static int pri_prune (struct arg *arg);
+static int pri_perm (struct arg *arg);
+static int pri_type (struct arg *arg);
+static int pri_links (struct arg *arg);
+static int pri_user (struct arg *arg);
+static int pri_group (struct arg *arg);
+static int pri_size (struct arg *arg);
+static int pri_atime (struct arg *arg);
+static int pri_ctime (struct arg *arg);
+static int pri_mtime (struct arg *arg);
+static int pri_exec (struct arg *arg);
+static int pri_ok (struct arg *arg);
+static int pri_print (struct arg *arg);
+static int pri_newer (struct arg *arg);
+static int pri_depth (struct arg *arg);
 
 /* Getargs */
-static char **get_name_arg (char *argv[], Extra *extra);
-static char **get_path_arg (char *argv[], Extra *extra);
-static char **get_xdev_arg (char *argv[], Extra *extra);
-static char **get_perm_arg (char *argv[], Extra *extra);
-static char **get_type_arg (char *argv[], Extra *extra);
-static char **get_n_arg (char *argv[], Extra *extra);
-static char **get_user_arg (char *argv[], Extra *extra);
-static char **get_group_arg(char *argv[], Extra *extra);
-static char **get_size_arg (char *argv[], Extra *extra);
-static char **get_exec_arg (char *argv[], Extra *extra);
-static char **get_ok_arg (char *argv[], Extra *extra);
-static char **get_print_arg(char *argv[], Extra *extra);
-static char **get_newer_arg(char *argv[], Extra *extra);
-static char **get_depth_arg(char *argv[], Extra *extra);
+static char **get_name_arg (char *argv[], union extra *extra);
+static char **get_path_arg (char *argv[], union extra *extra);
+static char **get_xdev_arg (char *argv[], union extra *extra);
+static char **get_perm_arg (char *argv[], union extra *extra);
+static char **get_type_arg (char *argv[], union extra *extra);
+static char **get_n_arg (char *argv[], union extra *extra);
+static char **get_user_arg (char *argv[], union extra *extra);
+static char **get_group_arg(char *argv[], union extra *extra);
+static char **get_size_arg (char *argv[], union extra *extra);
+static char **get_exec_arg (char *argv[], union extra *extra);
+static char **get_ok_arg (char *argv[], union extra *extra);
+static char **get_print_arg(char *argv[], union extra *extra);
+static char **get_newer_arg(char *argv[], union extra *extra);
+static char **get_depth_arg(char *argv[], union extra *extra);
 
 /* Freeargs */
-static void free_extra (Extra extra);
-static void free_exec_arg(Extra extra);
-static void free_ok_arg (Extra extra);
+static void free_extra (union extra extra);
+static void free_exec_arg(union extra extra);
+static void free_ok_arg (union extra extra);
 
 /* Parsing/Building/Running */
-static void fill_narg(char *s, Narg *n);
-static Pri_info *find_primary(char *name);
-static Op_info *find_op(char *name);
+static void fill_narg(char *s, struct narg *n);
+static struct pri_info *find_primary(char *name);
+static struct op_info *find_op(char *name);
 static void parse(int argc, char **argv);
-static int eval(Tok *tok, Arg *arg);
-static void find(char *path, Findhist *hist);
+static int eval(struct tok *tok, struct arg *arg);
+static void find(char *path, struct findhist *hist);
 static void usage(void);
 
-/* for comparisons with Narg */
+/* for comparisons with narg */
 static int cmp_gt(int a, int b) { return a > b; }
 static int cmp_eq(int a, int b) { return a == b; }
 static int cmp_lt(int a, int b) { return a < b; }
 
 /* order from find(1p), may want to alphabetize */
-static Pri_info primaries[] = {
+static struct pri_info primaries[] = {
         { "-name" , pri_name , get_name_arg , NULL , 1 },
         { "-path" , pri_path , get_path_arg , NULL , 1 },
         { "-nouser" , pri_nouser , NULL , NULL , 1 },
_AT_@ -194,7 +192,7 @@ static Pri_info primaries[] = {
         { NULL, NULL, NULL, NULL, 0 }
 };
 
-static Op_info ops[] = {
+static struct op_info ops[] = {
         { "(" , LPAR, 0, 0, 0 }, /* parens are handled specially */
         { ")" , RPAR, 0, 0, 0 },
         { "!" , NOT , 3, 1, 0 },
_AT_@ -206,8 +204,8 @@ static Op_info ops[] = {
 
 extern char **environ;
 
-static Tok *toks; /* holds allocated array of all Toks created while parsing */
-static Tok *root; /* points to root of expression tree, inside toks array */
+static struct tok *toks; /* holds allocated array of all toks created while parsing */
+static struct tok *root; /* points to root of expression tree, inside toks array */
 
 static struct timespec start; /* time find was started, used for -[acm]time */
 
_AT_@ -228,13 +226,13 @@ static struct {
  * Primaries
  */
 static int
-pri_name(Arg *arg)
+pri_name(struct arg *arg)
 {
         return !fnmatch((char *)arg->extra.p, basename(arg->path), 0);
 }
 
 static int
-pri_path(Arg *arg)
+pri_path(struct arg *arg)
 {
         return !fnmatch((char *)arg->extra.p, arg->path, 0);
 }
_AT_@ -242,39 +240,39 @@ pri_path(Arg *arg)
 /* FIXME: what about errors? find(1p) literally just says
  * "for which the getpwuid() function ... returns NULL" */
 static int
-pri_nouser(Arg *arg)
+pri_nouser(struct arg *arg)
 {
         return !getpwuid(arg->st->st_uid);
 }
 
 static int
-pri_nogroup(Arg *arg)
+pri_nogroup(struct arg *arg)
 {
         return !getgrgid(arg->st->st_gid);
 }
 
 static int
-pri_xdev(Arg *arg)
+pri_xdev(struct arg *arg)
 {
         return 1;
 }
 
 static int
-pri_prune(Arg *arg)
+pri_prune(struct arg *arg)
 {
         return gflags.prune = 1;
 }
 
 static int
-pri_perm(Arg *arg)
+pri_perm(struct arg *arg)
 {
- Permarg *p = (Permarg *)arg->extra.p;
+ struct permarg *p = (struct permarg *)arg->extra.p;
 
         return (arg->st->st_mode & 07777 & (p->exact ? -1U : p->mode)) == p->mode;
 }
 
 static int
-pri_type(Arg *arg)
+pri_type(struct arg *arg)
 {
         switch ((char)arg->extra.i) {
         default : return 0; /* impossible, but placate warnings */
_AT_@ -289,28 +287,28 @@ pri_type(Arg *arg)
 }
 
 static int
-pri_links(Arg *arg)
+pri_links(struct arg *arg)
 {
- Narg *n = arg->extra.p;
+ struct narg *n = arg->extra.p;
         return n->cmp(arg->st->st_nlink, n->n);
 }
 
 static int
-pri_user(Arg *arg)
+pri_user(struct arg *arg)
 {
         return arg->st->st_uid == (uid_t)arg->extra.i;
 }
 
 static int
-pri_group(Arg *arg)
+pri_group(struct arg *arg)
 {
         return arg->st->st_gid == (gid_t)arg->extra.i;
 }
 
 static int
-pri_size(Arg *arg)
+pri_size(struct arg *arg)
 {
- Sizearg *s = arg->extra.p;
+ struct sizearg *s = arg->extra.p;
         off_t size = arg->st->st_size;
 
         if (!s->bytes)
_AT_@ -321,37 +319,37 @@ pri_size(Arg *arg)
 
 /* FIXME: ignoring nanoseconds in atime, ctime, mtime */
 static int
-pri_atime(Arg *arg)
+pri_atime(struct arg *arg)
 {
- Narg *n = arg->extra.p;
+ struct narg *n = arg->extra.p;
         time_t time = (n->n - start.tv_sec) / 86400;
         return n->cmp(time, n->n);
 }
 
 static int
-pri_ctime(Arg *arg)
+pri_ctime(struct arg *arg)
 {
- Narg *n = arg->extra.p;
+ struct narg *n = arg->extra.p;
         time_t time = (n->n - start.tv_sec) / 86400;
         return n->cmp(time, n->n);
 }
 
 static int
-pri_mtime(Arg *arg)
+pri_mtime(struct arg *arg)
 {
- Narg *n = arg->extra.p;
+ struct narg *n = arg->extra.p;
         time_t time = (n->n - start.tv_sec) / 86400;
         return n->cmp(time, n->n);
 }
 
 static int
-pri_exec(Arg *arg)
+pri_exec(struct arg *arg)
 {
         int status;
         size_t len;
         pid_t pid;
         char **sp, ***brace;
- Execarg *e = arg->extra.p;
+ struct execarg *e = arg->extra.p;
 
         if (e->isplus) {
                 len = strlen(arg->path) + 1;
_AT_@ -406,12 +404,12 @@ pri_exec(Arg *arg)
 }
 
 static int
-pri_ok(Arg *arg)
+pri_ok(struct arg *arg)
 {
         int status;
         pid_t pid;
         char ***brace, reply, buf[256];
- Okarg *o = arg->extra.p;
+ struct okarg *o = arg->extra.p;
 
         fprintf(stderr, "%s: %s ?", *o->argv, arg->path);
         reply = fgetc(stdin);
_AT_@ -445,7 +443,7 @@ pri_ok(Arg *arg)
 }
 
 static int
-pri_print(Arg *arg)
+pri_print(struct arg *arg)
 {
         if (puts(arg->path) == EOF)
                 eprintf("puts failed:");
_AT_@ -454,13 +452,13 @@ pri_print(Arg *arg)
 
 /* FIXME: ignoring nanoseconds */
 static int
-pri_newer(Arg *arg)
+pri_newer(struct arg *arg)
 {
         return arg->st->st_mtime > (time_t)arg->extra.i;
 }
 
 static int
-pri_depth(Arg *arg)
+pri_depth(struct arg *arg)
 {
         return 1;
 }
_AT_@ -471,30 +469,30 @@ pri_depth(Arg *arg)
  * return pointer to last argument, the pointer will be incremented in parse()
  */
 static char **
-get_name_arg(char *argv[], Extra *extra)
+get_name_arg(char *argv[], union extra *extra)
 {
         extra->p = *argv;
         return argv;
 }
 
 static char **
-get_path_arg(char *argv[], Extra *extra)
+get_path_arg(char *argv[], union extra *extra)
 {
         extra->p = *argv;
         return argv;
 }
 
 static char **
-get_xdev_arg(char *argv[], Extra *extra)
+get_xdev_arg(char *argv[], union extra *extra)
 {
         gflags.xdev = 1;
         return argv;
 }
 
 static char **
-get_perm_arg(char *argv[], Extra *extra)
+get_perm_arg(char *argv[], union extra *extra)
 {
- Permarg *p = extra->p = emalloc(sizeof(*p));
+ struct permarg *p = extra->p = emalloc(sizeof(*p));
 
         if (**argv == '-')
                 (*argv)++;
_AT_@ -507,7 +505,7 @@ get_perm_arg(char *argv[], Extra *extra)
 }
 
 static char **
-get_type_arg(char *argv[], Extra *extra)
+get_type_arg(char *argv[], union extra *extra)
 {
         if (!strchr("bcdlpfs", **argv))
                 eprintf("invalid type %c for -type primary\n", **argv);
_AT_@ -517,15 +515,15 @@ get_type_arg(char *argv[], Extra *extra)
 }
 
 static char **
-get_n_arg(char *argv[], Extra *extra)
+get_n_arg(char *argv[], union extra *extra)
 {
- Narg *n = extra->p = emalloc(sizeof(*n));
+ struct narg *n = extra->p = emalloc(sizeof(*n));
         fill_narg(*argv, n);
         return argv;
 }
 
 static char **
-get_user_arg(char *argv[], Extra *extra)
+get_user_arg(char *argv[], union extra *extra)
 {
         char *end;
         struct passwd *p = getpwnam(*argv);
_AT_@ -541,7 +539,7 @@ get_user_arg(char *argv[], Extra *extra)
 }
 
 static char **
-get_group_arg(char *argv[], Extra *extra)
+get_group_arg(char *argv[], union extra *extra)
 {
         char *end;
         struct group *g = getgrnam(*argv);
_AT_@ -557,10 +555,10 @@ get_group_arg(char *argv[], Extra *extra)
 }
 
 static char **
-get_size_arg(char *argv[], Extra *extra)
+get_size_arg(char *argv[], union extra *extra)
 {
         char *p = *argv + strlen(*argv);
- Sizearg *s = extra->p = emalloc(sizeof(*s));
+ struct sizearg *s = extra->p = emalloc(sizeof(*s));
         /* if the number is followed by 'c', the size will by in bytes */
         if ((s->bytes = (p > *argv && *--p == 'c')))
                 *p = '\0';
_AT_@ -570,11 +568,11 @@ get_size_arg(char *argv[], Extra *extra)
 }
 
 static char **
-get_exec_arg(char *argv[], Extra *extra)
+get_exec_arg(char *argv[], union extra *extra)
 {
         char **arg, **new, ***braces;
         int nbraces = 0;
- Execarg *e = extra->p = emalloc(sizeof(*e));
+ struct execarg *e = extra->p = emalloc(sizeof(*e));
 
         for (arg = argv; *arg; arg++)
                 if (!strcmp(*arg, ";"))
_AT_@ -615,11 +613,11 @@ get_exec_arg(char *argv[], Extra *extra)
 }
 
 static char **
-get_ok_arg(char *argv[], Extra *extra)
+get_ok_arg(char *argv[], union extra *extra)
 {
         char **arg, ***braces;
         int nbraces = 0;
- Okarg *o = extra->p = emalloc(sizeof(*o));
+ struct okarg *o = extra->p = emalloc(sizeof(*o));
 
         for (arg = argv; *arg; arg++)
                 if (!strcmp(*arg, ";"))
_AT_@ -643,7 +641,7 @@ get_ok_arg(char *argv[], Extra *extra)
 }
 
 static char **
-get_print_arg(char *argv[], Extra *extra)
+get_print_arg(char *argv[], union extra *extra)
 {
         gflags.print = 0;
         return argv;
_AT_@ -651,7 +649,7 @@ get_print_arg(char *argv[], Extra *extra)
 
 /* FIXME: ignoring nanoseconds */
 static char **
-get_newer_arg(char *argv[], Extra *extra)
+get_newer_arg(char *argv[], union extra *extra)
 {
         struct stat st;
 
_AT_@ -663,7 +661,7 @@ get_newer_arg(char *argv[], Extra *extra)
 }
 
 static char **
-get_depth_arg(char *argv[], Extra *extra)
+get_depth_arg(char *argv[], union extra *extra)
 {
         gflags.depth = 1;
         return argv;
_AT_@ -673,18 +671,18 @@ get_depth_arg(char *argv[], Extra *extra)
  * Freeargs
  */
 static void
-free_extra(Extra extra)
+free_extra(union extra extra)
 {
         free(extra.p);
 }
 
 static void
-free_exec_arg(Extra extra)
+free_exec_arg(union extra extra)
 {
         int status;
         pid_t pid;
         char **arg;
- Execarg *e = extra.p;
+ struct execarg *e = extra.p;
 
         if (!e->isplus) {
                 free(e->u.s.braces);
_AT_@ -712,9 +710,9 @@ free_exec_arg(Extra extra)
 }
 
 static void
-free_ok_arg(Extra extra)
+free_ok_arg(union extra extra)
 {
- Okarg *o = extra.p;
+ struct okarg *o = extra.p;
 
         free(o->braces);
         free(o);
_AT_@ -724,7 +722,7 @@ free_ok_arg(Extra extra)
  * Parsing/Building/Running
  */
 static void
-fill_narg(char *s, Narg *n)
+fill_narg(char *s, struct narg *n)
 {
         char *end;
 
_AT_@ -738,10 +736,10 @@ fill_narg(char *s, Narg *n)
                 eprintf("bad number '%s'\n", s);
 }
 
-static Pri_info *
+static struct pri_info *
 find_primary(char *name)
 {
- Pri_info *p;
+ struct pri_info *p;
 
         for (p = primaries; p->name; p++)
                 if (!strcmp(name, p->name))
_AT_@ -749,10 +747,10 @@ find_primary(char *name)
         return NULL;
 }
 
-static Op_info *
+static struct op_info *
 find_op(char *name)
 {
- Op_info *o;
+ struct op_info *o;
 
         for (o = ops; o->name; o++)
                 if (!strcmp(name, o->name))
_AT_@ -761,32 +759,32 @@ find_op(char *name)
 }
 
 /* given the expression from the command line
- * 1) convert arguments from strings to Tok and place in an array duplicating
+ * 1) convert arguments from strings to tok and place in an array duplicating
  * the infix expression given, inserting "-a" where it was omitted
- * 2) allocate an array to hold the correct number of Tok, and convert from
+ * 2) allocate an array to hold the correct number of tok, and convert from
  * infix to rpn (using shunting-yard), add -a and -print if necessary
  * 3) evaluate the rpn filling in left and right pointers to create an
- * expression tree (Tok are still all contained in the rpn array, just
+ * expression tree (tok are still all contained in the rpn array, just
  * pointing at eachother)
  */
 static void
 parse(int argc, char **argv)
 {
- Tok infix[2 * argc + 1], *stack[argc], *tok, *rpn, *out, **top;
- Op_info *op;
- Pri_info *pri;
+ struct tok infix[2 * argc + 1], *stack[argc], *tok, *rpn, *out, **top;
+ struct op_info *op;
+ struct pri_info *pri;
         char **arg;
         int lasttype = -1;
         size_t ntok = 0;
- Tok and = { .u.oinfo = find_op("-a"), .type = AND };
+ struct tok and = { .u.oinfo = find_op("-a"), .type = AND };
 
         gflags.print = 1;
 
- /* convert argv to infix expression of Tok, inserting in *tok */
+ /* convert argv to infix expression of tok, inserting in *tok */
         for (arg = argv, tok = infix; *arg; arg++, tok++) {
                 pri = find_primary(*arg);
 
- if (pri) { /* token is a primary, fill out Tok and get arguments */
+ if (pri) { /* token is a primary, fill out tok and get arguments */
                         if (lasttype == PRIM || lasttype == RPAR) {
                                 *tok++ = and;
                                 ntok++;
_AT_@ -862,7 +860,7 @@ parse(int argc, char **argv)
          * if there was an expression but no -print, -exec, or -ok, add -a -print
          * in rpn, not infix */
         if (gflags.print)
- *out++ = (Tok){ .u.pinfo = find_primary("-print"), .type = PRIM };
+ *out++ = (struct tok){ .u.pinfo = find_primary("-print"), .type = PRIM };
         if (gflags.print == 2)
                 *out++ = and;
 
_AT_@ -897,7 +895,7 @@ parse(int argc, char **argv)
  * NOTE: operator NOT has NULL left side, expression on right side
  */
 static int
-eval(Tok *tok, Arg *arg)
+eval(struct tok *tok, struct arg *arg)
 {
         int ret;
 
_AT_@ -921,14 +919,14 @@ eval(Tok *tok, Arg *arg)
  * recurse
  */
 static void
-find(char *path, Findhist *hist)
+find(char *path, struct findhist *hist)
 {
         struct stat st;
         DIR *dir;
         struct dirent *de;
- Findhist *f, cur;
+ struct findhist *f, cur;
         size_t len = strlen(path) + 2; /* null and '/' */
- Arg arg = { path, &st, { NULL } };
+ struct arg arg = { path, &st, { NULL } };
 
         if ((gflags.l || (gflags.h && !hist) ? stat(path, &st) : lstat(path, &st)) < 0) {
                 weprintf("failed to stat %s:", path);
_AT_@ -1000,7 +998,7 @@ main(int argc, char **argv)
 {
         char **paths;
         int npaths;
- Tok *t;
+ struct tok *t;
 
         ARGBEGIN {
         case 'H': gflags.l = !(gflags.h = 1); break;
diff --git a/test.c b/test.c
index 1670904..367cec3 100644
--- a/test.c
+++ b/test.c
_AT_@ -38,12 +38,12 @@ static int binary_ge(char *s1, char *s2) { long long a = STOI(s1), b = STOI(s2);
 static int binary_lt(char *s1, char *s2) { long long a = STOI(s1), b = STOI(s2); return a < b; }
 static int binary_le(char *s1, char *s2) { long long a = STOI(s1), b = STOI(s2); return a <= b; }
 
-typedef struct {
+struct test {
         char *name;
         int (*func)();
-} Test;
+};
 
-static Test unary[] = {
+static struct test unary[] = {
         { "-b", unary_b },
         { "-c", unary_c },
         { "-d", unary_d },
_AT_@ -66,7 +66,7 @@ static Test unary[] = {
         { NULL, NULL },
 };
 
-static Test binary[] = {
+static struct test binary[] = {
         { "=" , binary_se },
         { "!=" , binary_sn },
         { "-eq", binary_eq },
_AT_@ -79,10 +79,10 @@ static Test binary[] = {
         { NULL, NULL },
 };
 
-static Test *
-find_test(Test *tests, char *name)
+static struct test *
+find_test(struct test *tests, char *name)
 {
- Test *t;
+ struct test *t;
 
         for (t = tests; t->name; t++)
                 if (!strcmp(t->name, name))
_AT_@ -105,7 +105,7 @@ onearg(char *argv[])
 static int
 twoarg(char *argv[])
 {
- Test *t;
+ struct test *t;
 
         if (!strcmp(argv[0], "!"))
                 return !onearg(argv + 1);
_AT_@ -120,7 +120,7 @@ twoarg(char *argv[])
 static int
 threearg(char *argv[])
 {
- Test *t = find_test(binary, argv[1]);
+ struct test *t = find_test(binary, argv[1]);
 
         if (t)
                 return t->func(argv[0], argv[2]);
Received on Tue Mar 17 2015 - 21:04:56 CET

This archive was generated by hypermail 2.3.0 : Tue Mar 17 2015 - 21:12:18 CET