Patch applied.  I also updated SET and RESET to be similar.

---------------------------------------------------------------------------

Nicolas Barbier wrote:
> 2006/4/15, Neil Conway <[EMAIL PROTECTED]>:
> 
> > On Sat, 2006-04-15 at 01:59 +0200, Nicolas Barbier wrote:
> >
> >> the following patch changes "parameter" to "gucname" in the grammar
> >> (and later references) for the SET syntax in ALTER ROLE and ALTER
> >> USER.
> >
> > Wouldn't "var_name", "varname", or something similar be more clear?
> > "GUC" is probably not an acronym we should be exposing prominently to
> > the user.
> 
> Indeed, that is what people on IRC also told me (after I sent it :-)).
> It now uses "varname". Updated patch attached.
> 
> Nicolas
> 
> --
> Nicolas Barbier
> http://www.gnu.org/philosophy/no-word-attachments.html

[ Attachment, skipping... ]

> 
> ---------------------------(end of broadcast)---------------------------
> TIP 6: explain analyze is your friend

-- 
  Bruce Momjian   http://candle.pha.pa.us
  EnterpriseDB    http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +
Index: doc/src/sgml/ref/alter_role.sgml
===================================================================
RCS file: /cvsroot/pgsql/doc/src/sgml/ref/alter_role.sgml,v
retrieving revision 1.3
diff -c -c -r1.3 alter_role.sgml
*** doc/src/sgml/ref/alter_role.sgml    18 Dec 2005 02:17:16 -0000      1.3
--- doc/src/sgml/ref/alter_role.sgml    25 Apr 2006 14:45:24 -0000
***************
*** 36,43 ****
  
  ALTER ROLE <replaceable class="PARAMETER">name</replaceable> RENAME TO 
<replaceable>newname</replaceable>
  
! ALTER ROLE <replaceable class="PARAMETER">name</replaceable> SET 
<replaceable>parameter</replaceable> { TO | = } { 
<replaceable>value</replaceable> | DEFAULT }
! ALTER ROLE <replaceable class="PARAMETER">name</replaceable> RESET 
<replaceable>parameter</replaceable>
  </synopsis>
   </refsynopsisdiv>
  
--- 36,43 ----
  
  ALTER ROLE <replaceable class="PARAMETER">name</replaceable> RENAME TO 
<replaceable>newname</replaceable>
  
! ALTER ROLE <replaceable class="PARAMETER">name</replaceable> SET 
<replaceable>varname</replaceable> { TO | = } { 
<replaceable>value</replaceable> | DEFAULT }
! ALTER ROLE <replaceable class="PARAMETER">name</replaceable> RESET 
<replaceable>varname</replaceable>
  </synopsis>
   </refsynopsisdiv>
  
***************
*** 143,149 ****
       </varlistentry>
  
       <varlistentry>
!       <term><replaceable>parameter</replaceable></term>
        <term><replaceable>value</replaceable></term>
        <listitem>
         <para>
--- 143,149 ----
       </varlistentry>
  
       <varlistentry>
!       <term><replaceable>varname</replaceable></term>
        <term><replaceable>value</replaceable></term>
        <listitem>
         <para>
Index: doc/src/sgml/ref/alter_user.sgml
===================================================================
RCS file: /cvsroot/pgsql/doc/src/sgml/ref/alter_user.sgml,v
retrieving revision 1.39
diff -c -c -r1.39 alter_user.sgml
*** doc/src/sgml/ref/alter_user.sgml    31 Jul 2005 17:19:17 -0000      1.39
--- doc/src/sgml/ref/alter_user.sgml    25 Apr 2006 14:45:25 -0000
***************
*** 36,43 ****
  
  ALTER USER <replaceable class="PARAMETER">name</replaceable> RENAME TO 
<replaceable>newname</replaceable>
  
! ALTER USER <replaceable class="PARAMETER">name</replaceable> SET 
<replaceable>parameter</replaceable> { TO | = } { 
<replaceable>value</replaceable> | DEFAULT }
! ALTER USER <replaceable class="PARAMETER">name</replaceable> RESET 
<replaceable>parameter</replaceable>
  </synopsis>
   </refsynopsisdiv>
  
--- 36,43 ----
  
  ALTER USER <replaceable class="PARAMETER">name</replaceable> RENAME TO 
<replaceable>newname</replaceable>
  
! ALTER USER <replaceable class="PARAMETER">name</replaceable> SET 
<replaceable>varname</replaceable> { TO | = } { 
<replaceable>value</replaceable> | DEFAULT }
! ALTER USER <replaceable class="PARAMETER">name</replaceable> RESET 
<replaceable>varname</replaceable>
  </synopsis>
   </refsynopsisdiv>
  
Index: doc/src/sgml/ref/reset.sgml
===================================================================
RCS file: /cvsroot/pgsql/doc/src/sgml/ref/reset.sgml,v
retrieving revision 1.28
diff -c -c -r1.28 reset.sgml
*** doc/src/sgml/ref/reset.sgml 25 Apr 2006 14:11:51 -0000      1.28
--- doc/src/sgml/ref/reset.sgml 25 Apr 2006 14:45:25 -0000
***************
*** 20,26 ****
  
   <refsynopsisdiv>
  <synopsis>
! RESET <replaceable class="PARAMETER">name</replaceable>
  RESET ALL
  </synopsis>
   </refsynopsisdiv>
--- 20,26 ----
  
   <refsynopsisdiv>
  <synopsis>
! RESET <replaceable class="PARAMETER">varname</replaceable>
  RESET ALL
  </synopsis>
   </refsynopsisdiv>
***************
*** 33,39 ****
     default values.  <command>RESET</command> is an alternative
     spelling for
  <synopsis>
! SET <replaceable class="parameter">parameter</replaceable> TO DEFAULT
  </synopsis>
     Refer to <xref linkend="sql-set" endterm="sql-set-title"> for
     details.
--- 33,39 ----
     default values.  <command>RESET</command> is an alternative
     spelling for
  <synopsis>
! SET <replaceable class="parameter">varname</replaceable> TO DEFAULT
  </synopsis>
     Refer to <xref linkend="sql-set" endterm="sql-set-title"> for
     details.
***************
*** 62,68 ****
  
    <variablelist>
     <varlistentry>
!     <term><replaceable class="PARAMETER">name</replaceable></term>
      <listitem>
       <para>
        The name of a run-time parameter. See <xref linkend="sql-set"
--- 62,68 ----
  
    <variablelist>
     <varlistentry>
!     <term><replaceable class="PARAMETER">varname</replaceable></term>
      <listitem>
       <para>
        The name of a run-time parameter. See <xref linkend="sql-set"
Index: doc/src/sgml/ref/set.sgml
===================================================================
RCS file: /cvsroot/pgsql/doc/src/sgml/ref/set.sgml,v
retrieving revision 1.86
diff -c -c -r1.86 set.sgml
*** doc/src/sgml/ref/set.sgml   10 Aug 2004 00:55:08 -0000      1.86
--- doc/src/sgml/ref/set.sgml   25 Apr 2006 14:45:25 -0000
***************
*** 20,26 ****
  
   <refsynopsisdiv>
  <synopsis>
! SET [ SESSION | LOCAL ] <replaceable class="PARAMETER">name</replaceable> { 
TO | = } { <replaceable class="PARAMETER">value</replaceable> | '<replaceable 
class="PARAMETER">value</replaceable>' | DEFAULT }
  SET [ SESSION | LOCAL ] TIME ZONE { <replaceable 
class="PARAMETER">timezone</replaceable> | LOCAL | DEFAULT }
  </synopsis>
   </refsynopsisdiv>
--- 20,26 ----
  
   <refsynopsisdiv>
  <synopsis>
! SET [ SESSION | LOCAL ] <replaceable class="PARAMETER">varname</replaceable> 
{ TO | = } { <replaceable class="PARAMETER">value</replaceable> | '<replaceable 
class="PARAMETER">value</replaceable>' | DEFAULT }
  SET [ SESSION | LOCAL ] TIME ZONE { <replaceable 
class="PARAMETER">timezone</replaceable> | LOCAL | DEFAULT }
  </synopsis>
   </refsynopsisdiv>
***************
*** 91,97 ****
     </varlistentry>
  
     <varlistentry>
!     <term><replaceable class="PARAMETER">name</replaceable></term>
      <listitem>
       <para>
        Name of a settable run-time parameter.  Available parameters are
--- 91,97 ----
     </varlistentry>
  
     <varlistentry>
!     <term><replaceable class="PARAMETER">varname</replaceable></term>
      <listitem>
       <para>
        Name of a settable run-time parameter.  Available parameters are
---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Reply via email to