[jboss-svn-commits] JBL Code SVN: r21502 - in labs/jbosslabs/trunk/stats-server/sources/kosmos: src/java/hu/midori/kosmos/server and 1 other directory.

[jboss-svn-commits at lists.jboss.org]

Author: wrzep
Date: 2008-08-12 23:52:46 -0400 (Tue, 12 Aug 2008)
New Revision: 21502

Modified:
   labs/jbosslabs/trunk/stats-server/sources/kosmos/project.xml
   labs/jbosslabs/trunk/stats-server/sources/kosmos/src/java/hu/midori/kosmos/server/AbstractKosmosService.java
Log:
fixed kosmos slow rendering issue


Modified: labs/jbosslabs/trunk/stats-server/sources/kosmos/project.xml
===================================================================
--- labs/jbosslabs/trunk/stats-server/sources/kosmos/project.xml	2008-08-13 01:53:00 UTC (rev 21501)
+++ labs/jbosslabs/trunk/stats-server/sources/kosmos/project.xml	2008-08-13 03:52:46 UTC (rev 21502)
@@ -41,7 +41,12 @@
 		    <groupId>javax.servlet</groupId>
 		    <artifactId>servlet-api</artifactId>
 		    <version>2.4</version>
-	</dependency>     	
+	</dependency>    
+	<dependency>
+	        <groupId>org.tmate</groupId>
+	            <artifactId>javasvn</artifactId>
+	                <version>1.0.3</version>
+    </dependency>   
     </dependencies>
     <build>
         <sourceDirectory>src/java</sourceDirectory>

Modified: labs/jbosslabs/trunk/stats-server/sources/kosmos/src/java/hu/midori/kosmos/server/AbstractKosmosService.java
===================================================================
--- labs/jbosslabs/trunk/stats-server/sources/kosmos/src/java/hu/midori/kosmos/server/AbstractKosmosService.java	2008-08-13 01:53:00 UTC (rev 21501)
+++ labs/jbosslabs/trunk/stats-server/sources/kosmos/src/java/hu/midori/kosmos/server/AbstractKosmosService.java	2008-08-13 03:52:46 UTC (rev 21502)
@@ -16,6 +16,7 @@
 import java.io.InputStream;
 import java.util.Date;
 import java.util.List;
+import java.util.ArrayList;
 import java.util.Locale;
 import java.util.ResourceBundle;
 
@@ -174,9 +175,23 @@
      * huge network transfers and unuseable web pages.
      */
     protected List trimList(List data) {
-        if((MAX_LIST_SIZE != -1) && (data.size() > MAX_LIST_SIZE))
-            return data.subList(0, MAX_LIST_SIZE);
-
+        if((MAX_LIST_SIZE != -1) && (data.size() > MAX_LIST_SIZE)) {
+        	
+        	List ret = new ArrayList();
+        	int i = 0;
+        	
+        	for (Object object : data) {
+				ret.add(data.get(i)); // Hope all passed lists perform get(i) in O(1)
+				i++;
+				if (i == MAX_LIST_SIZE)
+					return ret;
+			}
+        	
+        	//return data.subList(0, MAX_LIST_SIZE);
+        		// This returns a *view* of the portion of the list, hence does NOT solve the problem.
+        		// See AbstractList's javadoc.
+        }
+            
         return data;
     }
 }