splitting and trimming for list parameters

cbeust · Apr 8, 2015 · 2f76521 · 2f76521
1 parent f700d5a
commit 2f76521
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/com/beust/jcommander/JCommander.java b/src/main/java/com/beust/jcommander/JCommander.java
@@ -362,7 +362,7 @@ private String[] expandArgs(String[] originalArgv) {
 			if (isOption(v1, arg2)) {
 				String sep = getSeparatorFor(v1, arg2);
 				if (" ".equals(sep)) {
-					String[] sp = arg2.split("[" + sep + "]", 2);
+					String[] sp = arg2.split("[" + sep + "]+",2);
 					for (String ssp : sp) {
 						splitFileArgs.add(ssp);
 					}
@@ -388,7 +388,7 @@ private String[] expandArgs(String[] originalArgv) {
       if (isOption(v1, arg)) {
         String sep = getSeparatorFor(v1, arg);
         if (! " ".equals(sep)) {
-          String[] sp = arg.split("[" + sep + "]", 2);
+          String[] sp = arg.split("[" + sep + "]+", 2);
           for (String ssp : sp) {
             vResult2.add(ssp);
           }

diff --git a/src/test/java/com/beust/jcommander/args/ArgsList.java b/src/test/java/com/beust/jcommander/args/ArgsList.java
@@ -41,7 +41,7 @@ public List<String> convert(String value) {
   public static class SemiColonSplitter implements IParameterSplitter {
 
     public List<String> split(String value) {
-      return Arrays.asList(value.split(";"));
+      return Arrays.asList(value.split("\\s*;\\s*"));
     }
 
   }