Re: Few enhancements to ansi-term

[Stefan Monnier <monnier@iro.umontreal.ca>]

> * 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