Code cleanups

Author: marci4

src/main/java/org/java_websocket/AbstractWrappedByteChannel.java

@@ -30,9 +30,6 @@
 import java.nio.channels.ByteChannel;
 import java.nio.channels.SocketChannel;
 
-import javax.net.ssl.SSLException;
-
-
 public class AbstractWrappedByteChannel implements WrappedByteChannel {
 
 	private final ByteChannel channel;

src/main/java/org/java_websocket/WrappedByteChannel.java

@@ -29,8 +29,6 @@
 import java.nio.ByteBuffer;
 import java.nio.channels.ByteChannel;
 
-import javax.net.ssl.SSLException;
-
 public interface WrappedByteChannel extends ByteChannel {
 	/**
 	 * returns whether writeMore should be called write additional data.

src/main/java/org/java_websocket/client/WebSocketClient.java

@@ -249,7 +249,7 @@ public void run() {
 			}
 
 			// if the socket is set by others we don't apply any TLS wrapper
-			if (isNewSocket && uri.getScheme().equals("wss")) {
+			if (isNewSocket && "wss".equals( uri.getScheme())) {
 
 				SSLContext sslContext = SSLContext.getInstance("TLS");
 				sslContext.init(null, null, null);
@@ -292,7 +292,7 @@ private int getPort() {
 		int port = uri.getPort();
 		if( port == -1 ) {
 			String scheme = uri.getScheme();
-			if( scheme.equals( "wss" ) ) {
+			if( "wss".equals( scheme ) ) {
 				return WebSocket.DEFAULT_WSS_PORT;
 			} else if( scheme.equals( "ws" ) ) {
 				return WebSocket.DEFAULT_PORT;

src/main/java/org/java_websocket/drafts/Draft.java

@@ -82,7 +82,7 @@ public enum CloseHandshakeType {
 
 	public static ByteBuffer readLine( ByteBuffer buf ) {
 		ByteBuffer sbuf = ByteBuffer.allocate( buf.remaining() );
-		byte prev = '0';
+		byte prev;
 		byte cur = '0';
 		while ( buf.hasRemaining() ) {
 			prev = cur;
@@ -191,7 +191,7 @@ public List<Framedata> continuousFrame( Opcode op, ByteBuffer buffer, boolean fi
 		try {
 			bui.isValid();
 		} catch ( InvalidDataException e ) {
-			throw new RuntimeException( e ); // can only happen when one builds close frames(Opcode.Close)
+			throw new IllegalArgumentException( e ); // can only happen when one builds close frames(Opcode.Close)
 		}
 		if( fin ) {
 			continuousFrameType = null;
@@ -216,7 +216,7 @@ public List<ByteBuffer> createHandshake( Handshakedata handshakedata, Role ownro
 		} else if( handshakedata instanceof ServerHandshake ) {
 			bui.append("HTTP/1.1 101 ").append(((ServerHandshake) handshakedata).getHttpStatusMessage());
 		} else {
-			throw new RuntimeException( "unknown role" );
+			throw new IllegalArgumentException( "unknown role" );
 		}
 		bui.append( "\r\n" );
 		Iterator<String> it = handshakedata.iterateHttpFields();

src/main/java/org/java_websocket/drafts/Draft_6455.java

@@ -267,7 +267,9 @@ public Framedata translateSingleFrame( ByteBuffer buffer ) throws IncompleteExce
 			throw new IncompleteException( realpacketsize );
 		byte b1 = buffer.get( /*0*/ );
 		boolean FIN = b1 >> 8 != 0;
-		boolean rsv1 = false, rsv2 = false, rsv3 = false;
+		boolean rsv1 = false;
+		boolean rsv2 = false;
+		boolean rsv3 = false;
 		if( ( b1 & 0x40 ) != 0 ) {
 			rsv1 = true;
 		}
@@ -372,7 +374,6 @@ public List<Framedata> translateFrame( ByteBuffer buffer ) throws InvalidDataExc
 					incompleteframe = null;
 				} catch ( IncompleteException e ) {
 					// extending as much as suggested
-					int oldsize = incompleteframe.limit();
 					ByteBuffer extendedframe = ByteBuffer.allocate( checkAlloc( e.getPreferredSize() ) );
 					assert ( extendedframe.limit() > incompleteframe.limit() );
 					incompleteframe.rewind();
@@ -460,7 +461,7 @@ private String generateFinalKey( String in ) {
 		try {
 			sh1 = MessageDigest.getInstance( "SHA1" );
 		} catch ( NoSuchAlgorithmException e ) {
-			throw new RuntimeException( e );
+			throw new IllegalStateException( e );
 		}
 		return Base64.encodeBytes( sh1.digest( acc.getBytes() ) );
 	}
@@ -488,7 +489,7 @@ else if( opcode == Framedata.Opcode.PING )
 			return 9;
 		else if( opcode == Framedata.Opcode.PONG )
 			return 10;
-		throw new RuntimeException( "Don't know how to handle " + opcode.toString() );
+		throw new IllegalArgumentException( "Don't know how to handle " + opcode.toString() );
 	}
 
 

src/main/java/org/java_websocket/framing/Framedata.java

@@ -27,8 +27,6 @@
 
 import java.nio.ByteBuffer;
 
-import org.java_websocket.exceptions.InvalidFrameException;
-
 public interface Framedata {
 	/**
 	 * Enum which contains the different valid opcodes

src/main/java/org/java_websocket/framing/FramedataImpl1.java

@@ -27,10 +27,8 @@
 
 import org.java_websocket.exceptions.InvalidDataException;
 import org.java_websocket.util.ByteBufferUtils;
-import org.java_websocket.util.Charsetfunctions;
 
 import java.nio.ByteBuffer;
-import java.util.Arrays;
 
 public abstract class FramedataImpl1 implements Framedata {
 

src/main/java/org/java_websocket/server/DefaultSSLWebSocketServerFactory.java

@@ -25,7 +25,6 @@
 
 package org.java_websocket.server;
 import java.io.IOException;
-import java.net.Socket;
 import java.nio.channels.ByteChannel;
 import java.nio.channels.SelectionKey;
 import java.nio.channels.SocketChannel;

src/main/java/org/java_websocket/server/WebSocketServer.java

@@ -30,7 +30,6 @@
 import java.net.ServerSocket;
 import java.net.Socket;
 import java.nio.ByteBuffer;
-import java.nio.channels.ByteChannel;
 import java.nio.channels.CancelledKeyException;
 import java.nio.channels.ClosedByInterruptException;
 import java.nio.channels.SelectableChannel;