From: | "Nicolas Barbier" <nicolas(dot)barbier(at)gmail(dot)com> |
---|---|
To: | "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | "PostgreSQL Patches" <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Documentation patch: change a name in a grammar rule |
Date: | 2006-04-15 21:21:54 |
Message-ID: | b0f3f5a10604151421j1e24def1m3560ef35bb0fa229@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
2006/4/15, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>:
> s/prominently/at all/ ... I don't think the proposed patch is an
> improvement, and in fact see nothing wrong with the use of "parameter"
> here. If we want to abandon "parameter" as the official documentation
> term for GUC variables, then there are dozens or hundreds of changes
> to be made in config.sgml, to say nothing of other files.
In for example set.sgml, just "name" is used for the exact same
concept. I changed the patch so that "varname" is used, because "name"
was already in use by this specific grammar. Maybe the usage of "var"
makes it less clear indeed, for people that know that those things are
always referred to as "parameters" and never as "variables". But then,
leaving it as-is doesn't solve the apparently occuring confusion
between "parameter" the GUC and "parameter" the "parameter to the
statement", as we have had at least three people in #postgresql that
made that exact mistake.
Nicolas
--
Nicolas Barbier
http://www.gnu.org/philosophy/no-word-attachments.html
From | Date | Subject | |
---|---|---|---|
Next Message | Jaime Casanova | 2006-04-16 08:22:08 | Re: Proposed doc-patch: Identifying the Current WAL file |
Previous Message | Bruce Momjian | 2006-04-15 20:20:45 | Re: Proposed doc-patch: Identifying the Current WAL file |