From: Eli Zaretskii <eliz@gnu.org>
To: Michael Heerdegen <michael_heerdegen@web.de>
Cc: 17392@debbugs.gnu.org
Subject: bug#17392: 24.3.90; cursor blinks faster and faster
Date: Sat, 10 May 2014 14:06:38 +0300 [thread overview]
Message-ID: <83ppjlvqo1.fsf@gnu.org> (raw)
In-Reply-To: <8761ld3ota.fsf@web.de>
> From: Michael Heerdegen <michael_heerdegen@web.de>
> Date: Sat, 10 May 2014 12:34:09 +0200
> Cc: 17392@debbugs.gnu.org
>
> In emacs -Q, I did
>
> --8<---------------cut here---------------start------------->8---
> (progn
> (advice-add 'timer-event-handler :before
> (lambda (timer)
> (unless (or (memq timer timer-list)
> (memq timer timer-idle-list))
> (message "This should not happen"))))
> (defun provoke (&rest _)
> (run-with-idle-timer .001 nil
> (lambda () (sit-for 2.))))
> (defadvice handle-switch-frame (after provoke activate)
> (provoke)))
> --8<---------------cut here---------------end--------------->8---
>
> After switching frames a bit, I get "This should not happen". So an
> idle blinking timer not in timer-idle-list is called from C with
> timer-event-handler. At this point, there is already another blinking
> timer in timer-idle-list. timer-event-handler pushes the zombie timer
> to timer-idle-list as well, so then we have two of them there etc.
See how timers are run by keyboard.c: we first make a copy of the
timers' list, and then work on that copy. The comment there says:
/* We use copies of the timers' lists to allow a timer to add itself
again, without locking up Emacs if the newly added timer is
already ripe when added. */
Can this implementation detail explain what you see?
Btw, I don't understand what you say here, it sounds a contradiction:
> So an idle blinking timer not in timer-idle-list is called from C
> with timer-event-handler. At this point, there is already another
> blinking timer in timer-idle-list.
So is there a blinking time in timer-idle-list, or isn't there? The
first sentence seems to say there isn't, but then the next sentence
says there is.
next prev parent reply other threads:[~2014-05-10 11:06 UTC|newest]
Thread overview: 60+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-03 1:44 bug#17392: 24.3.90; cursor blinks faster and faster Michael Heerdegen
2014-05-03 18:33 ` Michael Heerdegen
2014-05-03 22:29 ` Michael Heerdegen
2014-05-05 12:25 ` Stefan Monnier
2014-05-05 13:14 ` Lennart Borgman
2014-05-05 20:17 ` Michael Heerdegen
2014-05-05 23:07 ` Michael Heerdegen
2014-05-05 23:30 ` Michael Heerdegen
2014-05-06 21:20 ` Michael Heerdegen
2014-05-07 2:02 ` Stefan Monnier
2014-05-07 2:14 ` Michael Heerdegen
2014-05-07 13:15 ` Stefan Monnier
2014-05-08 16:52 ` Michael Heerdegen
2014-05-08 17:04 ` Eli Zaretskii
2014-05-08 17:09 ` Lennart Borgman
2014-05-08 17:42 ` Michael Heerdegen
2014-05-08 18:40 ` Lennart Borgman
2014-05-10 10:34 ` Michael Heerdegen
2014-05-10 11:06 ` Eli Zaretskii [this message]
2014-05-10 11:40 ` Michael Heerdegen
2014-05-10 12:32 ` Eli Zaretskii
2014-05-10 16:16 ` Eli Zaretskii
2014-05-11 11:09 ` Michael Heerdegen
2014-05-11 16:40 ` Eli Zaretskii
2014-05-13 16:04 ` Michael Heerdegen
2014-05-13 17:50 ` Eli Zaretskii
2014-05-13 17:58 ` Lennart Borgman
2014-05-13 18:14 ` Eli Zaretskii
2014-05-13 18:25 ` Lennart Borgman
2014-05-13 18:44 ` Eli Zaretskii
2014-05-13 18:50 ` Lennart Borgman
2014-05-13 19:47 ` Eli Zaretskii
2014-05-13 20:10 ` Lennart Borgman
2014-05-14 2:40 ` Eli Zaretskii
2014-05-13 20:22 ` Stefan Monnier
2014-05-14 2:43 ` Eli Zaretskii
2014-05-14 14:59 ` Eli Zaretskii
2014-05-14 15:36 ` Michael Heerdegen
2014-05-14 16:00 ` Michael Heerdegen
2014-05-14 16:13 ` Eli Zaretskii
2014-05-14 17:23 ` Michael Heerdegen
2014-05-14 18:15 ` Eli Zaretskii
2014-05-14 19:11 ` Lennart Borgman
2014-05-14 19:30 ` Eli Zaretskii
2014-05-14 20:17 ` Lennart Borgman
2014-05-15 2:40 ` Eli Zaretskii
2014-05-15 3:31 ` Lennart Borgman
2014-05-15 17:04 ` Eli Zaretskii
2014-05-15 19:11 ` Lennart Borgman
2014-05-15 19:17 ` Eli Zaretskii
2014-05-15 20:22 ` Lennart Borgman
2014-05-16 5:30 ` Eli Zaretskii
2014-05-16 8:41 ` Lennart Borgman
2014-05-15 20:34 ` Glenn Morris
2014-05-15 20:44 ` Lennart Borgman
2014-05-07 2:27 ` Michael Heerdegen
2014-05-07 3:01 ` Michael Heerdegen
2014-05-07 13:57 ` Nicolas Richard
2014-05-18 13:05 ` Stefan Monnier
2014-05-18 13:41 ` Michael Heerdegen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=83ppjlvqo1.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=17392@debbugs.gnu.org \
--cc=michael_heerdegen@web.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.