|
From: | Stefan Monnier |
Subject: | bug#12107: 24.1.50; In Info mode, add the name of the manual as completion candidate for "bookmark-set" |
Date: | Wed, 01 Aug 2012 20:10:04 -0400 |
User-agent: | Gnus/5.13 (Gnus v5.13) Emacs/24.1.50 (gnu/linux) |
> - (default (car record))) > + (defaults (bookmark-prop-get record 'defaults)) > + (default (if (consp defaults) (car defaults) defaults))) > + > + (if defaults > + ;; Don't store default values in the record. > + (setq record (assq-delete-all 'defaults record)) > + ;; When no defaults in the record, use its first element. > + (setq defaults (car record) default defaults)) Is there a good reason why we can't use a list in (car record) and avoid using a new `defaults' field? If so, please add a comment explaining it. Stefan
[Prev in Thread] | Current Thread | [Next in Thread] |