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-18 06:27:41
Message-ID: 1108708061.32120.10.camel@localhost.localdomain
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

On Thu, 2005-02-10 at 23:32 -0500, Tom Lane wrote:
> Congratulations, you just reinvented the scheme we used before 8.0.
> It's *not* an improvement. The dot-dot business is better.

Right -- but it's not very good either, and I was trying to find a
proper solution. For now I've given up and reverted the code to use the
dot-dot technique.

> > Yes, that's a good point. I'll change the patch to just elide the
> > previous entry from the stack of callbacks, which is going to be
> > plpgsql_compile_error_callback (unfortunately we can't actually verify
> > that, AFAICS, since that callback is private to pl_comp.c)
>
> IMHO verifying that is well worth an extern.

Attached is a revised patch that incorporates your suggestions. Sorry
for the delay in posting this.

Barring any further objections, I'll commit this to HEAD tomorrow.

-Neil

Attachment Content-Type Size
plpgsql_cleanup-32.patch.gz application/x-gzip 24.8 KB

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Dave Page 2005-02-18 12:49:36 Re: UTF8 or Unicode
Previous Message Zhenbang Wei 2005-02-18 06:14:31 Update Traditional Chinese translations for 8.0 branch