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.