[前][次][番号順一覧][スレッド一覧]

ruby-changes:71012

From: Nobuyoshi <ko1@a...>
Date: Wed, 26 Jan 2022 05:57:49 +0900 (JST)
Subject: [ruby-changes:71012] 6e901939c6 (master): Revert "`ID` in `rb_id_table_foreach_with_replace` [Feature #18253]"

https://git.ruby-lang.org/ruby.git/commit/?id=6e901939c6

From 6e901939c67d7c1275c183b0bde4d5d1c78ef081 Mon Sep 17 00:00:00 2001
From: Nobuyoshi Nakada <nobu@r...>
Date: Wed, 26 Jan 2022 05:51:37 +0900
Subject: Revert "`ID` in `rb_id_table_foreach_with_replace` [Feature #18253]"

This reverts commit 530e485265dac6e2aea1d587a4a79a314cc772cf.
`rb_id_table_foreach_with_replace` is used during GC compaction,
and the global symbols array can have been moved at that time.
---
 id_table.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/id_table.c b/id_table.c
index 281a0fb50a7..b2ba6fae89e 100644
--- a/id_table.c
+++ b/id_table.c
@@ -274,14 +274,12 @@ rb_id_table_foreach_with_replace(struct rb_id_table *tbl, rb_id_table_foreach_fu https://github.com/ruby/ruby/blob/trunk/id_table.c#L274
 
     for (i=0; i<capa; i++) {
         if (ITEM_KEY_ISSET(tbl, i)) {
-            const id_key_t key = ITEM_GET_KEY(tbl, i);
-            ID id = key2id(key);
-            enum rb_id_table_iterator_result ret = (*func)(id, tbl->items[i].val, data);
-            assert(key != 0);
+            enum rb_id_table_iterator_result ret = (*func)((ID)0, tbl->items[i].val, data);
+            assert(ITEM_GET_KEY(tbl, i));
 
             if (ret == ID_TABLE_REPLACE) {
                 VALUE val = tbl->items[i].val;
-                ret = (*replace)(&id, &val, data, TRUE);
+                ret = (*replace)(NULL, &val, data, TRUE);
                 tbl->items[i].val = val;
             }
             else if (ret == ID_TABLE_STOP)
-- 
cgit v1.2.1


--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]