Title: [1319] trunk/qdox/src/main/java/com/thoughtworks/qdox/model: Use Collection for ClassLibrary.getJavaClasses(), ClassLibrary.getJavaSources() and ClassLibrary.getJavaPackages() as we cannot guarantee the order.
Revision
1319
Author
rfscholte
Date
2011-09-10 10:46:08 -0500 (Sat, 10 Sep 2011)

Log Message

Use Collection for ClassLibrary.getJavaClasses(), ClassLibrary.getJavaSources() and ClassLibrary.getJavaPackages() as we cannot guarantee the order.

Modified Paths

Diff

Modified: trunk/qdox/src/main/java/com/thoughtworks/qdox/library/ClassLibrary.java (1318 => 1319)

--- trunk/qdox/src/main/java/com/thoughtworks/qdox/library/ClassLibrary.java	2011-09-10 15:33:35 UTC (rev 1318)
+++ trunk/qdox/src/main/java/com/thoughtworks/qdox/library/ClassLibrary.java	2011-09-10 15:46:08 UTC (rev 1319)
@@ -20,7 +20,7 @@
  */
 
 import java.io.Serializable;
-import java.util.List;
+import java.util.Collection;
 
 import com.thoughtworks.qdox.model.JavaClass;
 import com.thoughtworks.qdox.model.JavaPackage;
@@ -68,7 +68,7 @@
      * 
      * @return all JavaClasses as a List, never <code>null</code>
      */
-    List<JavaClass> getJavaClasses();
+    Collection<JavaClass> getJavaClasses();
     
     /**
      * Return all JavaSources of the current library.
@@ -76,7 +76,7 @@
      * 
      * @return all JavaSources as a List, never <code>null</code>
      */
-    List<JavaSource> getJavaSources();
+    Collection<JavaSource> getJavaSources();
     
     /**
      * Get the JavaPackage or null if it's not possible
@@ -92,5 +92,5 @@
      * 
      * @return all JavaPackages as a List, never <code>null</code>
      */
-    List<JavaPackage> getJavaPackages();
+    Collection<JavaPackage> getJavaPackages();
 }

Modified: trunk/qdox/src/main/java/com/thoughtworks/qdox/model/DefaultJavaPackage.java (1318 => 1319)

--- trunk/qdox/src/main/java/com/thoughtworks/qdox/model/DefaultJavaPackage.java	2011-09-10 15:33:35 UTC (rev 1318)
+++ trunk/qdox/src/main/java/com/thoughtworks/qdox/model/DefaultJavaPackage.java	2011-09-10 15:46:08 UTC (rev 1319)
@@ -131,7 +131,7 @@
 
     public List<JavaPackage> getSubPackages() {
         String expected = name + ".";
-        List<JavaPackage> jPackages = classLibrary.getJavaPackages();
+        Collection<JavaPackage> jPackages = classLibrary.getJavaPackages();
         List<JavaPackage> retList = new LinkedList<JavaPackage>();
         for (JavaPackage jPackage : jPackages) {
             String pName = jPackage.getName();


To unsubscribe from this list please visit:

http://xircles.codehaus.org/manage_email

Reply via email to