Re: printTable API (was: Show INHERIT in \du)

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: printTable API (was: Show INHERIT in \du)
Дата
Msg-id 20080413200320.GB21979@alvh.no-ip.org
обсуждение исходный текст
Ответ на Re: printTable API (was: Show INHERIT in \du)  ("Brendan Jurd" <direvus@gmail.com>)
Ответы Re: printTable API (was: Show INHERIT in \du)  ("Brendan Jurd" <direvus@gmail.com>)
Список 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 по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Remove lossy-operator RECHECK flag?
Следующее
От: Josh Berkus
Дата:
Сообщение: Re: Commit fest queue