Re: [PATCHES] Bundle of patches - Mailing list pgsql-hackers

From Teodor Sigaev
Subject Re: [PATCHES] Bundle of patches
Date
Msg-id 45747A0B.5090903@sigaev.ru
Whole thread Raw
In response to Re: [PATCHES] Bundle of patches  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCHES] Bundle of patches  (Teodor Sigaev <teodor@sigaev.ru>)
List pgsql-hackers

> This one seems generally workable, but I really dislike the approach
> that's been used for passing typmod arguments to the typmod_in function.
> Representing them with an "internal" parameter means it'll be forever
> impossible to write typmod functions in anything but C, which seems an
> ugly restriction.  Perhaps an array of int4 would be better?  How much
I don't think that is a problem - I'll change that

> flexibility do we really want to provide for typmod arguments?  Allowing
> full "expr_list" in the grammar seems less than sane, considering the
> result is still going to have to pack into 32 bits.
As I remember, I tried to use some thing else but, I've got a lot conflicts
with
AexprConst:
    func_name '(' expr_list ')' Sconst

>
> The patch needs more cleanup before applying, too, eg make comments
> match code, get rid of unused keywords added to gram.y.
Ok.

--
Teodor Sigaev                                   E-mail: teodor@sigaev.ru
                                                    WWW: http://www.sigaev.ru/

pgsql-hackers by date:

Previous
From: Teodor Sigaev
Date:
Subject: Re: [PATCHES] Bundle of patches
Next
From: Teodor Sigaev
Date:
Subject: Re: [PATCHES] Bundle of patches