[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#11447: 24.1.50; notifications-notify eats keystrokes
From: |
Michael Albinus |
Subject: |
bug#11447: 24.1.50; notifications-notify eats keystrokes |
Date: |
Sun, 13 May 2012 11:11:43 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1.50 (gnu/linux) |
Stefan Monnier <monnier@iro.umontreal.ca> writes:
> I think I understand what you mean: you mean that while you do `sit-for'
> you actually want to run the code associated with the incoming events,
> since that code might be the one which changes dbus-return-values-table.
Yes.
> So, indeed sit-for and input-pending-p won't cut it. Your code looks
> OK, then, though you don't need to use a 0.1 timeout, AFAICT.
OK, I remove the timeout.
> And in order to avoid the (let (unread-command-events) ...), I'd do
> something like:
>
> (let ((seen ()))
> (unwind-protect
> (let ((event (read-event)))
> (when (and event (not (ignore-errors (dbus-check-event event))))
> (push event seen)))
> (setq unread-command-events
> (append unread-command-events (nreverse seen)))))
Unfortunately, it doesn't work either. When unread-command-events is
non-nil, read-event takes the events from there instead of reading them
from the input streams. This happens after the first time a character
has been read in the loop, before the awaited D-Bus event did arrive.
I've committed my original patch (minus the read-event timeout). Bug is
closed.
> -- Stefan
Best regards, Michael.
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Peter Münster, 2012/05/10
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Michael Albinus, 2012/05/11
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Peter Münster, 2012/05/11
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Stefan Monnier, 2012/05/11
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Michael Albinus, 2012/05/11
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Stefan Monnier, 2012/05/11
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Michael Albinus, 2012/05/11
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Stefan Monnier, 2012/05/11
- bug#11447: 24.1.50; notifications-notify eats keystrokes,
Michael Albinus <=
- Message not available
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Michael Albinus, 2012/05/13
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Stefan Monnier, 2012/05/13
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Michael Albinus, 2012/05/13
- bug#11447: 24.1.50; notifications-notify eats keystrokes, Stefan Monnier, 2012/05/13