Fix the builder.

2 files changed, 10 insertions, 4 deletions

diff --git a/src/main/java/com/beust/jcommander/JCommander.java b/src/main/java/com/beust/jcommander/JCommander.java
index db38d95..fd14432 100644
--- a/src/main/java/com/beust/jcommander/JCommander.java
+++ b/src/main/java/com/beust/jcommander/JCommander.java
@@ -968,6 +968,10 @@ public class JCommander {
         getConsole().println(sb.toString());
     }
 
+    public static Builder newBuilder() {
+        return new Builder();
+    }
+
     static class Builder {
         private JCommander jCommander = new JCommander();
         private String[] args = null;
@@ -1084,11 +1088,10 @@ public class JCommander {
         }
 
         public JCommander build() {
-            JCommander result = new JCommander();
             if (args != null) {
-                result.parse(args);
+                jCommander.parse(args);
             }
-            return result;
+            return jCommander;
         }
     }
 
diff --git a/src/test/java/com/beust/jcommander/JCommanderTest.java b/src/test/java/com/beust/jcommander/JCommanderTest.java
index 5b015fd..ac61d8c 100644
--- a/src/test/java/com/beust/jcommander/JCommanderTest.java
+++ b/src/test/java/com/beust/jcommander/JCommanderTest.java
@@ -1368,7 +1368,10 @@ public class JCommanderTest {
     }
 
     Parameters args = new Parameters();
-    new JCommander(args, new String[]{"--mv", "from", "to"});
+    JCommander.newBuilder()
+            .addObject(args)
+            .args(new String[]{"--mv", "from", "to"})
+            .build();
 
     Assert.assertNotNull(args.mvParameters);
     Assert.assertEquals(args.mvParameters.from, "from");