better keep the static nature of the MethodHandles.publicLookup

static in the sense of global lookup per classloader. related to #2091
jruby · Nov 23, 2014 · db10c66 · db10c66
1 parent 3bb5c15
commit db10c66
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/core/src/main/java/org/jruby/compiler/JITCompiler.java b/core/src/main/java/org/jruby/compiler/JITCompiler.java
@@ -173,18 +173,20 @@ public void jitThresholdReached(final InterpretedIRMethod method, final RubyInst
             jitTask.run();
         }
     }
-
+
+    private static final MethodHandles.Lookup PUBLIC_LOOKUP = MethodHandles.publicLookup().in(Ruby.class);
+
     private class JITTask implements Runnable {
         private final String className;
         private final InterpretedIRMethod method;
         private final String methodName;
-        
+
         public JITTask(String className, InterpretedIRMethod method, String methodName) {
             this.className = className;
             this.method = method;
             this.methodName = methodName;
         }
-        
+
         public void run() {
             try {
                 // Check if the method has been explicitly excluded
@@ -247,7 +249,7 @@ public void run() {
                     // only variable-arity
                     method.switchToJitted(
                             new CompiledIRMethod(
-                                    MethodHandles.publicLookup().in(Ruby.class).findStatic(sourceClass, jittedName, signatures.get(-1)),
+                                    PUBLIC_LOOKUP.findStatic(sourceClass, jittedName, signatures.get(-1)),
                                     method.getIRMethod(),
                                     method.getVisibility(),
                                     method.getImplementationClass()));
@@ -258,8 +260,8 @@ public void run() {
 
                         method.switchToJitted(
                                 new CompiledIRMethod(
-                                        MethodHandles.publicLookup().in(Ruby.class).findStatic(sourceClass, jittedName, signatures.get(-1)),
-                                        MethodHandles.publicLookup().in(Ruby.class).findStatic(sourceClass, jittedName, entry.getValue()),
+                                        PUBLIC_LOOKUP.findStatic(sourceClass, jittedName, signatures.get(-1)),
+                                        PUBLIC_LOOKUP.findStatic(sourceClass, jittedName, entry.getValue()),
                                         entry.getKey(),
                                         method.getIRMethod(),
                                         method.getVisibility(),