summaryrefslogtreecommitdiff
path: root/ractor.c
diff options
context:
space:
mode:
authorNobuyoshi Nakada <nobu@ruby-lang.org>2022-04-07 19:19:13 +0900
committerNobuyoshi Nakada <nobu@ruby-lang.org>2022-04-07 19:19:13 +0900
commit7b1ece9b9490a892861f2336ae41d611a06bdf2b (patch)
tree8d839a3fc151393bef5bfed1ee9a37bf465430b4 /ractor.c
parent9e58c3c5195f0745ba7f125a3478046c5eb3acd7 (diff)
Get rid of type-punning pointer casts
Diffstat (limited to 'ractor.c')
-rw-r--r--ractor.c24
1 files changed, 12 insertions, 12 deletions
diff --git a/ractor.c b/ractor.c
index 0c17d575ee..96c51eca70 100644
--- a/ractor.c
+++ b/ractor.c
@@ -2706,7 +2706,7 @@ obj_refer_only_shareables_p(VALUE obj)
static int
obj_traverse_replace_i(VALUE obj, struct obj_traverse_replace_data *data)
{
- VALUE replacement;
+ st_data_t replacement;
if (RB_SPECIAL_CONST_P(obj)) {
data->replacement = obj;
@@ -2719,14 +2719,14 @@ obj_traverse_replace_i(VALUE obj, struct obj_traverse_replace_data *data)
case traverse_stop: return 1; // stop search
}
- replacement = data->replacement;
+ replacement = (st_data_t)data->replacement;
- if (UNLIKELY(st_lookup(obj_traverse_replace_rec(data), (st_data_t)obj, (st_data_t *)&replacement))) {
- data->replacement = replacement;
+ if (UNLIKELY(st_lookup(obj_traverse_replace_rec(data), (st_data_t)obj, &replacement))) {
+ data->replacement = (VALUE)replacement;
return 0;
}
else {
- st_insert(obj_traverse_replace_rec(data), (st_data_t)obj, (st_data_t)replacement);
+ st_insert(obj_traverse_replace_rec(data), (st_data_t)obj, replacement);
}
if (!data->move) {
@@ -2872,7 +2872,7 @@ obj_traverse_replace_i(VALUE obj, struct obj_traverse_replace_data *data)
rb_bug("unreachable");
}
- data->replacement = replacement;
+ data->replacement = (VALUE)replacement;
if (data->leave_func(obj, data) == traverse_stop) {
return 1;
@@ -3052,9 +3052,9 @@ ractor_local_storage_mark(rb_ractor_t *r)
for (int i=0; i<freed_ractor_local_keys.cnt; i++) {
rb_ractor_local_key_t key = freed_ractor_local_keys.keys[i];
- st_data_t val;
- if (st_delete(r->local_storage, (st_data_t *)&key, &val) &&
- key->type->free) {
+ st_data_t val, k = (st_data_t)key;
+ if (st_delete(r->local_storage, &k, &val) &&
+ (key = (rb_ractor_local_key_t)k)->type->free) {
(*key->type->free)((void *)val);
}
}
@@ -3179,9 +3179,9 @@ ractor_local_set(rb_ractor_local_key_t key, void *ptr)
VALUE
rb_ractor_local_storage_value(rb_ractor_local_key_t key)
{
- VALUE val;
- if (ractor_local_ref(key, (void **)&val)) {
- return val;
+ void *val;
+ if (ractor_local_ref(key, &val)) {
+ return (VALUE)val;
}
else {
return Qnil;