Merge branch 'ide-1.5.x' into ide-1.5.x-jssc

Author: Federico Fissore

app/src/processing/app/preproc/PdePreprocessor.java

@@ -87,7 +87,7 @@ public int writePrefix(String program)
     // an OutOfMemoryError or NullPointerException will happen.
     // again, not gonna bother tracking this down, but here's a hack.
     // http://dev.processing.org/bugs/show_bug.cgi?id=16
-    scrubComments(program);
+    program = scrubComments(program);
     // If there are errors, an exception is thrown and this fxn exits.
 
     if (Preferences.getBoolean("preproc.substitute_unicode")) {
@@ -242,26 +242,27 @@ public int firstStatement(String in) {
    */
   public String strip(String in) {
     // XXX: doesn't properly handle special single-quoted characters
+    List<Pattern> patterns = new ArrayList<Pattern>();
     // single-quoted character
-    String p = "('.')";
-
-    p += "|('\\\\\"')";
-
-    // double-quoted string
-    p += "|(\"(?:[^\"\\\\]|\\\\.)*\")";
-    
+    patterns.add(Pattern.compile("('.')", Pattern.MULTILINE));
     // single and multi-line comment
-    //p += "|" + "(//\\s*?$)|(/\\*\\s*?\\*/)";
-    p += "|(//.*?$)|(/\\*[^*]*(?:\\*(?!/)[^*]*)*\\*/)";
-    
+    patterns.add(Pattern.compile("('\\\\\"')", Pattern.MULTILINE));
+    patterns.add(Pattern.compile("(//.*?$)", Pattern.MULTILINE));
+    patterns.add(Pattern.compile("(/\\*[^*]*(?:\\*(?!/)[^*]*)*\\*/)", Pattern.MULTILINE));
     // pre-processor directive
-    p += "|" + "(^\\s*#.*?$)";
+    patterns.add(Pattern.compile("(^\\s*#.*?$)", Pattern.MULTILINE));
+    // double-quoted string
+    patterns.add(Pattern.compile("(\"(?:[^\"\\\\]|\\\\.)*\")", Pattern.MULTILINE));
 
-    Pattern pattern = Pattern.compile(p, Pattern.MULTILINE);
-    Matcher matcher = pattern.matcher(in);
-    return matcher.replaceAll(" ");
+    String code = in;
+    for (Pattern p : patterns) {
+      Matcher matcher = p.matcher(code);
+      code = matcher.replaceAll(" ");
+    }
+
+    return code;
   }
-  
+
   /**
    * Removes the contents of all top-level curly brace pairs {}.
    * @param in the String to collapse

app/test/processing/app/preproc/IncludeBetweenMultilineComment.ino

@@ -0,0 +1,15 @@
+#include <CapacitiveSensorDue.h>
+/*
+#include <WiFi.h>
+*/
+CapacitiveSensorDue cs_13_8 = CapacitiveSensorDue(13,8);
+void setup()
+{
+	Serial.begin(9600);
+}
+void loop()
+{
+	long total1 = cs_13_8.read(30);
+	Serial.println(total1);
+	delay(100);
+}

app/test/processing/app/preproc/PdePreprocessorTest.java

@@ -18,4 +18,14 @@ public void testSourceWithQuoteAndDoubleQuotesEscapedAndFinalQuoteShouldNotRaise
 
     assertEquals(actualOutput, expectedOutput);
   }
-}
+
+  @Test
+  public void testIncludeInsideMultilineComment() throws Exception {
+    String s = FileUtils.readFileToString(new File(PdePreprocessorTest.class.getResource("IncludeBetweenMultilineComment.ino").getFile()));
+
+    PdePreprocessor pdePreprocessor = new PdePreprocessor();
+    pdePreprocessor.writePrefix(s);
+    assertEquals(1, pdePreprocessor.getExtraImports().size());
+    assertEquals("CapacitiveSensorDue.h", pdePreprocessor.getExtraImports().get(0));
+  }
+}