Re: Problem with the latest CVS build

[Jan Vroonhof <vroonhof@math.ethz.ch>]

Hrvoje Niksic <hniksic@iskon.hr> writes:

> What does this code do in nnheaderxmas.el, then:

<To play the devels-advocate:> It is duplicating code from timer.el

> (defun nnheader-xmas-run-at-time (time repeat function &rest args)
>   (start-itimer
>    "nnheader-run-at-time"
>    `(lambda ()
>       (,function ,@args))
>    time repeat))
> 
> (fset 'nnheader-run-at-time 'nnheader-xmas-run-at-time)
> (fset 'nnheader-cancel-timer 'delete-itimer)
> 
> AFAIK, Gnus has *never* required `timer.el' under XEmacs, probably
> because one hasn't existed.

This isn't supporting "itimer" at all. This is writing your own emulation
library. I don't see how that is preferable over using timer.el apart
from that we can then bitch a Lars if there are bugs.

We need namespaces...

</To play the devels-advocate>

>From gnus-async.el

	  (if (not (fboundp 'run-with-idle-timer))
	      ;; This is either an older Emacs or XEmacs, so we
	      ;; do this, which leads to slightly slower article
	      ;; buffer display.
	      (gnus-async-prefetch-article group next summary)
	    (run-with-idle-timer
	     0.1 nil 'gnus-async-prefetch-article group next summary)))))))

Note that you need a 21.1.9 for it to actually work because itimer is
broken (still lives in Emacs 18 times to much).

Jan