get debugger working when toggled from menu (fixes #282)

benfry · benfry · commit abb84e0a640a · 2021-10-04T21:57:33.000-04:00
diff --git a/java/src/processing/mode/java/debug/Debugger.java b/java/src/processing/mode/java/debug/Debugger.java
@@ -125,7 +125,7 @@ public void populateMenu(JMenu modeMenu) {
     debugItem = Toolkit.newJMenuItem(Language.text("menu.debug.enable"), 'D');
     debugItem.addActionListener(new ActionListener() {
       public void actionPerformed(ActionEvent e) {
-        toggleEnabled();
+        editor.toggleDebug();
       }
     });
     debugMenu.add(debugItem);
diff --git a/todo.txt b/todo.txt
@@ -19,6 +19,9 @@ X   https://github.com/processing/processing4/issues/254
 X   https://github.com/processing/processing4/issues/280 (duplicate)
 X Movie Maker broken with spaces in path (converted to %20)
 X   https://github.com/processing/processing4/issues/268
+X debugger not working
+X   https://github.com/processing/processing4/issues/282
+X   turned out to be a problem when using menu, not button to enable/disable
 
 contribs
 X fix minor typos in the PWM code for the IO library
@@ -46,8 +49,6 @@ _   probably related to second displays, need to hook one up and test
 _   https://github.com/processing/processing4/issues/194
 
 investigate
-_ debugger not working
-_   https://github.com/processing/processing4/issues/282
 _ Anti-aliasing/smoothing in createFont when disabled
 _   https://github.com/processing/processing4/issues/278
 X   couldn't reproduce, but probably an issue in there somewhere

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ public void populateMenu(JMenu modeMenu) {`
`125`	`125`	`debugItem = Toolkit.newJMenuItem(Language.text("menu.debug.enable"), 'D');`
`126`	`126`	`debugItem.addActionListener(new ActionListener() {`
`127`	`127`	`public void actionPerformed(ActionEvent e) {`
`128`		`- toggleEnabled();`
	`128`	`+ editor.toggleDebug();`
`129`	`129`	`}`
`130`	`130`	`});`
`131`	`131`	`debugMenu.add(debugItem);`