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

From: "Brendan Jurd" <direvus(at)gmail(dot)com>
To: "Alvaro Herrera" <alvherre(at)commandprompt(dot)com>
Cc: pgsql-patches(at)postgresql(dot)org, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: printTable API (was: Show INHERIT in \du)
Date: 2008-05-10 01:31:45
Message-ID: 37ed240d0805091831x34a41fb8y6c845bc3164a5bb7@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Hi guys,

Here's the latest version of the printTable API. This version is
against the current HEAD and merges in the changes made by the
recently committed psql wrap patch.

This version also includes Alvaro's fix for the issue of pg_strdup not
being available to programs in scripts/ (as quoted below).

Clean compile and all regression tests passed on amd64 gentoo.

Cheers,
BJ

On Thu, May 8, 2008 at 7:55 AM, Alvaro Herrera
<alvherre(at)commandprompt(dot)com> wrote:
> FWIW I just noticed something else. With this patch we add pg_strdup
> calls into print.c. pg_strdup lives in common.c.
>
> This is fine as psql is concerned, but we have another problem which is
> that in bin/scripts there are two programs that want to use
> printQuery(). The problem is that there's no pg_strdup there :-(
>
> The easy solution is to add pg_strdup to bin/scripts/common.c, but there
> we don't have a global progname, so the error report in the out of
> memory case cannot carry the name of the program crashing.
>
> I don't like that, but I don't see any other solution. Ideas welcome.
>
> --
> Alvaro Herrera http://www.CommandPrompt.com/
> PostgreSQL Replication, Consulting, Custom Development, 24x7 support
>

Attachment Content-Type Size
print-table-5.diff.bz2 application/x-bzip2 15.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2008-05-10 01:32:17 Re: Database owner installable modules patch
Previous Message Alex Hunsaker 2008-05-10 00:41:22 Re: [badalex@gmail.com: Re: [BUGS] Problem identifying constraints which should not be inherited]

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2008-05-10 01:32:17 Re: Database owner installable modules patch
Previous Message Bruce Momjian 2008-05-10 01:22:32 Re: [PATCH] sh: Add support Renesas SuperH