[mina-sshd] 02/02: Removed duplicate code from SftpFileSystemProvider#resolveSftpVersionSelector

[lgoldstein]

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

lgoldstein pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mina-sshd.git

commit 38fc6c45bccdd24a3938d075011a0c52bb554312
Author: Lyor Goldstein <lgoldst...@apache.org>
AuthorDate: Thu Nov 26 17:20:51 2020 +0200

    Removed duplicate code from 
SftpFileSystemProvider#resolveSftpVersionSelector
---
 .../sshd/sftp/client/fs/SftpFileSystemProvider.java     | 17 +++--------------
 1 file changed, 3 insertions(+), 14 deletions(-)

diff --git 
a/sshd-sftp/src/main/java/org/apache/sshd/sftp/client/fs/SftpFileSystemProvider.java
 
b/sshd-sftp/src/main/java/org/apache/sshd/sftp/client/fs/SftpFileSystemProvider.java
index f4b62b6..7c3e9c1 100644
--- 
a/sshd-sftp/src/main/java/org/apache/sshd/sftp/client/fs/SftpFileSystemProvider.java
+++ 
b/sshd-sftp/src/main/java/org/apache/sshd/sftp/client/fs/SftpFileSystemProvider.java
@@ -300,25 +300,14 @@ public class SftpFileSystemProvider extends 
FileSystemProvider {
             log.debug("resolveSftpVersionSelector({}) preference={}", uri, 
preference);
         }
 
+        // These are aliases for shorter parameters specification
         if ("max".equalsIgnoreCase(preference)) {
             return SftpVersionSelector.MAXIMUM;
         } else if ("min".equalsIgnoreCase(preference)) {
             return SftpVersionSelector.MINIMUM;
-        } else if ("current".equalsIgnoreCase(preference)) {
-            return SftpVersionSelector.CURRENT;
-        }
-
-        String[] values = GenericUtils.split(preference, ',');
-        if (values.length == 1) {
-            return 
SftpVersionSelector.fixedVersionSelector(Integer.parseInt(values[0]));
-        }
-
-        int[] preferred = new int[values.length];
-        for (int index = 0; index < values.length; index++) {
-            preferred[index] = Integer.parseInt(values[index]);
+        } else {
+            return SftpVersionSelector.resolveVersionSelector(preference);
         }
-
-        return SftpVersionSelector.preferredVersionSelector(preferred);
     }
 
     // NOTE: URI parameters override environment ones