Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Change Control.block () to raise Sys_error when forced to return #188

Merged
merged 1 commit into from
Jul 29, 2024

Conversation

polytypic
Copy link
Collaborator

@polytypic polytypic commented Jul 22, 2024

It is technically possible to finish the computation of a fiber from outside of the fiber when using the low level Picos interface so it is not strictly impossible for the Trigger.await inside Control.block () to return None.

@polytypic polytypic force-pushed the raise-sys_error-in-case-block-is-forced-to-return branch 11 times, most recently from 721479e to bc0a57c Compare July 29, 2024 15:20
It is technically possible to finish the computation of a fiber from outside of
the fiber when using the low level Picos interface so it is not strictly
impossible for the `Trigger.await` inside `Control.block ()` to return `None`.
@polytypic polytypic force-pushed the raise-sys_error-in-case-block-is-forced-to-return branch from bc0a57c to 72726a0 Compare July 29, 2024 16:00
@polytypic polytypic merged commit a255a1b into main Jul 29, 2024
6 checks passed
@polytypic polytypic deleted the raise-sys_error-in-case-block-is-forced-to-return branch July 29, 2024 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant