Hi,
In mail "Re: Proposed patch to add SSL support to net/pop.rb"
Daniel Hobe <daniel / nightrunner.com> wrote:
> > pop = Net::POP3.APOP($is_apop)
> > pop = pop.witH_ssl(certs, verify) if $use_ssl
> > pop.start(address, port, account, password) do |pop|
> > ...
> > end
>
> I like this solution best, but that's still a few more lines than the way it
> is now. I don't see why adding a few more arguments to the method is such a
> big deal. It is totally backward compatible and in the normal non-SSL case
> users won't even know they are there.
Because it is not a good design that a method have 7 parameters.
It is hard to write, hard to read. I cannot remember the order of
parameters...
Regards,
Minero Aoki