-
Notifications
You must be signed in to change notification settings - Fork 182
Add suspend mode for source breakpoints #610
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
base: main
Are you sure you want to change the base?
Conversation
|
@tsmaeder I wrote a simple program to test it, but got error: public class HelloWorld {
public static void main(String[] args) {
System.out.println("Starting application...");
String message = "Hello, World!";
System.out.println(message); // Set a breakpoint here
for (int i = 0; i < 3; i++) {
System.out.println("Count: " + i);
}
System.out.println("Application finished.");
}
}Error: Also please fix CI errors. |
|
@chagong how are you testing? I have a very similar program which works fine for me. Also, this PR does not contain changes that could cause a problem with the delegate command handler, IMO. Here's what I do:
|
Signed-off-by: Thomas Mäder <[email protected]>
Signed-off-by: Thomas Mäder <[email protected]>
d6f08d7 to
b3e14c0
Compare
|
@tsmaeder probably the problem of my setup, it works after I relaunch vscode and F5. The PR is good to go after your fix the format errors. |
Signed-off-by: Thomas Mäder <[email protected]>
Signed-off-by: Thomas Mäder <[email protected]>
|
@chagong running checkstyle:check locally spews out messages about |
Signed-off-by: Thomas Mäder <[email protected]>

Fixes #608
Note that this PR contains code for handling source breakpoints and exception breakpoints. But in fact, VS Code Java only properly handles source and instruction breakpoints. I've left the code in there, since it's not completely trivial and it should not affect the behavior if no suspend mode is sent when setting the exception breakpoints.
How to Test