hash: silence spurious clang warning

* lib/hash.c (hash_get_next): Remove unnecessary test against NULL.
Reported by Eric Blake.

diff --git a/ChangeLog b/ChangeLog
index a7b5e681da9caa896249a9f5327fe71fdf5adab2..a61bf9f4dd066a1ea5479be156c70533b46ca9da 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2010-08-30  Bruno Haible  <bruno@clisp.org>
+
+       hash: silence spurious clang warning
+       * lib/hash.c (hash_get_next): Remove unnecessary test against NULL.
+       Reported by Eric Blake.
+
 2010-08-30  Eric Blake  <eblake@redhat.com>
 
        strstr, memmem, strcasestr: avoid leaked shell message

diff --git a/lib/hash.c b/lib/hash.c
index 15630be21ed8569525fe17bd04e93457d960fd73..225865208883a5c9ad4aed524881edaf277459d7 100644 (file)
--- a/lib/hash.c
+++ b/lib/hash.c
@@ -307,9 +307,14 @@ hash_get_next (const Hash_table *table, const void *entry)
     abort ();
 
   /* Find next entry in the same bucket.  */
-  for (cursor = bucket; cursor; cursor = cursor->next)
-    if (cursor->data == entry && cursor->next)
-      return cursor->next->data;
+  cursor = bucket;
+  do
+    {
+      if (cursor->data == entry && cursor->next)
+        return cursor->next->data;
+      cursor = cursor->next;
+    }
+  while (cursor != NULL);
 
   /* Find first entry in any subsequent bucket.  */
   while (++bucket < table->bucket_limit)