Re: Revised patches to add table function support to PL/Tcl (TODO item) - Mailing list pgsql-hackers

From Karl Lehenbauer
Subject Re: Revised patches to add table function support to PL/Tcl (TODO item)
Date
Msg-id 26EA832F-C907-410B-846B-57EDC26F142A@gmail.com
Whole thread Raw
In response to Re: Revised patches to add table function support to PL/Tcl (TODO item)  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Revised patches to add table function support to PL/Tcl (TODO item)  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
On Dec 28, 2010, at 7:29 PM, Tom Lane wrote:
> This patch appears to be changing a whole lot of stuff that in fact
> pg_indent has never changed, so there's something wrong with the way you
> are doing it.  It looks like a bad typedef list from here.

You were right, Tom.  The problem was that typedefs "pltcl_interp_desc", "pltcl_proc_key", and "pltcl_proc_ptr" weren't
insrc/tools/pgindent/typedefs.list.  After adding them (and building and installing the netbsd-based, patched indent),
pgindentonly changes a handful of lines. 

pltcl-karl-try3-1-of-3-pgindent.patch patches typedefs.list with the three missing typedefs and pltcl.c with the small
changesmade by pgindent (it shifted some embedded comments left within their lines, mainly). 

As before, but "try3" now, pltcl-karl-try3-2-of-3-objects.patch converts pltcl.c to use the "Tcl objects" C API.

And as before, but "try3" now, pltcl-karl-try3-3-of-3-setof.patch adds returning record and SETOF record.







Attachment

pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: pg_dump --split patch
Next
From: Joel Jacobson
Date:
Subject: Re: pg_dump --split patch