Re: [PATCH] OptionParser < Hash

Hi,

At Wed, 13 Sep 2006 19:40:04 +0900,
greg weber wrote in [ruby-core:08856]:
> This is great, Nobu!

Though I forgot to mention the usage, I'm glad you to see it.

> I added lazy switch creation with new(), so if a program is called
> without any arguments or exits before calling the parsing functions,
> it will run much faster.  Also, I pass the result hash into the new
> block if user desires to assign keys to it in addition to the
> defaults.

In usual, I write as:

  OptionParser.new do |opt|
    opt.on ...
    opt.parse!
  end

or:

  ARGV.options do |opt|
    opt.on ...
    opt.parse!
  end

There is no chance to run the block.

Also, the registration isn't restricted in one place.
Application and libraries can share an OptionParser instance,
especially ARGV.options.

> 758c758
> <     yield self if block_given?
> ---
> >     @setup = block_given? ? (lambda &block) : nil

It's a very roundabout way equivalent to `@setup = block'.

BTW, could you use unified diff?

-- 
Nobu Nakada