diff mbox series

[Openvpn-devel] Remove unused void_ptr_hash_function and void_ptr_compare_function

Message ID 1537374009-11133-1-git-send-email-steffan.karger@fox-it.com
State Accepted
Headers
Series [Openvpn-devel] Remove unused void_ptr_hash_function and void_ptr_compare_function |

Commit Message

Steffan Karger Sept. 19, 2018, 6:20 a.m. UTC 
  Both functions are never used, so let's get rid of them.

Signed-off-by: Steffan Karger <steffan.karger@fox-it.com>
---
 src/openvpn/list.c | 12 ------------
 src/openvpn/list.h |  4 ----
 2 files changed, 16 deletions(-)

Comments

Gert Doering Sept. 19, 2018, 10:42 a.m. UTC | #1 
Acked-by: Gert Doering <gert@greenie.muc.de>

Because it's dead code.  Looking again, it actually seems correct
after all :-) (void_ptr_hash_function() is passed in a pointer *value*,
which is then stored locally, and the address of this local store is
handed to hash_func() - so it compares "memory addresses".  Misread the
code before.  It's James code so it must be correct.  But unused still!

Your patch has been applied to the master branch.

commit 01f7bb52ce510f0fc915c9aa6a14f79b2779a7f8
Author: Steffan Karger
Date:   Wed Sep 19 18:20:09 2018 +0200

     Remove unused void_ptr_hash_function and void_ptr_compare_function

     Signed-off-by: Steffan Karger <steffan.karger@fox-it.com>
     Acked-by: Gert Doering <gert@greenie.muc.de>
     Message-Id: <1537374009-11133-1-git-send-email-steffan.karger@fox-it.com>
     URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg17473.html
     Signed-off-by: Gert Doering <gert@greenie.muc.de>


--
kind regards,

Gert Doering
diff mbox series

Patch

diff --git a/src/openvpn/list.c b/src/openvpn/list.c
index 09e393a..6023ea6 100644
--- a/src/openvpn/list.c
+++ b/src/openvpn/list.c
@@ -222,18 +222,6 @@  hash_remove_marked(struct hash *hash, struct hash_bucket *bucket)
     }
 }
 
-uint32_t
-void_ptr_hash_function(const void *key, uint32_t iv)
-{
-    return hash_func((const void *)&key, sizeof(key), iv);
-}
-
-bool
-void_ptr_compare_function(const void *key1, const void *key2)
-{
-    return key1 == key2;
-}
-
 void
 hash_iterator_init_range(struct hash *hash,
                          struct hash_iterator *hi,
diff --git a/src/openvpn/list.h b/src/openvpn/list.h
index b67301c..6177157 100644
--- a/src/openvpn/list.h
+++ b/src/openvpn/list.h
@@ -116,10 +116,6 @@  void hash_iterator_free(struct hash_iterator *hi);
 
 uint32_t hash_func(const uint8_t *k, uint32_t length, uint32_t initval);
 
-uint32_t void_ptr_hash_function(const void *key, uint32_t iv);
-
-bool void_ptr_compare_function(const void *key1, const void *key2);
-
 #ifdef LIST_TEST
 void list_test(void);