From: Tom Lane <tgl@sss.pgh.pa.us>
To: Bruce Momjian <bruce@momjian.us>
Cc: troy@frericks.us
Cc: pgsql-docs@lists.postgresql.org
Subject: Re: Manua correction
Date: Tue, 20 Jul 2021 11:03:09 -0400
Message-ID: <400208.1626793389@sss.pgh.pa.us> (raw)
In-Reply-To: <20210719142355.GA12687@momjian.us>
References: <162614304115.701.2392941350859387646@wrigleys.postgresql.org>
<20210715213623.GC14027@momjian.us>
<3701102.1626405158@sss.pgh.pa.us>
<20210719142355.GA12687@momjian.us>
Bruce Momjian <bruce@momjian.us> writes:
> On Thu, Jul 15, 2021 at 11:12:38PM -0400, Tom Lane wrote:
>> At least two of these changes are flat out wrong. The places
>> that explicitly mention "substring(s)" are not doing so because
>> we failed to think about what that meant.
> I really don't understand the use of "(s)" except in place where we are
> really trying to point out the idea of one or multiple, and I don't see
> that being significant in these cases --- can you clarify?
See the example given for regexp_match:
regression=# select regexp_match('foobarbequebaz', '(bar)(beque)');
regexp_match
--------------
{bar,beque}
(1 row)
There's more than one parenthesized subpattern, so you get more than
one substring in the result. So I think that change is flat out
wrong.
The places where you changed "substring(s)" to "substrings" are maybe
not flat wrong, but I don't think they're improving the text either.
IIRC, in most of them you get one match if you didn't use the 'g'
flag, but possibly multiple matches if you did, and the "substring(s)"
wording is meant to allude to that without taking the space to spell
it out explicitly.
regards, tom lane
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-docs@postgresql.org
Cc: tgl@sss.pgh.pa.us, bruce@momjian.us, troy@frericks.us, pgsql-docs@lists.postgresql.org
Subject: Re: Manua correction
In-Reply-To: <400208.1626793389@sss.pgh.pa.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox