[svn:parrot] r48610 - trunk/include/parrot

mikehh at svn.parrot.org mikehh at svn.parrot.org
Mon Aug 23 08:22:46 UTC 2010


Author: mikehh
Date: Mon Aug 23 08:22:45 2010
New Revision: 48610
URL: https://trac.parrot.org/parrot/changeset/48610

Log:
fix codetest failures - unwrapped macro arguments and
 there should be one space or a newline after a comma

Modified:
   trunk/include/parrot/hash.h

Modified: trunk/include/parrot/hash.h
==============================================================================
--- trunk/include/parrot/hash.h	Mon Aug 23 07:13:45 2010	(r48609)
+++ trunk/include/parrot/hash.h	Mon Aug 23 08:22:45 2010	(r48610)
@@ -85,11 +85,11 @@
 {                                                                           \
     HashBucket *_bucket = (_hash)->buckets;                                 \
     UINTVAL     _found  = 0;                                                \
-    while (_found < _hash->entries){                                        \
+    while (_found < (_hash)->entries){                                      \
         if (_bucket->key){                                                  \
             _found++;                                                       \
             {                                                               \
-                _code                                                       \
+                (_code)                                                     \
             }                                                               \
         }                                                                   \
        _bucket++;                                                           \
@@ -102,14 +102,14 @@
     for (_loc = (_hash)->mask; _loc >= 0; --_loc) {                         \
         HashBucket *_bucket = (_hash)->index[_loc];                         \
         while (_bucket) {                                                   \
-            _code                                                           \
+            (_code)                                                         \
             _bucket = _bucket->next;                                        \
         }                                                                   \
     }                                                                       \
 }
 
 
-#define parrot_hash_iterator_advance(_hash,_bucket,_loc)                    \
+#define parrot_hash_iterator_advance(_hash, _bucket, _loc)                  \
 {                                                                           \
     /* Try to advance current bucket */                                     \
     if ((_bucket))                                                          \
@@ -118,7 +118,7 @@
         /* If there is no more buckets */                                   \
         if ((_loc) == (INTVAL)(_hash)->mask+1)                              \
             break;                                                          \
-        (_bucket) = (_hash)->index[_loc++];                                 \
+        (_bucket) = (_hash)->index[(_loc)++];                               \
     }                                                                       \
 }
 


More information about the parrot-commits mailing list