Skip to content

Fixup runtime#331

Merged
saghul merged 1 commit intomasterfrom
fixup-runtime
Nov 29, 2022
Merged

Fixup runtime#331
saghul merged 1 commit intomasterfrom
fixup-runtime

Conversation

@saghul
Copy link
Copy Markdown
Owner

@saghul saghul commented Nov 29, 2022

No description provided.

Now that the startup logic is in JS, we JS_Eval that file as part of
TJS_Run.

That file, in turn, may evaluate the given file as a module, which
results on abother JS_Eval before we are done with the first.

This seems to corrupt the vm state.

The solution here was to queue a microtask, thus breaking away from the
original problem. A hack to be sure...
@saghul saghul merged commit f3c8445 into master Nov 29, 2022
@saghul saghul deleted the fixup-runtime branch November 29, 2022 21:50
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