Tom Lane wrote:

Andrew Dunstan <[EMAIL PROTECTED]> writes:
I have just noticed this code in plperl.c:

       hv_store(plperl_proc_hash, internal_proname, proname_len,
                newSViv((IV) prodesc), 0);

basically, here prodesc is a pointer to a struct, and we are storing it as an integer in a perl hash for easy lookup by stringified oid.

How safe is this conversion on 64 bit platforms?

Not at all.  I'd vote for converting the whole thing to a dynahash
hashtable ...



Works for me. There are some other things about the procdesc stuff I'm trying to sort out (especially if we should be storing per-call info inside it).

Cleaning this up is definitely a TODO.

cheers

andrew

---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster

Reply via email to