A little more code cleanup #716

sleberknight · 2022-04-22T01:09:36Z

Remove unnecessary semicolon in KillTimeoutAction enum
Convert a 'var' (which was hiding the wrapper type Integer) to int
(IntelliJ found that, not me...)
Replace inefficient conversion of OutputStream to a String in
JSchSlf4jLoggerTest. Instead of getting the byte array and then
calling String's constructor, there is a toString(Charset) method
that was added to OutputStream in JDK 10 to directly convert to
a String from the stream. This avoids the overhead of doing an
OutputStream#toByteArray, which performs an array copy operation.

* Remove unnecessary semicolon in KillTimeoutAction enum * Convert a 'var' (which was hiding the wrapper type Integer) to int (IntelliJ found that, not me...) * Replace inefficient conversion of OutputStream to a String in JSchSlf4jLoggerTest. Instead of getting the byte array and then calling String's constructor, there is a toString(Charset) method that was added to OutputStream in JDK 10 to directly convert to a String from the stream. This avoids the overhead of doing an OutputStream#toByteArray, which performs an array copy operation.