procyberian/emacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

; Add a new TODO item

Browse source 
* etc/TODO: Add a new entry for :set-after.
* lisp/startup.el: Related comment.
This commit is contained in:
Glenn Morris 2017-02-20 18:08:48 -08:00
parent bfd1afb3da
commit 08c73cdc6e
2 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
etc/TODO
View file

@ -968,6 +968,17 @@ addition, toolkit builds create their menu bars in toolkit-specific
parts of code, bypassing xdisp.c, so those parts need to be enhanced
with toolkit-specific code to display the menu bar right to left.
** Custom
*** Extend :set-after to also mean initialize after.
If defcustom A specifies :set-after '(B), then if a user customizes
both A and B, custom will set A after B. But if the user only customizes
A, then if B is already defined, it gets left at its original setting.
Instead, if B has not been customized it should be re-initialized
(on the assumption that the default value depends on A).
See the places where we manually call custom-reevaluate-setting,
such as for mail-host-address and user-mail-address in startup.el.
** ImageMagick support
*** image-type-header-regexps priorities the jpeg loader over the

2
lisp/startup.el
View file

@ -1298,7 +1298,7 @@ the `--debug-init' option to view a complete error backtrace."
;; Yes, this is ugly, but slightly less so than leaving
;; user-mail-address uninitialized during init file processing.
;; Perhaps we should make :set-after do something like this?
;; Ie, extend it to also mean (re)initialize-after.
;; Ie, extend it to also mean (re)initialize-after. See etc/TODO.
(equal user-mail-address
(let (mail-host-address)
(ignore-errors