[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#21730: 25.0.50; Random errors in redisplay--pre-redisplay-functions

From: Philipp Stephani
Subject: bug#21730: 25.0.50; Random errors in redisplay--pre-redisplay-functions
Date: Wed, 14 Sep 2016 19:11:57 +0000

Philipp Stephani <address@hidden> schrieb am Mi., 14. Sep. 2016 um 18:54 Uhr:
Eli Zaretskii <address@hidden> schrieb am Sa., 31. Okt. 2015 um 14:34 Uhr:
> From: Michael Welsh Duggan <address@hidden>
> Date: Sat, 31 Oct 2015 09:07:05 -0400
> > I've changed the (bobp) to (= point 1).  I'll run it for a couple of
> > weeks and report back.
> This change seems to have done the trick.  I have not encountered the
> error in several days.

Thanks for testing.  I pushed the change, and I'm arking this bug as

This change was  reverted in 76ef52267cf887e3e1aa6d25b3b16dd0601dd459.
It also doesn't seem correct. cursor-sensor--detect is only used in pre-redisplay-functions, and the documentation of that variable says:
"Hook run just before redisplay.
It is called in each window that is to be redisplayed.  It takes one argument,
which is the window that will be redisplayed.  When run, the ‘current-buffer’
is set to the buffer displayed in that window."
That means that (bobp) is correct and (= point 1) cannot give a different result, unless narrowing is in effect (then only bobp is correct).
Given that replacing (bobp) with (= point 1) does solve this bug, the documentation of pre-redisplay-functions must be incorrect, i.e. the current buffer is not the buffer of the window passed as argument. I think the only way how this can happen is that a previous entry in pre-redisplay-functions has changed the current buffer. Probably the implementation of redisplay--pre-redisplay-functions should be changed from 
(with-current-buffer (window-buffer win)
          (run-hook-with-args 'pre-redisplay-functions win))
(run-hook-wrapped 'pre-redisplay-functions
  (lambda (func) (with-current-buffer (window-buffer win)
     (funcall func win)
or so.

I've attached a patch for this. 

Attachment: 0001-Restore-buffer-in-pre-redisplay-functions.txt
Description: Text document

reply via email to

[Prev in Thread] Current Thread [Next in Thread]