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)
