Re: printTable API (was: Show INHERIT in \du) - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: printTable API (was: Show INHERIT in \du)
Date
Msg-id 20080413200320.GB21979@alvh.no-ip.org
Whole thread Raw
In response to Re: printTable API (was: Show INHERIT in \du)  ("Brendan Jurd" <direvus@gmail.com>)
Responses Re: printTable API (was: Show INHERIT in \du)
List pgsql-hackers
Brendan Jurd escribió:

> I'd like to submit my first version of this patch for review.  I have
> introduced a new struct in print.h called printTableContent, which is
> used to compose the contents of a psql table.  The methods exposed for
> this struct are as follows:

Looks cool -- on a first read, I think you should add some more code
comments at the top of each function specifying whether the texts need
to be translated by the caller or done by the function itself.  Also it
would be good if it is consistent, too :-)

--
Alvaro Herrera                                http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Remove lossy-operator RECHECK flag?
Next
From: Josh Berkus
Date:
Subject: Re: Commit fest queue