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

Debugger can't handle map comprehensions #7914

Closed
ecpeterson opened this issue Nov 29, 2023 · 1 comment · Fixed by #7923
Closed

Debugger can't handle map comprehensions #7914

ecpeterson opened this issue Nov 29, 2023 · 1 comment · Fixed by #7923
Assignees
Labels
bug Issue is reported as a bug team:VM Assigned to OTP team VM

Comments

@ecpeterson
Copy link
Contributor

Describe the bug
The Debugger crashes when loading a file with a map comprehension. Map comprehensions show up as {mc, ...} tuples, for which there's no entry in this case expression.

To Reproduce
Compile, load, and inspect a module with a map comprehension in it.

Expected behavior
I expected the Debugger to at least load the file, even if its ability to manipulate it was diminished.

Affected versions
I checked this only with OTP 26.0. I also looked at the main branch in the source tree, where the bug appears to still be present.

@ecpeterson ecpeterson added the bug Issue is reported as a bug label Nov 29, 2023
@bjorng bjorng self-assigned this Nov 30, 2023
@bjorng bjorng added the team:VM Assigned to OTP team VM label Nov 30, 2023
@bjorng bjorng linked a pull request Dec 1, 2023 that will close this issue
@bjorng
Copy link
Contributor

bjorng commented Dec 1, 2023

Thanks! A fix will be included in OTP 26.2.

bjorng added a commit that referenced this issue Dec 4, 2023
Implement map comprehensions in the debugger
@bjorng bjorng closed this as completed in 26266f7 Dec 4, 2023
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is reported as a bug team:VM Assigned to OTP team VM
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants