bug#17439: 24.3.50; run-with-idle-timer runs on focus-out

[Stefan Monnier <monnier@iro.umontreal.ca>]

> +    (unless (and (consp last-input-event)
> +		 (symbolp (car last-input-event))
> +		 (memq (car last-input-event)

Aka (unless (memq (car-safe last-input-event)

> +		       '(
> +			 ;; `run-with-idle-timer' generates the `focus-out'

Huh?  `run-with-idle-timer' is not supposed to generate any event.

> +			 ;; event on switching out from the current frame,
> +			 ;; but `raise-frame' brings the same frame back,
> +			 ;; so ignore `focus-out' here.
> +			 focus-out

Running mouse avoidance after focus-out does not seem very useful, so we
should filter out focus-out elsewhere than here.

> +			 ;; Some window managers generate the `focus-in' event
> +			 ;; when showing the Window List,

What means "showing the Window List"?

> +			 ;;  but `raise-frame' forcibly switches to an Emacs frame when the Window List is active,

I don't know what this is talking about either.

> +			 ;; so `focus-in' needs to be ignored here too.

Shouldn't `switch-frame' be ignored for similar reasons, then?


        Stefan