cleanup code

Author: dota17

checkstyle-suppressions.xml

@@ -7,4 +7,8 @@
 <suppressions>
   <suppress checks="Javadoc" files="."/>
   <suppress checks="MagicNumberCheck" files="."/>
+  <suppress checks="PackageName" files="."/>
+  <suppress checks="AbbreviationAsWordInName" files="."/>
+  <suppress checks="LineLength" files="."/>
+  <suppress checks="VariableDeclarationUsageDistance" files="."/>
 </suppressions>

src/main/java/org/java_websocket/SSLSocketChannel2.java

@@ -22,6 +22,7 @@
  *  FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
  *  OTHER DEALINGS IN THE SOFTWARE.
  */
+
 package org.java_websocket;
 
 import java.io.EOFException;
@@ -275,10 +276,10 @@ public int write(ByteBuffer src) throws IOException {
       processHandshake();
       return 0;
     }
-    // assert ( bufferallocations > 1 ); //see #190
-    //if( bufferallocations <= 1 ) {
-    //	createBuffers( sslEngine.getSession() );
-    //}
+    // assert(bufferallocations > 1); // see #190
+    // if(bufferallocations <= 1) {
+    //   createBuffers(sslEngine.getSession());
+    // }
     int num = socketChannel.write(wrap(src));
     if (writeEngineResult.getStatus() == SSLEngineResult.Status.CLOSED) {
       throw new EOFException("Connection is closed");
@@ -311,10 +312,11 @@ public int read(ByteBuffer dst) throws IOException {
           }
         }
       }
-      // assert ( bufferallocations > 1 ); //see #190
-      //if( bufferallocations <= 1 ) {
-      //	createBuffers( sslEngine.getSession() );
-      //}
+      // assert(bufferallocations > 1); // see #190
+      // if (bufferallocations <= 1) {
+      //   createBuffers(sslEngine.getSession());
+      // }
+
       /* 1. When "dst" is smaller than "inData" readRemaining will fill "dst" with data decoded in a previous read call.
        * 2. When "inCrypt" contains more data than "inData" has remaining space, unwrap has to be called on more time(readRemaining)
        */

src/main/java/org/java_websocket/SocketChannelIOHelper.java

@@ -107,7 +107,7 @@ public static boolean batch(WebSocketImpl ws, ByteChannel sockchannel) throws IO
     }
 
     if (ws.outQueue.isEmpty() && ws.isFlushAndClose() && ws.getDraft() != null
-        && ws.getDraft().getRole() != null && ws.getDraft().getRole() == Role.SERVER) {//
+        && ws.getDraft().getRole() != null && ws.getDraft().getRole() == Role.SERVER) {
       ws.closeConnection();
     }
     return c == null || !((WrappedByteChannel) sockchannel).isNeedWrite();

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

@@ -498,7 +498,7 @@ public void run() {
       ostream = socket.getOutputStream();
 
       sendHandshake();
-    } catch ( /*IOException | SecurityException | UnresolvedAddressException | InvalidHandshakeException | ClosedByInterruptException | SocketTimeoutException */Exception e) {
+    } catch (/*IOException | SecurityException | UnresolvedAddressException | InvalidHandshakeException | ClosedByInterruptException | SocketTimeoutException */Exception e) {
       onWebsocketError(engine, e);
       engine.closeConnection(CloseFrame.NEVER_CONNECTED, e.getMessage());
       return;

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

@@ -522,12 +522,12 @@ private Framedata translateSingleFrame(ByteBuffer buffer)
     int maxpacketsize = buffer.remaining();
     int realpacketsize = 2;
     translateSingleFrameCheckPacketSize(maxpacketsize, realpacketsize);
-    byte b1 = buffer.get( /*0*/);
+    byte b1 = buffer.get(/*0*/);
     boolean fin = b1 >> 8 != 0;
     boolean rsv1 = (b1 & 0x40) != 0;
     boolean rsv2 = (b1 & 0x20) != 0;
     boolean rsv3 = (b1 & 0x10) != 0;
-    byte b2 = buffer.get( /*1*/);
+    byte b2 = buffer.get(/*1*/);
     boolean mask = (b2 & -128) != 0;
     int payloadlength = (byte) (b2 & ~(byte) 128);
     Opcode optcode = toOpcode((byte) (b1 & 15));
@@ -548,7 +548,7 @@ private Framedata translateSingleFrame(ByteBuffer buffer)
       byte[] maskskey = new byte[4];
       buffer.get(maskskey);
       for (int i = 0; i < payloadlength; i++) {
-        payload.put((byte) (buffer.get( /*payloadstart + i*/) ^ maskskey[i % 4]));
+        payload.put((byte) (buffer.get(/*payloadstart + i*/) ^ maskskey[i % 4]));
       }
     } else {
       payload.put(buffer.array(), buffer.position(), payload.limit());
@@ -599,15 +599,15 @@ private TranslatedPayloadMetaData translateSingleFramePayloadLength(ByteBuffer b
       realpacketsize += 2; // additional length bytes
       translateSingleFrameCheckPacketSize(maxpacketsize, realpacketsize);
       byte[] sizebytes = new byte[3];
-      sizebytes[1] = buffer.get( /*1 + 1*/);
-      sizebytes[2] = buffer.get( /*1 + 2*/);
+      sizebytes[1] = buffer.get(/*1 + 1*/);
+      sizebytes[2] = buffer.get(/*1 + 2*/);
       payloadlength = new BigInteger(sizebytes).intValue();
     } else {
       realpacketsize += 8; // additional length bytes
       translateSingleFrameCheckPacketSize(maxpacketsize, realpacketsize);
       byte[] bytes = new byte[8];
       for (int i = 0; i < 8; i++) {
-        bytes[i] = buffer.get( /*1 + i*/);
+        bytes[i] = buffer.get(/*1 + i*/);
       }
       long length = new BigInteger(bytes).longValue();
       translateSingleFrameCheckLengthLimit(length);
@@ -735,7 +735,8 @@ public List<Framedata> translateFrame(ByteBuffer buffer) throws InvalidDataExcep
         }
       }
 
-      while (buffer.hasRemaining()) {// Read as much as possible full frames
+      // Read as much as possible full frames
+      while (buffer.hasRemaining()) {
         buffer.mark();
         try {
           cur = translateSingleFrame(buffer);

src/main/java/org/java_websocket/extensions/permessage_deflate/PerMessageDeflateExtension.java

@@ -123,14 +123,14 @@ public void decodeFrame(Framedata inputFrame) throws InvalidDataException {
     try {
       decompress(inputFrame.getPayloadData().array(), output);
 
-            /*
-                If a message is "first fragmented and then compressed", as this project does, then the inflater
-                    can not inflate fragments except the first one.
-                This behavior occurs most likely because those fragments end with "final deflate blocks".
-                We can check the getRemaining() method to see whether the data we supplied has been decompressed or not.
-                And if not, we just reset the inflater and decompress again.
-                Note that this behavior doesn't occur if the message is "first compressed and then fragmented".
-             */
+      /*
+          If a message is "first fragmented and then compressed", as this project does, then the inflater
+              can not inflate fragments except the first one.
+          This behavior occurs most likely because those fragments end with "final deflate blocks".
+          We can check the getRemaining() method to see whether the data we supplied has been decompressed or not.
+          And if not, we just reset the inflater and decompress again.
+          Note that this behavior doesn't occur if the message is "first compressed and then fragmented".
+       */
       if (inflater.getRemaining() > 0) {
         inflater = new Inflater(true);
         decompress(inputFrame.getPayloadData().array(), output);
@@ -199,12 +199,12 @@ public void encodeFrame(Framedata inputFrame) {
     byte[] outputBytes = output.toByteArray();
     int outputLength = outputBytes.length;
 
-        /*
-            https://tools.ietf.org/html/rfc7692#section-7.2.1 states that if the final fragment's compressed
-                payload ends with 0x00 0x00 0xff 0xff, they should be removed.
-            To simulate removal, we just pass 4 bytes less to the new payload
-                if the frame is final and outputBytes ends with 0x00 0x00 0xff 0xff.
-        */
+    /*
+        https://tools.ietf.org/html/rfc7692#section-7.2.1 states that if the final fragment's compressed
+            payload ends with 0x00 0x00 0xff 0xff, they should be removed.
+        To simulate removal, we just pass 4 bytes less to the new payload
+            if the frame is final and outputBytes ends with 0x00 0x00 0xff 0xff.
+     */
     if (inputFrame.isFin()) {
       if (endsWithTail(outputBytes)) {
         outputLength -= TAIL_BYTES.length;