Merge pull request #259 from DevFactory/release/collection-isempty-fix-1

Code Quality Improvement - Collection.isEmpty() should be used to test for emptiness

Author: nkzawa

src/main/java/io/socket/client/Manager.java

@@ -416,7 +416,7 @@ public void call(Object... objects) {
 
     /*package*/ void destroy(Socket socket) {
         this.connected.remove(socket);
-        if (this.connected.size() > 0) return;
+        if (!this.connected.isEmpty()) return;
 
         this.close();
     }
@@ -447,7 +447,7 @@ public void call(Object[] encodedPackets) {
     }
 
     private void processPacketQueue() {
-        if (this.packetBuffer.size() > 0 && !this.encoding) {
+        if (!this.packetBuffer.isEmpty() && !this.encoding) {
             Packet pack = this.packetBuffer.remove(0);
             this.packet(pack);
         }

src/main/java/io/socket/client/Socket.java

@@ -325,7 +325,7 @@ private void onevent(Packet<JSONArray> packet) {
         }
 
         if (this.connected) {
-            if (args.size() == 0) return;
+            if (args.isEmpty()) return;
             String event = args.remove(0).toString();
             super.emit(event, args.toArray());
         } else {