[overlord-commits] Overlord SVN: r835 - sam/trunk/sam/examples/twitter/src/main/java/org/jboss/sam/examples/twitter.

[overlord-commits at lists.jboss.org]

Author: heiko.braun at jboss.com
Date: 2009-09-10 04:18:00 -0400 (Thu, 10 Sep 2009)
New Revision: 835

Modified:
   sam/trunk/sam/examples/twitter/src/main/java/org/jboss/sam/examples/twitter/InMemorySwapHandler.java
Log:
Switch to Queue interfaces

Modified: sam/trunk/sam/examples/twitter/src/main/java/org/jboss/sam/examples/twitter/InMemorySwapHandler.java
===================================================================
--- sam/trunk/sam/examples/twitter/src/main/java/org/jboss/sam/examples/twitter/InMemorySwapHandler.java	2009-09-09 13:54:53 UTC (rev 834)
+++ sam/trunk/sam/examples/twitter/src/main/java/org/jboss/sam/examples/twitter/InMemorySwapHandler.java	2009-09-10 08:18:00 UTC (rev 835)
@@ -25,6 +25,7 @@
 import org.jboss.sam.cvt.SwapHandler;
 
 import java.util.LinkedList;
+import java.util.Queue;
 
 /**
  * @author Heiko.Braun <heiko.braun at jboss.com>
@@ -32,7 +33,7 @@
 public class InMemorySwapHandler implements SwapHandler
 {
   
-  private LinkedList<CVTSnapshot> buffer = new LinkedList<CVTSnapshot>();
+  private Queue<CVTSnapshot> buffer = new LinkedList<CVTSnapshot>();
 
   private final static int MAX_SNAPSHOTS = 50;
 
@@ -48,15 +49,10 @@
   {
     synchronized(buffer)
     {
-      if(buffer.size()<MAX_SNAPSHOTS)
-      {
-        buffer.addLast(snapshot);
-      }
-      else
-      {
-        buffer.removeFirst(); // first is the oldest
-        buffer.addLast(snapshot);
-      }
+      if(buffer.size()>=MAX_SNAPSHOTS) // retain max capacity
+        buffer.poll();
+
+      buffer.add(snapshot);
     }
   }