On 7/12/11 2:31 PM, [email protected] wrote:
+ Charset encoding = params.getEncoding();if (encoding == null) { System.out.println(getHelp()); throw new TerminateToolException(1); }
The case encoding == null will never happen, since encoding is a mandatory parameter. So these three lines should be removed. The call to validateArguments will return false if the encoding is not specified, and the call to ArgumentParser.parse will fail if the Charset cannot be created, e.g invalid name, not support on platform. Jörn
