* void-variable gnus-face-properties-alist
@ 2006-04-15 19:06 Reiner Steib
2006-04-16 7:27 ` Lars Magne Ingebrigtsen
0 siblings, 1 reply; 3+ messages in thread
From: Reiner Steib @ 2006-04-15 19:06 UTC (permalink / raw)
Hi,
in the German-speaking Gnus newsgroup, I got the following report from
Wolfgang Bauer.
When displaying an article containing a Face header, Wolfgang got the
backtrace:
--8<---------------cut here---------------start------------->8---
Debugger entered--Lisp error: (void-variable gnus-face-properties-alist)
article-display-face()
apply(article-display-face nil)
(if interactive (call-interactively (quote article-display-face))
(apply (quote article-display-face) args))
(save-excursion (set-buffer gnus-article-buffer) (if interactive
(call-interactively ...) (apply ... args)))
gnus-article-display-face()
gnus-treat-article(head)
gnus-display-mime()
gnus-article-prepare-display()
gnus-article-prepare(23029 nil)
gnus-summary-display-article(23029 nil)
gnus-summary-select-article(nil nil pseudo)
gnus-summary-scroll-up(1)
* call-interactively(gnus-summary-scroll-up)
--8<---------------cut here---------------end--------------->8---
AFAICS, `article-display-face' uses `gnus-face-properties-alist', but
`gnus-fun' is not loaded:
--8<---------------cut here---------------start------------->8---
(eval-when-compile
(defvar gnus-face-properties-alist))
(defun article-display-face ()
"Display any Face headers in the header."
(interactive)
[...]
(apply 'gnus-create-image png 'png t
(cdr (assq 'png gnus-face-properties-alist))))
--8<---------------cut here---------------end--------------->8---
Should we add (require 'gnus-fun) inside the defun or move the
defcustom to `gnus-art.el'? (Or something else?)
Bye, Reiner.
--
,,,
(o o)
---ooO-(_)-Ooo--- | PGP key available | http://rsteib.home.pages.de/
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: void-variable gnus-face-properties-alist
2006-04-15 19:06 void-variable gnus-face-properties-alist Reiner Steib
@ 2006-04-16 7:27 ` Lars Magne Ingebrigtsen
2006-04-16 7:30 ` Lars Magne Ingebrigtsen
0 siblings, 1 reply; 3+ messages in thread
From: Lars Magne Ingebrigtsen @ 2006-04-16 7:27 UTC (permalink / raw)
Reiner Steib <reinersteib+gmane@imap.cc> writes:
> Should we add (require 'gnus-fun) inside the defun or move the
> defcustom to `gnus-art.el'? (Or something else?)
I don't really like having `require's in functions, but in this case,
I guess it's the right thing to do. I'll commit the change.
--
(domestic pets only, the antidote for overdose, milk.)
larsi@gnus.org * Lars Magne Ingebrigtsen
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: void-variable gnus-face-properties-alist
2006-04-16 7:27 ` Lars Magne Ingebrigtsen
@ 2006-04-16 7:30 ` Lars Magne Ingebrigtsen
0 siblings, 0 replies; 3+ messages in thread
From: Lars Magne Ingebrigtsen @ 2006-04-16 7:30 UTC (permalink / raw)
Lars Magne Ingebrigtsen <larsi@gnus.org> writes:
>> Should we add (require 'gnus-fun) inside the defun or move the
>> defcustom to `gnus-art.el'? (Or something else?)
>
> I don't really like having `require's in functions, but in this case,
> I guess it's the right thing to do. I'll commit the change.
I changed my mind. I moved the defcustom to gnus-art.el.
--
(domestic pets only, the antidote for overdose, milk.)
larsi@gnus.org * Lars Magne Ingebrigtsen
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-04-16 7:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-04-15 19:06 void-variable gnus-face-properties-alist Reiner Steib
2006-04-16 7:27 ` Lars Magne Ingebrigtsen
2006-04-16 7:30 ` Lars Magne Ingebrigtsen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).