tweakmode: place tweak init function right after the size function

galsasson · galsasson · commit 3d37a7a28ca6 · 2015-08-09T11:20:56.000-04:00
diff --git a/java/src/processing/mode/java/JavaEditor.java b/java/src/processing/mode/java/JavaEditor.java
@@ -2898,8 +2898,8 @@ protected boolean automateSketch(Sketch sketch, SketchParser parser) {
       return false;
     }
 
-    int setupEndPos = SketchParser.getSetupEnd(baseCode[0]);
-    if (setupEndPos < 0) {
+    int afterSizePos = SketchParser.getAfterSizePos(baseCode[0]);
+    if (afterSizePos < 0) {
       return false;
     }
 
@@ -2992,12 +2992,14 @@ protected boolean automateSketch(Sketch sketch, SketchParser parser) {
 
     // add call to our initAllVars and initOSC functions
     // from the setup() function.
-    String addToSetup = "\n"+
-      " tweakmode_initAllVars();\n"+
-      " tweakmode_initCommunication();\n\n";
-
-    setupEndPos = SketchParser.getSetupEnd(c);
-    c = replaceString(c, setupEndPos, setupEndPos, addToSetup);
+    String addToSetup = "\n\n\n"+
+      "  /* TWEAKMODE */\n"+
+      "    tweakmode_initAllVars();\n"+
+      "    tweakmode_initCommunication();\n"+
+      "  /* TWEAKMODE */\n\n";
+
+    afterSizePos = SketchParser.getAfterSizePos(c);
+    c = replaceString(c, afterSizePos, afterSizePos, addToSetup);
 
     // Server code defines a class, so it should go later in the sketch
     String serverCode =
diff --git a/java/src/processing/mode/java/tweak/SketchParser.java b/java/src/processing/mode/java/tweak/SketchParser.java
@@ -39,7 +39,8 @@ public class SketchParser {
 
   List<List<Range>> scientificNotations;
 
-  Range setupFunction;
+  // currently is used to ignore numbers in 'setup' and 'settings' functions
+  List<Range> ignoreFunctions;
 
   List<List<Range>> commentBlocks;
   List<int[]> curlyScopes;
@@ -57,8 +58,12 @@ public SketchParser(String[] codeTabs, boolean requiresComment) {
       commentBlocks.add(getCommentBlocks(code));
     }
 
-    // get setup function range (to ignore all numbers there)
-    setupFunction = new Range(getSetupStart(codeTabs[0]), getSetupEnd(codeTabs[0]));
+    // add 'settings' and 'setup' to ignore list (to ignore all numbers there)
+    ignoreFunctions = new ArrayList<Range>();
+    ignoreFunctions.add(new Range(getVoidFunctionStart(codeTabs[0], "settings"),
+                                  getVoidFunctionEnd(codeTabs[0], "settings")));
+    ignoreFunctions.add(new Range(getVoidFunctionStart(codeTabs[0], "setup"),
+                                  getVoidFunctionEnd(codeTabs[0], "setup")));
 
     // build curly scope for every character in the code
     curlyScopes = new ArrayList<>();
@@ -125,8 +130,8 @@ private void addAllDecimalNumbers() {
           continue;
         }
 
-        if (setupFunction.contains(start)) {
-          // ignore numbers in setup
+        if (isInRangeList(start, ignoreFunctions)) {
+          // ignore numbers in predefined functions
           continue;
         }
 
@@ -217,8 +222,8 @@ private void addAllHexNumbers() {
           continue;
         }
 
-        if (setupFunction.contains(start)) {
-          // ignore number in setup
+        if (isInRangeList(start, ignoreFunctions)) {
+          // ignore numbers in predefined functions
           continue;
         }
 
@@ -278,8 +283,8 @@ private void addAllWebColorNumbers() {
           continue;
         }
 
-        if (setupFunction.contains(start)) {
-          // ignore number in setup
+        if (isInRangeList(start, ignoreFunctions)) {
+          // ignore numbers in predefined functions
           continue;
         }
 
@@ -382,8 +387,8 @@ private void createColorBoxes() {
           continue;
         }
 
-        if (setupFunction.contains(m.start())) {
-          // ignore number in setup
+        if (isInRangeList(m.start(), ignoreFunctions)) {
+          // ignore numbers in predefined functions
           continue;
         }
 
@@ -466,8 +471,8 @@ private void createColorBoxesForLights() {
           continue;
         }
 
-        if (setupFunction.contains(m.start())) {
-          // ignore number in setup
+        if (isInRangeList(m.start(), ignoreFunctions)) {
+          // ignore numbers in predefined functions
           continue;
         }
 
@@ -827,6 +832,50 @@ else if (readObject) {
     return obj;
   }
 
+  static public int getVoidFunctionStart(String code, String functionName) {
+    Pattern p = Pattern.compile("void[\\s\\t\\r\\n]*"+functionName+"[\\s\\t]*\\(\\)[\\s\\t\\r\\n]*\\{");
+    Matcher m = p.matcher(code);
+
+    if (m.find()) {
+      return m.end();
+    }
+
+    return -1;
+  }
+
+  static public int getVoidFunctionEnd(String code, String functionName) {
+    List<Range> comments = getCommentBlocks(code);
+
+    int start = getVoidFunctionStart(code, functionName);
+    if (start == -1) {
+      return -1;
+    }
+
+    // count brackets to look for setup end
+    int bracketCount=1;
+    int pos = start;
+    while (bracketCount > 0 && pos < code.length()) {
+      if (isInRangeList(pos, comments)) {
+        // in a comment, ignore and move on
+        pos++;
+        continue;
+      }
+
+      if (code.charAt(pos) == '{') {
+        bracketCount++;
+      }
+      else if (code.charAt(pos) == '}') {
+        bracketCount--;
+      }
+      pos++;
+    }
+
+    if (bracketCount == 0) {
+      return pos-1;
+    }
+    return -1;
+  }
+
 
   static public int getSetupStart(String code) {
     Pattern p = Pattern.compile("void[\\s\\t\\r\\n]*setup[\\s\\t]*\\(\\)[\\s\\t\\r\\n]*\\{");
@@ -839,7 +888,6 @@ static public int getSetupStart(String code) {
     return -1;
   }
 
-
   static public int getSetupEnd(String code) {
     List<Range> comments = getCommentBlocks(code);
 
@@ -873,6 +921,67 @@ else if (code.charAt(pos) == '}') {
     return -1;
   }
 
+  static public int getAfterSizePos(String code) {
+    List<Range> comments = getCommentBlocks(code);
+
+    // find the size function
+    Pattern p = Pattern.compile("size[\\s\\t]*\\(");
+    Matcher m = p.matcher(code);
+
+    while (m.find()) {
+      if (isInRangeList(m.start(), comments) ||
+            isInRangeList(m.end(), comments)) {
+        // this is a comment, next
+        continue;
+      }
+
+      // count brackets to look for size call end
+      int bracketCount=1;
+      int pos = m.end();
+      while (bracketCount > 0 && pos < code.length()) {
+        if (isInRangeList(pos, comments)) {
+          // in a comment, ignore and move on
+          pos++;
+          continue;
+        }
+
+        if (code.charAt(pos) == '(') {
+          bracketCount++;
+        }
+        else if (code.charAt(pos) == ')') {
+          bracketCount--;
+        }
+        pos++;
+      }
+
+      if (bracketCount != 0) {
+        // could not find closing ')'. next
+        continue;
+      }
+
+      // find ';' sign
+      boolean found = false;
+      while (pos < code.length()) {
+        if (code.charAt(pos) == ';' &&
+              !isInRangeList(pos, comments)) {
+          found = true;
+          break;
+        }
+        pos++;
+      }
+
+      if (!found) {
+        // didn't find the ';'. next
+        continue;
+      }
+
+      // success! we found the place
+      return pos+1;
+    }
+
+    // nothing was found
+    return -1;
+  }
 
   static class Range {
     int start;
