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