Re: suspicious pointer/integer coersion

Tom Lane wrote:

>Andrew Dunstan <andrew@dunslane.net> 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