Fixed pre processor bug. Closes #1245

Author: Federico Fissore

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

@@ -112,13 +112,13 @@ public int writePrefix(String program)
 //    }
 
     prototypes = prototypes(program);
-    
+
     // store # of prototypes so that line number reporting can be adjusted
     prototypeCount = prototypes.size();
 
     // do this after the program gets re-combobulated
     this.program = program;
-    
+
     return headerCount + prototypeCount;
   }
 
@@ -244,7 +244,9 @@ public String strip(String in) {
     // XXX: doesn't properly handle special single-quoted characters
     // single-quoted character
     String p = "('.')";
-    
+
+    p += "|('\\\\\"')";
+
     // double-quoted string
     p += "|(\"(?:[^\"\\\\]|\\\\.)*\")";
 
@@ -294,7 +296,7 @@ private String collapseBraces(String in) {
 
   public ArrayList<String> prototypes(String in) {
     in = collapseBraces(strip(in));
-    
+
     // XXX: doesn't handle ... varargs
     // XXX: doesn't handle function pointers
     Pattern prototypePattern = Pattern.compile("[\\w\\[\\]\\*]+\\s+[&\\[\\]\\*\\w\\s]+\\([&,\\[\\]\\*\\w\\s]*\\)(?=\\s*;)");

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

@@ -0,0 +1,21 @@
+package processing.app.preproc;
+
+import org.junit.Test;
+import processing.app.helpers.FileUtils;
+
+import java.io.File;
+
+import static org.junit.Assert.assertEquals;
+
+public class PdePreprocessorTest {
+
+  @Test
+  public void testSourceWithQuoteAndDoubleQuotesEscapedAndFinalQuoteShouldNotRaiseException() throws Exception {
+    String s = FileUtils.readFileToString(new File(PdePreprocessorTest.class.getResource("RemoteCallLogger_v1e0.ino").getFile()));
+
+    String actualOutput = new PdePreprocessor().strip(s);
+    String expectedOutput = FileUtils.readFileToString(new File(PdePreprocessorTest.class.getResource("RemoteCallLogger_v1e0.stripped.ino").getFile()));
+
+    assertEquals(actualOutput, expectedOutput);
+  }
+}