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

Partly revert d53a197 & 36fc539 to workaround #233 #238

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

shhyou
Copy link
Collaborator

@shhyou shhyou commented Dec 16, 2024

A more appropriate fix should be applied in the future.

To workaround #233, this partly reverts commits

  • "highlight universe when set-up fails at run-time". d53a197
  • "fix resource administration in run and launch-many-worlds". 36fc539

Otherwise, when the object [((new-world (if #,rec? aworld% world%)) w #,@args)] and [((new-universe universe%) u #,@args)] are created in the main thread, the continuation marks of the stepper are somehow broken.

The two follow up commits forward the check-with exceptions from (o) in esp/thd to the main thread and repackages it with big-bang's stack frames.

To workaround racket#233, this partly reverts commits

- "highlight universe when set-up fails at run-time".
   d53a197
- "fix resource administration in run and launch-many-worlds".
   36fc539

Otherwise, when the object `[((new-world (if #,rec? aworld% world%)) w #,@Args)]`
and `[((new-universe universe%) u #,@Args)]` are created in the main thread, the
continuation marks of the stepper are somehow broken.

A more appropriate fix should be applied in the future.
@shhyou shhyou added stepper teachpack [2]htdp teachpack labels Dec 16, 2024
@shhyou shhyou marked this pull request as draft December 28, 2024 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stepper teachpack [2]htdp teachpack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant