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/02/25 16:24:47 UTC
svn commit: r1074576 - in
/felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver:
Candidates.java ResolverImpl.java
Author: rickhall
Date: Fri Feb 25 15:24:47 2011
New Revision: 1074576
URL: http://svn.apache.org/viewvc?rev=1074576&view=rev
Log:
Add a couple "to do" comments.
Modified:
felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/Candidates.java
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/Candidates.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/Candidates.java?rev=1074576&r1=1074575&r2=1074576&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/Candidates.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/resolver/Candidates.java Fri Feb 25 15:24:47 2011
@@ -245,6 +245,8 @@ public class Candidates
}
else
{
+// TODO: FRAGMENT RESOLVER - Fragments should only be removed when they no longer
+// match any hosts, not immediately.
unselectedFragments.add(m);
}
}
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=1074576&r1=1074575&r2=1074576&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 Fri Feb 25 15:24:47 2011
@@ -446,6 +446,7 @@ public class ResolverImpl implements Res
}
// TODO: FELIX3 - Modify to not be recursive.
+// TODO: FRAGMENT RESOLVER - Does it make sense to move this to Candidates?
private void populateCandidates(
ResolverState state, Module module, Candidates allCandidates,
Map<Module, Object> resultCache)