pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Alvaro Herrera from 2ndQuadrant <alvherre@alvh.no-ip.org>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Andrew Gierth <andrew@tao11.riddles.org.uk>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: SQL-spec incompatibilities in similar_escape() and related stuff
Date: Fri, 6 Sep 2019 15:54:34 -0400
Message-ID: <20190906195434.GA19525@alvherre.pgsql> (raw)
In-Reply-To: <32617.1558649424@sss.pgh.pa.us>

On 2019-May-23, Tom Lane wrote:

> +   <para>
> +    Another nonstandard extension is that following the escape character
> +    with a letter or digit provides access to the same escape sequences
> +    defined for POSIX regular expressions, below (see
> +    <xref linkend="posix-character-entry-escapes-table"/>,
> +    <xref linkend="posix-class-shorthand-escapes-table"/>, and
> +    <xref linkend="posix-constraint-escapes-table"/>).
>     </para>

I think the word "same" in this para is more confusing than helpful;
also the tables are an integral part of this rather than just an
illustration, so they should not be in parenthesis but after only a
semicolon or such.  So:

> +    Another nonstandard extension is that following the escape character
> +    with a letter or digit provides access to the escape sequences
> +    defined for POSIX regular expressions; see
> +    <xref linkend="posix-character-entry-escapes-table"/>,
> +    <xref linkend="posix-class-shorthand-escapes-table"/>, and
> +    <xref linkend="posix-constraint-escapes-table"/> below.

I think it would be useful to provide a trivial example that illustrates
this in the <para> below; say '\mabc\M' not matching "zabc".

All in all, these are pretty trivial points and I would certainly not be
mad if it's committed without these changes.

Marked ready for committer.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services





view thread (16+ messages)  latest in thread

Message-ID: <20190906195434.GA19525@alvherre.pgsql>
Permalink:  ../20190906195434.GA19525@alvherre.pgsql/
Also on:    postgresql.org/message-id/20190906195434.GA19525@alvherre.pgsql

 · 

reply

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-hackers@postgresql.org
  Cc: alvherre@alvh.no-ip.org, tgl@sss.pgh.pa.us, andrew@tao11.riddles.org.uk, pgsql-hackers@lists.postgresql.org
  Subject: Re: SQL-spec incompatibilities in similar_escape() and related stuff
  In-Reply-To: <20190906195434.GA19525@alvherre.pgsql>

* 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