GNU bug report logs - #20701
25.0.50; [PATCH] (vertical-motion 0) broken at display strings after newline

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Wolfgang Jenkner <wjenkner <at> inode.at>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#20701: closed (25.0.50; [PATCH] (vertical-motion 0) broken at
 display strings after newline)
Date: Wed, 03 Jun 2015 13:58:03 +0000

[Message part 1 (text/plain, inline)]

Your message dated Wed, 03 Jun 2015 15:56:12 +0200
with message-id <85wpzk3nf2.fsf <at> iznogoud.viz>
and subject line Re: bug#20701: 25.0.50; [PATCH] (vertical-motion 0) broken at display strings after newline
has caused the debbugs.gnu.org bug report #20701,
regarding 25.0.50; [PATCH] (vertical-motion 0) broken at display strings after newline
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
20701: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=20701
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Wolfgang Jenkner <wjenkner <at> inode.at>
To: bug-gnu-emacs <at> gnu.org
Subject: 25.0.50;
 [PATCH] (vertical-motion 0) broken at display strings after newline
Date: Sun, 31 May 2015 16:19:46 +0200

If you run the following snippet in emacs -Q point will end up at the
first line of the "Test" buffer, but it should stay at the second line.

(with-current-buffer (generate-new-buffer "Test")
  (pop-to-buffer (current-buffer))
  (insert "x\n")
  (save-excursion
    (insert (propertize "x" 'display "x")))
  (sit-for 1)
  (vertical-motion 0))

On the other hand, non-zero arguments for vertical-motion seem to work
correctly, as long as the buffer contains only ASCII characters before
the propertized "x".

Otherwise

(with-current-buffer (generate-new-buffer "Test")
  (pop-to-buffer (current-buffer))
  (insert "\né\n")
  (save-excursion
    (insert (propertize "x" 'display "x")))
  (sit-for 1)
  (vertical-motion -1))

Here, point should go to the second line but it goes to the first line.

The following patch tries to fix both issues.

-- >8 --
Subject: [PATCH] * src/indent.c (Fvertical_motion): Fix a case of motion by 0
 lines.

Starting from a display string after a newline, point went to the
previous line.

Also, correct an inadvertent use of a buffer position with FETCH_BYTE.
---
 src/indent.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/indent.c b/src/indent.c
index ce78308..b4e6d74 100644
--- a/src/indent.c
+++ b/src/indent.c
@@ -2134,17 +2134,20 @@ whether or not it is currently displayed in some window.  */)
 	  overshoot_handled = 1;
 	}
       else if (IT_CHARPOS (it) == PT - 1
-	       && FETCH_BYTE (PT - 1) == '\n'
-	       && nlines < 0)
+	       && FETCH_BYTE (PT_BYTE - 1) == '\n'
+	       && nlines <= 0)
 	{
 	  /* The position we started from was covered by a display
 	     property, so we moved to position before the string, and
-	     backed up one line, because the character at PT - 1 is a
-	     newline.  So we need one less line to go up.  */
+	     backed up one line, because the character at PT - 1 is
+	     a newline.  So we need one less line to go up (or exactly
+	     one line to go down if nlines == 0).  */
 	  nlines++;
 	  /* But we still need to record that one line, in order to
 	     return the correct value to the caller.  */
 	  vpos_init = -1;
+
+	  overshoot_handled = 1;
 	}
       if (lcols_given)
 	to_x = window_column_x (w, window, extract_float (lcols), lcols);
@@ -2159,7 +2162,7 @@ whether or not it is currently displayed in some window.  */)
 	}
       else if (overshoot_handled)
 	{
-	  it.vpos = 0;
+	  it.vpos = vpos_init;
 	  move_it_by_lines (&it, min (PTRDIFF_MAX, nlines));
 	}
       else
-- 
2.4.1

[Message part 3 (message/rfc822, inline)]

From: Wolfgang Jenkner <wjenkner <at> inode.at>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 20701-done <at> debbugs.gnu.org
Subject: Re: bug#20701: 25.0.50;
 [PATCH] (vertical-motion 0) broken at display strings after newline
Date: Wed, 03 Jun 2015 15:56:12 +0200

Version: 25.1

On Sun, May 31 2015, Eli Zaretskii wrote:

>> Subject: [PATCH] * src/indent.c (Fvertical_motion): Fix a case of motion by 0
>>  lines.
>> 
>> Starting from a display string after a newline, point went to the
>> previous line.
>> 
>> Also, correct an inadvertent use of a buffer position with FETCH_BYTE.
>
> Looks good to me, please push.

Thank you very much for reviewing the patch.

Pushed to master.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.