* Changes in revno 100600 on the emacs-23 branch [not found] <mailman.65.1308412813.15703.emacs-diffs@gnu.org> @ 2011-06-18 16:40 ` Eli Zaretskii 2011-06-18 23:46 ` YAMAMOTO Mitsuharu 0 siblings, 1 reply; 2+ messages in thread From: Eli Zaretskii @ 2011-06-18 16:40 UTC (permalink / raw) To: YAMAMOTO Mitsuharu; +Cc: emacs-devel > From: emacs-diffs-request@gnu.org > Date: Sat, 18 Jun 2011 12:00:13 -0400 > > Date: Sat, 18 Jun 2011 09:37:38 +0900 > From: YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp> > To: emacs-diffs@gnu.org > Subject: [Emacs-diffs] /srv/bzr/emacs/emacs-23 r100600: * dispnew.c > (scrolling_window): Before scrolling, turn off a mouse-highlight in > the window being scrolled. > > === modified file 'src/dispnew.c' > --- a/src/dispnew.c 2011-05-25 03:06:05 +0000 > +++ b/src/dispnew.c 2011-06-18 00:37:38 +0000 > @@ -5206,6 +5206,7 @@ > /* Copy on the display. */ > if (r->current_y != r->desired_y) > { > + rif->clear_window_mouse_face (w); > rif->scroll_run_hook (w, r); Why do you do this inside the loop? Mouse highlight needs only be turned off once, right? And no code inside the loop seems to turn it back on, right? And btw, where's the code that will turn mouse highlight back on after this? ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Changes in revno 100600 on the emacs-23 branch 2011-06-18 16:40 ` Changes in revno 100600 on the emacs-23 branch Eli Zaretskii @ 2011-06-18 23:46 ` YAMAMOTO Mitsuharu 0 siblings, 0 replies; 2+ messages in thread From: YAMAMOTO Mitsuharu @ 2011-06-18 23:46 UTC (permalink / raw) To: Eli Zaretskii; +Cc: emacs-devel >>>>> On Sat, 18 Jun 2011 19:40:44 +0300, Eli Zaretskii <eliz@gnu.org> said: >> === modified file 'src/dispnew.c' >> --- a/src/dispnew.c 2011-05-25 03:06:05 +0000 >> +++ b/src/dispnew.c 2011-06-18 00:37:38 +0000 >> @@ -5206,6 +5206,7 @@ >> /* Copy on the display. */ >> if (r->current_y != r->desired_y) >> { >> + rif->clear_window_mouse_face (w); >> rif->scroll_run_hook (w, r); > Why do you do this inside the loop? Mouse highlight needs only be > turned off once, right? And no code inside the loop seems to turn it > back on, right? The first call clears dpyinfo->mouse_face_window to Qnil inside clear_mouse_face, and thus the subsequent calls return early. Of course one could save the function call overhead by introducing a local variable that differentiates the first call, but scroll_run_hook involves graphics operations and is not light in the first place, and I chose code similarity with the corresponding parts in try_window_id and try_window_reusing_current_matrix. > And btw, where's the code that will turn mouse highlight back on after > this? Just as in the case for try_window_id and try_window_reusing_current_matrix, frame_up_to_date_hook will call note_mouse_highlight. YAMAMOTO Mitsuharu mituharu@math.s.chiba-u.ac.jp ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-06-18 23:46 UTC | newest] Thread overview: 2+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <mailman.65.1308412813.15703.emacs-diffs@gnu.org> 2011-06-18 16:40 ` Changes in revno 100600 on the emacs-23 branch Eli Zaretskii 2011-06-18 23:46 ` YAMAMOTO Mitsuharu
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.