diff options
author | Tom Tromey <tom@tromey.com> | 2021-12-22 10:30:16 -0700 |
---|---|---|
committer | Tom Tromey <tom@tromey.com> | 2021-12-29 10:24:05 -0700 |
commit | ba0084966cfee307631aba6d0f052a39ad773909 (patch) | |
tree | 63012fc81a9af3914b8f5368803dc52a76383b8f /gdb/utils.c | |
parent | 003aae076207dbf32f98ba846158fc32669ef85f (diff) | |
download | binutils-ba0084966cfee307631aba6d0f052a39ad773909.zip binutils-ba0084966cfee307631aba6d0f052a39ad773909.tar.gz binutils-ba0084966cfee307631aba6d0f052a39ad773909.tar.bz2 |
Remove unusual use of core_addr_eq and core_addr_hash
gdbtypes.h uses core_addr_eq and core_addr_hash in a weird way: taking
the address of a member and then passing this (as a void*) to these
functions.
It seems better to simply inline the ordinary code here. CORE_ADDR is
a scalar so it can be directly compared, and the identity hash
function seems safe to assume as well.
After this, core_addr_eq and core_addr_hash are unused, so this patch
removes them.
Diffstat (limited to 'gdb/utils.c')
-rw-r--r-- | gdb/utils.c | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/gdb/utils.c b/gdb/utils.c index 620ae9f..eba1acb 100644 --- a/gdb/utils.c +++ b/gdb/utils.c @@ -2837,27 +2837,6 @@ print_core_address (struct gdbarch *gdbarch, CORE_ADDR address) return hex_string_custom (address, 16); } -/* Callback hash_f for htab_create_alloc or htab_create_alloc_ex. */ - -hashval_t -core_addr_hash (const void *ap) -{ - const CORE_ADDR *addrp = (const CORE_ADDR *) ap; - - return *addrp; -} - -/* Callback eq_f for htab_create_alloc or htab_create_alloc_ex. */ - -int -core_addr_eq (const void *ap, const void *bp) -{ - const CORE_ADDR *addr_ap = (const CORE_ADDR *) ap; - const CORE_ADDR *addr_bp = (const CORE_ADDR *) bp; - - return *addr_ap == *addr_bp; -} - /* Convert a string back into a CORE_ADDR. */ CORE_ADDR string_to_core_addr (const char *my_string) |