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 is not aware of process exits #454

Closed
lukaszsamson opened this issue Jan 8, 2021 · 0 comments · Fixed by #457
Closed

Debugger is not aware of process exits #454

lukaszsamson opened this issue Jan 8, 2021 · 0 comments · Fixed by #457
Labels
bug Something isn't working

Comments

@lukaszsamson
Copy link
Collaborator

Environment

  • Editor Plugin/LSP Client name: 0.6.2

Description

Debugger is not aware when a process that is stopped on a breakpoint is being terminated. Attempting to continue, etc results in a crash.

@lukaszsamson lukaszsamson added the bug Something isn't working label Jan 8, 2021
lukaszsamson added a commit that referenced this issue Jan 10, 2021
add test for mix task exit
Fixes #454
lukaszsamson added a commit that referenced this issue Jan 10, 2021
add test for mix task exit
Fixes #454
axelson pushed a commit that referenced this issue Jan 14, 2021
* remove legacy io_request handlers

we don't support OTP < R15B

* rescue MatchError in :int calls

Fixes #455

* make output device better conform to erlang I/O protocol

see https://erlang.org/doc/apps/stdlib/io_protocol.html for details

* return WireProtocol.send error to the caller

no need to IO.warn if write fails

* we are redirection stderr to stdout, use stdout as underlying device

* inspect error

* monitor debugged processes

add test for mix task exit
Fixes #454

* avoid debugger crashes when handling requests for no longer existing thread, frame and variable ids

Fixes #452

* add test

* forbid changes of underlying device opts

* refactor and add tests coverage to invalid requests

* Map.pop! is available since elixir 1.10

* run formatter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant