Remove unused error logging

JakubValtar · JakubValtar · commit 885a53e0a12e · 2015-10-22T12:25:22.000+02:00
diff --git a/java/src/processing/mode/java/JavaEditor.java b/java/src/processing/mode/java/JavaEditor.java
@@ -1296,9 +1296,9 @@ public void statusError(String what) {
 
   public void internalCloseRunner() {
     // Added temporarily to dump error log. TODO: Remove this later [mk29]
-    if (JavaMode.errorLogsEnabled) {
-      writeErrorsToFile();
-    }
+    //if (JavaMode.errorLogsEnabled) {
+    //  writeErrorsToFile();
+    //}
     handleStop();
   }
 
@@ -1328,10 +1328,12 @@ public void dispose() {
   }
 
 
+  // Not sure how this was supposed to work, tempErrorLog is always empty [jv]
   /**
    * Writes all error messages to a csv file.
    * For analytics purposes only.
    */
+  /*
   private void writeErrorsToFile() {
     if (errorCheckerService.tempErrorLog.size() == 0) return;
 
@@ -1369,7 +1371,7 @@ private void writeErrorsToFile() {
       System.err.println("Failed to save log file for sketch " + getSketch().getName());
       e.printStackTrace();
     }
-  }
+  }*/
 
 
   /*
diff --git a/java/src/processing/mode/java/pdex/ErrorCheckerService.java b/java/src/processing/mode/java/pdex/ErrorCheckerService.java
@@ -30,7 +30,6 @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.TreeMap;
 import java.util.concurrent.ArrayBlockingQueue;
 import java.util.concurrent.BlockingQueue;
 import java.util.concurrent.Executors;
@@ -514,8 +513,6 @@ public boolean hasErrors(){
     return lastCodeCheckResult.containsErrors;
   }
 
-  public TreeMap<String, IProblem> tempErrorLog = new TreeMap<>();
-
   protected URLClassLoader classLoader;
 
 
