Re: revised patch for PL/PgSQL table functions

Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
> Neil Conway <neilc(at)samurai(dot)com> writes:
> > Here's another improved patch. I got the parser to accept 'RETURN
> > NEXT' properly

> Consider doing it the way that the main parser converts "UNION JOIN"
> into a single token --- viz, there's an outer filter that calls the
> lexer an extra time to look ahead one token when necessary. Doing this
> in the lexer is really quite messy if you want to do it right (eg,
> deal correctly with comments between the two keywords).

Ah, ok -- thanks for the tip.

I've attached a revised patch: rediffed against CVS HEAD, changed the
RETURN NEXT parsing as suggested by Tom above, and made some more
minor cleanups to PL/PgSQL code.

Cheers,

Neil

--
Neil Conway <neilc(at)samurai(dot)com> || PGP Key ID: DB3C29FC