Code Improvement: collection itteration, invoke performance methods, remove unnecesssary constructor

src/main/java/net/spy/memcached/MemcachedClient.java

@@ -1758,7 +1758,7 @@ public void receivedStatus(OperationStatus s) {
           // XXX: Potential abstraction leak.
           // The handling of incr/decr in the binary protocol
           // Allows us to avoid string processing.
-          rv.set(new Long(s.isSuccess() ? s.getMessage() : "-1"));
+          rv.set(Long.valueOf(s.isSuccess() ? s.getMessage() : "-1"));
         }
 
         @Override
@@ -1993,7 +1993,7 @@ private OperationFuture<Long> asyncMutate(Mutator m, String key, long by,
         new OperationCallback() {
           @Override
           public void receivedStatus(OperationStatus s) {
-            rv.set(new Long(s.isSuccess() ? s.getMessage() : "-1"), s);
+            rv.set(Long.valueOf(s.isSuccess() ? s.getMessage() : "-1"), s);
           }
 
           @Override

src/main/java/net/spy/memcached/compat/SpyObject.java

@@ -32,13 +32,6 @@ public class SpyObject extends Object {
 
   private transient Logger logger = null;
 
-  /**
-   * Get an instance of SpyObject.
-   */
-  public SpyObject() {
-    super();
-  }
-
   /**
    * Get a Logger instance for this class.
    *

src/main/java/net/spy/memcached/protocol/binary/MultiGetOperationImpl.java

@@ -72,7 +72,7 @@ protected int addKey(String k) {
       bkeys.put(rv, KeyUtil.getKeyBytes(k));
       rkeys.put(k, rv);
       synchronized (vbmap) {
-        vbmap.put(k, new Short((short) 0));
+        vbmap.put(k, Short.valueOf((short) 0));
       }
     }
     return rv;

src/main/java/net/spy/memcached/tapmessage/RequestMessage.java

@@ -179,9 +179,9 @@ public ByteBuffer getBytes() {
     }
     if (hasVBucketCheckpoints) {
       bb.putShort((short)vBucketCheckpoints.size());
-      for (Short vBucket : vBucketCheckpoints.keySet()) {
-        bb.putShort(vBucket);
-        bb.putLong(vBucketCheckpoints.get(vBucket));
+      for (Map.Entry<Short, Long> vBucket : vBucketCheckpoints.entrySet()){
+    	bb.putShort(vBucket.getKey());
+    	bb.putLong(vBucket.getValue());
       }
     }