GNU bug report logs - #3494
23.0.94; line-move-visual: temporary goal column confused by hscrolling

Previous Next

Full log

View this message in rfc822 format

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Chong Yidong <cyd <at> stupidchicken.com>
Cc: 3494 <at> debbugs.gnu.org
Subject: bug#3494: Review line-move-visual patch
Date: Tue, 14 Jul 2009 22:05:59 -0400

> I just checked in a patch into the trunk, which makes line-move-visual
> play properly with truncated lines (Bugs 3494 and 3805).  I think it
> should go to the branch as well.  Could you review it?

> 2009-07-11  Chong Yidong  <cyd <at> stupidchicken.com>

> 	* simple.el (temporary-goal-column): Change the value for
> 	line-move-visual to a cons cell.
> 	(line-move-visual): Record or set the window hscroll, if
> 	necessary (Bug#3494).
> 	(line-move-1): Handle cons value of temporary-goal-column.

The problem with this code is that vertical-motion is defined as:

   LINES can optionally take the form (COLS . LINES), in which case
   the motion will not stop at the start of a screen line but on
   its column COLS (if such exists on that line, that is).

but without specifying what is meant by COLS w.r.t hscroll.
Similarly the docstring refers to "start of the screen line" without
making it clear what it means in the case of hscroll.

So, I'm not sure if doing it like you've done is right or not.
Maybe a better solution is to use COLS + HSCROLL as the
temporary-goal-column"; it would at least avoid having to change
temporary-goal-column to accept a cons cell rather than only a number.



        Stefan

Previous Next

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