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

Re-send breakpoints when source file has been changed #92

Merged
merged 8 commits into from
Jan 19, 2025

Conversation

RemcoSmitsDev
Copy link
Owner

@RemcoSmitsDev RemcoSmitsDev commented Jan 18, 2025

This PR should fix an issue, when you modify a file while you are debugging that the debugger not always sends the right row back for the current stack frame. So by re-sending the breakpoints when you saved the file, we should get the right row back for the current stack frame, so we also show the right current debug line after you modified the file.

TODO:

  • Add tests

@RemcoSmitsDev RemcoSmitsDev force-pushed the resend-breakpoints-when-source-is-modified branch from 68f128f to e532fea Compare January 18, 2025 21:34
We send the breakpoints twice, because we already send them when the FileHandleChanged event was received and that is received after the Saved event itself. And the Saved event does not guarantee that the source is already changed on disk. Which it has to be so we can send the new breakpoints.
@RemcoSmitsDev RemcoSmitsDev changed the title Resend breakpoints when the editor has been saved Re-send breakpoints when source file has been changed Jan 19, 2025
@RemcoSmitsDev RemcoSmitsDev marked this pull request as ready for review January 19, 2025 11:57
@RemcoSmitsDev RemcoSmitsDev merged commit a193efd into debugger Jan 19, 2025
11 checks passed
@RemcoSmitsDev RemcoSmitsDev deleted the resend-breakpoints-when-source-is-modified branch January 19, 2025 11:58
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