summaryrefslogtreecommitdiff
path: root/weakmap.c
diff options
context:
space:
mode:
authorYusuke Endoh <mame@ruby-lang.org>2024-12-04 16:33:49 +0900
committerYusuke Endoh <mame@ruby-lang.org>2024-12-04 17:12:26 +0900
commitd2c55e33fb4cf22859ef5c70fe532342d7bb2c62 (patch)
tree0f0a7d8db70a149ed48cd1e22bba73d2e3cd0482 /weakmap.c
parent3b27818062f5bd4723a3b76f7fb7e963f4dd1f49 (diff)
Remove meaningless NULL checks
The functions dereference `*dead_entry` without a NULL check
Notes
Notes: Merged: https://github.com/ruby/ruby/pull/12258
Diffstat (limited to 'weakmap.c')
-rw-r--r--weakmap.c12
1 files changed, 4 insertions, 8 deletions
diff --git a/weakmap.c b/weakmap.c
index 32acf2d6ec..f8a6ae3747 100644
--- a/weakmap.c
+++ b/weakmap.c
@@ -636,10 +636,8 @@ static int
wkmap_mark_table_i(st_data_t key, st_data_t val_obj, st_data_t data)
{
VALUE **dead_entry = (VALUE **)data;
- if (dead_entry != NULL) {
- ruby_sized_xfree(*dead_entry, sizeof(VALUE));
- *dead_entry = NULL;
- }
+ ruby_sized_xfree(*dead_entry, sizeof(VALUE));
+ *dead_entry = NULL;
VALUE *key_ptr = (VALUE *)key;
@@ -702,10 +700,8 @@ static int
wkmap_compact_table_i(st_data_t key, st_data_t val_obj, st_data_t data, int _error)
{
VALUE **dead_entry = (VALUE **)data;
- if (dead_entry != NULL) {
- ruby_sized_xfree(*dead_entry, sizeof(VALUE));
- *dead_entry = NULL;
- }
+ ruby_sized_xfree(*dead_entry, sizeof(VALUE));
+ *dead_entry = NULL;
VALUE *key_ptr = (VALUE *)key;