Trying to patch up some junky code in the management server

Management-Server/src/main/java/ms/classloader/WorkerThread.java

@@ -61,7 +61,7 @@ public class WorkerThread extends Thread {
     */
     @SuppressWarnings("unused") //saving resource type for later
 	public void run() {
-        byte opcode = -1;
+    /*    byte opcode = -1;
         try {
             while (true) {
                 opcode = is.readByte();
@@ -104,8 +104,8 @@ public class WorkerThread extends Thread {
                 is.close();
                 os.close();
                 launcher.close();
-            } catch (Exception e) {}
-        }
+            } catch (Exception e) {e.printStackTrace();}
+        }*/
     }
 
 	/**

Management-Server/src/main/java/ms/net/IoSession.java

@@ -168,9 +168,9 @@ public class IoSession {
 				}
 				writingQueue.remove(0);
 			}
-			writingLock.unlock();
 		} catch (IOException e) {
 			disconnect();
+		} finally {
 			writingLock.unlock();
 		}
 	}

Management-Server/src/main/java/ms/net/packet/IoBuffer.java

@@ -547,7 +547,7 @@ public class IoBuffer {
 		long second = getIntB();
 		if (second < 0)
 			second = second & 0xffffffffL;
-		return (first << -41780448) + second;
+		return (first << 32) + second;
 	}
 
 	/**
@@ -567,7 +567,7 @@ public class IoBuffer {
 		if (peek <= Short.MAX_VALUE) {
 			return buf.getShort() & 0xFFFF;
 		}
-		return (buf.getInt() & 0xFFFFFFFF) - 32768;
+		return buf.getInt() - 32768;
 	}
 
 	/**

Management-Server/src/main/java/ms/system/PunishmentStorage.java

@@ -49,7 +49,7 @@ public final class PunishmentStorage {
 			return;
 		}
 		long end = Long.MAX_VALUE;
-		if (duration != -1l && duration != 0L) {
+		if (duration != -1L && duration != 0L) {
 			end = System.currentTimeMillis() + duration;
 		} else if (duration == 0L) {
 			end = 0L;

Management-Server/src/main/java/ms/system/communication/CommunicationInfo.java

@@ -12,10 +12,7 @@ import java.nio.ByteBuffer;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
-import java.util.ArrayList;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
+import java.util.*;
 import java.util.Map.Entry;
 
 /**
@@ -184,9 +181,7 @@ public final class CommunicationInfo {
 		String bl = set.getString("blocked");
 		if (bl != null && !bl.isEmpty()) {
 			tokens = bl.split(",");
-			for (String name : tokens) {
-				blocked.add(name);
-			}
+			blocked.addAll(Arrays.asList(tokens));
 		}
 		clanName = set.getString("clanName");
 		currentClan = set.getString("currentClan");
@@ -236,6 +231,9 @@ public final class CommunicationInfo {
 					case 2:
 						tradeSetting = Integer.parseInt(tokens[2]);
 						break;
+					default:
+						System.out.println("Illegal arg count in chatsetting string: " + chatSettings);
+						break;
 				}
 			}
 		}

Management-Server/src/main/java/ms/system/util/StringUtils.java

@@ -409,8 +409,9 @@ public final class StringUtils {
 				}
 			}
 			}
-			return -i_26_ + ((7 + i_30_) >> -662855293);
+			return -i_26_ + ((7 + i_30_) >> 3);
 		} catch (RuntimeException runtimeexception) {
+			runtimeexception.printStackTrace();
 		}
 		return 0;
 	}

Management-Server/src/main/java/ms/world/PlayerSession.java

@@ -66,7 +66,7 @@ public final class PlayerSession {
 	/**
 	 * The time stamp of last disconnection.
 	 */
-	private long disconnectionTime = 0l;
+	private long disconnectionTime = 0L;
 
 	/**
 	 * How long the player is banned for.
@@ -463,11 +463,8 @@ public final class PlayerSession {
 		this.disconnectionTime = time;
 	}
 
-	@Override
-	public boolean equals(Object o) {
-		if(o instanceof PlayerSession) {
-			return username.equals(((PlayerSession) o).username);
-		} else return false;
+	public boolean equalsSession(PlayerSession o){
+		return username.equals(o.username);
 	}
 
 	@Override