On Wed, 2011-06-08 at 06:56 -0400, Martyn Taylor wrote:
Ack, Config Parser works fine, the only issue with it is that it
throws a horrible exception when an invalid option is supplied. But
I'm sure we can catch that at a later date.
Structure looks fine, nice modular code.
Good Stuff,
Cheers
Martyn
Good catch, guess I was so focused on getting all the flags covered, I
forgot the case where one was bad :) I just replaced this:
@optparse.parse!(@args)
with this:
begin
@optparse.parse!(@args)
rescue OptionParser::InvalidOption
puts "Warning: Invalid option"
exit(0)
end
Happy to resend if you like, otherwise I'll add this (plus a test) and
push shortly.
-j