[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: bookmark.el and lisp/gnus/gnus-bookmark.el
From: |
Karl Fogel |
Subject: |
Re: bookmark.el and lisp/gnus/gnus-bookmark.el |
Date: |
Fri, 07 Mar 2008 21:43:53 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (gnu/linux) |
Stefan Monnier <address@hidden> writes:
> I think we should do as I suggest above: a single function of no
> arguments that is called first that returns a bookmark record, including
> a suggested name (which can be nil so as to use the default heuristic
> to decide the default bookmark name).
>
> This has another advantage: since the function is called first, it can
> signal an error if the bookmark can't be created (e.g. if
> buffer-file-name is nil), so we can drop the (or (local-variable-p ...)
> (bookmark-buffer-file-name) (error ...)) test which is currently forced
> to guess whether the actual bookmark construction function will work.
I think this way is fine too. I almost proposed it in my first
response, in fact -- well, something close to it. What I'd suggest is
that `bookmark-make-record-function's value return an alist:
((`name' . NAME) (`record' . RECORD) ...)
That's extensible, and if you don't want to supply a name, just leave
out that alist element entirely.
-Karl
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, (continued)
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Bastien, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Karl Fogel, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Bastien Guerry, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Karl Fogel, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Bastien, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Bastien Guerry, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Stefan Monnier, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Stefan Monnier, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Bastien, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Stefan Monnier, 2008/03/07
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el,
Karl Fogel <=
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Bastien, 2008/03/08
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Stefan Monnier, 2008/03/08
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Bastien, 2008/03/08
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Stefan Monnier, 2008/03/08
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Karl Fogel, 2008/03/09
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Reiner Steib, 2008/03/08
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Bastien, 2008/03/08
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, Bastien, 2008/03/07