unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: 22036@debbugs.gnu.org
Subject: bug#22036: 25.0.50; (emacs) Lax Search
Date: Fri, 27 Nov 2015 20:30:00 -0800 (PST)	[thread overview]
Message-ID: <6e6838d0-d8c3-4a19-a09f-39d2578b5a0b@default> (raw)

Mostly minor suggestions.  And I realize that some of the problems
mentioned here are not new - they were already present in node `Special
Isearch', for example.  But things have deteriorated a bit more, I
think.  If you can make improvements, that will help readers.

1. "Normally, you'd want", "are normally perceived as equivalent",
"letter-case differences usually don't matter", "normally ignore the
case", "Searches in Emacs normally perform"...

Please rephrase such text.  There is nothing "normal" about one behavior
or abnormal about the opposite behavior.  There is no norm, here.
Simply say what the behaviors are and what the default behavior is,
without claiming that one is "normal".  And it is not the case for all
(even most?) users that "letter-case differences usually don't matter".
There is no such "usually".

Emacs users and use cases are all over the map.  Saying "normally" or
"usually" can make a reader think that it is important to stick with
this behavior, whereas it is just a default that we chose because we
thought it would be helpful.

2. Please cross-reference node `Replacement and Lax Matches' (which
cross-references node `Lax Search').

3. Why is regexp search not handled here as well?  Why send someone off
to node `Regexp Search' for info about lax matching during regexp
search?  Maybe there is a good reason, but as one user I would expect,
a priori, to read all about lax matching here.

4. There is a problem with this text:

"Hence, `foo bar' matches `foo bar', `foo bar', `foo bar', and so on
(but not `foobar')."  Those three are rendered identically - a single
SPC char is used to separate the words in all cases.

5. Missing word "off" here? "you can turn lax space matching by typing"

6. Unclear: "To disable this feature entirely".  When I read that far, I
thought it was talking about disabling the feature of toggling.  Also,
just say that `M-s <SPC>' toggles lax whitespace search - no need to
describe what toggling means (off, on again, etc.).

7. Everything in this manual is "in Emacs", so you can drop that phrase
everywhere.

8. The explanation of case matching here seems to repeat what is said
about it in node `Special Isearch'.  More factoring needed, perhaps.

9. "This applies to regular expression search as well as to string
search."  Literal search, not string search.

10. The same paragraph is also repetitive: you explain the effect of an
upper-case char, then you say that if it is removed the effect goes away
(OK, but not needed), and then you describe the effect of option
`search-upper-case' by repeating the description of the non-nil
behavior.

11. "does not extend beyond the current incremental search to the next
one" is repetitive.  Presumably "to the next" is what was meant by
"extend beyond".

12. Choose whether you want the term "character folding" to refer to (1)
the general category of folding of chars, including specific kinds of
character folding such as case folding and whitespace folding, or (2)
what is called by the search and replace UI "character folding",
which is just another specific kind of (character) folding.

The equivalence classes recognized by this "character folding" (#2) do
not include case folding - they are specifically about a set of
predefined equivalences other than letter case, equivalences which
involve ignoring diacritical marks plus a few others (quote marks etc.).

We need to find a clear way to talk about this now, because now is when
we are introducing more character folding than just case and whitespace.
If you choose #1 then some more specific term is need here and in the UI
for #2.

It is also the case that, for the moment, at least, the equivalences
defined by character-fold.el are a motley crew - diacriticals and
compositions plus some ad hoc extras (e.g., quotations).  This should
all be sorted out before we go too far with this (useful) feature.

13. It's better to drop the future tense and use the present: "will
match" -> "matches".

14. "the `nil' value" -> just "`nil'".  We don't say "the 4" value; we
say "4".  Same with `nil', `t', etc.

15. "typing an explicit variant of a character, such as `ä'".  Consider
adding the char name after this char, or do something else to help
readers see that what is quoted is not just the letter "a".  This is not
obvious.  Maybe a different char choice would be more obvious; dunno.

In GNU Emacs 25.0.50.1 (i686-pc-mingw32)
 of 2015-11-27
Bzr revision: a5f2970207d792e5f5d40160485007f282a0569d
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --prefix=/c/Devel/emacs/snapshot/emacs-25 --with-modules
 --enable-checking=yes --enable-check-lisp-object-type
 --without-compress-install 'CFLAGS=-Og -ggdb3'
 LDFLAGS=-Lc:/Devel/emacs/lib 'CPPFLAGS=-DGC_MCHECK=1
 -Ic:/Devel/emacs/include''





             reply	other threads:[~2015-11-28  4:30 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-28  4:30 Drew Adams [this message]
2015-11-28  9:40 ` bug#22036: 25.0.50; (emacs) Lax Search Eli Zaretskii
2015-11-28 14:30   ` Drew Adams

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=6e6838d0-d8c3-4a19-a09f-39d2578b5a0b@default \
    --to=drew.adams@oracle.com \
    --cc=22036@debbugs.gnu.org \
    /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).