Re: [PATCHES] 2 line patch to allow plpythonu functions to return void ...

2006-02-25 Thread James Robinson
On Feb 25, 2006, at 12:10 PM, Tom Lane wrote: James Robinson <[EMAIL PROTECTED]> writes: Shamelessly cloned from the parallel code in pltcl, an exception for void in denying pseudotypes being returned. Pl/tcl didn't reference VOIDOID anywhere else, so ... . This sort of thing normally requir

Re: [PATCHES] 2 line patch to allow plpythonu functions to return void ...

2006-02-25 Thread Harald Armin Massa
Tom,This sort of thing normally requires more thought than just removingthe safety check.  What happens when the python code does/doesn't return a value, in both cases (declared return type void or not)?python functions are specified to return "None", if no return is given. I recommend to also see

Re: [PATCHES] 2 line patch to allow plpythonu functions to return void ...

2006-02-25 Thread Tom Lane
James Robinson <[EMAIL PROTECTED]> writes: > Shamelessly cloned from the parallel code in pltcl, an exception for > void in denying pseudotypes being returned. Pl/tcl didn't reference > VOIDOID anywhere else, so ... . This sort of thing normally requires more thought than just removing the saf

[PATCHES] 2 line patch to allow plpythonu functions to return void ...

2006-02-24 Thread James Robinson
Shamelessly cloned from the parallel code in pltcl, an exception for void in denying pseudotypes being returned. Pl/tcl didn't reference VOIDOID anywhere else, so ... . Allowed following trivial test function to succeed: create or replace function set_gd(int) returns void as $$ GD['g