Author: jm
Date: 2012-06-28 13:59:16 -0700 (Thu, 28 Jun 2012)
New Revision: 29719

Modified:
   
core3/api/trunk/service-api/src/main/java/org/cytoscape/service/util/internal/utils/RegisterUtil.java
Log:
Fixes #1179: getAllInterfaces no longer returns duplicates

Modified: 
core3/api/trunk/service-api/src/main/java/org/cytoscape/service/util/internal/utils/RegisterUtil.java
===================================================================
--- 
core3/api/trunk/service-api/src/main/java/org/cytoscape/service/util/internal/utils/RegisterUtil.java
       2012-06-28 18:56:10 UTC (rev 29718)
+++ 
core3/api/trunk/service-api/src/main/java/org/cytoscape/service/util/internal/utils/RegisterUtil.java
       2012-06-28 20:59:16 UTC (rev 29719)
@@ -36,8 +36,11 @@
 package org.cytoscape.service.util.internal.utils;
 
 
+import java.util.ArrayList;
+import java.util.HashSet;
 import java.util.List;
-import java.util.ArrayList;
+import java.util.Set;
+
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -49,12 +52,12 @@
        private static final Logger logger = 
LoggerFactory.getLogger(RegisterUtil.class);
 
        public static List<Class<?>> getAllInterfaces(Class<?> clazz) {
-               List<Class<?>> interfaces = new ArrayList<Class<?>>();
+               Set<Class<?>> interfaces = new HashSet<Class<?>>();
                addAllInterfaces(interfaces, clazz);
-               return interfaces;
+               return new ArrayList<Class<?>>(interfaces);
        }
        
-       private static void addAllInterfaces(List<Class<?>> list, Class<?> 
clazz) {
+       private static void addAllInterfaces(Set<Class<?>> list, Class<?> 
clazz) {
                for ( Class<?> c : clazz.getInterfaces() ) {
                        list.add(c);
                        addAllInterfaces(list, c);

-- 
You received this message because you are subscribed to the Google Groups 
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/cytoscape-cvs?hl=en.

Reply via email to