all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Blinking cursor on w32
@ 2013-07-27 11:57 Eli Zaretskii
  2013-07-27 12:16 ` Juanma Barranquero
  0 siblings, 1 reply; 3+ messages in thread
From: Eli Zaretskii @ 2013-07-27 11:57 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: emacs-devel

It looks like the port to w32 of the latest changes related to
blinking cursor is incomplete.  If I put breakpoints on
handle-focus-in/out, and do the mouse gestures that switch focus to
and from the Emacs frame, I see handle-focus-in being called, but
handle-focus-out never is.  (On my system, just moving the mouse
pointer off the Emacs frames switches focus out of the frame; on other
systems, you'd actually need to click on some other frame.  But I
don't think this matters.)

The result is that setting blink-cursor-blinks to zero does not work
as intended: the timer continues to run even when none of Emacs frames
have focus.

The changes below fix this for me; OK to commit?

=== modified file 'src/w32term.c'
--- src/w32term.c	2013-07-26 09:59:59 +0000
+++ src/w32term.c	2013-07-27 11:48:35 +0000
@@ -4923,16 +4923,11 @@ w32_read_socket (struct terminal *termin
 	  break;
 
 	case WM_KILLFOCUS:
+	  w32_detect_focus_change (dpyinfo, &msg, &inev);
 	  f = x_top_window_to_frame (dpyinfo, msg.msg.hwnd);
 
           if (f)
             {
-              if (f == dpyinfo->w32_focus_event_frame)
-                dpyinfo->w32_focus_event_frame = 0;
-
-              if (f == dpyinfo->w32_focus_frame)
-                x_new_focus_frame (dpyinfo, 0);
-
               if (f == hlinfo->mouse_face_mouse_frame)
                 {
                   /* If we move outside the frame, then we're




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Blinking cursor on w32
  2013-07-27 11:57 Blinking cursor on w32 Eli Zaretskii
@ 2013-07-27 12:16 ` Juanma Barranquero
  2013-07-27 13:08   ` Eli Zaretskii
  0 siblings, 1 reply; 3+ messages in thread
From: Juanma Barranquero @ 2013-07-27 12:16 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Emacs developers

On Sat, Jul 27, 2013 at 1:57 PM, Eli Zaretskii <eliz@gnu.org> wrote:

> The changes below fix this for me; OK to commit?

Sure.

   J



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Blinking cursor on w32
  2013-07-27 12:16 ` Juanma Barranquero
@ 2013-07-27 13:08   ` Eli Zaretskii
  0 siblings, 0 replies; 3+ messages in thread
From: Eli Zaretskii @ 2013-07-27 13:08 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: emacs-devel

> From: Juanma Barranquero <lekktu@gmail.com>
> Date: Sat, 27 Jul 2013 14:16:52 +0200
> Cc: Emacs developers <emacs-devel@gnu.org>
> 
> On Sat, Jul 27, 2013 at 1:57 PM, Eli Zaretskii <eliz@gnu.org> wrote:
> 
> > The changes below fix this for me; OK to commit?
> 
> Sure.

Done.



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-07-27 13:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-27 11:57 Blinking cursor on w32 Eli Zaretskii
2013-07-27 12:16 ` Juanma Barranquero
2013-07-27 13:08   ` Eli Zaretskii

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.