Initial refactoring of plperl.c [PATCH] - Mailing list pgsql-hackers

From Tim Bunce
Subject Initial refactoring of plperl.c [PATCH]
Date
Msg-id 20091125153625.GA55857@timac.local
Whole thread Raw
In response to Initial refactoring of plperl.c - draft [PATCH]  (Tim Bunce <Tim.Bunce@pobox.com>)
Responses Re: Initial refactoring of plperl.c [PATCH]
Re: Initial refactoring of plperl.c [PATCH]
List pgsql-hackers
Following on from my earlier draft plperl.c refactoring patch, here's a
new version that's complete (from my perspective at least).

I've started work on the enhancements to plperl I outlined on pg-general
(in the "Wishlist of PL/Perl Enhancements for 8.5" thread).
I have a working implementation of those changes, plus some performance
enhancements, that I'm now re-working into a clean set of tested and
polished patches.

This patch is a first step that doesn't add any extra functionality.
It refactors the internals to make adding the extra functionality
easier (and more clearly visible).

Changes in this patch:

- Changed MULTIPLICITY check from runtime to compiletime.
    No loads the large Config module.
- Changed plperl_init_interp() to return new interp
    and not alter the global interp_state
- Moved plperl_safe_init() call into check_interp().
- Removed plperl_safe_init_done state variable
    as interp_state now covers that role.
- Changed plperl_create_sub() to take a plperl_proc_desc argument.
- Simplified return value handling in plperl_create_sub.
- Added a test for the effect of the utf8fix function.
- Changed perl.com link in the docs to perl.org and tweaked
    wording to clarify that require, not use, is what's blocked.
- Moved perl code in large multi-line C string literal macros
    out to plc_*.pl files.
- Added a test2macro.pl utility to convert the plc_*.pl files to
    macros in a perlchunks.h file which is #included

I'd appreciate any feedback on the patch.

Tim.


Attachment

pgsql-hackers by date:

Previous
From: Magnus Hagander
Date:
Subject: cvs chapters in our docs
Next
From: Stefan Kaltenbrunner
Date:
Subject: Re: garbage in psql -l