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

eval command: Escape HTML on the evaluated value. #212

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jvilk-stripe
Copy link

Many things that evaluates to a string do not work properly without this escape. For example:

  • ""
  • " "
  • [].join (which produces " ")

Fixes a bug where the debug console in VS Code can hang the debugger when inserting one of those types of input.

I could not find tests for this routine. If I missed any, let me know!

Many things that evaluates to a string do not work properly without this escape. For example:

* `""`
* `" "`
* `[].join` (which produces `" "`)

Fixes a bug where the debug console in VS Code can hang the debugger when inserting one of those types of input.
@jvilk-stripe
Copy link
Author

cc @hurricup

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