* free variable new-handle
@ 2001-06-05 9:30 Alex Schroeder
2001-06-05 18:47 ` Simon Josefsson
0 siblings, 1 reply; 2+ messages in thread
From: Alex Schroeder @ 2001-06-05 9:30 UTC (permalink / raw)
The following looks like a minor bug in current cvs:
While compiling mm-handle-set-external-undisplayer in file /home/alex/elisp/gnus/lisp/mm-decode.el:
** assignment to free variable new-handle
** reference to free variable new-handle
Something like the following should do just as fine, I think.
Untested.
(defun mm-handle-set-external-undisplayer (handle function)
"Set the undisplayer for this handle; postpone undisplaying of viewers
for types in mm-keep-viewer-alive-types."
(if (mm-keep-viewer-alive-p handle)
(let ((new-handle (copy-sequence handle)))
(mm-handle-set-undisplayer new-handle function)
(mm-handle-set-undisplayer handle nil)
(push new-handle mm-postponed-undisplay-list))
(mm-handle-set-undisplayer handle function)))
Alex.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: free variable new-handle
2001-06-05 9:30 free variable new-handle Alex Schroeder
@ 2001-06-05 18:47 ` Simon Josefsson
0 siblings, 0 replies; 2+ messages in thread
From: Simon Josefsson @ 2001-06-05 18:47 UTC (permalink / raw)
Cc: ding
Alex Schroeder <alex@gnu.org> writes:
> Something like the following should do just as fine, I think.
Me too. I committed it.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2001-06-05 18:47 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-06-05 9:30 free variable new-handle Alex Schroeder
2001-06-05 18:47 ` Simon Josefsson
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).