bug#38185: isearch-yank-word-or-char grabs periods at ends of words when used on man pages

bug#38185: isearch-yank-word-or-char grabs periods at ends of words when used on man pages

[積丹尼 Dan Jacobson]

When used on man pages, isearch-yank-word-or-char also gets periods at
the ends of words, along with the word. Commas, however, thankfully, are
not gotten.

    C-w runs the command isearch-yank-word-or-char (found in
    overriding-terminal-local-map), which is an interactive compiled Lisp
    function in ‘isearch.el’.

    (isearch-yank-word-or-char)

    Pull next character, subword or word from buffer into search string.
    Subword is used when ‘subword-mode’ is activated.

M-x man cat
Put cursor here:

  blarf.
  ^
Hit C-s C-w

Cursor now here:
  blarf.
        ^
Should be here:
  blarf.
       ^

emacs-version "26.3".

bug#38185: isearch-yank-word-or-char grabs periods at ends of words when used on man pages

[Lars Ingebrigtsen]

積丹尼 Dan Jacobson <jidanni@jidanni.org> writes:

> When used on man pages, isearch-yank-word-or-char also gets periods at
> the ends of words, along with the word. Commas, however, thankfully, are
> not gotten.
>
>     C-w runs the command isearch-yank-word-or-char (found in
>     overriding-terminal-local-map), which is an interactive compiled Lisp
>     function in ‘isearch.el’.
>
>     (isearch-yank-word-or-char)
>
>     Pull next character, subword or word from buffer into search string.
>     Subword is used when ‘subword-mode’ is activated.

The command calls

(isearch--yank-char-or-syntax '(?w) 'forward-word arg)

so I guess this means that "." has word syntax in Man buffers, but ","
doesn't?

I guess that makes sense -- you can have stuff like "foo.bar" that
should be considered a single "word", but I agree that it doesn't make
much sense to have a terminating "." be part of a word.

I'm not sure how to fix that, though -- switch isearch-yank-word-or-char
to something like ffap instead of using syntax classes?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

bug#38185: isearch-yank-word-or-char grabs periods at ends of words when used on man pages

[Lars Ingebrigtsen]

Lars Ingebrigtsen <larsi@gnus.org> writes:

> I'm not sure how to fix that, though -- switch isearch-yank-word-or-char
> to something like ffap instead of using syntax classes?

I mean thingatp.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

bug#38185: isearch-yank-word-or-char grabs periods at ends of words when used on man pages

[Juri Linkov]

> (isearch--yank-char-or-syntax '(?w) 'forward-word arg)
>
> so I guess this means that "." has word syntax in Man buffers, but ","
> doesn't?
>
> I guess that makes sense -- you can have stuff like "foo.bar" that
> should be considered a single "word", but I agree that it doesn't make
> much sense to have a terminating "." be part of a word.
>
> I'm not sure how to fix that, though -- switch isearch-yank-word-or-char
> to something like ffap instead of using syntax classes?

isearch-yank-word-or-char was intended to yank exactly the same text
as navigated by forward-word.

However, I can't reproduce this issue in Man buffers: forward-word
doesn't consider "." as part of a word.  Ah, this is because I fixed
this long ago in the init file ~/.emacs as:

  ;; Don't use `man-mode-syntax-table' that sets word syntax to `.', `_', `:'.
  (add-hook 'Man-mode-hook
            (lambda ()
              (set-syntax-table text-mode-syntax-table)))

bug#38185: isearch-yank-word-or-char grabs periods at ends of words when used on man pages

[Lars Ingebrigtsen]

Juri Linkov <juri@linkov.net> writes:

> isearch-yank-word-or-char was intended to yank exactly the same text
> as navigated by forward-word.

And I guess we don't really want to change the syntax class of "." in
Man buffers, so there isn't really anything to fix here, and I'm closing
this bug report.

> However, I can't reproduce this issue in Man buffers: forward-word
> doesn't consider "." as part of a word.  Ah, this is because I fixed
> this long ago in the init file ~/.emacs as:
>
>   ;; Don't use `man-mode-syntax-table' that sets word syntax to `.', `_', `:'.
>   (add-hook 'Man-mode-hook
>             (lambda ()
>               (set-syntax-table text-mode-syntax-table)))

(And Dan can do this if he wants a more text-mode like behaviour.)

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no