all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Drew Adams <drew.adams@oracle.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 31807@debbugs.gnu.org
Subject: bug#31807: 27.0; `info-apropos' bad name or bad matching
Date: Thu, 14 Jun 2018 07:11:48 -0700 (PDT)	[thread overview]
Message-ID: <a4bf1dd4-d411-4a74-b5dd-9b7abaef9250@default> (raw)
In-Reply-To: <83o9gd33dj.fsf@gnu.org>

> > `info-apropos', as its doc string says, apparently does NOT do
> > "apropos" matching, i.e., regexp and keyword matching.  It
> apparently does literal
> > string matching against index entries in all of the manuals.
> >
> > Perhaps either the name should be changed (to not use the word
> > "apropos") or the behavior should be changed, to allow apropos
> > matching.
> 
> This command is the Emacs implementation of the "info --apropos"
> feature present in the Texinfo's stand-alone Info reader, which also
> looks for substring matches.  Apparently, "apropos" doesn't
> necessarily mean "regexps and keywords".

What matters, I think, is what Emacs means by "apropos", not
what Info might mean by it, especially not just what some
Info switch might be called.

In Emacs, "apropos" has always meant pattern matching, in
particular, regexp matching.  (Later, keyword matching was
added.)

> > I threw this together quickly, as a POC.  It does the job, but
> 
> Thanks.  The original command was introduced 11 years ago, so I don't
> think we can make radical changes in the user-visible behavior by
> default after all this time.

I disagree, here.  I think we _can_ make such a change,
and I don't think it's radical.

Plus, most literal string matches that someone might want
to make, in practice, do not involve any special regexp
chars, so they would still just "work".

If you want to keep the current behavior then I'd suggest
having two different commands AND, for the one that does
literal string matching, change the name to something that
does not include "apropos" in the name.

It's fine with me that any key-bindings (e.g., menu items)
for the literal-matching command be kept for the (newly
named) literal-matching command, if you like.  I don't
insist that that command or its keys be _supplanted_ by
a real apropos command.

The point is that (1) we can and should have an
apropos-matching command (as a replacement or in addition
to the current literal-matching command - I don't care
which), and (2) a literal-match command should not be
named `...-apropos' (in Emacs).

> This should be an opt-in optional
> behavior, controlled either by a prefix argument or by a (new) user
> variable.  Would you like to modify your suggestion along these lines?

No, I disagree that that is the right approach.
See above, for an alternative approach.

> Btw, does this really work as intended without changes to
> Info-apropos-matches?  That function calls regexp-quote on its
> argument.

Dunno.  When I get some time I'll take a closer look.
I don't insist on this code, which I threw together quickly.
Clearly, the apropos command should be made to work well.

The points are as stated above: provide a real apropos
command, and ensure that a command that matches only string
literals is not called `...-apropos'.





  reply	other threads:[~2018-06-14 14:11 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-12 23:01 bug#31807: 27.0; `info-apropos' bad name or bad matching Drew Adams
2018-06-12 23:21 ` Drew Adams
2018-06-12 23:39   ` Drew Adams
2018-06-12 23:59     ` Drew Adams
2018-06-14 13:29       ` Eli Zaretskii
2018-06-13  0:04     ` Noam Postavsky
2018-06-13  0:16       ` Drew Adams
2018-06-13 18:24         ` Drew Adams
     [not found]     ` <<eb245b09-a156-4585-8021-7a3e0e173dbc@default>
     [not found]       ` <<83muvx33ab.fsf@gnu.org>
2018-06-14 14:16         ` Drew Adams
2018-06-17  7:44           ` Eli Zaretskii
     [not found]     ` <<<eb245b09-a156-4585-8021-7a3e0e173dbc@default>
     [not found]       ` <<<83muvx33ab.fsf@gnu.org>
     [not found]         ` <<f7814f7b-d9c0-48d5-83aa-b354b047db10@default>
     [not found]           ` <<83muvtuab9.fsf@gnu.org>
2018-06-17 14:26             ` Drew Adams
2018-06-14 13:27 ` Eli Zaretskii
2018-06-14 14:11   ` Drew Adams [this message]
2018-06-14 14:28     ` Eli Zaretskii
     [not found] <<<78156de2-86c8-4593-92b5-59edf7a8996e@default>
     [not found] ` <<<83o9gd33dj.fsf@gnu.org>
     [not found]   ` <<<a4bf1dd4-d411-4a74-b5dd-9b7abaef9250@default>
     [not found]     ` <<<83in6l30ir.fsf@gnu.org>
     [not found]       ` <<5fc07c84-b8d2-493c-a17b-774ad1c213bf@default>
     [not found]         ` <<83fu1p2ym7.fsf@gnu.org>
2018-06-14 15:33           ` Drew Adams
2018-06-14 15:50             ` Eli Zaretskii
     [not found] <<<<78156de2-86c8-4593-92b5-59edf7a8996e@default>
     [not found] ` <<<<83o9gd33dj.fsf@gnu.org>
     [not found]   ` <<<<a4bf1dd4-d411-4a74-b5dd-9b7abaef9250@default>
     [not found]     ` <<<<83in6l30ir.fsf@gnu.org>
     [not found]       ` <<<5fc07c84-b8d2-493c-a17b-774ad1c213bf@default>
     [not found]         ` <<<83fu1p2ym7.fsf@gnu.org>
     [not found]           ` <<73c2c21d-9fba-4ab5-a0b0-8b36e9a23bcf@default>
     [not found]             ` <<83efh92wr2.fsf@gnu.org>
2018-06-14 15:58               ` Drew Adams
     [not found] <<78156de2-86c8-4593-92b5-59edf7a8996e@default>
     [not found] ` <<83o9gd33dj.fsf@gnu.org>
     [not found]   ` <<a4bf1dd4-d411-4a74-b5dd-9b7abaef9250@default>
     [not found]     ` <<83in6l30ir.fsf@gnu.org>
2018-06-14 14:42       ` Drew Adams
2018-06-14 15:09         ` Eli Zaretskii
2018-06-14 20:45       ` Drew Adams
2022-04-21 15:05         ` Lars Ingebrigtsen
2022-04-21 20:04           ` 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

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

  git send-email \
    --in-reply-to=a4bf1dd4-d411-4a74-b5dd-9b7abaef9250@default \
    --to=drew.adams@oracle.com \
    --cc=31807@debbugs.gnu.org \
    --cc=eliz@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 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.