This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-net.git

commit 467711fcd8424c26600160a57bbeed104eae4924
Author: Sebb <s...@apache.org>
AuthorDate: Thu Jan 28 15:41:40 2021 +0000

    Tidy
    
    Array designator belongs with type
    One declaration per line
---
 .../commons/net/io/ToNetASCIIInputStream.java      |  4 ++--
 .../java/org/apache/commons/net/telnet/Telnet.java | 22 +++++++++++++---------
 2 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/src/main/java/org/apache/commons/net/io/ToNetASCIIInputStream.java 
b/src/main/java/org/apache/commons/net/io/ToNetASCIIInputStream.java
index 61d744f..3b453a8 100644
--- a/src/main/java/org/apache/commons/net/io/ToNetASCIIInputStream.java
+++ b/src/main/java/org/apache/commons/net/io/ToNetASCIIInputStream.java
@@ -107,7 +107,7 @@ public final class ToNetASCIIInputStream extends 
FilterInputStream
      *            stream.
      */
     @Override
-    public int read(final byte buffer[]) throws IOException
+    public int read(final byte[] buffer) throws IOException
     {
         return read(buffer, 0, buffer.length);
     }
@@ -128,7 +128,7 @@ public final class ToNetASCIIInputStream extends 
FilterInputStream
      *            stream.
      */
     @Override
-    public int read(final byte buffer[], int offset, int length) throws 
IOException
+    public int read(final byte[] buffer, int offset, int length) throws 
IOException
     {
         int ch;
         final int off;
diff --git a/src/main/java/org/apache/commons/net/telnet/Telnet.java 
b/src/main/java/org/apache/commons/net/telnet/Telnet.java
index 17658d8..cf82ad8 100644
--- a/src/main/java/org/apache/commons/net/telnet/Telnet.java
+++ b/src/main/java/org/apache/commons/net/telnet/Telnet.java
@@ -55,13 +55,17 @@ class Telnet extends SocketClient
                                           (byte)TelnetCommand.IAC, 
(byte)TelnetCommand.SE
                                       };
 
-    static final int WILL_MASK = 0x01, DO_MASK = 0x02,
-                                  REQUESTED_WILL_MASK = 0x04, 
REQUESTED_DO_MASK = 0x08;
+    static final int WILL_MASK = 0x01;
+    static final int DO_MASK = 0x02;
+    static final int REQUESTED_WILL_MASK = 0x04;
+    static final int REQUESTED_DO_MASK = 0x08;
 
     /* public */
     static final int DEFAULT_PORT =  23;
 
-    private final int[] doResponse, willResponse, options;
+    private final int[] doResponse;
+    private final int[] willResponse;
+    private final int[] options;
 
     /* TERMINAL-TYPE option (start)*/
     /**
@@ -96,7 +100,7 @@ class Telnet extends SocketClient
     /**
      * Array of option handlers
      */
-    private final TelnetOptionHandler optionHandlers[];
+    private final TelnetOptionHandler[] optionHandlers;
 
     /* open TelnetOptionHandler functionality (end)*/
 
@@ -272,7 +276,7 @@ class Telnet extends SocketClient
             {
                 optionHandlers[option].setWill(true);
 
-                final int subneg[] =
+                final int[] subneg =
                     optionHandlers[option].startSubnegotiationLocal();
 
                 if (subneg != null)
@@ -301,7 +305,7 @@ class Telnet extends SocketClient
             {
                 optionHandlers[option].setDo(true);
 
-                final int subneg[] =
+                final int[] subneg =
                     optionHandlers[option].startSubnegotiationRemote();
 
                 if (subneg != null)
@@ -684,7 +688,7 @@ class Telnet extends SocketClient
      * @param suboptionLength - length of data received
      * @throws IOException - Exception in I/O.
      **/
-    void processSuboption(final int suboption[], final int suboptionLength)
+    void processSuboption(final int[] suboption, final int suboptionLength)
     throws IOException
     {
         if (debug)
@@ -697,7 +701,7 @@ class Telnet extends SocketClient
         {
             if (optionHandlers[suboption[0]] != null)
             {
-                final int responseSuboption[] =
+                final int[] responseSuboption =
                   optionHandlers[suboption[0]].answerSubnegotiation(suboption,
                   suboptionLength);
                 _sendSubnegotiation(responseSuboption);
@@ -756,7 +760,7 @@ class Telnet extends SocketClient
      * @param subn - subnegotiation data to be sent
      * @throws IOException - Exception in I/O.
      **/
-    final synchronized void _sendSubnegotiation(final int subn[])
+    final synchronized void _sendSubnegotiation(final int[] subn)
     throws IOException
     {
         if (debug)

Reply via email to