all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Noam Postavsky <npostavs@gmail.com>
To: Aaron Jensen <aaronjensen@gmail.com>
Cc: 30815@debbugs.gnu.org
Subject: bug#30815: 26.0.91; unicode right single quote mark with syntax entry of w not respected by forward-word
Date: Tue, 13 Mar 2018 22:24:46 -0400	[thread overview]
Message-ID: <87y3ivwg0h.fsf@gmail.com> (raw)
In-Reply-To: <CAHyO48ymfyWWOX5apkKz1vPwwX-Pw0j7Os4-UJi4U92rh1aThQ@mail.gmail.com> (Aaron Jensen's message of "Tue, 13 Mar 2018 19:07:02 -0700")

[-- Attachment #1: Type: text/plain, Size: 339 bytes --]

Aaron Jensen <aaronjensen@gmail.com> writes:

> Thank you for the pointer. FWIW, I checked the docs of forward-word
> which only talks about buffer syntax tables. Perhaps that should
> either reference the Word Motion docs or it should also describe the
> whole picture.

I think leaving the whole picture to the manual makes more sense.


[-- Attachment #2: patch --]
[-- Type: text/plain, Size: 1773 bytes --]

From 4c0a6f91b72aae7f43a93f2d286731fcd939fecd Mon Sep 17 00:00:00 2001
From: Noam Postavsky <npostavs@gmail.com>
Date: Tue, 13 Mar 2018 22:17:43 -0400
Subject: [PATCH] Improve word motion docs (Bug#30815)

* doc/lispref/positions.texi (Word Motion): Fix reference to
`char-script-table'.
* src/syntax.c (forward-word): Add link to 'Word Motion' manual
section.
---
 doc/lispref/positions.texi | 2 +-
 src/syntax.c               | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/doc/lispref/positions.texi b/doc/lispref/positions.texi
index 0a03e246c0..fdc8bb96ae 100644
--- a/doc/lispref/positions.texi
+++ b/doc/lispref/positions.texi
@@ -210,7 +210,7 @@ Word Motion
 modes can override that by setting up a suitable
 @code{find-word-boundary-function-table}, described below.  Characters
 that belong to different scripts (as defined by
-@code{char-syntax-table}), also define a word boundary
+@code{char-script-table}), also define a word boundary
 (@pxref{Character Properties}).  In any case, this function cannot
 move point past the boundary of the accessible portion of the buffer,
 or across a field boundary (@pxref{Fields}).  The most common case of
diff --git a/src/syntax.c b/src/syntax.c
index 378064611c..d3dd79229a 100644
--- a/src/syntax.c
+++ b/src/syntax.c
@@ -1555,7 +1555,8 @@ DEFUN ("forward-word", Fforward_word, Sforward_word, 0, 1, "^p",
 table, but `find-word-boundary-function-table', such as set up
 by `subword-mode', can change that.  If a Lisp program needs to
 move by words determined strictly by the syntax table, it should
-use `forward-word-strictly' instead.  */)
+use `forward-word-strictly' instead.  See Info node `(elisp) Word
+Motion' for details.  */)
   (Lisp_Object arg)
 {
   Lisp_Object tmp;
-- 
2.11.0


[-- Attachment #3: Type: text/plain, Size: 417 bytes --]


> Also, as I understand the thread it seems like this is "by design" and
> there's no good/accepted way to get it to behave as I want without
> risking breaking unknown things.

Well, just taking the syntax entry as the sole source of truth and
ignoring the character script is probably the wrong thing.  

Maybe there is some other way to get the wanted behaviour though.  I
also found Bug#13129 asking about this.

  reply	other threads:[~2018-03-14  2:24 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-14  0:14 bug#30815: 26.0.91; unicode right single quote mark with syntax entry of w not respected by forward-word Aaron Jensen
2018-03-14  1:11 ` Noam Postavsky
2018-03-14  2:07   ` Aaron Jensen
2018-03-14  2:24     ` Noam Postavsky [this message]
2018-03-14  2:43       ` Aaron Jensen
2018-03-14 16:09         ` Eli Zaretskii
2018-03-14 17:43           ` Aaron Jensen
2018-03-14 17:51             ` Eli Zaretskii
2018-03-14 15:59       ` Eli Zaretskii
2018-03-14 23:19         ` Noam Postavsky
2018-03-15  6:39           ` Eli Zaretskii
2018-03-17  0:03             ` Noam Postavsky

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87y3ivwg0h.fsf@gmail.com \
    --to=npostavs@gmail.com \
    --cc=30815@debbugs.gnu.org \
    --cc=aaronjensen@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.