From 41dcc725f34575f7c32be7dedbdbb95b0364d55f Mon Sep 17 00:00:00 2001
From: Jakub Valtar <jakub.valtar@gmail.com>
Date: Mon, 21 Sep 2015 17:23:53 -0400
Subject: [PATCH] Revert "Implemented code folder imports. Fixes #3732"

This reverts commit 24508804ddac585a9356e64f214a4bb8c4a92a45.

Conflicts:
	java/src/processing/mode/java/pdex/ASTGenerator.java
	java/src/processing/mode/java/pdex/ErrorCheckerService.java
	java/src/processing/mode/java/pdex/XQPreprocessor.java
---
 .../mode/java/pdex/ASTGenerator.java          | 13 ------------
 .../mode/java/pdex/ErrorCheckerService.java   | 20 ++++---------------
 .../mode/java/pdex/XQPreprocessor.java        | 19 ++----------------
 3 files changed, 6 insertions(+), 46 deletions(-)

diff --git a/java/src/processing/mode/java/pdex/ASTGenerator.java b/java/src/processing/mode/java/pdex/ASTGenerator.java
index 1f5df5badc..2105767b20 100644
--- a/java/src/processing/mode/java/pdex/ASTGenerator.java
+++ b/java/src/processing/mode/java/pdex/ASTGenerator.java
@@ -1370,19 +1370,6 @@ protected Class<?> findClassIfExists(String className){
       //log("Doesn't exist in imp package: " + impS.getImportName());
     }
 
-    for (ImportStatement impS : errorCheckerService.codeFolderImports) {
-      String temp = impS.getPackageName();
-      if (impS.isStarredImport() && className.indexOf('.') == -1) {
-        temp = impS.getPackageName() + "." + className;
-      }
-      tehClass = loadClass(temp);
-      if (tehClass != null) {
-        log(tehClass.getName() + " located.");
-        return tehClass;
-      }
-      //log("Doesn't exist in (code folder) imp package: " + impS.getImportName());
-    }
-
     PdePreprocessor p = new PdePreprocessor(null);
     for (String impS : p.getCoreImports()) {
       tehClass = loadClass(impS.substring(0,impS.length()-1) + className);
diff --git a/java/src/processing/mode/java/pdex/ErrorCheckerService.java b/java/src/processing/mode/java/pdex/ErrorCheckerService.java
index 3e3745a3db..1040d760f1 100644
--- a/java/src/processing/mode/java/pdex/ErrorCheckerService.java
+++ b/java/src/processing/mode/java/pdex/ErrorCheckerService.java
@@ -202,11 +202,6 @@ public class ErrorCheckerService implements Runnable {
    */
   protected ArrayList<ImportStatement> previousImports = new ArrayList<ImportStatement>();
 
-  /**
-   * List of import statements for any .jar files in the code folder.
-   */
-  protected ArrayList<ImportStatement> codeFolderImports = new ArrayList<ImportStatement>();
-
   /**
    * Teh Preprocessor
    */
@@ -248,7 +243,7 @@ public ErrorCheckerService(JavaEditor debugEditor) {
 
     initParser();
     //initializeErrorWindow();
-    xqpreproc = new XQPreprocessor(this);
+    xqpreproc = new XQPreprocessor();
     PdePreprocessor pdePrepoc = new PdePreprocessor(null);
     defaultImportsOffset = pdePrepoc.getCoreImports().length +
         pdePrepoc.getDefaultImports().length + 1;
@@ -663,10 +658,6 @@ protected void compileCheck() {
         loadCompClass = false;
       }
 
-//      for(URL cpUrl: classPath) {
-//        Messages.log("CP jar: " + cpUrl.getPath());
-//      }
-
       if (compilerSettings == null) {
         prepareCompilerSetting();
       }
@@ -877,18 +868,15 @@ protected void prepareCompilerClasspath() {
           Messages.log("Sketch has a code folder. Adding its jars");
           String codeFolderPath[] =
                   PApplet.split(codeFolderClassPath.substring(1).trim(),
-                          File.pathSeparatorChar);
+                                File.pathSeparatorChar);
           try {
             for (String pathItem : codeFolderPath) {
               classpathJars.add(new File(pathItem).toURI().toURL());
-              Messages.log("Addind cf jar: " + pathItem);
             }
           } catch (Exception e2) {
             e2.printStackTrace();
           }
         }
-
-
       }
 
       // Also add jars specified in mode's search path
@@ -929,11 +917,11 @@ protected boolean ignorableSuggestionImport(String impName) {
       }
     }
 
-    for (ImportStatement impS : codeFolderImports) {
+    /*for (ImportStatement impS : codeFolderImports) {
       if (impNameLc.startsWith(impS.getPackageName().toLowerCase())) {
         return false;
       }
-    }
+    }*/
 
     final String include = "include";
     final String exclude = "exclude";
diff --git a/java/src/processing/mode/java/pdex/XQPreprocessor.java b/java/src/processing/mode/java/pdex/XQPreprocessor.java
index dcce79c9ce..20a4d3af2f 100644
--- a/java/src/processing/mode/java/pdex/XQPreprocessor.java
+++ b/java/src/processing/mode/java/pdex/XQPreprocessor.java
@@ -38,7 +38,6 @@
 import org.eclipse.text.edits.MalformedTreeException;
 import org.eclipse.text.edits.TextEdit;
 
-import processing.app.Util;
 import processing.data.StringList;
 import processing.mode.java.preproc.PdePreprocessor;
 
@@ -51,13 +50,12 @@
 public class XQPreprocessor {
   private ASTRewrite rewrite = null;
   private List<ImportStatement> extraImports;
-  private ErrorCheckerService ecs;
+
   private String[] coreImports;
   private String[] defaultImports;
 
 
-  public XQPreprocessor(ErrorCheckerService errorCheckerService) {
-    ecs = errorCheckerService;
+  public XQPreprocessor() {
     PdePreprocessor p = new PdePreprocessor(null);
     defaultImports = p.getDefaultImports();
     coreImports = p.getCoreImports();
@@ -113,19 +111,6 @@ String prepareImports(List<ImportStatement> programImports) {
     for (String imp : defaultImports) {
       imports.append("import " + imp + ";");
     }
-    if (ecs.getEditor().getSketch().getCodeFolder().exists()) {
-      StringList codeFolderPackages = null;
-      String codeFolderClassPath = Util.contentsToClassPath(ecs.getEditor().getSketch().getCodeFolder());
-      codeFolderPackages = Util.packageListFromClassPath(codeFolderClassPath);
-      if (codeFolderPackages != null) {
-        ecs.codeFolderImports.clear();
-        for (String item : codeFolderPackages) {
-          // Messages.log("CF import " + item);
-          imports.append("import " + item + ".*;");
-          ecs.codeFolderImports.add(new ImportStatement("import " + item + ".*;",0,0));
-        }
-      }
-    }
     return imports.join("\n") + "\n";
   }