From c88ef1f1e28aaf750b52052d6bdda1455251ac0e Mon Sep 17 00:00:00 2001
From: Ben Fry <fry@processing.org>
Date: Sun, 25 Jan 2015 12:55:57 -0500
Subject: [PATCH] removing more underbrush

---
 java/src/processing/mode/java/JavaEditor.java | 34 ++-----------------
 .../mode/java/pdex/ErrorCheckerService.java   |  2 +-
 todo.txt                                      |  3 ++
 3 files changed, 6 insertions(+), 33 deletions(-)

diff --git a/java/src/processing/mode/java/JavaEditor.java b/java/src/processing/mode/java/JavaEditor.java
index 2543e59e9..4f85fd918 100644
--- a/java/src/processing/mode/java/JavaEditor.java
+++ b/java/src/processing/mode/java/JavaEditor.java
@@ -60,31 +60,11 @@ public class JavaEditor extends Editor {
   protected LineHighlight currentLine; // line the debugger is currently suspended at
   protected final String breakpointMarkerComment = " //<>//"; // breakpoint marker comment
 
-  protected JMenu debugMenu; // the debug menu
-
-//  protected JMenuItem debugMenuItem;
-//  protected JMenuItem continueMenuItem;
-//  protected JMenuItem stopMenuItem;
-
-//  protected JMenuItem toggleBreakpointMenuItem;
-//  protected JMenuItem listBreakpointsMenuItem;
-
-//  protected JMenuItem stepOverMenuItem;
-//  protected JMenuItem stepIntoMenuItem;
-//  protected JMenuItem stepOutMenuItem;
-
-//  protected JMenuItem printStackTraceMenuItem;
-//  protected JMenuItem printLocalsMenuItem;
-//  protected JMenuItem printThisMenuItem;
-//  protected JMenuItem printSourceMenuItem;
-//  protected JMenuItem printThreads;
-
-//  protected JMenuItem toggleVariableInspectorMenuItem;
+  protected JMenu debugMenu;
 
   protected Debugger debugger; // the debugger
   protected DebugTray tray; // the variable inspector frame
 
-//  public JavaTextArea ta; // the text area
   public ErrorBar errorBar;
     
   protected XQConsoleToggle btnShowConsole;
@@ -94,16 +74,7 @@ public class JavaEditor extends Editor {
   protected XQErrorTable errorTable;
 
   // TODO how is this different from hasJavaTabs?
-  public boolean compilationCheckEnabled = true;
-
-//  protected JCheckBoxMenuItem showWarnings;
-//  public JCheckBoxMenuItem problemWindowMenuCB;
-//  protected JCheckBoxMenuItem debugMessagesEnabled;
-//  protected JMenuItem showOutline; 
-//  protected JMenuItem showTabOutline;
-//  protected JCheckBoxMenuItem writeErrorLog;
-//  protected JCheckBoxMenuItem completionsEnabled;
-    
+//  public boolean compilationCheckEnabled = true;
   private boolean hasJavaTabs;
   
 
@@ -2428,7 +2399,6 @@ public class JavaEditor extends Editor {
     private boolean checkForJavaTabs() {
       for (SketchCode code : getSketch().getCode()) {
         if (code.getExtension().equals("java")) {
-          compilationCheckEnabled = false;
           final String msg = 
             getSketch().getName() + " contains .java tabs. Some editor " +
             "features are not supported for .java tabs and will be disabled.";
diff --git a/java/src/processing/mode/java/pdex/ErrorCheckerService.java b/java/src/processing/mode/java/pdex/ErrorCheckerService.java
index 461fc9bb8..8fcf10f59 100644
--- a/java/src/processing/mode/java/pdex/ErrorCheckerService.java
+++ b/java/src/processing/mode/java/pdex/ErrorCheckerService.java
@@ -504,7 +504,7 @@ public class ErrorCheckerService implements Runnable {
       if (!hasSyntaxErrors()) {
              
       }
-      if (problems.length == 0 && editor.compilationCheckEnabled) {
+      if (problems.length == 0 && !editor.hasJavaTabs()) {
         //mainClassOffset++; // just a hack.
         
         sourceCode = xqpreproc.doYourThing(sourceCode, programImports);
diff --git a/todo.txt b/todo.txt
index 50d805471..566531afd 100644
--- a/todo.txt
+++ b/todo.txt
@@ -94,6 +94,9 @@ X remove public 'ta' object in DebugEditor, also dmode
 _ fix hasJavaTabs() function
 _   almost always used as a negative, or tied to a 'return' from fxn
 _   name isn't tied to its function, but the symptom
+_   only checked once (when the editor is created)
+X   how does it relate to compilationCheckEnabled?
+X     removed, seems to be duplicate/used the same way
 _ crashed on startup w/ JavaScript mode as default b/c PdeKeyListener not found
 _   because it's in the other ClassLoader, can no longer rely on it