[Emacs-diffs] emacs-24 r117274: Improve documentation of how vertical-motion interprets columns.

[Eli Zaretskii]

------------------------------------------------------------
revno: 117274
revision-id: address@hidden
parent: address@hidden
committer: Eli Zaretskii <address@hidden>
branch nick: emacs-24
timestamp: Sat 2014-06-21 11:17:29 +0300
message:
  Improve documentation of how vertical-motion interprets columns.
  
   src/indent.c (Fvertical_motion): Doc fix.
  
   doc/lispref/positions.texi (Screen Lines): Clarify how columns are counted
   by vertical-motion.
modified:
  doc/lispref/ChangeLog          changelog-20091113204419-o5vbwnq5f7feedwu-6155
  doc/lispref/positions.texi     
positions.texi-20091113204419-o5vbwnq5f7feedwu-6206
  src/ChangeLog                  changelog-20091113204419-o5vbwnq5f7feedwu-1438
  src/indent.c                   indent.c-20091113204419-o5vbwnq5f7feedwu-181

=== modified file 'doc/lispref/ChangeLog'
--- a/doc/lispref/ChangeLog     2014-06-14 16:37:15 +0000
+++ b/doc/lispref/ChangeLog     2014-06-21 08:17:29 +0000
@@ -1,3 +1,8 @@
+2014-06-21  Eli Zaretskii  <address@hidden>
+
+       * positions.texi (Screen Lines): Clarify how columns are counted
+       by vertical-motion.
+
 2014-06-14  Eli Zaretskii  <address@hidden>
 
        * commands.texi (Accessing Mouse): Improve the wording of the

=== modified file 'doc/lispref/positions.texi'
--- a/doc/lispref/positions.texi        2014-02-06 04:27:26 +0000
+++ b/doc/lispref/positions.texi        2014-06-21 08:17:29 +0000
@@ -500,7 +500,11 @@
 The @var{count} argument can be a cons cell, @code{(@var{cols}
 . @var{lines})}, instead of an integer.  Then the function moves by
 @var{lines} screen lines, and puts point @var{cols} columns from the
-start of that screen line.
+visual start of that screen line.  Note that @var{cols} are counted
+from the @emph{visual} start of the line; if the window is scrolled
+horizontally (@pxref{Horizontal Scrolling}), the column on which point
+will end is in addition to the number of columns by which the text is
+scrolled.
 
 The return value is the number of screen lines over which point was
 moved.  The value may be less in absolute value than @var{count} if

=== modified file 'src/ChangeLog'
--- a/src/ChangeLog     2014-06-21 07:03:41 +0000
+++ b/src/ChangeLog     2014-06-21 08:17:29 +0000
@@ -1,3 +1,7 @@
+2014-06-21  Eli Zaretskii  <address@hidden>
+
+       * indent.c (Fvertical_motion): Doc fix.
+
 2014-06-21  Paul Eggert  <address@hidden>
 
        Port to OS X ACLs (Bug#17810).

=== modified file 'src/indent.c'
--- a/src/indent.c      2014-04-12 11:21:47 +0000
+++ b/src/indent.c      2014-06-21 08:17:29 +0000
@@ -1944,9 +1944,12 @@
 parameters such as width, horizontal scrolling, and so on.
 The default is to use the selected window's parameters.
 
-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).
+LINES can optionally take the form (COLS . LINES), in which case the
+motion will not stop at the start of a screen line but COLS column
+from the visual start of the line (if such exists on that line, that
+is).  If the line is scrolled horizontally, COLS is interpreted
+visually, i.e., as addition to the columns of text beyond the left
+edge of the window.
 
 `vertical-motion' always uses the current buffer,
 regardless of which buffer is displayed in WINDOW.