Insert synthetic settings() function at the end of the module body, r…

…ather than at the beginning.

This avoids inserting before import __future__.

Fixes jdf/Processing.py-Bugs#148

runtime/src/jycessing/pyde_preprocessor.py

@@ -78,9 +78,9 @@ def pyde_preprocessor(module):
 			settingsArgs = ast.arguments(args=[], vararg=None, kwarg=None, defaults=[])
 			settingsFunc = ast.FunctionDef("settings", settingsArgs, [], [])
 			info.insert(settingsFunc.body)
-			# Place the newly defined settings() function within the module body. 
+			# Place the newly defined settings() function at the end of the module body. 
 			# It's like it's been there the whole time... 
-			module.body.insert(0, settingsFunc)
+			module.body.insert(len(module.body), settingsFunc)
 
 module = ast.parse(__processing_source__ + "\n\n", filename=__file__)
 pyde_preprocessor(module)

testing/resources/test_from_future_with_settings.py

@@ -0,0 +1,8 @@
+from __future__ import division
+
+def setup():
+    size(20, 20)
+
+def draw():
+    print 'OK'
+    exit()

testing/src/test/jycessing/JycessingTests.java

@@ -12,15 +12,15 @@
 import java.nio.file.Paths;
 import java.nio.file.StandardCopyOption;
 
-import org.junit.Test;
-
 import jycessing.MixedModeError;
 import jycessing.PAppletJythonDriver;
 import jycessing.Printer;
 import jycessing.PythonSketchError;
 import jycessing.Runner;
 import jycessing.StreamPrinter;
 
+import org.junit.Test;
+
 public class JycessingTests {
   private static class CapturingPrinter implements Printer {
     private final ByteArrayOutputStream baos = new ByteArrayOutputStream();
@@ -270,4 +270,10 @@ public void randintDomainRegression() throws Exception {
   public void syspathappend() throws Exception {
     expectOK("syspathappend");
   }
+
+  // https://github.com/jdf/Processing.py-Bugs/issues/148
+  @Test
+  public void from_future_with_settings() throws Exception {
+    expectOK("from_future_with_settings");
+  }
 }