You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@felix.apache.org by ri...@apache.org on 2006/09/01 21:09:31 UTC

svn commit: r439422 - /incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ContentClassLoader.java

Author: rickhall
Date: Fri Sep  1 12:09:30 2006
New Revision: 439422

URL: http://svn.apache.org/viewvc?rev=439422&view=rev
Log:
Added a "to do" comment to remind me to look into a synchronization issue.

Modified:
    incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ContentClassLoader.java

Modified: incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ContentClassLoader.java
URL: http://svn.apache.org/viewvc/incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ContentClassLoader.java?rev=439422&r1=439421&r2=439422&view=diff
==============================================================================
--- incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ContentClassLoader.java (original)
+++ incubator/felix/trunk/framework/src/main/java/org/apache/felix/framework/searchpolicy/ContentClassLoader.java Fri Sep  1 12:09:30 2006
@@ -83,6 +83,8 @@
 
     protected Class findClass(String name) throws ClassNotFoundException
     {
+// TODO: ML - We probably need to do this check closer to defineClass() and
+//       protect it with a synchronized block.
         Class clazz = findLoadedClass(name);
 
         // Search for class in module.