Re: [PATCH] trivial patch, jumping cursor in term

[Ivan Kanis <expire-by-2009-09-30@kanis.fr>]

Hi Dan, Stefan,


Stefan Monnier <monnier@iro.umontreal.ca> wrote:

> Hmm... it should have *exactly* the same effect as your patch.

Does it work for you? It didn't seem to work for me...

Dan Nicolaescu <dann@ics.uci.edu> wrote:

> After your changes yesterday an error is thrown when using the above scenario:
> error in process filter: Wrong type argument: number-or-marker-p, nil

Yeah I've seen it too, the following trivial patch fixes it.

--- a/lisp/misc/term.el Thu Sep 24 12:01:01 2009 +0200
+++ b/lisp/misc/term.el Fri Sep 25 10:53:21 2009 +0200
@@ -3730,7 +3730,7 @@
   (let ((start-column (term-horizontal-column)))
     (when (and check-for-scroll (or term-scroll-with-delete term-pager-count))
       (setq down (term-handle-scroll down)))
-    (unless (and (= term-current-row 0) (< down 0))
+    (unless (and term-current-row (= term-current-row 0) (< down 0))
       (term-adjust-current-row-cache down)
       (when (or (/= (point) (point-max)) (< down 0))
        (setq down (- down (term-vertical-motion down)))))


Take care,
-- 
Ivan
Kanis http://kanis.fr

The cause of these complaints lies in WinZip, which turns an all
upper-case directory into an all lower case one in a fit of
helpfulness
    -- Ant Documentation