Re: Package namespace and Safe init cleanup for plperl [PATCH]

Tim Bunce <tim.bunce@pobox.com>

From: Tim Bunce <Tim.Bunce@pobox.com>
To: pgsql-hackers@postgresql.org
Cc: Tim Bunce <Tim.Bunce@pobox.com>
Date: 2010-01-25T19:53:26Z
Lists: pgsql-hackers

Attachments

On Fri, Jan 15, 2010 at 04:02:02AM +0000, Tim Bunce wrote:
> This is the final plperl patch in the series from me.
> 
> Changes in this patch:
> 
> - Moved internal functions out of main:: namespace
>     into PostgreSQL::InServer and PostgreSQL::InServer::safe
> 
> - Restructured Safe compartment setup code
>     to generalize and separate the data from the logic.
> 
> Neither change has any user visible effects.

This is a revised version of the patch with the following additional
changes:

- Further generalized the 'what to load into Safe compartment' logic.

- Added the 'warnings' pragma to the list of modules to load into Safe.
  So plperl functions can now "use warnings;" - added test for that.

- Added 'use 5.008001;' to plc_perlboot.pl as a run-time check to
  complement the configure-time check added by Tom Lane recently.

Tim.