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 9d17fd7f7b9821a24638d0de2256a8b62092e0e0 Author: Sebb <s...@apache.org> AuthorDate: Thu Jan 28 15:42:09 2021 +0000 Simplify by inverting condition --- src/main/java/org/apache/commons/net/ftp/FTPClient.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/main/java/org/apache/commons/net/ftp/FTPClient.java b/src/main/java/org/apache/commons/net/ftp/FTPClient.java index f01c928..7490693 100644 --- a/src/main/java/org/apache/commons/net/ftp/FTPClient.java +++ b/src/main/java/org/apache/commons/net/ftp/FTPClient.java @@ -619,8 +619,7 @@ implements Configurable delim3 = reply.charAt(2); delim4 = reply.charAt(reply.length()-1); - if (!(delim1 == delim2) || !(delim2 == delim3) - || !(delim3 == delim4)) { + if ((delim1 != delim2) || (delim2 != delim3) || (delim3 != delim4)) { throw new MalformedServerReplyException( "Could not parse extended passive host information.\nServer Reply: " + reply); }