Re: PL/Python warnings in CVS HEAD

Patch applied. Thanks.

---------------------------------------------------------------------------

Marko Kreen wrote:
> On 4/3/07, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> > Great, patch applied and TODO item removed.
>
> If you are already patching, please apply the following patch also.
>
> It removes last remaining casts inside struct definitions.
> Such usage is bad practice, as it hides problems from compiler.
>
> Reason for the cast is popular practice in some circles
> to define functions as foo(MyObj *) instead of foo(PyObject *)
> thus avoiding a local variable inside functions and make
> direct calling easier. As pl/python does not use such style,
> the casts were unnecessary from the start.
>
> --
> marko

[ Attachment, skipping... ]

>
> ---------------------------(end of broadcast)---------------------------
> TIP 9: In versions below 8.0, the planner will ignore your desire to
> choose an index scan if your joining column's datatypes do not
> match

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +