Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fiber.yield is not allowed on transferred fiber #410

Merged
merged 1 commit into from
Oct 8, 2020
Merged

Fiber.yield is not allowed on transferred fiber #410

merged 1 commit into from
Oct 8, 2020

Conversation

ko1
Copy link
Contributor

@ko1 ko1 commented Oct 8, 2020

https://bugs.ruby-lang.org/issues/17221 will change the behavior of Fiber#transfer,
so that Fiber#transfer's usage should be changed.

(1) Transferred fiber can not Fiber.yield
(2) Fiber.yield should be used with Fiber#resume

@ko1
Fiber.yield is not allowed on transferred fiber
0f9a6b3 
https://bugs.ruby-lang.org/issues/17221 will change the behavior of Fiber#transfer,
so that `Fiber#transfer`'s usage should be changed.

(1) Transferred fiber can not `Fiber.yield` 
(2) `Fiber.yield` should be used with `Fiber#resume`
@soutaro soutaro merged commit aa488d9 into ruby:master Oct 8, 2020
@soutaro soutaro mentioned this pull request Oct 8, 2020
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ko1 @soutaro