From 19a1c2ba47932ee3bb1512cef732b05fad24b65a Mon Sep 17 00:00:00 2001 From: LoRd_MuldeR Date: Thu, 1 Dec 2022 16:53:36 +0100 Subject: [PATCH] Small code clean-up. --- libhashset/src/generic_hash_map.h | 30 ++++++------------------------ libhashset/src/generic_hash_set.h | 25 +++++-------------------- 2 files changed, 11 insertions(+), 44 deletions(-) diff --git a/libhashset/src/generic_hash_map.h b/libhashset/src/generic_hash_map.h index 5d8342b..240ee99 100644 --- a/libhashset/src/generic_hash_map.h +++ b/libhashset/src/generic_hash_map.h @@ -378,14 +378,8 @@ errno_t DECLARE(hash_map_iterate)(const hash_map_t *const instance, size_t *cons { if (IS_VALID(instance->data, index)) { - if (key) - { - *key = instance->data.keys[index]; - } - if (value) - { - *value = instance->data.values[index]; - } + SAFE_SET(key, instance->data.keys[index]); + SAFE_SET(value, instance->data.values[index]); *cursor = index + 1U; return 0; } @@ -407,22 +401,10 @@ errno_t DECLARE(hash_map_info)(const hash_map_t *const instance, size_t *const c return EINVAL; } - if (capacity) - { - *capacity = instance->data.capacity; - } - if (valid) - { - *valid = instance->valid; - } - if (deleted) - { - *deleted = instance->deleted; - } - if (limit) - { - *limit = instance->limit; - } + SAFE_SET(capacity, instance->data.capacity); + SAFE_SET(valid, instance->valid); + SAFE_SET(deleted,instance->deleted); + SAFE_SET(limit, instance->limit); return 0; } diff --git a/libhashset/src/generic_hash_set.h b/libhashset/src/generic_hash_set.h index 00c68ff..0dc566c 100644 --- a/libhashset/src/generic_hash_set.h +++ b/libhashset/src/generic_hash_set.h @@ -348,10 +348,7 @@ errno_t DECLARE(hash_set_iterate)(const hash_set_t *const instance, size_t *cons { if (IS_VALID(instance->data, index)) { - if (item) - { - *item = instance->data.items[index]; - } + SAFE_SET(item, instance->data.items[index]); *cursor = index + 1U; return 0; } @@ -373,22 +370,10 @@ errno_t DECLARE(hash_set_info)(const hash_set_t *const instance, size_t *const c return EINVAL; } - if (capacity) - { - *capacity = instance->data.capacity; - } - if (valid) - { - *valid = instance->valid; - } - if (deleted) - { - *deleted = instance->deleted; - } - if (limit) - { - *limit = instance->limit; - } + SAFE_SET(capacity, instance->data.capacity); + SAFE_SET(valid, instance->valid); + SAFE_SET(deleted, instance->deleted); + SAFE_SET(limit, instance->limit); return 0; }