Skip to content

Add watchpoint support #710

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

Merged
merged 1 commit into from
Jul 8, 2021

Conversation

isc-bsaviano
Copy link
Contributor

This PR fixes #697

@isc-bsaviano isc-bsaviano merged commit 75f3ea4 into intersystems-community:master Jul 8, 2021
@isc-solon
Copy link
Collaborator

I remember this being completed, but I don't see a way in VS Code to actually declare a watchpoint, and this capability is not documented. Adding @rajrsingh

@gjsjohnmurray
Copy link
Contributor

@isc-solon does the information at https://code.visualstudio.com/Docs/editor/debugging#_data-breakpoints get you anywhere?

@isc-solon
Copy link
Collaborator

Thanks, John. I didn't think to check the code doc itself. Very clear. But...my context menu only shows "Add to Watch" and none of the "Break on" options.

@gjsjohnmurray
Copy link
Contributor

Here's what I get:

image

And after I select it a new entry appears in the BREAKPOINTS section:

image

Then after I continue (F5) here's where execution next stops:

image

@isc-solon
Copy link
Collaborator

isc-solon commented Oct 13, 2023

Fascinating! Whether I'm in a class method or a .Mac routine, and whether or not I right-click on the variable in the Variables section or the Watch section, I don't see "Break On Value Change".

I'm on the latest versions of VS Code and the extensions, and I'm running v2023.2 of IRIS for Health in a container. I don't have that "proposed APIs" thing enabled.

@isc-rsingh
Copy link
Member

isc-rsingh commented Oct 13, 2023

Using VS Code "regular" edition (v1.82.2) I don't have a "Break on value change" menu item either, but using VS Code Insiders (v1.84.0-insider) with proposed APIs I do see that option.

VS Code "regular":
image

Insiders:
image

@isc-solon
Copy link
Collaborator

Do "proposed APIs" ever become "standard APIs?"

@isc-rsingh
Copy link
Member

We're hoping so...

@gjsjohnmurray
Copy link
Contributor

What you're seeing is unrelated to proposed API. Please see #1252 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support watchpoints in debugger
4 participants