[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: |
Stefan Monnier |
Subject: |
Re: bookmark.el and lisp/gnus/gnus-bookmark.el |
Date: |
Sat, 08 Mar 2008 18:20:30 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (gnu/linux) |
>>> If we get rid of the annotation argument for this single function, it
>>> means the annotation has to be done before this function returns sth
>>> that can be used as the suggested name for the bookmark.
>>
>> AFAICT getting rid of the annotation argument means nothing at all,
>> other than that the annotation will have to be added by the generic
>> bookmark-set code rather than being done by every single
>> bookmark-make-record-function.
> For now `bookmark-set' asks for an annotation (if required), sends it to
> `bookmark-make' which calls `bookmark-make-record-function' with the
> annotation as an argument.
> IIUC Tassilo suggested that the annotation should be directly handled by
> `bookmark-make-record-function', which seems correct to me. What you
> propose is to handle annotation in `bookmark-set' (or preferrably in
> `bookmark-make') -- this is _feasible_, but it looks weird to me since
> the annotation is really part of the record.
The record has various parts: a name to select it, an annotation,
a bunch of data to find the place, there could be more (date the record
was created, when it was last used, ...).
The bookmark-make-record-function should only concern itself with
providing the data to find the place. It may also provide a default
name, a default annotation, ... but it's better if it doesn't need to
handle annotations since those are (or rather should be) handled
identically for *all* bookmark-make-record-function.
Stefan
- Re: bookmark.el and lisp/gnus/gnus-bookmark.el, (continued)
- 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, 2008/03/07
- 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 <=
- 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