From: "Mattias Engdegård" <mattiase@acm.org>
To: Stefan Kangas <stefan@marxist.se>
Cc: 46910-done@debbugs.gnu.org
Subject: bug#46910: 27.1; (elisp) RX doc
Date: Sun, 26 Sep 2021 14:44:42 +0200 [thread overview]
Message-ID: <1CB22E13-F19B-4B9B-B31C-6F5CFC9E8A51@acm.org> (raw)
In-Reply-To: <CADwFkm=WsHnt55E0otrBLBjW_403TuGzL1o2h0gH_2pEWEX22A@mail.gmail.com>
25 sep. 2021 kl. 17.25 skrev Stefan Kangas <stefan@marxist.se>:
>
> Drew Adams <drew.adams@oracle.com> writes:
>
>> 1. `Rx Constructs' doesn't explain that the "constructs" are used in
>> arguments to macro `rx'.
Yes, I've added an explanation.
>> 2. `Rx Constructs': Don't show things like `\sCODE' without saying what
>> CODE means. If the column title for the table explaining RX names is
>> `Syntax character' or `Category character' then that's apparently what
>> CODE corresponds to for `\sCODE' or `\cCODE'. But there's no way to
>> know that unless you're already familiar with `\s' and `\c' syntax.
That's a valid point; now elaborated.
If anything I'd like to move the tables of syntax and (especially) category names to a separate page since they are rarely consulted and take up a lot of space. Not sure how to best do that.
>> 3. `Rx Constructs', `Zero-width assertions': Call this `Zero-width
>> matching' or some such.
Thank you, but it's established terminology and immediately explained in the first sentence.
>> 4. `Rx Functions': RX-EXPR is used as parameter name for what is, I
>> guess an Rx construct
That was a clear mistake, now fixed -- thanks.
>> 5. `Rx Functions': A couple places talk about `'regexp' forms', but
>> those are specified nowhere
The `regexp` and `literal` forms are both documented on the previous page.
For good measure, I also added some examples to illustrate the difference between `rx` and `rx-to-string`.
prev parent reply other threads:[~2021-09-26 12:44 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-03 23:15 bug#46910: 27.1; (elisp) RX doc Drew Adams
2021-09-25 15:25 ` Stefan Kangas
2021-09-26 12:44 ` Mattias Engdegård [this message]
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=1CB22E13-F19B-4B9B-B31C-6F5CFC9E8A51@acm.org \
--to=mattiase@acm.org \
--cc=46910-done@debbugs.gnu.org \
--cc=stefan@marxist.se \
/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.