| 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: | Whole Thread | Raw Message | 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 |
| 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 |