Re: [PATCH] Add option to ERC to make all connections use SSL.

[John Zorn <jzornn@gmail.com>]

On Sat, 2 Aug 2014 23:17:13 +0530, 
Vibhav Pant <vibhavp@gmail.com> wrote:
> Agreed. I have redone the patch and added some more features:
> 1. erc-ssl-servers: Alist of servers (ADDRESS PORT) for which ERC will
> always connect to using SSL.
> 2. erc-use-ssl: If non-nill, all connections will be made using SSL.
> 3. erc-ssl-blacklist: Alist of server ERC will connect to without SSL.
> Only useful when erc-use-ssl is non-nil.
>
> erc-ssl-servers is also used to compute the initial port number
> displayed on the minibuffer during user input when ERC is run
> interactively.
>

All this extra code and complexity, for what exactly?
You said it yourself, `erc-tls' is already there,
it's even interactive!

If you think it's not "prominent enough", one can certainly
make it more prominent instead of producing the tangled
mess that is your patch. Similar situations have
led to ERC being bloated and messy as hell.

John