Thread: cleanup PL/PgSQL PERFORM
This patch makes a minor cleanup to the implementation of PERFORM in PL/PgSQL. Previously, it had been bundled together with the assign statement implementation, for some reason that wasn't clear to me (they certainly don't share any code with one another). So I separated them and made PERFORM a statement like any other. No changes in functionality. Along the way, I added some regression tests for PERFORM, added a bunch more SGML tags to the PL/PgSQL docs, and removed an obsolete comment relating to the implementation of RETURN NEXT. This is for 7.4; all the regression tests pass. Cheers, Neil -- Neil Conway <neilc@samurai.com> || PGP Key ID: DB3C29FC
Attachment
This has been saved for the 7.4 release:
http:/momjian.postgresql.org/cgi-bin/pgpatches2
---------------------------------------------------------------------------
Neil Conway wrote:
> This patch makes a minor cleanup to the implementation of PERFORM in
> PL/PgSQL. Previously, it had been bundled together with the assign
> statement implementation, for some reason that wasn't clear to me
> (they certainly don't share any code with one another). So I separated
> them and made PERFORM a statement like any other. No changes in
> functionality.
>
> Along the way, I added some regression tests for PERFORM, added a
> bunch more SGML tags to the PL/PgSQL docs, and removed an obsolete
> comment relating to the implementation of RETURN NEXT.
>
> This is for 7.4; all the regression tests pass.
>
> Cheers,
>
> Neil
>
> --
> Neil Conway <neilc@samurai.com> || PGP Key ID: DB3C29FC
[ Attachment, skipping... ]
>
> ---------------------------(end of broadcast)---------------------------
> TIP 6: Have you searched our list archives?
>
> http://archives.postgresql.org
--
Bruce Momjian | http://candle.pha.pa.us
pgman@candle.pha.pa.us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
Patch applied. Thanks. --------------------------------------------------------------------------- Neil Conway wrote: > This patch makes a minor cleanup to the implementation of PERFORM in > PL/PgSQL. Previously, it had been bundled together with the assign > statement implementation, for some reason that wasn't clear to me > (they certainly don't share any code with one another). So I separated > them and made PERFORM a statement like any other. No changes in > functionality. > > Along the way, I added some regression tests for PERFORM, added a > bunch more SGML tags to the PL/PgSQL docs, and removed an obsolete > comment relating to the implementation of RETURN NEXT. > > This is for 7.4; all the regression tests pass. > > Cheers, > > Neil > > -- > Neil Conway <neilc@samurai.com> || PGP Key ID: DB3C29FC [ Attachment, skipping... ] > > ---------------------------(end of broadcast)--------------------------- > TIP 6: Have you searched our list archives? > > http://archives.postgresql.org -- Bruce Momjian | http://candle.pha.pa.us pgman@candle.pha.pa.us | (610) 359-1001 + If your life is a hard drive, | 13 Roberts Road + Christ can be your backup. | Newtown Square, Pennsylvania 19073