You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by ds...@apache.org on 2010/08/26 15:49:22 UTC
svn commit: r989687 -
/felix/trunk/sigil/common/core/src/org/apache/felix/sigil/common/repository/AbstractRepositoryManager.java
Author: dsavage
Date: Thu Aug 26 13:49:21 2010
New Revision: 989687
URL: http://svn.apache.org/viewvc?rev=989687&view=rev
Log:
make addRepository private and discard unused removeRepository method
Modified:
felix/trunk/sigil/common/core/src/org/apache/felix/sigil/common/repository/AbstractRepositoryManager.java
Modified: felix/trunk/sigil/common/core/src/org/apache/felix/sigil/common/repository/AbstractRepositoryManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/sigil/common/core/src/org/apache/felix/sigil/common/repository/AbstractRepositoryManager.java?rev=989687&r1=989686&r2=989687&view=diff
==============================================================================
--- felix/trunk/sigil/common/core/src/org/apache/felix/sigil/common/repository/AbstractRepositoryManager.java (original)
+++ felix/trunk/sigil/common/core/src/org/apache/felix/sigil/common/repository/AbstractRepositoryManager.java Thu Aug 26 13:49:21 2010
@@ -119,7 +119,7 @@ public abstract class AbstractRepository
}
}
- protected void addRepository(IBundleRepository rep, int level)
+ private void addRepository(IBundleRepository rep, int level)
{
Type type = null;
@@ -156,27 +156,6 @@ public abstract class AbstractRepository
notifyListeners(new RepositoryChangeEvent(rep, type));
}
- protected void removeRepository(IBundleRepository rep)
- {
- Type type = null;
-
- synchronized (repositories)
- {
- if (repositories.remove(rep.getId()) != null)
- {
- order.remove(rep);
- type = Type.REMOVED;
- clearLevel(rep);
- resetLevels();
- }
- }
-
- if (type != null)
- {
- notifyListeners(new RepositoryChangeEvent(rep, type));
- }
- }
-
private void clearLevel(IBundleRepository rep)
{
for (Iterator<Map.Entry<Integer, HashSet<IBundleRepository>>> iter = levelMap.entrySet().iterator(); iter.hasNext();)