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 2011/03/07 17:03:01 UTC

svn commit: r1078824 - /felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java

Author: rickhall
Date: Mon Mar  7 16:03:00 2011
New Revision: 1078824

URL: http://svn.apache.org/viewvc?rev=1078824&view=rev
Log:
Remove comments since moving candidate population to Candidates resolved
this commented issue. (FELIX-2858)

Modified:
    felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java

Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java?rev=1078824&r1=1078823&r2=1078824&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/ResolverImpl.java Mon Mar  7 16:03:00 2011
@@ -73,8 +73,6 @@ public class ResolverImpl implements Res
                     Candidates allCandidates = new Candidates(state, module);
 
                     // Try to populate optional fragments.
-// TODO: SINGLETON RESOLVER - These optional modules will not be considered
-//       for singleton calculation -- fix this.
                     for (Module fragment : fragments)
                     {
                         try
@@ -213,8 +211,6 @@ public class ResolverImpl implements Res
                 try
                 {
                     // Try to populate optional fragments.
-// TODO: SINGLETON RESOLVER - These optional modules will not be considered
-//       for singleton calculation -- fix this.
                     for (Module fragment : fragments)
                     {
                         try