Re: Revised patches to add table function support to PL/Tcl (TODO item)

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Karl Lehenbauer <karllehenbauer(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: Revised patches to add table function support to PL/Tcl (TODO item)
Date: 2011-02-09 01:37:57
Message-ID: 4D51EFF5.4020907@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 02/07/2011 11:30 PM, Robert Haas wrote:
> On Tue, Dec 28, 2010 at 9:23 PM, Karl Lehenbauer
> <karllehenbauer(at)gmail(dot)com> wrote:
>> 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 in src/tools/pgindent/typedefs.list. After adding them (and building and installing the netbsd-based, patched indent), pgindent only 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 changes made 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.
> This patch did not get reviewed, because the person who originally
> planned to review it had a hardware failure that prevented him from
> doing so. Can anyone pick this up?

I will have a look at it.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alex Hunsaker 2011-02-09 01:44:00 Re: arrays as pl/perl input arguments [PATCH]
Previous Message Itagaki Takahiro 2011-02-09 01:31:00 Re: pg_dump directory archive format / parallel pg_dump