FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

don't keep menu items around that aren't used outside the constructor… · sarvex/processing@70819be · GitHub

  1. sarvex
  2. processing

Repository navigation

Commit 70819be

Browse files
benfry
committed
don't keep menu items around that aren't used outside the constructor scope
1 parent f1432be commit 70819be

1 file changed

Lines changed: 60 additions & 60 deletions

File tree

‎java/src/processing/mode/java/JavaEditor.java‎

Lines changed: 60 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -63,23 +63,23 @@ public class JavaEditor extends Editor {
6363
protected JMenu debugMenu; // the debug menu
6464

6565
// protected JMenuItem debugMenuItem;
66-
protected JMenuItem continueMenuItem;
67-
protected JMenuItem stopMenuItem;
66+
// protected JMenuItem continueMenuItem;
67+
// protected JMenuItem stopMenuItem;
6868

6969
// protected JMenuItem toggleBreakpointMenuItem;
70-
protected JMenuItem listBreakpointsMenuItem;
70+
// protected JMenuItem listBreakpointsMenuItem;
7171

72-
protected JMenuItem stepOverMenuItem;
73-
protected JMenuItem stepIntoMenuItem;
74-
protected JMenuItem stepOutMenuItem;
72+
// protected JMenuItem stepOverMenuItem;
73+
// protected JMenuItem stepIntoMenuItem;
74+
// protected JMenuItem stepOutMenuItem;
7575

76-
protected JMenuItem printStackTraceMenuItem;
77-
protected JMenuItem printLocalsMenuItem;
78-
protected JMenuItem printThisMenuItem;
79-
protected JMenuItem printSourceMenuItem;
80-
protected JMenuItem printThreads;
76+
// protected JMenuItem printStackTraceMenuItem;
77+
// protected JMenuItem printLocalsMenuItem;
78+
// protected JMenuItem printThisMenuItem;
79+
// protected JMenuItem printSourceMenuItem;
80+
// protected JMenuItem printThreads;
8181

82-
protected JMenuItem toggleVariableInspectorMenuItem;
82+
// protected JMenuItem toggleVariableInspectorMenuItem;
8383

8484
protected Debugger debugger; // the debugger
8585
protected DebugTray tray; // the variable inspector frame
@@ -95,12 +95,13 @@ public class JavaEditor extends Editor {
9595

9696
public boolean compilationCheckEnabled = true;
9797

98-
protected JCheckBoxMenuItem showWarnings;
98+
// protected JCheckBoxMenuItem showWarnings;
9999
public JCheckBoxMenuItem problemWindowMenuCB;
100-
protected JCheckBoxMenuItem debugMessagesEnabled;
101-
protected JMenuItem showOutline, showTabOutline;
102-
protected JCheckBoxMenuItem writeErrorLog;
103-
protected JCheckBoxMenuItem completionsEnabled;
100+
// protected JCheckBoxMenuItem debugMessagesEnabled;
101+
// protected JMenuItem showOutline;
102+
// protected JMenuItem showTabOutline;
103+
// protected JCheckBoxMenuItem writeErrorLog;
104+
// protected JCheckBoxMenuItem completionsEnabled;
104105

105106
// TODO no way should this be public; make an accessor or protected
106107
public boolean hasJavaTabs;
@@ -1434,23 +1435,23 @@ public void actionPerformed(ActionEvent e) {
14341435
});
14351436
debugMenu.add(item);
14361437

1437-
continueMenuItem = Toolkit.newJMenuItem(Language.text("menu.debug.continue"), KeyEvent.VK_U);
1438-
continueMenuItem.addActionListener(new ActionListener() {
1438+
item = Toolkit.newJMenuItem(Language.text("menu.debug.continue"), KeyEvent.VK_U);
1439+
item.addActionListener(new ActionListener() {
14391440
public void actionPerformed(ActionEvent e) {
14401441
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Continue' menu item");
14411442
debugger.continueDebug();
14421443
}
14431444
});
1444-
debugMenu.add(continueMenuItem);
1445+
debugMenu.add(item);
14451446

1446-
stopMenuItem = new JMenuItem(Language.text("menu.debug.stop"));
1447-
stopMenuItem.addActionListener(new ActionListener() {
1447+
item = new JMenuItem(Language.text("menu.debug.stop"));
1448+
item.addActionListener(new ActionListener() {
14481449
public void actionPerformed(ActionEvent e) {
14491450
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Stop' menu item");
14501451
debugger.stopDebug();
14511452
}
14521453
});
1453-
debugMenu.add(stopMenuItem);
1454+
debugMenu.add(item);
14541455

14551456
debugMenu.addSeparator();
14561457

@@ -1464,120 +1465,119 @@ public void actionPerformed(ActionEvent e) {
14641465
});
14651466
debugMenu.add(item);
14661467

1467-
listBreakpointsMenuItem = new JMenuItem(Language.text("menu.debug.list_breakpoints"));
1468-
listBreakpointsMenuItem.addActionListener(new ActionListener() {
1468+
item = new JMenuItem(Language.text("menu.debug.list_breakpoints"));
1469+
item.addActionListener(new ActionListener() {
14691470
public void actionPerformed(ActionEvent e) {
14701471
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'List Breakpoints' menu item");
14711472
debugger.listBreakpoints();
14721473
}
14731474
});
1474-
debugMenu.add(listBreakpointsMenuItem);
1475+
debugMenu.add(item);
14751476

14761477
debugMenu.addSeparator();
14771478

1478-
stepOverMenuItem = Toolkit.newJMenuItem(Language.text("menu.debug.step"), KeyEvent.VK_J);
1479-
stepOverMenuItem.addActionListener(new ActionListener() {
1479+
item = Toolkit.newJMenuItem(Language.text("menu.debug.step"), KeyEvent.VK_J);
1480+
item.addActionListener(new ActionListener() {
14801481
public void actionPerformed(ActionEvent e) {
14811482
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Step Over' menu item");
14821483
debugger.stepOver();
14831484
}
14841485
});
1485-
debugMenu.add(stepOverMenuItem);
1486+
debugMenu.add(item);
14861487

1487-
stepIntoMenuItem = Toolkit.newJMenuItemShift(Language.text("menu.debug.step_into"), KeyEvent.VK_J);
1488-
stepIntoMenuItem.addActionListener(new ActionListener() {
1488+
item = Toolkit.newJMenuItemShift(Language.text("menu.debug.step_into"), KeyEvent.VK_J);
1489+
item.addActionListener(new ActionListener() {
14891490
public void actionPerformed(ActionEvent e) {
14901491
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Step Into' menu item");
14911492
debugger.stepInto();
14921493
}
14931494
});
1494-
debugMenu.add(stepIntoMenuItem);
1495+
debugMenu.add(item);
14951496

1496-
stepOutMenuItem = Toolkit.newJMenuItemAlt(Language.text("menu.debug.step_out"), KeyEvent.VK_J);
1497-
stepOutMenuItem.addActionListener(new ActionListener() {
1497+
item = Toolkit.newJMenuItemAlt(Language.text("menu.debug.step_out"), KeyEvent.VK_J);
1498+
item.addActionListener(new ActionListener() {
14981499
public void actionPerformed(ActionEvent e) {
14991500
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Step Out' menu item");
15001501
debugger.stepOut();
15011502
}
15021503
});
1503-
debugMenu.add(stepOutMenuItem);
1504+
debugMenu.add(item);
15041505

15051506
debugMenu.addSeparator();
15061507

1507-
printStackTraceMenuItem = new JMenuItem(Language.text("menu.debug.print_stack_trace"));
1508-
printStackTraceMenuItem.addActionListener(new ActionListener() {
1508+
item = new JMenuItem(Language.text("menu.debug.print_stack_trace"));
1509+
item.addActionListener(new ActionListener() {
15091510
public void actionPerformed(ActionEvent e) {
15101511
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Print Stack Trace' menu item");
15111512
debugger.printStackTrace();
15121513
}
15131514
});
1514-
debugMenu.add(printStackTraceMenuItem);
1515+
debugMenu.add(item);
15151516

1516-
printLocalsMenuItem = new JMenuItem(Language.text("menu.debug.print_locals"));
1517-
printLocalsMenuItem.addActionListener(new ActionListener() {
1517+
item = new JMenuItem(Language.text("menu.debug.print_locals"));
1518+
item.addActionListener(new ActionListener() {
15181519
public void actionPerformed(ActionEvent e) {
15191520
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Print Locals' menu item");
15201521
debugger.printLocals();
15211522
}
15221523
});
1523-
debugMenu.add(printLocalsMenuItem);
1524+
debugMenu.add(item);
15241525

1525-
printThisMenuItem = new JMenuItem(Language.text("menu.debug.print_fields"));
1526-
printThisMenuItem.addActionListener(new ActionListener() {
1526+
item = new JMenuItem(Language.text("menu.debug.print_fields"));
1527+
item.addActionListener(new ActionListener() {
15271528
public void actionPerformed(ActionEvent e) {
15281529
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Print This' menu item");
15291530
debugger.printThis();
15301531
}
15311532
});
1532-
debugMenu.add(printThisMenuItem);
1533+
debugMenu.add(item);
15331534

1534-
printSourceMenuItem = new JMenuItem(Language.text("menu.debug.print_source_location"));
1535-
printSourceMenuItem.addActionListener(new ActionListener() {
1535+
item = new JMenuItem(Language.text("menu.debug.print_source_location"));
1536+
item.addActionListener(new ActionListener() {
15361537
public void actionPerformed(ActionEvent e) {
15371538
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Print Source' menu item");
15381539
debugger.printSource();
15391540
}
15401541
});
1541-
debugMenu.add(printSourceMenuItem);
1542+
debugMenu.add(item);
15421543

1543-
printThreads = new JMenuItem(Language.text("menu.debug.print_threads"));
1544-
printThreads.addActionListener(new ActionListener() {
1544+
item = new JMenuItem(Language.text("menu.debug.print_threads"));
1545+
item.addActionListener(new ActionListener() {
15451546
public void actionPerformed(ActionEvent e) {
15461547
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Print Threads' menu item");
15471548
debugger.printThreads();
15481549
}
15491550
});
1550-
debugMenu.add(printThreads);
1551+
debugMenu.add(item);
15511552

15521553
debugMenu.addSeparator();
15531554

1554-
toggleVariableInspectorMenuItem = Toolkit.newJMenuItem(Language.text("menu.debug.toggle_variable_inspector"), KeyEvent.VK_I);
1555-
toggleVariableInspectorMenuItem.addActionListener(new ActionListener() {
1555+
item = Toolkit.newJMenuItem(Language.text("menu.debug.toggle_variable_inspector"), KeyEvent.VK_I);
1556+
item.addActionListener(new ActionListener() {
15561557
public void actionPerformed(ActionEvent e) {
15571558
Logger.getLogger(JavaEditor.class.getName()).log(Level.INFO, "Invoked 'Toggle Variable Inspector' menu item");
15581559
toggleVariableInspector();
15591560
}
15601561
});
1562+
debugMenu.add(item);
15611563

1562-
debugMenu.add(toggleVariableInspectorMenuItem);
1563-
1564-
showOutline = Toolkit.newJMenuItem(Language.text("menu.debug.show_sketch_outline"), KeyEvent.VK_L);
1565-
showOutline.addActionListener(new ActionListener() {
1564+
item = Toolkit.newJMenuItem(Language.text("menu.debug.show_sketch_outline"), KeyEvent.VK_L);
1565+
item.addActionListener(new ActionListener() {
15661566
public void actionPerformed(ActionEvent e) {
15671567
Base.log("Show Sketch Outline:");
15681568
errorCheckerService.getASTGenerator().showSketchOutline();
15691569
}
15701570
});
1571-
debugMenu.add(showOutline);
1571+
debugMenu.add(item);
15721572

1573-
showTabOutline = Toolkit.newJMenuItem(Language.text("menu.debug.show_tabs_list"), KeyEvent.VK_Y);
1574-
showTabOutline.addActionListener(new ActionListener() {
1573+
item = Toolkit.newJMenuItem(Language.text("menu.debug.show_tabs_list"), KeyEvent.VK_Y);
1574+
item.addActionListener(new ActionListener() {
15751575
public void actionPerformed(ActionEvent e) {
15761576
Base.log("Show Tab Outline:");
15771577
errorCheckerService.getASTGenerator().showTabOutline();
15781578
}
15791579
});
1580-
debugMenu.add(showTabOutline);
1580+
debugMenu.add(item);
15811581

15821582
return debugMenu;
15831583
}

0 commit comments

Comments
 (0)
Back | FazBrowse Home | New Git URL