geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gno...@apache.org
Subject svn commit: r784076 - in /geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint: container/AbstractServiceReferenceRecipe.java di/Recipe.java
Date Fri, 12 Jun 2009 11:16:53 GMT
Author: gnodet
Date: Fri Jun 12 11:16:53 2009
New Revision: 784076

URL: http://svn.apache.org/viewvc?rev=784076&view=rev
Log:
Add some TODOs

Modified:
    geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/container/AbstractServiceReferenceRecipe.java
    geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/di/Recipe.java

Modified: geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/container/AbstractServiceReferenceRecipe.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/container/AbstractServiceReferenceRecipe.java?rev=784076&r1=784075&r2=784076&view=diff
==============================================================================
--- geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/container/AbstractServiceReferenceRecipe.java
(original)
+++ geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/container/AbstractServiceReferenceRecipe.java
Fri Jun 12 11:16:53 2009
@@ -96,7 +96,9 @@
         this.explicitDependencies = explicitDependencies;
         // Create a ClassLoader delegating to the bundle, but also being able to see our
bundle classes
         // so that the created proxy can access cglib classes.
-        // TODO: use a doPrivileged block 
+        // TODO: use a doPrivileged block
+        // TODO: we should be able to get rid of this classloader when using JDK 1.4 proxies
with a single interface
+        //         (the case defined by the spec) and use the interface classloader instead
         this.proxyClassLoader = new BundleDelegatingClassLoader(blueprintContainer.getBundleContext().getBundle(),
                                                                 getClass().getClassLoader());
 

Modified: geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/di/Recipe.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/di/Recipe.java?rev=784076&r1=784075&r2=784076&view=diff
==============================================================================
--- geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/di/Recipe.java
(original)
+++ geronimo/sandbox/blueprint/blueprint-core/src/main/java/org/apache/geronimo/blueprint/di/Recipe.java
Fri Jun 12 11:16:53 2009
@@ -31,8 +31,10 @@
 
     List<Recipe> getNestedRecipes();
 
+    // TODO: replace with destroy(Object instance)
     Destroyable getDestroyable(Object instance);
 
+    // TODO: Remove this method
     void postCreate();
 
 }



Mime
View raw message