public inbox for [email protected]  
help / color / mirror / Atom feed
From: Bruce Momjian <[email protected]>
To: Alvaro Herrera <[email protected]>
Cc: Dave Stewart <[email protected]>
Cc: David G. Johnston <[email protected]>
Cc: Tom Lane <[email protected]>
Cc: Pg Docs <[email protected]>
Subject: Re: Conventions
Date: Wed, 26 Jan 2022 18:32:04 -0500
Message-ID: <YfHZ9Ecpiy/[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>

On Wed, Jan 26, 2022 at 02:01:40PM -0300, Álvaro Herrera wrote:
> On 2022-Jan-26, Bruce Momjian wrote:
> 
> > On Tue, Jan 25, 2022 at 06:45:47PM -0800, Dave Stewart wrote:
> > > I'm good with it.  Thank you!
> > 
> > OK, updated patch attached.  I don't think we even show TCL syntax
> > anywhere anymore, so I removed that text, rather than moving it.
> 
> We do here:
> https://www.postgresql.org/docs/14/pltcl-dbaccess.html
> Maybe not worth bothering, though.  I think it should be obvious to
> someone already familiar with the language; and for those who aren't,
> they probably won't be reading this page.

Oh, that helps.  I looked for '?' in pltcl.sgml and when I couldn't find
any, I think the use of the question-mark syntax was gone, but I do see
it in the output:

	https://www.postgresql.org/docs/14/pltcl-dbaccess.html

and then I found this in stylesheet-common.xsl:

	<!-- Special support for Tcl synopses -->
	
	<xsl:template match="optional[@role='tcl']">
-->	  <xsl:text>?</xsl:text>
	  <xsl:call-template name="inline.charseq"/>
	  <xsl:text>?</xsl:text>
	</xsl:template>

The attached patch moves the TCL syntax mention to a more appropriate
place.

-- 
  Bruce Momjian  <[email protected]>        https://momjian.us
  EDB                                      https://enterprisedb.com

  If only the physical world exists, free will is an illusion.



Attachments:

  [text/x-diff] paren.diff (1.3K, 2-paren.diff)
  download | inline diff:
diff --git a/doc/src/sgml/notation.sgml b/doc/src/sgml/notation.sgml
index bd1e8f629a..c3798d5f1b 100644
--- a/doc/src/sgml/notation.sgml
+++ b/doc/src/sgml/notation.sgml
@@ -6,12 +6,12 @@
  <para>
   The following conventions are used in the synopsis of a command:
   brackets (<literal>[</literal> and <literal>]</literal>) indicate
-  optional parts.  (In the synopsis of a Tcl command, question marks
-  (<literal>?</literal>) are used instead, as is usual in Tcl.)  Braces
+  optional parts.  Braces
   (<literal>{</literal> and <literal>}</literal>) and vertical lines
   (<literal>|</literal>) indicate that you must choose one
   alternative.  Dots (<literal>...</literal>) mean that the preceding element
-  can be repeated.
+  can be repeated.  All other symbols, including parentheses, should be
+  taken literally.
  </para>
 
  <para>
diff --git a/doc/src/sgml/pltcl.sgml b/doc/src/sgml/pltcl.sgml
index 1759fc4498..6ef968e078 100644
--- a/doc/src/sgml/pltcl.sgml
+++ b/doc/src/sgml/pltcl.sgml
@@ -321,7 +321,8 @@ $$ LANGUAGE pltcl;
 
     <para>
      The following commands are available to access the database from
-     the body of a PL/Tcl function:
+     the body of a PL/Tcl function (brackets (<literal>[</literal> and
+     <literal>]</literal>) are represented here as question marks):
 
     <variablelist>
 


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: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Conventions
  In-Reply-To: <YfHZ9Ecpiy/[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox