Clean up code

Author: warriordog

.idea/vcs.xml

@@ -21,5 +21,15 @@
         <SOURCES />
       </library>
     </orderEntry>
+    <orderEntry type="module-library" scope="TEST">
+      <library name="JUnit4">
+        <CLASSES>
+          <root url="jar://$APPLICATION_HOME_DIR$/lib/junit-4.12.jar!/" />
+          <root url="jar://$APPLICATION_HOME_DIR$/lib/hamcrest-core-1.3.jar!/" />
+        </CLASSES>
+        <JAVADOC />
+        <SOURCES />
+      </library>
+    </orderEntry>
   </component>
 </module>

LoginLogs.iml

@@ -4,7 +4,6 @@
 
 import java.io.BufferedReader;
 import java.io.IOException;
-import java.io.Reader;
 import java.io.Writer;
 import java.util.*;
 import java.util.logging.Logger;
@@ -15,25 +14,11 @@ public class PlayerList {
     private final List<LLPlayer> playerList = new ArrayList<>();
     private final Map<String, LLPlayer> uuidMap = new HashMap<>();
     private final Map<String, LLPlayer> nameMap = new HashMap<>();
-    //private final LLPlayer[] recentLogins = new LLPlayer[MAX_LOGINS];
     private final List<LLPlayer> loginList = new ArrayList<>(MAX_LOGINS + 1);
-    //private final Set<LLPlayer> recentPlayers = new HashSet<>();
     private final Logger logger;
-    //private final Comparator<LLPlayer> playerSorter;
 
     public PlayerList(Logger logger) {
         this.logger = logger;
-        /*
-        playerSorter = (o1, o2) -> {
-            if (o1 != null && o2 != null) {
-                return (int)o1.getLoginTime() - (int)o2.getLoginTime();
-            } else if (o1 == null) {
-                return -1; //null goes below not null
-            } else {
-                return 1; //not null goes above null
-            }
-        };
-        */
     }
 
     public void load(BufferedReader in) {
@@ -116,13 +101,5 @@ public void updateRecentLogins(LLPlayer player) {
         while (loginList.size() > MAX_LOGINS) {
             loginList.remove(loginList.size() - 1);
         }
-        /*if (!recentPlayers.contains(player)) {
-            recentPlayers.add(player);
-            LLPlayer last = recentLogins[MAX_LOGINS - 1];
-            recentPlayers.remove(last);
-            recentLogins[MAX_LOGINS - 1] = player;
-            Arrays.sort(recentLogins, playerSorter);
-        }
-        */
     }
 }