diff options
author | Yusuke Endoh <mame@ruby-lang.org> | 2022-02-09 18:24:17 +0900 |
---|---|---|
committer | Yusuke Endoh <mame@ruby-lang.org> | 2022-02-10 00:14:27 +0900 |
commit | 496591de96b261b8789332c7f8b2bfbd17658955 (patch) | |
tree | 4bfeafcfd1086faa26c1cb2893790cebec404328 /st.c | |
parent | 8013250136b61e0ae5a8d27a4ec73936cd7647eb (diff) |
st.c: Do not clear entries_bound when calling Hash#shift for empty hash
tab->entries_bound is used to check if the bins are full in
rebuild_table_if_necessary.
Hash#shift against an empty hash assigned 0 to tab->entries_bound, but
didn't clear the bins. Thus, the table is not rebuilt even when the bins
are full. Attempting to add a new element into full-bin hash gets stuck.
This change stops clearing tab->entries_bound in Hash#shift.
[Bug #18578]
Notes
Notes:
Merged: https://github.com/ruby/ruby/pull/5539
Diffstat (limited to 'st.c')
-rw-r--r-- | st.c | 1 |
1 files changed, 0 insertions, 1 deletions
@@ -1363,7 +1363,6 @@ st_shift(st_table *tab, st_data_t *key, st_data_t *value) return 1; } } - tab->entries_start = tab->entries_bound = 0; if (value != 0) *value = 0; return 0; } |