fix index

`i` should not be used here because `i` will be `alen` when jumped here by "goto restart".
ruby · Dec 4, 2020 · 82bbce8 · 82bbce8
1 parent f1757a8
commit 82bbce8
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/ractor.c b/ractor.c
@@ -802,6 +802,7 @@ ractor_select(rb_execution_context_t *ec, const VALUE *rs, int alen, VALUE yield
     bool interrupted = false;
     enum ractor_wait_status wait_status = 0;
     bool yield_p = (yielded_value != Qundef) ? true : false;
+    const int rs_len = alen;
 
     struct ractor_select_action {
         enum ractor_select_action_type {
@@ -840,8 +841,8 @@ ractor_select(rb_execution_context_t *ec, const VALUE *rs, int alen, VALUE yield
   restart:
 
     if (yield_p) {
-        actions[i].type = ractor_select_action_yield;
-        actions[i].v = Qundef;
+        actions[rs_len].type = ractor_select_action_yield;
+        actions[rs_len].v = Qundef;
         wait_status |= wait_yielding;
         alen++;