From: markt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Date: Wed, 24 Dec 2008 16:35:46 +0000 (+0000)
Subject: Some more simple generics and other Eclipse warnings
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=5c336fb7a0b0dfd1bacfa2449eecbde40eda110a;p=tomcat7.0

Some more simple generics and other Eclipse warnings

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@729332 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java b/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java
index c75f90a1c..34e7e92bd 100644
--- a/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java
+++ b/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java
@@ -260,14 +260,14 @@ public abstract class AbstractReplicatedMap extends ConcurrentHashMap implements
         }
         //update our map of members, expire some if we didn't receive a ping back
         synchronized (mapMembers) {
-            Iterator it = mapMembers.entrySet().iterator();
+            Iterator<Map.Entry<Member, Long>> it = mapMembers.entrySet().iterator();
             long now = System.currentTimeMillis();
             while ( it.hasNext() ) {
-                Map.Entry entry = (Map.Entry)it.next();
-                long access = ((Long)entry.getValue()).longValue(); 
+                Map.Entry<Member,Long> entry = it.next();
+                long access = entry.getValue().longValue(); 
                 if ( (now - access) > timeout ) {
                     it.remove();
-                    memberDisappeared( (Member) entry.getKey());
+                    memberDisappeared(entry.getKey());
                 }
             }
         }//synch
@@ -377,7 +377,7 @@ public abstract class AbstractReplicatedMap extends ConcurrentHashMap implements
         MapEntry entry = (MapEntry)super.get(key);
         if ( entry == null ) return;
         if ( !entry.isSerializable() ) return;
-        if (entry != null && entry.isPrimary() && entry.getBackupNodes()!= null && entry.getBackupNodes().length > 0) {
+        if (entry.isPrimary() && entry.getBackupNodes()!= null && entry.getBackupNodes().length > 0) {
             Object value = entry.getValue();
             //check to see if we need to replicate this object isDirty()||complete
             boolean repl = complete || ( (value instanceof ReplicatedMapEntry) && ( (ReplicatedMapEntry) value).isDirty());
@@ -1055,10 +1055,6 @@ public abstract class AbstractReplicatedMap extends ConcurrentHashMap implements
             return counter;
         }
     
-        protected boolean removeEldestEntry(Map.Entry eldest) {
-            return false;
-        }
-    
         public boolean isEmpty() {
             return size()==0;
         }