unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#17533: 24.4.50; Word Isearch bug
@ 2014-05-20  8:30 Dani Moncayo
  2014-05-20 21:21 ` Juri Linkov
  0 siblings, 1 reply; 6+ messages in thread
From: Dani Moncayo @ 2014-05-20  8:30 UTC (permalink / raw)
  To: 17533

From "emacs -Q":
  f o o RET - > f o o C-s M-s w - > f o o

Expected: Only the string "->foo" is matched.

Observed: There are two (quite strange) matches:
1. From the final dot in the *scratch* buffer to the end of the first
   "foo".
2. From the end of the first "foo" to the end of the second "foo".


In GNU Emacs 24.4.50.1 (i686-pc-mingw32)
 of 2014-04-21 on LEG570
Repository revision: 117001 dancol@dancol.org-20140421012855-xu7gwqdl59pgkgur
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'

Configured features:
XPM JPEG TIFF GIF PNG RSVG NOTIFY ACL GNUTLS LIBXML2 ZLIB

Important settings:
  value of $LANG: ESN
  locale-coding-system: cp1252

-- 
Dani Moncayo





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#17533: 24.4.50; Word Isearch bug
  2014-05-20  8:30 bug#17533: 24.4.50; Word Isearch bug Dani Moncayo
@ 2014-05-20 21:21 ` Juri Linkov
  2014-05-21 10:07   ` Dani Moncayo
  2014-05-21 15:59   ` Dani Moncayo
  0 siblings, 2 replies; 6+ messages in thread
From: Juri Linkov @ 2014-05-20 21:21 UTC (permalink / raw)
  To: Dani Moncayo; +Cc: 17533

> From "emacs -Q":
>   f o o RET - > f o o C-s M-s w - > f o o
>
> Expected: Only the string "->foo" is matched.
>
> Observed: There are two (quite strange) matches:
> 1. From the final dot in the *scratch* buffer to the end of the first
>    "foo".
> 2. From the end of the first "foo" to the end of the second "foo".

In word search mode, non-word characters are interpreted as whitespace.





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#17533: 24.4.50; Word Isearch bug
  2014-05-20 21:21 ` Juri Linkov
@ 2014-05-21 10:07   ` Dani Moncayo
  2014-05-21 10:50     ` Dani Moncayo
  2014-05-21 15:59   ` Dani Moncayo
  1 sibling, 1 reply; 6+ messages in thread
From: Dani Moncayo @ 2014-05-21 10:07 UTC (permalink / raw)
  To: Juri Linkov; +Cc: 17533

> In word search mode, non-word characters are interpreted as whitespace.

IMO, that rule should have an exception: the characters in the current
search string should be interpreted as word characters (for the
purpose of the search), because IMO, what the user wants is to search
for occurrences of _that_ string (regardless of the type of its
characters), where the _surrounding_ characters are non-words.


-- 
Dani Moncayo





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#17533: 24.4.50; Word Isearch bug
  2014-05-21 10:07   ` Dani Moncayo
@ 2014-05-21 10:50     ` Dani Moncayo
  2014-05-21 13:07       ` Dani Moncayo
  0 siblings, 1 reply; 6+ messages in thread
From: Dani Moncayo @ 2014-05-21 10:50 UTC (permalink / raw)
  To: Juri Linkov; +Cc: 17533

On Wed, May 21, 2014 at 12:07 PM, Dani Moncayo <dmoncayo@gmail.com> wrote:
>> In word search mode, non-word characters are interpreted as whitespace.
>
> IMO, that rule should have an exception: the characters in the current
> search string should be interpreted as word characters (for the
> purpose of the search), because IMO, what the user wants is to search
> for occurrences of _that_ string (regardless of the type of its
> characters), where the _surrounding_ characters are non-words.

Though admittedly, it's a bit odd to put non-word characters in a
word-type Isearch.

My use case is this: I was editing a C source code file, which had
things like: "pointer->foo", "pointer->foobar", and also stand-alone
variables "foo" and "foobar".

I wanted to find only the occurrences of "pointer->foo", so I tried to
do a word-type Isearch with "->foo" as search string.

Therefore, I think that I intuitively expected the following behavior
for word-type Isearch: find matches of (literally) the search string
which have a "change in type of character" in its boundaries, i.e.:

1. If the first character in my search string is word-type, its
previous character in the buffer must be non-word-type (and vice-versa).

2. If the last character in my search string is word-type, its
following character in the buffer must be non-word-type (and vice-versa).



-- 
Dani Moncayo





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#17533: 24.4.50; Word Isearch bug
  2014-05-21 10:50     ` Dani Moncayo
@ 2014-05-21 13:07       ` Dani Moncayo
  0 siblings, 0 replies; 6+ messages in thread
From: Dani Moncayo @ 2014-05-21 13:07 UTC (permalink / raw)
  To: Juri Linkov; +Cc: 17533

> Therefore, I think that I intuitively expected the following behavior
> for word-type Isearch: find matches of (literally) the search string
> which have a "change in type of character" in its boundaries, i.e.:
>
> 1. If the first character in my search string is word-type, its
> previous character in the buffer must be non-word-type (and vice-versa).
>
> 2. If the last character in my search string is word-type, its
> following character in the buffer must be non-word-type (and vice-versa).

Mmmm... perphaps the "vice-versa"s above should be removed.


-- 
Dani Moncayo





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#17533: 24.4.50; Word Isearch bug
  2014-05-20 21:21 ` Juri Linkov
  2014-05-21 10:07   ` Dani Moncayo
@ 2014-05-21 15:59   ` Dani Moncayo
  1 sibling, 0 replies; 6+ messages in thread
From: Dani Moncayo @ 2014-05-21 15:59 UTC (permalink / raw)
  To: Juri Linkov; +Cc: 17533-done

> In word search mode, non-word characters are interpreted as whitespace.

Indeed, I now remember the philosophy behind word-type Isearch: you
can search for "foo, bar" (with punctuation between the words) and
you'll find that sequence of words, regardless of the whitespace or
punctuation between the words.

So my expectations were wrong.

I'm closing the bug report.  Sorry for the noise.

-- 
Dani Moncayo





^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-05-21 15:59 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-20  8:30 bug#17533: 24.4.50; Word Isearch bug Dani Moncayo
2014-05-20 21:21 ` Juri Linkov
2014-05-21 10:07   ` Dani Moncayo
2014-05-21 10:50     ` Dani Moncayo
2014-05-21 13:07       ` Dani Moncayo
2014-05-21 15:59   ` Dani Moncayo

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).