performance in context not as good as the microbenchmark.

Sam Pullara · Sam Pullara · commit 7f7a8b37a34d · 2012-03-06T00:04:51.000-08:00
diff --git a/indy/src/test/java/IndyDemo.java b/indy/src/test/java/IndyDemo.java
@@ -1,11 +1,16 @@
-import java.lang.invoke.*;
-import java.lang.reflect.*;
+import com.github.mustachejava.indy.IndyWrapper;
+import com.github.mustachejava.reflect.ReflectionObjectHandler;
+import com.github.mustachejava.reflect.ReflectionWrapper;
+import com.github.mustachejava.util.Wrapper;
 
 public class IndyDemo {
   public static void main(String[] args) throws Throwable {
+    IndyDemo indyDemo = new IndyDemo();
+    REFLECTED = new ReflectionObjectHandler().find("someMethod", new Object[] { indyDemo });
+    INDY = IndyWrapper.create((ReflectionWrapper) REFLECTED);
     for (int i = 0; i < 10; i++) {
-      timeReflection();
-      timeIndy();
+      timeReflection(indyDemo);
+      timeIndy(indyDemo);
     }
   }
 
@@ -18,42 +23,30 @@ public static void main(String[] args) throws Throwable {
     }
   }
 
-  public static void timeReflection() throws Throwable {
+  public static void timeReflection(IndyDemo indyDemo) throws Throwable {
     long start = System.currentTimeMillis();
+    Object[] scopes = {indyDemo};
     for (int i = 0; i < 100000000; i++) {
-      REFLECTED.invoke(null, strings[i % 100]);
+      REFLECTED.call(scopes);
     }
     System.out.println("reflected: " + (System.currentTimeMillis() - start));
   }
 
-  public static void timeIndy() throws Throwable {
+  public static void timeIndy(IndyDemo indyDemo) throws Throwable {
     long start = System.currentTimeMillis();
+    Object[] scopes = {indyDemo};
     for (int i = 0; i < 100000000; i++) {
-      HANDLE.invokeExact(strings[i % 100]);
+      INDY.call(scopes);
     }
     System.out.println("indy: " + (System.currentTimeMillis() - start));
   }
 
-  private static final Method REFLECTED;
-  private static final MethodHandle HANDLE;
+  private static Wrapper REFLECTED;
+  private static Wrapper INDY;
 
-  static {
-    Method method = null;
-    try {
-      method = IndyDemo.class.getMethod("someMethod", String.class);
-    } catch (Exception e) {}
-    REFLECTED = method;
-
-    MethodHandle handle = null;
-    try {
-      handle = MethodHandles.lookup().unreflect(REFLECTED);
-    } catch (Exception e) {}
-    HANDLE = handle;
-  }
-
-  public static int length = 0;
+  private int length = 0;
 
-  public static void someMethod(String a) {
-    length = a.length();
+  public int someMethod() {
+    return length++;
   }
 }
diff --git a/indy/src/test/java/com/github/mustachejava/ComplexObject.java b/indy/src/test/java/com/github/mustachejava/ComplexObject.java
@@ -3,42 +3,35 @@
 import java.util.Arrays;
 import java.util.List;
 
-/**
-* Created by IntelliJ IDEA.
-* User: spullara
-* Date: 1/17/12
-* Time: 8:17 PM
-* To change this template use File | Settings | File Templates.
-*/
 public class ComplexObject {
-String header = "Colors";
-List<Color> item = Arrays.asList(
-        new Color("red", true, "#Red"),
-        new Color("green", false, "#Green"),
-        new Color("blue", false, "#Blue")
-);
+  String header = "Colors";
+  List<Color> item = Arrays.asList(
+          new Color("red", true, "#Red"),
+          new Color("green", false, "#Green"),
+          new Color("blue", false, "#Blue")
+  );
 
-boolean list() {
-  return item.size() != 0;
-}
-
-boolean empty() {
-  return item.size() == 0;
-}
-
-private static class Color {
-  boolean link() {
-    return !current;
+  boolean list() {
+    return item.size() != 0;
   }
 
-  Color(String name, boolean current, String url) {
-    this.name = name;
-    this.current = current;
-    this.url = url;
+  boolean empty() {
+    return item.size() == 0;
   }
 
-  String name;
-  boolean current;
-  String url;
-}
+  private static class Color {
+    boolean link() {
+      return !current;
+    }
+
+    Color(String name, boolean current, String url) {
+      this.name = name;
+      this.current = current;
+      this.url = url;
+    }
+
+    String name;
+    boolean current;
+    String url;
+  }
 }
diff --git a/indy/src/test/java/com/github/mustachejava/ParallelComplexObject.java b/indy/src/test/java/com/github/mustachejava/ParallelComplexObject.java
@@ -4,46 +4,39 @@
 import java.util.List;
 import java.util.concurrent.Callable;
 
-/**
-* Created by IntelliJ IDEA.
-* User: spullara
-* Date: 1/17/12
-* Time: 8:19 PM
-* To change this template use File | Settings | File Templates.
-*/
 public class ParallelComplexObject {
-String header = "Colors";
-Callable<List<Color>> item = new Callable<List<Color>>() {
-  @Override
-  public List<Color> call() throws Exception {
-    return Arrays.asList(
-            new Color("red", true, "#Red"),
-            new Color("green", false, "#Green"),
-            new Color("blue", false, "#Blue"));
+  String header = "Colors";
+  Callable<List<Color>> item = new Callable<List<Color>>() {
+    @Override
+    public List<Color> call() throws Exception {
+      return Arrays.asList(
+              new Color("red", true, "#Red"),
+              new Color("green", false, "#Green"),
+              new Color("blue", false, "#Blue"));
+    }
+  };
+
+  boolean list() {
+    return true;
   }
-};
 
-boolean list() {
-  return true;
-}
+  boolean empty() {
+    return false;
+  }
 
-boolean empty() {
-  return false;
-}
+  private static class Color {
+    boolean link() {
+      return !current;
+    }
 
-private static class Color {
-  boolean link() {
-    return !current;
-  }
+    Color(String name, boolean current, String url) {
+      this.name = name;
+      this.current = current;
+      this.url = url;
+    }
 
-  Color(String name, boolean current, String url) {
-    this.name = name;
-    this.current = current;
-    this.url = url;
+    String name;
+    boolean current;
+    String url;
   }
-
-  String name;
-  boolean current;
-  String url;
-}
 }
