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

Feature/pass args to debugger #1665

Merged
merged 1 commit into from
Nov 19, 2024
Merged

Conversation

wycats
Copy link
Contributor

@wycats wycats commented Nov 19, 2024

This commit eliminates debugger infrastructure that derived from the old partial/eval infrastructure.

Instead, it passes the in-scope values as a constant to the debugger wire format opcode and uses it directly in the scope inspector.

This will be followed up with support for accessing strict-mode lexical variables.

This commit eliminates debugger infrastructure that derived from the old
partial/eval infrastructure.

Instead, it passes the in-scope values as a constant to the `debugger`
wire format opcode and uses it directly in the scope inspector.

This will be followed up with support for accessing strict-mode lexical
variables.
@wycats wycats force-pushed the feature/pass-args-to-debugger branch from 721888c to 99d0232 Compare November 19, 2024 21:46
@NullVoxPopuli
Copy link
Contributor

local perf test shows no difference compared to main 🎉

Copy link
Contributor

@NullVoxPopuli NullVoxPopuli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dang, this is way cleaner!

@NullVoxPopuli NullVoxPopuli merged commit 589945a into main Nov 19, 2024
5 checks passed
@NullVoxPopuli NullVoxPopuli deleted the feature/pass-args-to-debugger branch November 19, 2024 22:59
@github-actions github-actions bot mentioned this pull request Nov 19, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants