diff options
| author | Yusuke Endoh <mame@ruby-lang.org> | 2024-12-04 16:33:49 +0900 |
|---|---|---|
| committer | Yusuke Endoh <mame@ruby-lang.org> | 2024-12-04 17:12:26 +0900 |
| commit | d2c55e33fb4cf22859ef5c70fe532342d7bb2c62 (patch) | |
| tree | 0f0a7d8db70a149ed48cd1e22bba73d2e3cd0482 /weakmap.c | |
| parent | 3b27818062f5bd4723a3b76f7fb7e963f4dd1f49 (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.c | 12 |
1 files changed, 4 insertions, 8 deletions
@@ -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; |
