Re: plperl features

From: Sergej Sergeev <sergej(at)commandprompt(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: plperl features
Date: 2004-09-30 16:28:25
Message-ID: 415C3429.7040904@commandprompt.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches


>Sergej Sergeev <sergej(at)commandprompt(dot)com> writes:
>
>
>>>What happens if you feed other pseudotypes, like cstring or
>>>language_handler? Shouldn't that be disallowed or something?
>>>
>>>
>
>
>
>>Other pseudo-types are disallowed (no-change)
>>
>>
>
>No, because you diked out the check at lines 1452ff, rather than
>upgrading it to something correct.
>
>I find the "fn_retispseudo" and "arg_is_p" flags pretty bogus anyway
>since they fail to indicate *which* pseudotype it is. You might as
>well just test for the specific type OID.
>
> regards, tom lane
>
>
New patch. I have added the check pseudo-type argumetns.
Specific type is substituted in runtime, during function call.

--
g.gRay: PL/perl, PL/PHP ;)

Attachment Content-Type Size
plperl.patch text/plain 6.5 KB

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2004-09-30 16:37:39 consultant
Previous Message JEDIDIAH 2004-09-30 15:54:50 Re: [HACKERS] Point in Time Recovery