Re: authinfo encryption through netrc.el works now

[Simon Josefsson <jas@extundo.com>]

> +(defun password-read-and-add (prompt &optional key)
> +  "Read password, for use with KEY, from user, or from cache if wanted.
> +Then store the password in the cache.  Uses `password-read' and
> +`password-cache-add'."
> +  (let ((password (password-read prompt key)))
> +    (when (and password key)
> +      (password-cache-add key password))
> +    password))

FWIW:

The point of separating reading the password and caching it, was that
code using password.el would typically look like:

1. Read password.
2. Try to authenticate against server, invoke GPG using the password, etc.
3. If authentication failed, goto 1.
4. Cache the password.

If the password is cached directly, and if the user type the wrong
password, that logic wouldn't work, and it would infloop steps 1, 2,
and 3.

I realize that perhaps a better logic would be:

1. Read and cache password.
2. Try to authenticate against server, invoke GPG using the password, etc.
3. If authentication failed, clear password, and goto 1.

And if the gnus-encrypt/netrc stuff use this logic, then fine, and
ignore this message.