You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by gs...@apache.org on 2010/05/28 20:39:35 UTC

svn commit: r949255 - /mahout/trunk/core/src/main/java/org/apache/mahout/common/AbstractJob.java

Author: gsingers
Date: Fri May 28 18:39:34 2010
New Revision: 949255

URL: http://svn.apache.org/viewvc?rev=949255&view=rev
Log:
change the private scope, add another option for required, but no default value

Modified:
    mahout/trunk/core/src/main/java/org/apache/mahout/common/AbstractJob.java

Modified: mahout/trunk/core/src/main/java/org/apache/mahout/common/AbstractJob.java
URL: http://svn.apache.org/viewvc/mahout/trunk/core/src/main/java/org/apache/mahout/common/AbstractJob.java?rev=949255&r1=949254&r2=949255&view=diff
==============================================================================
--- mahout/trunk/core/src/main/java/org/apache/mahout/common/AbstractJob.java (original)
+++ mahout/trunk/core/src/main/java/org/apache/mahout/common/AbstractJob.java Fri May 28 18:39:34 2010
@@ -82,8 +82,13 @@ public abstract class AbstractJob extend
   protected static Option buildOption(String name, String shortName, String description, String defaultValue) {
     return buildOption(name, shortName, description, false, defaultValue);
   }
+
+  protected static Option buildOption(String name, String shortName, String description,
+                                      boolean required) {
+    return buildOption(name, shortName, description, required, null);
+  }
   
-  private static Option buildOption(String name,
+  protected static Option buildOption(String name,
                                     String shortName,
                                     String description,
                                     boolean required,