Make it a function
authorDale Weiler <weilercdale@gmail.com>
Sat, 27 Sep 2014 08:50:50 +0000 (04:50 -0400)
committerDale Weiler <weilercdale@gmail.com>
Sat, 27 Sep 2014 08:50:50 +0000 (04:50 -0400)
hash.c

diff --git a/hash.c b/hash.c
index e136d10..41d77bf 100644 (file)
--- a/hash.c
+++ b/hash.c
@@ -301,14 +301,16 @@ static GMQCC_FORCEINLINE uint32_t hash_murmur(const void *GMQCC_RESTRICT key, si
 #define STRLEN_ONES ((size_t)-1/UCHAR_MAX)
 #define STRLEN_HIGHS (STRLEN_ONES * (UCHAR_MAX/2+1))
 #define STRLEN_HASZERO(X) (((X)-STRLEN_ONES) & ~(X) & STRLEN_HIGHS)
-ASAN_DISABLE size_t hash(const char *key) {
+
+static ASAN_DISABLE size_t hash_strlen(const char *key) {
     const char *s = key;
     const char *a = s;
     const size_t *w;
 
     for (; (uintptr_t)s % STRLEN_ALIGN; s++) {
-        if (!*s)
-            return hash_murmur((const void *)key, s-a);
+        if (*s)
+            continue;
+        return s-a;
     }
 
     VALGRIND_MAKE_MEM_DEFINED(s, STRLEN_ALIGN);
@@ -321,12 +323,14 @@ ASAN_DISABLE size_t hash(const char *key) {
 
     /* It's not legal to access this area anymore */
     VALGRIND_MAKE_MEM_NOACCESS(s + 1, STRLEN_ALIGN);
-
-    return hash_murmur((const void *)key, s-a);
+    return s-a;
 }
 #else
-/* No way to disable asan instrumenting, use strlen. */
+static GMQCC_INLINE size_t hash_strlen(const char *key) {
+    return strlen(key);
+}
+#endif
+
 size_t hash(const char *key) {
-    return hash_murmur((const void *)key, strlen(key));
+    return hash_murmur((const void *)key, hash_strlen(key));
 }
-#endif /*! HAS_ASAN_DISABLE */