You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by da...@apache.org on 2015/10/12 18:28:01 UTC
svn commit: r1708157 -
/felix/trunk/framework/src/main/java/org/apache/felix/framework/HookRegistry.java
Author: davidb
Date: Mon Oct 12 16:28:00 2015
New Revision: 1708157
URL: http://svn.apache.org/viewvc?rev=1708157&view=rev
Log:
FELIX-5075 Framework hooks not re-sorted on service ranking update
Added slightly better comment
Modified:
felix/trunk/framework/src/main/java/org/apache/felix/framework/HookRegistry.java
Modified: felix/trunk/framework/src/main/java/org/apache/felix/framework/HookRegistry.java
URL: http://svn.apache.org/viewvc/felix/trunk/framework/src/main/java/org/apache/felix/framework/HookRegistry.java?rev=1708157&r1=1708156&r2=1708157&view=diff
==============================================================================
--- felix/trunk/framework/src/main/java/org/apache/felix/framework/HookRegistry.java (original)
+++ felix/trunk/framework/src/main/java/org/apache/felix/framework/HookRegistry.java Mon Oct 12 16:28:00 2015
@@ -161,7 +161,7 @@ public class HookRegistry
{
TreeSet<ServiceReference<?>> newHooks = new TreeSet<ServiceReference<?>>(Collections.reverseOrder());
for (ServiceReference<?> hook : hooks) {
- newHooks.add(hook); // addAll() does not re-sort
+ newHooks.add(hook); // copy constructor / addAll() does not re-sort
}
m_allHooks.put(serviceName, newHooks);