args4j icon indicating copy to clipboard operation
args4j copied to clipboard

Fix deprecation in SampleMain.java

Open ghost opened this issue 7 years ago • 2 comments

@sfuhrm , this is the updated version of my previous pull request. I had to create a new one because I deleted the repository I used.

I removed CmdLineException and matched the formatting of argument.isEmpty() to the other if statements.

ghost avatar Jan 26 '17 23:01 ghost

Removing the help message on parsing error was a mistake, thanks for correcting that.

The second change depends on whether the program should run given no arguments. However, programs that use args4j will likely require some arguments to run correctly. I agree with this change as well.

ghost avatar Jan 28 '17 20:01 ghost

Thanks for fixing the deprecations!

sfuhrm avatar Jan 29 '17 20:01 sfuhrm