Remove unused field

jboss-modules · Mar 5, 2012 · 46115a3 · 46115a3
1 parent 8cbbf83
commit 46115a3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/src/main/java/org/jboss/modules/Module.java b/src/main/java/org/jboss/modules/Module.java
@@ -151,11 +151,6 @@ public Void run() {
      * The module loader which created this module.
      */
     private final ModuleLoader moduleLoader;
-    /**
-     * This reference exists solely to prevent the {@code FutureModule} from getting GC'd prematurely.
-     */
-    @SuppressWarnings({ "unused" })
-    private final Object myKey;
     /**
      * The fallback local loader, if any is defined.
      */
@@ -185,11 +180,9 @@ public Void run() {
      *
      * @param spec the module specification
      * @param moduleLoader the module loader
-     * @param myKey the key to keep a strong reference to
      */
-    Module(final ConcreteModuleSpec spec, final ModuleLoader moduleLoader, final Object myKey) {
+    Module(final ConcreteModuleSpec spec, final ModuleLoader moduleLoader) {
         this.moduleLoader = moduleLoader;
-        this.myKey = myKey;
 
         // Initialize state from the spec.
         identifier = spec.getModuleIdentifier();

diff --git a/src/main/java/org/jboss/modules/ModuleLoader.java b/src/main/java/org/jboss/modules/ModuleLoader.java
@@ -379,7 +379,7 @@ private Module defineModule(final ConcreteModuleSpec moduleSpec, final FutureMod
         final ModuleLogger log = Module.log;
         final ModuleIdentifier moduleIdentifier = moduleSpec.getModuleIdentifier();
 
-        final Module module = new Module(moduleSpec, this, futureModule);
+        final Module module = new Module(moduleSpec, this);
         module.getClassLoaderPrivate().recalculate();
         module.setDependencies(Arrays.asList(moduleSpec.getDependencies()));
         log.moduleDefined(moduleIdentifier, this);