Removing typename from A_Const (was: Empty arrays with ARRAY[])

Поиск
Список
Период
Сортировка
От Brendan Jurd
Тема Removing typename from A_Const (was: Empty arrays with ARRAY[])
Дата
Msg-id 37ed240d0804141423y36a29943r20bab71df9531140@mail.gmail.com
обсуждение исходный текст
Ответы Re: Removing typename from A_Const (was: Empty arrays with ARRAY[])  (Alvaro Herrera <alvherre@commandprompt.com>)
Список pgsql-patches
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Fri, Mar 21, 2008 at 7:47 AM, Tom Lane  wrote:
>  I didn't do anything about removing A_Const's typename field, but I'm
>  thinking that would be a good cleanup patch.
>

Here's my attempt to remove the typename field from A_Const.  There
were a few places (notably flatten_set_variable_args() in guc.c, and
typenameTypeMod() in parse_type.c) where the code expected to see an
A_Const with a typename, and I had to adjust for an A_Const within a
TypeCast.  Nonetheless, there was an overall net reduction of 34 lines
of code, so I think this was a win.

All regression tests passed on x86_64 gentoo.

Added to May CommitFest.

Cheers,
BJ
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.7 (GNU/Linux)
Comment: http://getfiregpg.org

iD8DBQFIA8s/5YBsbHkuyV0RAlMdAJ0dWdoZd5cypvInAR2msO8XA8qqxACeILSw
bCI2TGAQI3m3TBoJspvV4OQ=
=dGP9
-----END PGP SIGNATURE-----

Вложения

В списке pgsql-patches по дате отправления:

Предыдущее
От: Bruce Momjian
Дата:
Сообщение: Terminating a backend
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: explicit tracking of ActiveSnapshot