Re: address@hidden: Eamcs 22.1 comint-scroll-show-maximum-output bug]

[Richard Stallman]

    The bug _is_ caused by setting scroll-conservatively to 10000 in
    comint buffers as a buffer local variable. Let-binding it back to 0 in
    the relevant command does not help (because of when redisplay
    happens).

I thought `recenter' would actually do redisplay.  However, I was
wrong -- it does that only when called with no argument.

    1) Don't set it to 10000 in comint buffers, since it doesn't seem to
       be needed.

I am sure it WAS needed.  If you search for bug reports in a few
weeks before I made that fix, you will probably find the bug report
which I fixed this way.

    2) Make it a variable that should not be set directly, only via M-x
       customize or a function that sets scroll-conservatively at the same
       time.

You mean, if the user sets comint-scroll-show-maximum-output,
that should zero out scroll-conservatively?

Maybe that is correct, but I am not sure.  You see,
comint-scroll-show-maximum-output is only supposed to affect
output.

Perhaps the thing to do is bind scroll-conservatively in this code and
force a redisplay there:

                         (save-excursion
                           (goto-char (point-max))
                           (recenter (- -1 scroll-margin))))