Skip to content

service/dap: Change event reason to "entry" when stopping on entry#1976

Merged
derekparker merged 1 commit intogo-delve:masterfrom
polinasok:stop-on-entry
Apr 1, 2020
Merged

service/dap: Change event reason to "entry" when stopping on entry#1976
derekparker merged 1 commit intogo-delve:masterfrom
polinasok:stop-on-entry

Conversation

@polinasok
Copy link
Copy Markdown
Collaborator

@polinasok polinasok commented Mar 31, 2020

Existing code matched vscode-go, setting reason to 'breakpoint', but that's misleading and according to the spec, there is a special option for entry. Use that option. This impacts what is displayed in the UI ('paused on breakpoint' vs 'paused on entry').
pause-on-entry
Updates #1515

@polinasok polinasok changed the title Change reason to "entry" when stopping on entry service/dap: Change reason to "entry" when stopping on entry Mar 31, 2020
@polinasok
Copy link
Copy Markdown
Collaborator Author

cc @eliben

@polinasok polinasok changed the title service/dap: Change reason to "entry" when stopping on entry service/dap: Change event reason to "entry" when stopping on entry Mar 31, 2020
Copy link
Copy Markdown
Member

@aarzilli aarzilli left a comment

Choose a reason for hiding this comment

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

lgtm

@derekparker derekparker merged commit c97c042 into go-delve:master Apr 1, 2020
@polinasok
Copy link
Copy Markdown
Collaborator Author

Matching PR to vscode-go was accepted as well: microsoft/vscode-go#3150.

@polinasok polinasok deleted the stop-on-entry branch July 2, 2020 21:44
RenovZ pushed a commit to RenovZ/delve that referenced this pull request Mar 25, 2022
…e#1976)

Co-authored-by: Polina Sokolova <polinasok@users.noreply.github.com>
abner-chenc pushed a commit to loongson/delve that referenced this pull request Mar 1, 2024
…e#1976)

Co-authored-by: Polina Sokolova <polinasok@users.noreply.github.com>
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.

3 participants