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

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
Cc: Brendan Jurd <direvus(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org
Subject: Re: Removing typename from A_Const (was: Empty arrays with ARRAY[])
Date: 2008-04-28 16:50:18
Message-ID: 13756.1209401418@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Alvaro Herrera <alvherre(at)commandprompt(dot)com> writes:
> Brendan Jurd escribi:
>> 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.

> Do say ... why don't we do away with A_Const altogether and just replace
> it with Value? After this patch, I don't see what's the difference.

They're logically different things, and after I get done putting a parse
location field into A_Const, they'll still be physically different too.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2008-04-28 16:54:34 Re: [HACKERS] Removing typename from A_Const (was: Empty arrays with ARRAY[])
Previous Message Alvaro Herrera 2008-04-28 16:27:02 Re: Removing typename from A_Const (was: Empty arrays with ARRAY[])

Browse pgsql-patches by date

  From Date Subject
Next Message Alvaro Herrera 2008-04-28 16:54:34 Re: [HACKERS] Removing typename from A_Const (was: Empty arrays with ARRAY[])
Previous Message Alvaro Herrera 2008-04-28 16:27:02 Re: Removing typename from A_Const (was: Empty arrays with ARRAY[])