bug#27752: Custom mode-line face + smart eshell triggers interesting freeze

[Eli Zaretskii]

> Date: Tue, 18 Jul 2017 18:58:01 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: 27752@debbugs.gnu.org
> 
> > From: Yegor Timoshenko <yegortimoshenko@gmail.com>
> > Date: Tue, 18 Jul 2017 13:46:54 +0300
> > 
> > Emacs config required to reproduce this bug:
> > 
> > (custom-set-faces
> >  '(mode-line ((t (:box 1)))))
> > 
> > (require 'em-smart)
> > (add-hook 'eshell-mode-hook 'eshell-smart-initialize)
> > 
> > Instructions:
> > 
> > M-x eshell, M-x split-window-horizontally, type "find-file foo" in the 
> > eshell buffer and hit ENTER.
> > Emacs freezes, CPU consumption is around 100%.
> 
> This happens because Eshell abuses window-scroll-functions.  I'm
> working on a fix.

Should be fixed now.

eshell-smart-redisplay called pos-visible-in-window-p, which is not a
very "smart" thing to do from a function that runs in the middle of
redisplay: how can the caller expect to get a reliable result, if the
window is being scrolled, and no one yet knows what will be inside the
window when redisplay ends.  And there are other atrocities in
eshell-smart-redisplay.

Emacs now tries to protect itself against that, so that it won't
infloop.

Btw, I think this is the same bug as 26980, it's just that there I
never could have a reproducer.