[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Few enhancements to ansi-term
From: |
Stefan Monnier |
Subject: |
Re: Few enhancements to ansi-term |
Date: |
Tue, 30 Apr 2019 14:34:48 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) |
> * lisp/term.el (term-set-scroll-region): Do not set
> term-scroll-with-delete when the region is set to the height of the
> terminal.
> * lisp/term.el (term-reset-terminal): Reset term-scroll-with-delete.
Better write these as:
> * lisp/term.el (term-set-scroll-region): Do not set
> term-scroll-with-delete when the region is set to the height of the
> terminal.
> (term-reset-terminal): Reset term-scroll-with-delete.
[ I read the first entry and thought the next was about another file so
I didn't even look at it. ]
The patch also changes term-reset-size but the ChangeLog doesn't explain
what this change does.
> @@ -3439,7 +3440,7 @@ term-set-scroll-region
> (setq term-scroll-with-delete
> (or (term-using-alternate-sub-buffer)
> (not (and (= term-scroll-start 0)
> - (= term-scroll-end term-height)))))
> + (= term-scroll-end (1- term-height))))))
The code just above does:
(setq term-scroll-end
(if (or (<= bottom term-scroll-start) (> bottom term-height))
term-height
bottom))
Should this also be changed to (1- term-height)?
And/or should the test use <= as in:
(not (and (<= term-scroll-start 0)
(>= term-scroll-end (1- term-height))))))
?
> * lisp/term.el (term-unwrap-line, term-emulate-terminal): Add
> rear-nonsticky text property to the newlines used for line wrapping.
The diff basically say that already, so I'd rather write it as "Prevent
the `term-line-wrap` property of newlines from spreading accidentally
when inserting text next to it" or something like that.
As for the code, it looks good to me.
Stefan