From: Daiki Ueno <ueno@unixuser.org>
To: Stefan Monnier <monnier@iro.umontreal.ca>
Cc: simon@josefsson.org, rms@gnu.org, emacs-devel@gnu.org
Subject: Re: Moving files from lisp/gnus/ to lisp/net/?
Date: Sun, 11 Nov 2007 09:04:19 +0900 [thread overview]
Message-ID: <87hcjtfy0s.fsf@broken.deisui.org> (raw)
In-Reply-To: <jwvlk9550q8.fsf-monnier+emacs@gnu.org> (Stefan Monnier's message of "Sat, 10 Nov 2007 15:03:20 -0500")
Stefan Monnier <monnier@iro.umontreal.ca> writes:
>>> password.el itself is in lisp/gnus/. But we need a new function which
>>> does password caching, whose arglist is similar to read-passwd's, and
>>> whose name is longer than `read-passwd'.
>
>>> What about this patch?
>
>> Sorry for consecutive post but this might be better regarding argument
>> order.
>
> I think the docstring should make it very clear that the caller should
> be careful to remove the password from the cache in case it turns out
> not to be right.
> Or even make it more clear that this is not the thing to use because the
> password should only be added to the cache after checking its validity.
Agreed. I perhaps misunderstood Simon's arguments in the first post
correctly.
In that case, I think the most part of the existing API of password.el
is sufficient enough:
- password-read-from-cache KEY
Obtain passphrase for KEY from time-limited passphrase cache.
- password-cache-add KEY PASSWORD
Add password to cache.
- password-cache-remove KEY
Remove password indexed by KEY from password cache.
However, `password-read-and-add' and `password-read' look awkward as
interface function. Because `password-read-and-add' is not capable of
checking the validity of the password as you say, and `password-read'
has somewhat misleading name to use. In addition, they does not provide
a way to pass CONFIRM and DEFAULT args to `read-passwd' while they call
`read-passwd' internally.
- 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'.
- password-read PROMPT &optional KEY
Read password, for use with KEY, from user, or from cache if wanted.
How about just making these two functions obsolete?
Regards,
--
Daiki Ueno
next prev parent reply other threads:[~2007-11-11 0:04 UTC|newest]
Thread overview: 100+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-10-15 19:26 Moving files from lisp/gnus/ to lisp/net/? Simon Josefsson
2004-10-15 19:37 ` Simon Josefsson
2004-10-15 23:27 ` Miles Bader
2004-10-16 2:08 ` Ted Zlatanov
2004-10-16 13:52 ` Richard Stallman
2004-10-17 13:45 ` Simon Josefsson
2004-10-18 13:58 ` Richard Stallman
2004-10-24 12:24 ` Simon Josefsson
2004-10-25 13:13 ` Richard Stallman
2004-10-25 14:13 ` Simon Josefsson
2004-10-27 10:47 ` Richard Stallman
2004-10-27 19:08 ` Simon Josefsson
2004-10-29 4:32 ` Richard Stallman
2004-10-29 21:25 ` Simon Josefsson
2004-11-01 20:09 ` Ted Zlatanov
2004-11-01 20:31 ` Simon Josefsson
2004-11-02 18:30 ` Ted Zlatanov
2004-11-02 21:15 ` Stefan Monnier
2007-11-02 15:02 ` Richard Stallman
2007-11-05 10:16 ` Simon Josefsson
2007-11-06 8:38 ` Richard Stallman
2007-11-06 14:01 ` Daiki Ueno
2007-11-07 7:55 ` Richard Stallman
2007-11-08 0:46 ` Daiki Ueno
2007-11-08 17:40 ` Richard Stallman
[not found] ` <54a15d860711081633k364caf30r3b75464be8a060fa@mail.gmail.com>
2007-11-09 13:40 ` Fwd: " Daiki Ueno
[not found] ` <E1IqbtM-0003DR-P8@fencepost.gnu.org>
2007-11-10 4:53 ` Daiki Ueno
2007-11-10 7:07 ` Daiki Ueno
2007-11-10 20:03 ` Stefan Monnier
2007-11-11 0:04 ` Daiki Ueno [this message]
2007-11-11 19:33 ` Richard Stallman
2007-11-12 8:54 ` Daiki Ueno
2007-11-13 5:10 ` Richard Stallman
2007-11-14 3:45 ` Daiki Ueno
2007-11-14 5:44 ` Michael Albinus
2007-11-14 14:54 ` Stefan Monnier
2007-11-14 15:13 ` Michael Albinus
2007-11-14 17:30 ` Stefan Monnier
2007-11-15 3:08 ` Richard Stallman
2007-11-15 3:47 ` Stefan Monnier
2007-11-16 4:28 ` Richard Stallman
2007-11-15 0:23 ` Miles Bader
2007-11-15 3:08 ` Richard Stallman
2007-11-11 5:22 ` Richard Stallman
2007-11-10 19:53 ` Tom Tromey
2004-10-15 20:05 ` Miles Bader
2004-10-15 21:15 ` Simon Josefsson
2004-10-16 7:43 ` Thien-Thi Nguyen
2004-10-16 13:44 ` Miles Bader
2004-10-15 20:44 ` Stefan Monnier
2004-10-15 21:06 ` Simon Josefsson
2004-10-15 21:32 ` Lars Magne Ingebrigtsen
2004-10-15 21:47 ` Simon Josefsson
2004-10-15 21:52 ` Stefan Monnier
2004-10-18 1:04 ` Miles Bader
2004-10-18 13:59 ` Richard Stallman
2004-10-20 10:27 ` Lars Magne Ingebrigtsen
2004-10-20 10:50 ` Miles Bader
2004-10-20 11:22 ` Ralf Angeli
2004-10-20 11:39 ` Ralf Angeli
2004-10-21 1:09 ` Miles Bader
2004-10-20 11:32 ` Lars Magne Ingebrigtsen
2004-10-20 11:56 ` Karl Fogel
2004-10-20 12:38 ` David Kastrup
2004-10-20 13:06 ` Stefan Monnier
2004-10-20 12:34 ` David Kastrup
2004-10-21 1:28 ` Miles Bader
2004-10-21 5:03 ` Lars Magne Ingebrigtsen
2004-10-21 5:49 ` Miles Bader
2004-10-21 10:05 ` Kim F. Storm
2004-10-23 7:48 ` Lars Magne Ingebrigtsen
2004-10-25 1:02 ` Miles Bader
2004-10-25 10:14 ` Simon Josefsson
2004-10-25 17:46 ` Lars Magne Ingebrigtsen
2004-10-25 22:41 ` Miles Bader
2004-10-27 17:21 ` Lars Magne Ingebrigtsen
2004-10-28 2:21 ` Miles Bader
2004-10-26 7:28 ` Kai Grossjohann
2004-10-27 17:21 ` Lars Magne Ingebrigtsen
2004-10-16 13:52 ` Richard Stallman
2004-10-18 17:54 ` Ted Zlatanov
2004-10-18 21:36 ` Reiner Steib
2004-10-21 18:45 ` Ted Zlatanov
2004-10-22 0:25 ` Miles Bader
2004-10-24 12:13 ` Simon Josefsson
2004-10-24 12:20 ` Miles Bader
2004-10-24 13:41 ` Simon Josefsson
2004-10-24 22:18 ` Stefan
2004-10-25 10:08 ` Simon Josefsson
2004-10-25 11:55 ` Stefan
2004-10-25 13:07 ` Simon Josefsson
2004-10-26 9:05 ` Richard Stallman
2004-10-26 9:26 ` Simon Josefsson
2004-10-19 16:45 ` Richard Stallman
2004-10-21 18:45 ` Ted Zlatanov
2004-10-23 4:49 ` Richard Stallman
2004-10-24 12:59 ` Simon Josefsson
2004-10-24 13:05 ` Move sha1.el and hex-util.el from lisp/gnus/ to lisp/net/ (was: Re: Moving files from lisp/gnus/ to lisp/net/?) Simon Josefsson
2004-10-25 13:13 ` Richard Stallman
2004-10-25 14:23 ` Move sha1.el and hex-util.el from lisp/gnus/ to lisp/net/ Simon Josefsson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87hcjtfy0s.fsf@broken.deisui.org \
--to=ueno@unixuser.org \
--cc=emacs-devel@gnu.org \
--cc=monnier@iro.umontreal.ca \
--cc=rms@gnu.org \
--cc=simon@josefsson.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.