unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Tassilo Horn <tsdh@gnu.org>
To: Richard Stallman <rms@gnu.org>
Cc: Andreas Schwab <schwab@suse.de>, 21277@debbugs.gnu.org
Subject: bug#21277: 25.0.50; char-fold search is a screw, and I can't see how to disable
Date: Tue, 18 Aug 2015 14:30:35 +0200	[thread overview]
Message-ID: <87pp2k22z8.fsf@gnu.org> (raw)
In-Reply-To: <E1ZRXmc-0002cL-Hp@fencepost.gnu.org> (Richard Stallman's message of "Mon, 17 Aug 2015 23:41:54 -0400")

Richard Stallman <rms@gnu.org> writes:

>   > That has nothing to do with char-fold, this feature already exists since
>   > 24.3 and is called lax-whitespace.
>
> How come it did not occur in the Emacs I was running before last week?
> I built that from master a few months ago.
>
> Anyway, it is a total screw.  Who likes this as a default?

I like that behavior because I frequently search for "some text" in
prose, and then it allows me to find also the matches where the two
words are wrapped around.

But I think the `isearch-lax-whitespace' feature can still be improved:

1. The `isearch-lax-whitespace' docstring speaks of "ordinary
   incremental search" which I've read as "non-regex isearch".  But it
   seems that regex isearch is also affected.  I'd just strip the
   "ordinary".

2. More importantly, isearching for C-j SPC matches any sequence of two
   whitespace characters, e.g., it matches between here  and  here, too.
                                                       ^^   ^^
I had expected that only space characters in the search string are
replaced by `search-whitespace-regexp' but it seems that any whitespace
character is replaced by it.  I think this is not good.

And IMHO, it would be a good idea to disable lax-whitespace for the
current search as soon as the search string contains a non-space
whitespace character.  I think if the user explicitly types C-j, C-i,
etc. she's probably not interested in treating spaces as any-whitespace
but wants to distinguish newlines from tabs and spaces.

Bye,
Tassilo





  reply	other threads:[~2015-08-18 12:30 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-17  5:04 bug#21277: 25.0.50; char-fold search is a screw, and I can't see how to disable Richard Stallman
2015-08-17  7:23 ` Andreas Schwab
2015-08-18  3:41   ` Richard Stallman
2015-08-18 12:30     ` Tassilo Horn [this message]
2015-08-18 14:24       ` Drew Adams
2015-08-18 21:58       ` Artur Malabarba
2015-08-18 21:58         ` Artur Malabarba
2015-08-19 18:14         ` Richard Stallman
2015-08-19 22:17       ` Juri Linkov
2015-08-18  3:41   ` Richard Stallman
2015-08-19 22:08     ` Juri Linkov
2015-08-20 16:56       ` Richard Stallman
2015-08-17 19:02 ` Mark Oteiza
2015-08-18 22:29 ` Artur Malabarba
2015-08-19 22:02 ` Juri Linkov
2015-08-20  9:36   ` Artur Malabarba

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=87pp2k22z8.fsf@gnu.org \
    --to=tsdh@gnu.org \
    --cc=21277@debbugs.gnu.org \
    --cc=rms@gnu.org \
    --cc=schwab@suse.de \
    /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 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).