From: Daniel Hobe <daniel@...>
Date: 2004-04-03T04:42:59+09:00
Subject: Re: Proposed patch to add SSL support to net/pop.rb

I'll reimplement based on nobu's suggestion.

The reason I chose to just add more arguments was that this was the way
net/imap.rb did it.  Would it be useful to rework net/imap|smtp|pop so
that they all handle SSL the same way?

> 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
>
>
>


-- 
Daniel Hobe <daniel@nightrunner.com>