cmd: Tidy header includes
[openafs.git] / src / cmd / cmd.c
index 9aae393..13277e6 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * Copyright 2000, International Business Machines Corporation and others.
  * All Rights Reserved.
- * 
+ *
  * This software has been released under the terms of the IBM Public
  * License.  For details, see the LICENSE file in the top-level source
  * directory or online at http://www.openafs.org/dl/license10.html
 #include <afsconfig.h>
 #include <afs/param.h>
 
+#include <roken.h>
 
-#include <sys/types.h>
 #include <ctype.h>
-#include "cmd.h"
-#include <stdio.h>
-#include <string.h>
-#include <stdlib.h>
 #include <assert.h>
 
+#include "cmd.h"
+
 /* declaration of private token type */
 struct cmd_token {
     struct cmd_token *next;
     char *key;
 };
 
-static int dummy;              /* non-null ptr used for flag existence */
+static struct cmd_item dummy;          /* non-null ptr used for flag existence */
 static struct cmd_syndesc *allSyntax = 0;
 static int noOpcodes = 0;
 static int (*beforeProc) (struct cmd_syndesc * ts, void *beforeRock) = NULL;
@@ -51,10 +49,10 @@ NName(char *a1, char *a2)
 
 /* return true if asub is a substring of amain */
 static int
-SubString(register char *amain, register char *asub)
+SubString(char *amain, char *asub)
 {
     int mlen, slen;
-    register int i, j;
+    int i, j;
     mlen = (int) strlen(amain);
     slen = (int) strlen(asub);
     j = mlen - slen;
@@ -69,13 +67,18 @@ SubString(register char *amain, register char *asub)
 }
 
 static int
-FindType(register struct cmd_syndesc *as, register char *aname)
+FindType(struct cmd_syndesc *as, char *aname)
 {
-    register int i;
+    int i;
     size_t cmdlen;
     int ambig;
     int best;
 
+    /* Allow --long-style options. */
+    if (aname[0] == '-' && aname[1] == '-' && aname[2] && aname[3]) {
+        aname++;
+    }
+
     cmdlen = strlen(aname);
     ambig = 0;
     best = -1;
@@ -103,7 +106,7 @@ FindType(register struct cmd_syndesc *as, register char *aname)
 static struct cmd_syndesc *
 FindSyntax(char *aname, int *aambig)
 {
-    register struct cmd_syndesc *ts;
+    struct cmd_syndesc *ts;
     struct cmd_syndesc *best;
     size_t cmdLen;
     int ambig;
@@ -146,7 +149,7 @@ FindSyntax(char *aname, int *aambig)
 
 /* print the help for a single parameter */
 static void
-PrintParmHelp(register struct cmd_parmdesc *aparm)
+PrintParmHelp(struct cmd_parmdesc *aparm)
 {
     if (aparm->type == CMD_FLAG) {
 #ifdef notdef
@@ -167,17 +170,17 @@ PrintParmHelp(register struct cmd_parmdesc *aparm)
 extern char *AFSVersion;
 
 static int
-VersionProc(register struct cmd_syndesc *as, void *arock)
+VersionProc(struct cmd_syndesc *as, void *arock)
 {
     printf("%s\n", AFSVersion);
     return 0;
 }
 
 void
-PrintSyntax(register struct cmd_syndesc *as)
+PrintSyntax(struct cmd_syndesc *as)
 {
-    register int i;
-    register struct cmd_parmdesc *tp;
+    int i;
+    struct cmd_parmdesc *tp;
 
     /* now print usage, from syntax table */
     if (noOpcodes)
@@ -208,9 +211,9 @@ PrintSyntax(register struct cmd_syndesc *as)
 
 /* must print newline in any case, to terminate preceding line */
 static void
-PrintAliases(register struct cmd_syndesc *as)
+PrintAliases(struct cmd_syndesc *as)
 {
-    register struct cmd_syndesc *ts;
+    struct cmd_syndesc *ts;
 
     if (as->flags & CMD_ALIAS) {
        ts = as->aliasOf;
@@ -228,10 +231,10 @@ PrintAliases(register struct cmd_syndesc *as)
 }
 
 void
-PrintFlagHelp(register struct cmd_syndesc *as)
+PrintFlagHelp(struct cmd_syndesc *as)
 {
-    register int i;
-    register struct cmd_parmdesc *tp;
+    int i;
+    struct cmd_parmdesc *tp;
     int flag_width;
     char *flag_prefix;
 
@@ -271,9 +274,9 @@ PrintFlagHelp(register struct cmd_syndesc *as)
 }
 
 static int
-AproposProc(register struct cmd_syndesc *as, void *arock)
+AproposProc(struct cmd_syndesc *as, void *arock)
 {
-    register struct cmd_syndesc *ts;
+    struct cmd_syndesc *ts;
     char *tsub;
     int didAny;
 
@@ -296,10 +299,10 @@ AproposProc(register struct cmd_syndesc *as, void *arock)
 }
 
 static int
-HelpProc(register struct cmd_syndesc *as, void *arock)
+HelpProc(struct cmd_syndesc *as, void *arock)
 {
-    register struct cmd_syndesc *ts;
-    register struct cmd_item *ti;
+    struct cmd_syndesc *ts;
+    struct cmd_item *ti;
     int ambig;
     int code = 0;
 
@@ -364,7 +367,7 @@ cmd_SetAfterProc(int (*aproc) (struct cmd_syndesc * ts, void *afterRock),
 static int
 SortSyntax(struct cmd_syndesc *as)
 {
-    register struct cmd_syndesc **ld, *ud;
+    struct cmd_syndesc **ld, *ud;
 
     for (ld = &allSyntax, ud = *ld; ud; ld = &ud->next, ud = *ld) {
        if (strcmp(ud->name, as->name) > 0) {   /* next guy is bigger than us */
@@ -382,7 +385,7 @@ cmd_CreateSyntax(char *aname,
                 int (*aproc) (struct cmd_syndesc * ts, void *arock),
                 void *arock, char *ahelp)
 {
-    register struct cmd_syndesc *td;
+    struct cmd_syndesc *td;
 
     /* can't have two cmds in no opcode mode */
     if (noOpcodes)
@@ -425,9 +428,9 @@ cmd_CreateSyntax(char *aname,
 }
 
 int
-cmd_CreateAlias(register struct cmd_syndesc *as, char *aname)
+cmd_CreateAlias(struct cmd_syndesc *as, char *aname)
 {
-    register struct cmd_syndesc *td;
+    struct cmd_syndesc *td;
 
     td = (struct cmd_syndesc *)malloc(sizeof(struct cmd_syndesc));
     assert(td);
@@ -450,14 +453,14 @@ cmd_CreateAlias(register struct cmd_syndesc *as, char *aname)
 }
 
 int
-cmd_IsAdministratorCommand(register struct cmd_syndesc *as)
+cmd_IsAdministratorCommand(struct cmd_syndesc *as)
 {
     as->flags |= CMD_ADMIN;
     return 0;
 }
 
 int
-cmd_Seek(register struct cmd_syndesc *as, int apos)
+cmd_Seek(struct cmd_syndesc *as, int apos)
 {
     if (apos >= CMD_MAXPARMS)
        return CMD_EXCESSPARMS;
@@ -466,10 +469,10 @@ cmd_Seek(register struct cmd_syndesc *as, int apos)
 }
 
 int
-cmd_AddParm(register struct cmd_syndesc *as, char *aname, int atype,
+cmd_AddParm(struct cmd_syndesc *as, char *aname, int atype,
            afs_int32 aflags, char *ahelp)
 {
-    register struct cmd_parmdesc *tp;
+    struct cmd_parmdesc *tp;
 
     if (as->nParms >= CMD_MAXPARMS)
        return CMD_EXCESSPARMS;
@@ -492,9 +495,9 @@ cmd_AddParm(register struct cmd_syndesc *as, char *aname, int atype,
 
 /* add a text item to the end of the parameter list */
 static int
-AddItem(register struct cmd_parmdesc *aparm, register char *aval)
+AddItem(struct cmd_parmdesc *aparm, char *aval)
 {
-    register struct cmd_item *ti, *ni;
+    struct cmd_item *ti, *ni;
     ti = (struct cmd_item *)calloc(1, sizeof(struct cmd_item));
     assert(ti);
     ti->data = (char *)malloc(strlen(aval) + 1);
@@ -513,10 +516,10 @@ AddItem(register struct cmd_parmdesc *aparm, register char *aval)
 
 /* skip to next non-flag item, if any */
 static int
-AdvanceType(register struct cmd_syndesc *as, register afs_int32 aval)
+AdvanceType(struct cmd_syndesc *as, afs_int32 aval)
 {
-    register afs_int32 next;
-    register struct cmd_parmdesc *tp;
+    afs_int32 next;
+    struct cmd_parmdesc *tp;
 
     /* first see if we should try to grab rest of line for this dude */
     if (as->parms[aval].flags & CMD_EXPANDS)
@@ -533,11 +536,11 @@ AdvanceType(register struct cmd_syndesc *as, register afs_int32 aval)
 
 /* discard parameters filled in by dispatch */
 static void
-ResetSyntax(register struct cmd_syndesc *as)
+ResetSyntax(struct cmd_syndesc *as)
 {
     int i;
-    register struct cmd_parmdesc *tp;
-    register struct cmd_item *ti, *ni;
+    struct cmd_parmdesc *tp;
+    struct cmd_item *ti, *ni;
 
     tp = as->parms;
     for (i = 0; i < CMD_MAXPARMS; i++, tp++) {
@@ -561,10 +564,10 @@ ResetSyntax(register struct cmd_syndesc *as)
 
 /* move the expands flag to the last one in the list */
 static int
-SetupExpandsFlag(register struct cmd_syndesc *as)
+SetupExpandsFlag(struct cmd_syndesc *as)
 {
-    register struct cmd_parmdesc *tp;
-    register int last, i;
+    struct cmd_parmdesc *tp;
+    int last, i;
 
     last = -1;
     /* find last CMD_LIST type parameter, optional or not, and make it expandable
@@ -623,10 +626,10 @@ InsertInitOpcode(int *aargc, char **aargv)
 }                              /*InsertInitOpcode */
 
 static int
-NoParmsOK(register struct cmd_syndesc *as)
+NoParmsOK(struct cmd_syndesc *as)
 {
-    register int i;
-    register struct cmd_parmdesc *td;
+    int i;
+    struct cmd_parmdesc *td;
 
     for (i = 0; i < CMD_MAXPARMS; i++) {
        td = &as->parms[i];
@@ -795,7 +798,7 @@ cmd_Dispatch(int argc, char **argv)
                return (CMD_INTERNALERROR);
            }
            if (ts->parms[j].type == CMD_FLAG) {
-               ts->parms[j].items = (struct cmd_item *)&dummy;
+               ts->parms[j].items = &dummy;
            } else {
                positional = 0;
                curType = j;
@@ -879,7 +882,7 @@ cmd_Dispatch(int argc, char **argv)
     }
 
     /*
-     * Before calling the beforeProc and afterProc and all the implications 
+     * Before calling the beforeProc and afterProc and all the implications
      * from those calls, check if the help procedure was called and call it now.
      */
     if ((ts->proc == HelpProc) || (ts->proc == AproposProc)) {
@@ -906,9 +909,9 @@ cmd_Dispatch(int argc, char **argv)
 
 /* free token list returned by parseLine */
 static int
-FreeTokens(register struct cmd_token *alist)
+FreeTokens(struct cmd_token *alist)
 {
-    register struct cmd_token *nlist;
+    struct cmd_token *nlist;
     for (; alist; alist = nlist) {
        nlist = alist->next;
        free(alist->key);
@@ -919,9 +922,9 @@ FreeTokens(register struct cmd_token *alist)
 
 /* free an argv list returned by parseline */
 int
-cmd_FreeArgv(register char **argv)
+cmd_FreeArgv(char **argv)
 {
-    register char *tp;
+    char *tp;
     for (tp = *argv; tp; argv++, tp = *argv)
        free(tp);
     return 0;
@@ -932,10 +935,10 @@ cmd_FreeArgv(register char **argv)
     later on */
 #define INITSTR ""
 static int
-CopyBackArgs(register struct cmd_token *alist, register char **argv,
+CopyBackArgs(struct cmd_token *alist, char **argv,
             afs_int32 * an, afs_int32 amaxn)
 {
-    register struct cmd_token *next;
+    struct cmd_token *next;
     afs_int32 count;
 
     count = 0;
@@ -965,7 +968,7 @@ CopyBackArgs(register struct cmd_token *alist, register char **argv,
 }
 
 static int
-quote(register int x)
+quote(int x)
 {
     if (x == '"' || x == 39 /* single quote */ )
        return 1;
@@ -974,7 +977,7 @@ quote(register int x)
 }
 
 static int
-space(register int x)
+space(int x)
 {
     if (x == 0 || x == ' ' || x == '\t' || x == '\n')
        return 1;
@@ -986,11 +989,11 @@ int
 cmd_ParseLine(char *aline, char **argv, afs_int32 * an, afs_int32 amaxn)
 {
     char tbuffer[256];
-    register char *tptr = 0;
+    char *tptr = 0;
     int inToken, inQuote;
     struct cmd_token *first, *last;
-    register struct cmd_token *ttok;
-    register int tc;
+    struct cmd_token *ttok;
+    int tc;
 
     inToken = 0;               /* not copying token chars at start */
     first = (struct cmd_token *)0;