fix placement of highlighted error line (processing#3654)

benfry · benfry · commit 3218390e8a6e · 2015-08-19T08:54:03.000-04:00
diff --git a/java/src/processing/mode/java/JavaBuild.java b/java/src/processing/mode/java/JavaBuild.java
@@ -263,11 +263,14 @@ public String preprocess(File srcFolder,
     if (sizeInfo != null && sizeInfo.hasSettings()) {
 //      String sizeStatement = sizeInfo.getStatement();
       for (String stmt : sizeInfo.getStatements()) {
-//      if (sizeStatement != null) {
         //System.out.format("size stmt is '%s'%n", sizeStatement);
+        // Don't remove newlines (and while you're at it, just keep spaces)
+        // https://github.com/processing/processing/issues/3654
+        stmt = stmt.trim();
         int index = bigCode.indexOf(stmt);
         if (index != -1) {
           bigCode.delete(index, index + stmt.length());
+          System.out.println("code now " + bigCode);
         } else {
           // TODO remove once we hit final; but prevent an exception like in
           // https://github.com/processing/processing/issues/3531
diff --git a/todo.txt b/todo.txt
@@ -2,6 +2,8 @@
 X processing-java isn't working in OS X 10.11 El Capitan
 X   https://github.com/processing/processing/issues/3497
 o   probably have to add the script/Processing.app location to user's path
+X line selected for errors is off by one or two
+X   https://github.com/processing/processing/issues/3654
 
 contribs
 X Undo does not move to the correct location in the editor window
