Removing typename from A_Const (was: Empty arrays with ARRAY[]) - Mailing list pgsql-patches

From Brendan Jurd
Subject Removing typename from A_Const (was: Empty arrays with ARRAY[])
Date
Msg-id 37ed240d0804141423y36a29943r20bab71df9531140@mail.gmail.com
Whole thread Raw
Responses Re: Removing typename from A_Const (was: Empty arrays with ARRAY[])
List 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-----

Attachment

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Terminating a backend
Next
From: Alvaro Herrera
Date:
Subject: Re: explicit tracking of ActiveSnapshot