[svn:parrot] r47842 - branches/hash_faster/include/parrot

[whiteknight at svn.parrot.org]

Author: whiteknight
Date: Fri Jun 25 22:34:03 2010
New Revision: 47842
URL: https://trac.parrot.org/parrot/changeset/47842

Log:
[hash] small cleanup. Remove/rework comments for clarity and modernity

Modified:
   branches/hash_faster/include/parrot/hash.h

Modified: branches/hash_faster/include/parrot/hash.h
==============================================================================
--- branches/hash_faster/include/parrot/hash.h	Fri Jun 25 22:11:27 2010	(r47841)
+++ branches/hash_faster/include/parrot/hash.h	Fri Jun 25 22:34:03 2010	(r47842)
@@ -9,10 +9,6 @@
 #ifndef PARROT_HASH_H_GUARD
 #define PARROT_HASH_H_GUARD
 
-/* Prototypes */
-
-/* Hash is really a hashtable, but 'hash' is standard perl nomenclature. */
-
 typedef enum {
     enum_hash_undef,
     enum_hash_int = enum_type_INTVAL,
@@ -22,14 +18,6 @@
     enum_hash_ptr = enum_type_ptr
 } HashEntryType;
 
-/*
- * hash_entry is currently unused in the hash structure
-
-typedef struct _hash_entry {
-    HashEntryType type;
-    UnionVal val;
-} HashEntry;
-*/
 
 /* A BucketIndex is an index into the pool of available buckets. */
 typedef UINTVAL BucketIndex;
@@ -64,8 +52,8 @@
     HashBucket *free_list;      /* empty buckets */
     UINTVAL entries;            /* Number of values stored in hashtable */
     UINTVAL mask;               /* alloced - 1 */
-    PMC *container;             /* e.g. the PerlHash PMC */
-    Hash_key_type key_type;     /* cstring, ascii-string, utf8-string */
+    PMC *container;             /* The owner PMC */
+    Hash_key_type key_type;     /* the type of key object this hash uses */
     PARROT_DATA_TYPE entry_type;/* type of value */
     size_t seed;                /* randomizes the hash_key generation
                                    updated for each new hash */