bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#17585: 24.4.50; vertical-motion erroneously adds points


From: Keith David Bershatsky
Subject: bug#17585: 24.4.50; vertical-motion erroneously adds points
Date: Thu, 29 May 2014 23:21:05 -0700
User-agent: / () / () APEL/10.8 Emacs/24.4.50 (x86_64-apple-darwin10.8.0) MULE/6.0 (HANACHIRUSATO)

The `test` function yields the same result with Emacs built (--with-ns) from 
the Trunk as of May 29, 2014 at 11:18 P.S.T.  Approximately when, please, 
should I expect to see the fix merged into the Trunk?

I'm using macports / baazar:

/macports/bin/bzr branch --stacked bzr://bzr.savannah.gnu.org/emacs/trunk 
emacs-trunk

Thanks,

Keith

;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;

At Sun, 25 May 2014 17:23:31 +0300,
Eli Zaretskii wrote:
> 
> > Date: Sun, 25 May 2014 00:23:20 -0700
> > From: Keith David Bershatsky <address@hidden>
> > 
> > The following test function demonstrates how vertical-motion erroneously 
> > adds points when dealing with long lines without spaces that go beyond the 
> > window-width:
> > 
> >     (defun test ()
> >     (interactive)
> >       (switch-to-buffer (get-buffer-create "foo.txt"))
> >       (text-mode)
> >       (linum-mode t)
> >       (visual-line-mode t)
> >       (dotimes (i (* (window-width) 4))
> >         (insert ";"))
> >      (dotimes (i 10)
> >         (insert "\n"))
> >       (goto-char (point-min))
> >       (vertical-motion 1)
> >       (message "Second Row + 2 points:  %s" (current-column))
> >       (vertical-motion 1)
> >       (message "Third Row + 4 points:  %s" (current-column)))
> 
> Fixed in revision 117154 on the emacs-24 branch.  The patch is below,
> in case you want to try it without waiting for it to be merged to the
> trunk.
> 
> Thanks.
> 
> --- src/xdisp.c       2014-05-24 21:28:43 +0000
> +++ src/xdisp.c       2014-05-25 14:18:46 +0000
> @@ -8556,7 +8556,7 @@ move_it_in_display_line_to (struct it *i
>       }
>        else
>       {
> -       if (it->line_wrap == WORD_WRAP)
> +       if (it->line_wrap == WORD_WRAP && it->area == TEXT_AREA)
>           {
>             if (IT_DISPLAYING_WHITESPACE (it))
>               may_wrap = 1;





reply via email to

[Prev in Thread] Current Thread [Next in Thread]