[jacli commit] r24 - in trunk/src/main/java/com/google/jacli: . compilers

[codesite...@google.com]

Author: maomaode
Date: Wed Jul 23 00:17:48 2008
New Revision: 24

Modified:
trunk/src/main/java/com/google/jacli/CommandLine.java
trunk/src/main/java/com/google/jacli/CommandLineParser.java
trunk/src/main/java/com/google/jacli/compilers/AbstractCompiler.java

Log:
* Support Enum type
* Fix the checks

Modified: trunk/src/main/java/com/google/jacli/CommandLine.java
==============================================================================
--- trunk/src/main/java/com/google/jacli/CommandLine.java (original)
+++ trunk/src/main/java/com/google/jacli/CommandLine.java Wed Jul 23
00:17:48 2008
@@ -41,7 +41,7 @@
usage.print(this);
}

- public void setPattern(Pattern s) {
+ public final void setPattern(Pattern s) {
this.pattern = s;
Pattern.setCommandLine(this);
}

Modified: trunk/src/main/java/com/google/jacli/CommandLineParser.java
==============================================================================
--- trunk/src/main/java/com/google/jacli/CommandLineParser.java (original)
+++ trunk/src/main/java/com/google/jacli/CommandLineParser.java Wed Jul
23 00:17:48 2008
@@ -18,8 +18,6 @@
CommandLine command = new CommandLine();
command.setBean(target);

- Class clz = target.getClass();
-
List<Field> fields = getAllDeclaredFields(target.getClass());

List<Field> fieldList = command.getFields();

Modified: trunk/src/main/java/com/google/jacli/compilers/AbstractCompiler.java
==============================================================================
---
trunk/src/main/java/com/google/jacli/compilers/AbstractCompiler.java (original)
+++
trunk/src/main/java/com/google/jacli/compilers/AbstractCompiler.java
Wed Jul 23 00:17:48 2008
@@ -123,6 +123,7 @@
return null;
}

+ @SuppressWarnings("unchecked")
protected Object getValue(final Class clz, final String value) {
if (clz == String.class) {
return value;
@@ -132,8 +133,11 @@
return Integer.parseInt(value);
} else if (clz == boolean.class) {
return Boolean.parseBoolean(value);
+ } else if (clz.isEnum()) {
+ return Enum.valueOf(clz, value);
}
- return null;
+
+ return value;
}

protected boolean hasOption(final Argument argument, final String
arg) {