gcc lint
[binutils-gdb.git] / gas / hash.c
index b57ba9ef25b874457eeba82e134e0a7105eb3a9a..e8894f8ce5ebb605145bffa9c693b75fb7b0db08 100644 (file)
@@ -1,23 +1,21 @@
 /* hash.c - hash table lookup strings -
-   Copyright (C) 1987, 1990, 1991 Free Software Foundation, Inc.
+   Copyright (C) 1987, 1990, 1991, 1992 Free Software Foundation, Inc.
 
-This file is part of GAS, the GNU Assembler.
+   This file is part of GAS, the GNU Assembler.
 
-GAS is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 1, or (at your option)
-any later version.
+   GAS is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2, or (at your option)
+   any later version.
 
-GAS is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+   GAS is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with GAS; see the file COPYING.  If not, write to
-the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
-
-/* static const char rcsid[] = "$Id$"; */
+   You should have received a copy of the GNU General Public License
+   along with GAS; see the file COPYING.  If not, write to
+   the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 /*
  * BUGS, GRIPES, APOLOGIA etc.
@@ -69,12 +67,13 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 \f
 /*
  *  The code and its structures are re-enterent.
+ *
  *  Before you do anything else, you must call hash_new() which will
- *  return the address of a hash-table-control-block (or NULL if there
- *  is not enough memory). You then use this address as a handle of the
- *  symbol table by passing it to all the other hash_...() functions.
- *  The only approved way to recover the memory used by the symbol table
- *  is to call hash_die() with the handle of the symbol table.
+ *  return the address of a hash-table-control-block.  You then use
+ *  this address as a handle of the symbol table by passing it to all
+ *  the other hash_...() functions.  The only approved way to recover
+ *  the memory used by the symbol table is to call hash_die() with the
+ *  handle of the symbol table.
  *
  *  Before you call hash_die() you normally delete anything pointed to
  *  by individual symbols. After hash_die() you can't use that symbol
@@ -105,8 +104,7 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
  *    (total hashes,collisions) for (reads,writes) (*)
  *  All of the above values vary in time.
  *  (*) some of these numbers will not be meaningful if we change the
- *  internals.
- */
+ *  internals. */
 \f
 /*
  *  I N T E R N A L
@@ -138,23 +136,21 @@ the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA.  */
 
 #define error  as_fatal
 
-#define DELETED     ((char *)1)        /* guarenteed invalid address */
-#define START_POWER    (11)    /* power of two: size of new hash table *//* JF was 6 */
-/* JF These next two aren't used any more. */
-/* #define START_SIZE    (64)  / * 2 ** START_POWER */
-/* #define START_FULL    (32)      / * number of entries before table expands */
+#define DELETED     ((PTR)1)   /* guarenteed invalid address */
+#define START_POWER    (11)    /* power of two: size of new hash table */
+
+/* TRUE if a symbol is in entry @ ptr.  */
 #define islive(ptr) (ptr->hash_string && ptr->hash_string!=DELETED)
-                               /* above TRUE if a symbol is in entry @ ptr */
 
-#define STAT_SIZE      (0)      /* number of slots in hash table */
-                               /* the wall does not count here */
-                               /* we expect this is always a power of 2 */
+/* Number of slots in hash table.  The wall does not count here.
+   We expect this is always a power of 2.  */
+#define STAT_SIZE      (0)
 #define STAT_ACCESS    (1)     /* number of hash_ask()s */
-#define STAT__READ     (0)      /* reading */
-#define STAT__WRITE    (1)      /* writing */
-#define STAT_COLLIDE   (3)     /* number of collisions (total) */
-                               /* this may exceed STAT_ACCESS if we have */
-                               /* lots of collisions/access */
+#define STAT__READ     (0)     /* reading */
+#define STAT__WRITE    (1)     /* writing */
+/* Number of collisions (total).  This may exceed STAT_ACCESS if we
+   have lots of collisions/access.  */
+#define STAT_COLLIDE   (3)
 #define STAT_USED      (5)     /* slots used right now */
 #define STATLENGTH     (6)     /* size of statistics block */
 #if STATLENGTH != HASH_STATLENGTH
@@ -164,7 +160,8 @@ Panic! Please make #include "stat.h" agree with previous definitions!
 /* #define SUSPECT to do runtime checks */
 /* #define TEST to be a test jig for hash...() */
 
-#ifdef TEST                    /* TEST: use smaller hash table */
+#ifdef TEST
+/* TEST: use smaller hash table */
 #undef  START_POWER
 #define START_POWER (3)
 #undef  START_SIZE
@@ -175,123 +172,105 @@ Panic! Please make #include "stat.h" agree with previous definitions!
 \f
 /*------------------ plan ---------------------------------- i = internal
 
-struct hash_control * c;
-struct hash_entry   * e;                                                    i
-int                   b[z];     buffer for statistics
-                      z         size of b
-char                * s;        symbol string (address) [ key ]
-char                * v;        value string (address)  [datum]
-boolean               f;        TRUE if we found s in hash table            i
-char                * t;        error string; "" means OK
-int                   a;        access type [0...n)                         i
-
-c=hash_new       ()             create new hash_control
-
-hash_die         (c)            destroy hash_control (and hash table)
-                                table should be empty.
-                                doesn't check if table is empty.
-                                c has no meaning after this.
-
-hash_say         (c,b,z)        report statistics of hash_control.
-                                also report number of available statistics.
-
-v=hash_delete    (c,s)          delete symbol, return old value if any.
-    ask()                       NULL means no old value.
-    f
-
-v=hash_replace   (c,s,v)        replace old value of s with v.
-    ask()                       NULL means no old value: no table change.
-    f
-
-t=hash_insert    (c,s,v)        insert (s,v) in c.
-    ask()                       return error string.
-    f                           it is an error to insert if s is already
-                                in table.
-                                if any error, c is unchanged.
-
-t=hash_jam       (c,s,v)        assert that new value of s will be v.       i
-    ask()                       it may decide to GROW the table.            i
-    f                                                                       i
-    grow()                                                                  i
-t=hash_grow      (c)            grow the hash table.                        i
-    jam()                       will invoke JAM.                            i
-
-?=hash_apply     (c,y)          apply y() to every symbol in c.
-    y                           evtries visited in 'unspecified' order.
-
-v=hash_find      (c,s)          return value of s, or NULL if s not in c.
-    ask()
-    f
-
-f,e=hash_ask()   (c,s,a)        return slot where s SHOULD live.            i
-    code()                      maintain collision stats in c.              i
-
-.=hash_code      (c,s)          compute hash-code for s,                    i
-                                from parameters of c.                       i
-
-*/
+  struct hash_control * c;
+  struct hash_entry   * e;                                                    i
+  int                   b[z];     buffer for statistics
+  z         size of b
+  char                * s;        symbol string (address) [ key ]
+  char                * v;        value string (address)  [datum]
+  boolean               f;        TRUE if we found s in hash table            i
+  char                * t;        error string; 0 means OK
+  int                   a;        access type [0...n)                         i
+
+  c=hash_new       ()             create new hash_control
+
+  hash_die         (c)            destroy hash_control (and hash table)
+  table should be empty.
+  doesn't check if table is empty.
+  c has no meaning after this.
+
+  hash_say         (c,b,z)        report statistics of hash_control.
+  also report number of available statistics.
+
+  v=hash_delete    (c,s)          delete symbol, return old value if any.
+  ask()                       NULL means no old value.
+  f
+
+  v=hash_replace   (c,s,v)        replace old value of s with v.
+  ask()                       NULL means no old value: no table change.
+  f
+
+  t=hash_insert    (c,s,v)        insert (s,v) in c.
+  ask()                       return error string.
+  f                           it is an error to insert if s is already
+  in table.
+  if any error, c is unchanged.
+
+  t=hash_jam       (c,s,v)        assert that new value of s will be v.       i
+  ask()                       it may decide to GROW the table.            i
+  f                                                                       i
+  grow()                                                                  i
+  t=hash_grow      (c)            grow the hash table.                        i
+  jam()                       will invoke JAM.                            i
+
+  ?=hash_apply     (c,y)          apply y() to every symbol in c.
+  y                           evtries visited in 'unspecified' order.
+
+  v=hash_find      (c,s)          return value of s, or NULL if s not in c.
+  ask()
+  f
+
+  f,e=hash_ask()   (c,s,a)        return slot where s SHOULD live.            i
+  code()                      maintain collision stats in c.              i
+
+  .=hash_code      (c,s)          compute hash-code for s,                    i
+  from parameters of c.                       i
+
+  */
 \f
-static char hash_found;                /* returned by hash_ask() to stop extra */
-                               /* testing. hash_ask() wants to return both */
-                               /* a slot and a status. This is the status. */
-                               /* TRUE: found symbol */
-                               /* FALSE: absent: empty or deleted slot */
-                               /* Also returned by hash_jam(). */
-                               /* TRUE: we replaced a value */
-                               /* FALSE: we inserted a value */
-
-static struct hash_entry * hash_ask();
-static int hash_code ();
-static char * hash_grow();
+/* Returned by hash_ask() to stop extra testing. hash_ask() wants to
+   return both a slot and a status. This is the status.  TRUE: found
+   symbol FALSE: absent: empty or deleted slot Also returned by
+   hash_jam().  TRUE: we replaced a value FALSE: we inserted a value.  */
+static char hash_found;
+
+static struct hash_entry *hash_ask PARAMS ((struct hash_control *,
+                                           const char *, int));
+static int hash_code PARAMS ((struct hash_control *, const char *));
+static const char *hash_grow PARAMS ((struct hash_control *));
 \f
-/*
- *             h a s h _ n e w ( )
- *
- */
+/* Create a new hash table.  Return NULL if failed; otherwise return handle
+   (address of struct hash).  */
 struct hash_control *
-hash_new()                     /* create a new hash table */
-                               /* return NULL if failed */
-                               /* return handle (address of struct hash) */
+hash_new ()
 {
-  register struct hash_control * retval;
-  register struct hash_entry *   room; /* points to hash table */
-  register struct hash_entry *   wall;
-  register struct hash_entry *   entry;
-  register int *                 ip;   /* scan stats block of struct hash_control */
-  register int *                 nd;   /* limit of stats block */
-
-  if (( room = (struct hash_entry *) malloc( sizeof(struct
-                                                   hash_entry)*((1<<START_POWER) + 1) ) ) != NULL)
-                               /* +1 for the wall entry */
-    {
-      if (( retval = (struct hash_control *) malloc(sizeof(struct
-                                                          hash_control)) ) != NULL)
-       {
-         nd = retval->hash_stat + STATLENGTH;
-         for (ip=retval->hash_stat; ip<nd; ip++)
-           {
-             *ip = 0;
-           }
-
-         retval -> hash_stat[STAT_SIZE]  = 1<<START_POWER;
-         retval -> hash_mask             = (1<<START_POWER) - 1;
-         retval -> hash_sizelog          = START_POWER;
-                               /* works for 1's compl ok */
-         retval -> hash_where            = room;
-         retval -> hash_wall             =
-           wall                          = room + (1<<START_POWER);
-         retval -> hash_full             = (1<<START_POWER)/2;
-         for (entry=room; entry<=wall; entry++)
-           {
-             entry->hash_string = NULL;
-           }
-       }
-    }
-  else
-    {
-      retval = NULL;           /* no room for table: fake a failure */
-    }
-  return(retval);              /* return NULL or set-up structs */
+  struct hash_control *retval;
+  struct hash_entry *room;     /* points to hash table */
+  struct hash_entry *wall;
+  struct hash_entry *entry;
+  int *ip;             /* scan stats block of struct hash_control */
+  int *nd;             /* limit of stats block */
+
+  room = (struct hash_entry *) xmalloc (sizeof (struct hash_entry)
+                                       /* +1 for the wall entry */
+                                       * ((1 << START_POWER) + 1));
+  retval = (struct hash_control *) xmalloc (sizeof (struct hash_control));
+
+  nd = retval->hash_stat + STATLENGTH;
+  for (ip = retval->hash_stat; ip < nd; ip++)
+    *ip = 0;
+
+  retval->hash_stat[STAT_SIZE] = 1 << START_POWER;
+  retval->hash_mask = (1 << START_POWER) - 1;
+  retval->hash_sizelog = START_POWER;
+  /* works for 1's compl ok */
+  retval->hash_where = room;
+  retval->hash_wall =
+    wall = room + (1 << START_POWER);
+  retval->hash_full = (1 << START_POWER) / 2;
+  for (entry = room; entry <= wall; entry++)
+    entry->hash_string = NULL;
+  return retval;
 }
 
 /*
@@ -305,11 +284,11 @@ hash_new()                        /* create a new hash table */
  * No errors are recoverable.
  */
 void
-hash_die(handle)
-     struct hash_control * handle;
+hash_die (handle)
+     struct hash_control *handle;
 {
-  free((char *)handle->hash_where);
-  free((char *)handle);
+  free ((char *) handle->hash_where);
+  free ((char *) handle);
 }
 \f
 /*
@@ -326,20 +305,20 @@ hash_die(handle)
  * until your buffer or hash_stat[] is exausted.
  */
 void
-hash_say(handle,buffer,bufsiz)
-     register struct hash_control * handle;
-     register int                   buffer[/*bufsiz*/];
-     register int                   bufsiz;
+hash_say (handle, buffer, bufsiz)
+     struct hash_control *handle;
+     int buffer[ /*bufsiz*/ ];
+     int bufsiz;
 {
-  register int * nd;                   /* limit of statistics block */
-  register int * ip;                   /* scan statistics */
+  int *nd;             /* limit of statistics block */
+  int *ip;             /* scan statistics */
 
-  ip = handle -> hash_stat;
-  nd = ip + min(bufsiz-1,STATLENGTH);
-  if (bufsiz>0)                        /* trust nothing! bufsiz<=0 is dangerous */
+  ip = handle->hash_stat;
+  nd = ip + min (bufsiz - 1, STATLENGTH);
+  if (bufsiz > 0)              /* trust nothing! bufsiz<=0 is dangerous */
     {
       *buffer++ = STATLENGTH;
-      for (; ip<nd; ip++,buffer++)
+      for (; ip < nd; ip++, buffer++)
        {
          *buffer = *ip;
        }
@@ -355,33 +334,33 @@ hash_say(handle,buffer,bufsiz)
  * Anyway, the symbol is not present after this function.
  *
  */
-char *                         /* NULL if string not in table, else */
-                               /* returns value of deleted symbol */
-hash_delete(handle,string)
-     register struct hash_control * handle;
-     register char *                string;
+PTR                            /* NULL if string not in table, else */
+/* returns value of deleted symbol */
+hash_delete (handle, string)
+     struct hash_control *handle;
+     const char *string;
 {
-  register char *                   retval; /* NULL if string not in table */
-  register struct hash_entry *      entry; /* NULL or entry of this symbol */
+  PTR retval;
+  struct hash_entry *entry;
 
-  entry = hash_ask(handle,string,STAT__WRITE);
+  entry = hash_ask (handle, string, STAT__WRITE);
   if (hash_found)
     {
-         retval = entry -> hash_value;
-         entry -> hash_string = DELETED; /* mark as deleted */
-         handle -> hash_stat[STAT_USED] -= 1; /* slots-in-use count */
+      retval = entry->hash_value;
+      entry->hash_string = DELETED;
+      handle->hash_stat[STAT_USED] -= 1;
 #ifdef SUSPECT
-         if (handle->hash_stat[STAT_USED]<0)
-           {
-             error("hash_delete");
-           }
+      if (handle->hash_stat[STAT_USED] < 0)
+       {
+         error ("hash_delete");
+       }
 #endif /* def SUSPECT */
     }
   else
     {
       retval = NULL;
     }
-  return(retval);
+  return (retval);
 }
 \f
 /*
@@ -392,66 +371,66 @@ hash_delete(handle,string)
  * Return NULL and don't change the table if the symbol is not already
  * in the table.
  */
-char *
-hash_replace(handle,string,value)
-     register struct hash_control * handle;
-     register char *                string;
-     register char *                value;
+PTR
+hash_replace (handle, string, value)
+     struct hash_control *handle;
+     const char *string;
+     PTR value;
 {
-  register struct hash_entry *      entry;
-  register char *                   retval;
+  struct hash_entry *entry;
+  char *retval;
 
-  entry = hash_ask(handle,string,STAT__WRITE);
+  entry = hash_ask (handle, string, STAT__WRITE);
   if (hash_found)
     {
-      retval = entry -> hash_value;
-      entry -> hash_value = value;
+      retval = entry->hash_value;
+      entry->hash_value = value;
     }
   else
     {
       retval = NULL;
     }
   ;
-  return (retval);
+  return retval;
 }
 \f
 /*
  *                   h a s h _ i n s e r t ( )
  *
  * Insert a (symbol-string, value) into the hash table.
- * Return an error string, "" means OK.
+ * Return an error string, 0 means OK.
  * It is an 'error' to insert an existing symbol.
  */
 
-char *                         /* return error string */
-hash_insert(handle,string,value)
-     register struct hash_control * handle;
-     register char *                string;
-     register char *                value;
+const char *                   /* return error string */
+hash_insert (handle, string, value)
+     struct hash_control *handle;
+     const char *string;
+     PTR value;
 {
-  register struct hash_entry * entry;
-  register char *              retval;
+  struct hash_entry *entry;
+  const char *retval;
 
-  retval = "";
+  retval = 0;
   if (handle->hash_stat[STAT_USED] > handle->hash_full)
     {
-      retval = hash_grow(handle);
+      retval = hash_grow (handle);
     }
-  if ( ! * retval)
+  if (!retval)
     {
-      entry = hash_ask(handle,string,STAT__WRITE);
+      entry = hash_ask (handle, string, STAT__WRITE);
       if (hash_found)
        {
          retval = "exists";
        }
       else
        {
-         entry -> hash_value  = value;
-         entry -> hash_string = string;
-         handle-> hash_stat[STAT_USED]  += 1;
+         entry->hash_value = value;
+         entry->hash_string = string;
+         handle->hash_stat[STAT_USED] += 1;
        }
     }
-  return(retval);
+  return retval;
 }
 \f
 /*
@@ -460,7 +439,7 @@ hash_insert(handle,string,value)
  * Regardless of what was in the symbol table before, after hash_jam()
  * the named symbol has the given value. The symbol is either inserted or
  * (its value is) relpaced.
- * An error message string is returned, "" means OK.
+ * An error message string is returned, 0 means OK.
  *
  * WARNING: this may decide to grow the hashed symbol table.
  * To do this, we call hash_grow(), WHICH WILL recursively CALL US.
@@ -468,31 +447,31 @@ hash_insert(handle,string,value)
  * We report status internally: hash_found is TRUE if we replaced, but
  * false if we inserted.
  */
-char *
-hash_jam(handle,string,value)
-     register struct hash_control * handle;
-     register char *                string;
-     register char *                value;
+const char *
+hash_jam (handle, string, value)
+     struct hash_control *handle;
+     const char *string;
+     PTR value;
 {
-  register char *                   retval;
-  register struct hash_entry *      entry;
+  const char *retval;
+  struct hash_entry *entry;
 
-  retval = "";
+  retval = 0;
   if (handle->hash_stat[STAT_USED] > handle->hash_full)
     {
-      retval = hash_grow(handle);
+      retval = hash_grow (handle);
     }
-  if (! * retval)
+  if (!retval)
     {
-      entry = hash_ask(handle,string,STAT__WRITE);
-      if ( ! hash_found)
+      entry = hash_ask (handle, string, STAT__WRITE);
+      if (!hash_found)
        {
-         entry -> hash_string = string;
+         entry->hash_string = string;
          handle->hash_stat[STAT_USED] += 1;
        }
-      entry -> hash_value = value;
+      entry->hash_value = value;
     }
-  return(retval);
+  return retval;
 }
 
 /*
@@ -500,98 +479,95 @@ hash_jam(handle,string,value)
  *
  * Grow a new (bigger) hash table from the old one.
  * We choose to double the hash table's size.
- * Return a human-scrutible error string: "" if OK.
+ * Return a human-scrutible error string: 0 if OK.
  * Warning! This uses hash_jam(), which had better not recurse
  * back here! Hash_jam() conditionally calls us, but we ALWAYS
  * call hash_jam()!
  * Internal.
  */
-static char *
-hash_grow(handle)                      /* make a hash table grow */
-     struct hash_control * handle;
+static const char *
+hash_grow (handle)             /* make a hash table grow */
+     struct hash_control *handle;
 {
-  register struct hash_entry *      newwall;
-  register struct hash_entry *      newwhere;
-  struct hash_entry *      newtrack;
-  register struct hash_entry *      oldtrack;
-  register struct hash_entry *      oldwhere;
-  register struct hash_entry *      oldwall;
-  register int                      temp;
-  int                      newsize;
-  char *                   string;
-  char *                   retval;
+  struct hash_entry *newwall;
+  struct hash_entry *newwhere;
+  struct hash_entry *newtrack;
+  struct hash_entry *oldtrack;
+  struct hash_entry *oldwhere;
+  struct hash_entry *oldwall;
+  int temp;
+  int newsize;
+  const char *string;
+  const char *retval;
 #ifdef SUSPECT
-  int                      oldused;
+  int oldused;
 #endif
 
   /*
    * capture info about old hash table
    */
-  oldwhere = handle -> hash_where;
-  oldwall  = handle -> hash_wall;
+  oldwhere = handle->hash_where;
+  oldwall = handle->hash_wall;
 #ifdef SUSPECT
-  oldused  = handle -> hash_stat[STAT_USED];
+  oldused = handle->hash_stat[STAT_USED];
 #endif
   /*
    * attempt to get enough room for a hash table twice as big
    */
   temp = handle->hash_stat[STAT_SIZE];
-  if (( newwhere = (struct hash_entry *)
-       xmalloc((long)((temp+temp+1)*sizeof(struct hash_entry)))) != NULL)
-                               /* +1 for wall slot */
+  if ((newwhere = ((struct hash_entry *)
+                  xmalloc ((unsigned long) ((temp + temp + 1)
+                                            * sizeof (struct hash_entry)))))
+      != NULL)
+    /* +1 for wall slot */
     {
-      retval = "";             /* assume success until proven otherwise */
+      retval = 0;              /* assume success until proven otherwise */
       /*
        * have enough room: now we do all the work.
        * double the size of everything in handle,
        * note: hash_mask frob works for 1's & for 2's complement machines
        */
-      handle->hash_mask              = handle->hash_mask + handle->hash_mask + 1;
+      handle->hash_mask = handle->hash_mask + handle->hash_mask + 1;
       handle->hash_stat[STAT_SIZE] <<= 1;
-      newsize                        = handle->hash_stat[STAT_SIZE];
-      handle->hash_where             = newwhere;
-      handle->hash_full            <<= 1;
-      handle->hash_sizelog         += 1;
-      handle->hash_stat[STAT_USED]   = 0;
-      handle->hash_wall              =
-      newwall                        = newwhere + newsize;
+      newsize = handle->hash_stat[STAT_SIZE];
+      handle->hash_where = newwhere;
+      handle->hash_full <<= 1;
+      handle->hash_sizelog += 1;
+      handle->hash_stat[STAT_USED] = 0;
+      handle->hash_wall =
+       newwall = newwhere + newsize;
       /*
        * set all those pesky new slots to vacant.
        */
-      for (newtrack=newwhere; newtrack <= newwall; newtrack++)
+      for (newtrack = newwhere; newtrack <= newwall; newtrack++)
        {
-         newtrack -> hash_string = NULL;
+         newtrack->hash_string = NULL;
        }
       /*
        * we will do a scan of the old table, the hard way, using the
        * new control block to re-insert the data into new hash table.
        */
-      handle -> hash_stat[STAT_USED] = 0;      /* inserts will bump it up to correct */
-      for (oldtrack=oldwhere; oldtrack < oldwall; oldtrack++)
-       {
-         if ( ((string=oldtrack->hash_string) != NULL) && string!=DELETED )
-           {
-             if ( * (retval = hash_jam(handle,string,oldtrack->hash_value) ) )
-               {
-                 break;
-               }
-           }
-       }
+      handle->hash_stat[STAT_USED] = 0;        /* inserts will bump it up to correct */
+      for (oldtrack = oldwhere; oldtrack < oldwall; oldtrack++)
+       if (((string = oldtrack->hash_string) != NULL) && string != DELETED)
+         if ((retval = hash_jam (handle, string, oldtrack->hash_value)))
+           break;
+
 #ifdef SUSPECT
-      if ( !*retval && handle->hash_stat[STAT_USED] != oldused)
+      if (!retval && handle->hash_stat[STAT_USED] != oldused)
        {
          retval = "hash_used";
        }
 #endif
-      if (!*retval)
+      if (!retval)
        {
          /*
           * we have a completely faked up control block.
           * return the old hash table.
           */
-         free((char *)oldwhere);
+         free ((char *) oldwhere);
          /*
-          * Here with success. retval is already "".
+          * Here with success. retval is already 0.
           */
        }
     }
@@ -599,7 +575,7 @@ hash_grow(handle)                   /* make a hash table grow */
     {
       retval = "no room";
     }
-  return(retval);
+  return retval;
 }
 \f
 /*
@@ -649,19 +625,19 @@ hash_grow(handle)                 /* make a hash table grow */
  * yet. (The function has no graceful failures.)
  */
 char *
-hash_apply(handle,function)
-     struct hash_control * handle;
-     char*                 (*function)();
+hash_apply (handle, function)
+     struct hash_control *handle;
+     char *(*function) ();
 {
-  register struct hash_entry *      entry;
-  register struct hash_entry *      wall;
+  struct hash_entry *entry;
+  struct hash_entry *wall;
 
   wall = handle->hash_wall;
   for (entry = handle->hash_where; entry < wall; entry++)
     {
-      if (islive(entry))       /* silly code: tests entry->string twice! */
+      if (islive (entry))      /* silly code: tests entry->string twice! */
        {
-         (*function)(entry->hash_string,entry->hash_value);
+         (*function) (entry->hash_string, entry->hash_value);
        }
     }
   return (NULL);
@@ -673,24 +649,18 @@ hash_apply(handle,function)
  * Given symbol string, find value (if any).
  * Return found value or NULL.
  */
-char *
-hash_find(handle,string)       /* return char* or NULL */
-     struct hash_control * handle;
-     char *                string;
+PTR
+hash_find (handle, string)
+     struct hash_control *handle;
+     const char *string;
 {
-  register struct hash_entry *      entry;
-  register char *                   retval;
+  struct hash_entry *entry;
 
-  entry = hash_ask(handle,string,STAT__READ);
+  entry = hash_ask (handle, string, STAT__READ);
   if (hash_found)
-    {
-      retval = entry->hash_value;
-    }
+    return entry->hash_value;
   else
-    {
-      retval = NULL;
-    }
-  return(retval);
+    return NULL;
 }
 \f
 /*
@@ -703,34 +673,27 @@ hash_find(handle,string)  /* return char* or NULL */
  * Internal.
  */
 static struct hash_entry *     /* string slot, may be empty or deleted */
-hash_ask(handle,string,access)
-     struct hash_control * handle;
-     char *                string;
-     int                   access; /* access type */
+hash_ask (handle, string, access)
+     struct hash_control *handle;
+     const char *string;
+     int access;               /* access type */
 {
-  register char        *string1;       /* JF avoid strcmp calls */
-  register char *                   s;
-  register int                      c;
-  register struct hash_entry *      slot;
-  register int                      collision; /* count collisions */
-
-  slot = handle->hash_where + hash_code(handle,string); /* start looking here */
-  handle->hash_stat[STAT_ACCESS+access] += 1;
+  const char *s;
+  struct hash_entry *slot;
+  int collision;       /* count collisions */
+
+  /* start looking here */
+  slot = handle->hash_where + hash_code (handle, string);
+
+  handle->hash_stat[STAT_ACCESS + access] += 1;
   collision = 0;
   hash_found = FALSE;
-  while ( ((s = slot->hash_string) != NULL) && s!=DELETED )
+  while (((s = slot->hash_string) != NULL) && s != DELETED)
     {
-       for(string1=string;;) {
-               if((c= *s++) == 0) {
-                       if(!*string1)
-                               hash_found = TRUE;
-                       break;
-               }
-               if(*string1++!=c)
-                       break;
-       }
-       if(hash_found)
-               break;
+      if (string == s || !strcmp (string, s))
+       hash_found = TRUE;
+      if (hash_found)
+       break;
       collision++;
       slot++;
     }
@@ -742,19 +705,13 @@ hash_ask(handle,string,access)
    *                   in table:       dig here                  slot
    *       at DELETED: dig here                                  slot
    */
-  if (slot==handle->hash_wall)
+  if (slot == handle->hash_wall)
     {
-      slot = handle->hash_where; /* now look again */
-      while( ((s = slot->hash_string) != NULL) && s!=DELETED )
+      slot = handle->hash_where;/* now look again */
+      while (((s = slot->hash_string) != NULL) && s != DELETED)
        {
-         for(string1=string;*s;string1++,s++) {
-           if(*string1!=*s)
-               break;
-         }
-         if(*s==*string1) {
-             hash_found = TRUE;
-             break;
-           }
+         if (string == s || !strcmp (string, s))
+           hash_found = TRUE;
          collision++;
          slot++;
        }
@@ -766,9 +723,8 @@ hash_ask(handle,string,access)
        *       DELETED:dig here                                   slot
        */
     }
-/*   fprintf(stderr,"hash_ask(%s)->%d(%d)\n",string,hash_code(handle,string),collision); */
-  handle -> hash_stat[STAT_COLLIDE+access] += collision;
-  return(slot);                        /* also return hash_found */
+  handle->hash_stat[STAT_COLLIDE + access] += collision;
+  return (slot);               /* also return hash_found */
 }
 \f
 /*
@@ -778,22 +734,38 @@ hash_ask(handle,string,access)
  * Internal.
  */
 static int
-hash_code(handle,string)
-     struct hash_control * handle;
-     register char *                string;
+hash_code (handle, string)
+     struct hash_control *handle;
+     const char *string;
 {
-  register long                 h;      /* hash code built here */
-  register long                 c;      /* each character lands here */
-  register int                    n;      /* Amount to shift h by */
+#if 0
+  long h;              /* hash code built here */
+  long c;              /* each character lands here */
+  int n;               /* Amount to shift h by */
 
   n = (handle->hash_sizelog - 3);
   h = 0;
   while ((c = *string++) != 0)
     {
       h += c;
-      h = (h<<3) + (h>>n) + c;
+      h = (h << 3) + (h >> n) + c;
     }
   return (h & handle->hash_mask);
+#else
+  unsigned long h = 0;
+  unsigned int len = 0;
+  unsigned int c;
+
+  while ((c = *string++) != 0)
+    {
+      h += c + (c << 17);
+      h ^= h >> 2;
+      ++len;
+    }
+  h += len + (len << 17);
+  h ^= h >> 2;
+  return h & handle->hash_mask;
+#endif
 }
 \f
 /*
@@ -802,183 +774,182 @@ hash_code(handle,string)
 #ifdef TEST
 
 #define TABLES (6)             /* number of hash tables to maintain */
-                               /* (at once) in any testing */
+/* (at once) in any testing */
 #define STATBUFSIZE (12)       /* we can have 12 statistics */
 
 int statbuf[STATBUFSIZE];      /* display statistics here */
 char answer[100];              /* human farts here */
-char * hashtable[TABLES];      /* we test many hash tables at once */
-char * h;                      /* points to curent hash_control */
-char ** pp;
-char *  p;
-char *  name;
-char *  value;
-int     size;
-int     used;
-char    command;
-int     number;                        /* number 0:TABLES-1 of current hashed */
-                               /* symbol table */
-
-main()
+char *hashtable[TABLES];       /* we test many hash tables at once */
+char *h;                       /* points to curent hash_control */
+char **pp;
+char *p;
+char *name;
+char *value;
+int size;
+int used;
+char command;
+int number;                    /* number 0:TABLES-1 of current hashed */
+/* symbol table */
+
+main ()
 {
-  char (*applicatee());
-  char * hash_find();
-  char * destroy();
-  char * what();
-  struct hash_control * hash_new();
-  char * hash_replace();
-  int *  ip;
+  char (*applicatee ());
+  char *destroy ();
+  char *what ();
+  int *ip;
 
   number = 0;
   h = 0;
-  printf("type h <RETURN> for help\n");
-  for(;;)
+  printf ("type h <RETURN> for help\n");
+  for (;;)
     {
-      printf("hash_test command: ");
-      gets(answer);
+      printf ("hash_test command: ");
+      gets (answer);
       command = answer[0];
-      if (isupper(command)) command = tolower(command);        /* ecch! */
+      if (isupper (command))
+       command = tolower (command);    /* ecch! */
       switch (command)
        {
        case '#':
-         printf("old hash table #=%d.\n",number);
-         whattable();
+         printf ("old hash table #=%d.\n", number);
+         whattable ();
          break;
        case '?':
-         for (pp=hashtable; pp<hashtable+TABLES; pp++)
+         for (pp = hashtable; pp < hashtable + TABLES; pp++)
            {
-             printf("address of hash table #%d control block is %xx\n"
-                    ,pp-hashtable,*pp);
+             printf ("address of hash table #%d control block is %xx\n"
+                     ,pp - hashtable, *pp);
            }
          break;
        case 'a':
-         hash_apply(h,applicatee);
+         hash_apply (h, applicatee);
          break;
        case 'd':
-         hash_apply(h,destroy);
-         hash_die(h);
+         hash_apply (h, destroy);
+         hash_die (h);
          break;
        case 'f':
-         p = hash_find(h,name=what("symbol"));
-         printf("value of \"%s\" is \"%s\"\n",name,p?p:"NOT-PRESENT");
+         p = hash_find (h, name = what ("symbol"));
+         printf ("value of \"%s\" is \"%s\"\n", name, p ? p : "NOT-PRESENT");
          break;
        case 'h':
-         printf("# show old, select new default hash table number\n");
-         printf("? display all hashtable control block addresses\n");
-         printf("a apply a simple display-er to each symbol in table\n");
-         printf("d die: destroy hashtable\n");
-         printf("f find value of nominated symbol\n");
-         printf("h this help\n");
-         printf("i insert value into symbol\n");
-         printf("j jam value into symbol\n");
-         printf("n new hashtable\n");
-         printf("r replace a value with another\n");
-         printf("s say what %% of table is used\n");
-         printf("q exit this program\n");
-         printf("x delete a symbol from table, report its value\n");
+         printf ("# show old, select new default hash table number\n");
+         printf ("? display all hashtable control block addresses\n");
+         printf ("a apply a simple display-er to each symbol in table\n");
+         printf ("d die: destroy hashtable\n");
+         printf ("f find value of nominated symbol\n");
+         printf ("h this help\n");
+         printf ("i insert value into symbol\n");
+         printf ("j jam value into symbol\n");
+         printf ("n new hashtable\n");
+         printf ("r replace a value with another\n");
+         printf ("s say what %% of table is used\n");
+         printf ("q exit this program\n");
+         printf ("x delete a symbol from table, report its value\n");
          break;
        case 'i':
-         p = hash_insert(h,name=what("symbol"),value=what("value"));
-         if (*p)
+         p = hash_insert (h, name = what ("symbol"), value = what ("value"));
+         if (p)
            {
-             printf("symbol=\"%s\"  value=\"%s\"  error=%s\n",name,value,p);
+             printf ("symbol=\"%s\"  value=\"%s\"  error=%s\n", name, value,
+                     p);
            }
          break;
        case 'j':
-         p = hash_jam(h,name=what("symbol"),value=what("value"));
-         if (*p)
+         p = hash_jam (h, name = what ("symbol"), value = what ("value"));
+         if (p)
            {
-             printf("symbol=\"%s\"  value=\"%s\"  error=%s\n",name,value,p);
+             printf ("symbol=\"%s\"  value=\"%s\"  error=%s\n", name, value, p);
            }
          break;
        case 'n':
-         h = hashtable[number] = (char *) hash_new();
+         h = hashtable[number] = (char *) hash_new ();
          break;
        case 'q':
-         exit();
+         exit (EXIT_SUCCESS);
        case 'r':
-         p = hash_replace(h,name=what("symbol"),value=what("value"));
-         printf("old value was \"%s\"\n",p?p:"{}");
+         p = hash_replace (h, name = what ("symbol"), value = what ("value"));
+         printf ("old value was \"%s\"\n", p ? p : "{}");
          break;
        case 's':
-         hash_say(h,statbuf,STATBUFSIZE);
-         for (ip=statbuf; ip<statbuf+STATBUFSIZE; ip++)
+         hash_say (h, statbuf, STATBUFSIZE);
+         for (ip = statbuf; ip < statbuf + STATBUFSIZE; ip++)
            {
-             printf("%d ",*ip);
+             printf ("%d ", *ip);
            }
-         printf("\n");
+         printf ("\n");
          break;
        case 'x':
-         p = hash_delete(h,name=what("symbol"));
-         printf("old value was \"%s\"\n",p?p:"{}");
+         p = hash_delete (h, name = what ("symbol"));
+         printf ("old value was \"%s\"\n", p ? p : "{}");
          break;
        default:
-         printf("I can't understand command \"%c\"\n",command);
+         printf ("I can't understand command \"%c\"\n", command);
          break;
        }
     }
 }
 
 char *
-what(description)
-     char * description;
+what (description)
+     char *description;
 {
-  char * retval;
-  char * malloc();
+  char *retval;
+  char *malloc ();
 
-  printf("   %s : ",description);
-  gets(answer);
+  printf ("   %s : ", description);
+  gets (answer);
   /* will one day clean up answer here */
-  retval = malloc(strlen(answer)+1);
+  retval = malloc (strlen (answer) + 1);
   if (!retval)
     {
-      error("room");
+      error ("room");
     }
-  (void)strcpy(retval,answer);
-  return(retval);
+  (void) strcpy (retval, answer);
+  return (retval);
 }
 
 char *
-destroy(string,value)
-     char * string;
-     char * value;
+destroy (string, value)
+     char *string;
+     char *value;
 {
-  free(string);
-  free(value);
-  return(NULL);
+  free (string);
+  free (value);
+  return (NULL);
 }
 
 
 char *
-applicatee(string,value)
-     char * string;
-     char * value;
+applicatee (string, value)
+     char *string;
+     char *value;
 {
-  printf("%.20s-%.20s\n",string,value);
-  return(NULL);
+  printf ("%.20s-%.20s\n", string, value);
+  return (NULL);
 }
 
-whattable()                    /* determine number: what hash table to use */
-                               /* also determine h: points to hash_control */
+whattable ()                   /* determine number: what hash table to use */
+     /* also determine h: points to hash_control */
 {
 
   for (;;)
     {
-      printf("   what hash table (%d:%d) ?  ",0,TABLES-1);
-      gets(answer);
-      sscanf(answer,"%d",&number);
-      if (number>=0 && number<TABLES)
+      printf ("   what hash table (%d:%d) ?  ", 0, TABLES - 1);
+      gets (answer);
+      sscanf (answer, "%d", &number);
+      if (number >= 0 && number < TABLES)
        {
          h = hashtable[number];
          if (!h)
            {
-             printf("warning: current hash-table-#%d. has no hash-control\n",number);
+             printf ("warning: current hash-table-#%d. has no hash-control\n", number);
            }
          return;
        }
       else
        {
-         printf("invalid hash table number: %d\n",number);
+         printf ("invalid hash table number: %d\n", number);
        }
     }
 }
@@ -987,4 +958,4 @@ whattable()                 /* determine number: what hash table to use */
 
 #endif /* #ifdef TEST */
 
-/* end: hash.c */
+/* end of hash.c */