Re: WIP: pl/pgsql cleanup

From: Neil Conway <neilc(at)samurai(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: WIP: pl/pgsql cleanup
Date: 2005-02-11 00:36:00
Message-ID: 1108082160.1286.198.camel@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

On Thu, 2005-02-10 at 10:15 -0500, Tom Lane wrote:
> Yeah, looks better, though I question the use of "embedded" in the
> message --- seems a bit jargony. Are you going to post a revised patch?

Actually the code to present error messages as in the previous message
was in the previous patch, just #if 0'd out :)

Attached is a revised patch that removes "embedded" and updates the
regression tests. I'll apply this to HEAD later today barring any
further suggestions for improvement.

-Neil

Attachment Content-Type Size
plpgsql_cleanup-27.patch.gz application/x-gzip 24.1 KB

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2005-02-11 00:36:44 Re: Clarify use of NOW() in pl/pgsql docs
Previous Message Neil Conway 2005-02-11 00:05:10 Re: Clarify use of NOW() in pl/pgsql docs