Patches welcome! :-) Gary
On Sun, Apr 5, 2015 at 4:13 AM, Romain Manni-Bucau <[email protected]> wrote: > +1 (from tomee community as a user) > Le 5 avr. 2015 12:42, "David Goodenough" <[email protected]> > a > écrit : > > > Currently the code in org.apache.commons.cli.OptionValidator generates an > > IllegalArgumentException which contains the message:- > > > > "illegal option value '" + ch + "'" > > > > This is generated in two places, one for single character options and one > > for > > multi character options. > > > > However this does not tell the caller which option it is contains this > > character, which makes debugging the error without the source (i.e most > > users) extremely difficult. > > > > Would it not be possible to change the message to:- > > > > "illegal option '" + opt + "' value '" + ch + "'" > > > > which would then at least identify the offending option. > > > > David > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > > > -- E-Mail: [email protected] | [email protected] Java Persistence with Hibernate, Second Edition <http://www.manning.com/bauer3/> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/> Spring Batch in Action <http://www.manning.com/templier/> Blog: http://garygregory.wordpress.com Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory
