Patches welcome! :-) Gary
On Sun, Apr 5, 2015 at 4:13 AM, Romain Manni-Bucau <rmannibu...@gmail.com> wrote: > +1 (from tomee community as a user) > Le 5 avr. 2015 12:42, "David Goodenough" <david.goodeno...@btconnect.com> > 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: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > -- E-Mail: garydgreg...@gmail.com | ggreg...@apache.org 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
