Re: ALTER INDEX - Mailing list pgsql-patches

From Stefan Kaltenbrunner
Subject Re: ALTER INDEX
Date
Msg-id 411C8021.3050708@kaltenbrunner.cc
Whole thread Raw
In response to Re: ALTER INDEX  (Gavin Sherry <swm@linuxworld.com.au>)
Responses Re: ALTER INDEX  (Gavin Sherry <swm@linuxworld.com.au>)
Re: ALTER INDEX  (Bruce Momjian <pgman@candle.pha.pa.us>)
Re: ALTER INDEX  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-patches
Gavin Sherry wrote:


> Index: src/bin/psql/tab-complete.c
> ===================================================================
> RCS file: /usr/local/cvsroot/pgsql-server/src/bin/psql/tab-complete.c,v
> retrieving revision 1.109
> diff -2 -c -r1.109 tab-complete.c
> *** src/bin/psql/tab-complete.c    28 Jul 2004 14:23:30 -0000    1.109
> --- src/bin/psql/tab-complete.c    13 Aug 2004 06:34:55 -0000
> ***************
> *** 633,637 ****
>       {
>           static const char *const list_ALTER[] =
> !         {"DATABASE", "GROUP", "SCHEMA", "TABLE", "TRIGGER", "USER", NULL};
>
>           COMPLETE_WITH_LIST(list_ALTER);
> --- 633,638 ----
>       {
>           static const char *const list_ALTER[] =
> !         {"DATABASE", "GROUP", "SCHEMA", "TABLE", "TRIGGER", "USER", "INDEX",
> !              NULL};
>
>           COMPLETE_WITH_LIST(list_ALTER);
> ***************
> *** 647,650 ****
> --- 648,661 ----
>           COMPLETE_WITH_LIST(list_ALTERDATABASE);
>       }
> +     /* ALTER INDEX <name> */
> +     else if (pg_strcasecmp(prev3_wd, "ALTER") == 0 &&
> +              pg_strcasecmp(prev2_wd, "INDEX") == 0)
> +     {
> +         static const char *const list_ALTERDATABASE[] =
> +         {"SET TABLESPACE", "OWNER TO", "RENAME TO", NULL};
> +
> +         COMPLETE_WITH_LIST(list_ALTERDATABASE);

minor issue/nit(?): reusing list_ALTERDATABASE for the ALTER INDEX part
looks a little strange ...


Stefan(who could really need some feedback on his own tab-complete patch
*g*)

pgsql-patches by date:

Previous
From: Gavin Sherry
Date:
Subject: Re: ALTER INDEX
Next
From: Gavin Sherry
Date:
Subject: Re: ALTER INDEX