Message-ID: <4EE4CBA9.2020507@gmail.com>
Date: Sun, 11 Dec 2011 16:26:33 +0100
From: Yeb Havinga <yebhavinga@gmail.com>
User-Agent: Mozilla/5.0 (Windows NT 5.1;
 rv:8.0) Gecko/20111105 Thunderbird/8.0
MIME-Version: 1.0
To: Kevin Grittner <Kevin.Grittner@wicourts.gov>
CC: pgsql-hackers@postgresql.org
Subject: Re: [REVIEW] Patch for cursor calling with named parameters
References: <4EDA37F7020000250004380D@gw.wicourts.gov>
	<4EDC97B7.6030508@gmail.com>
	<4EDC9042020000250004382F@gw.wicourts.gov>
	<4EDDF56D0200002500043914@gw.wicourts.gov>
In-Reply-To: <4EDDF56D0200002500043914@gw.wicourts.gov>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

On 2011-12-06 17:58, Kevin Grittner wrote:
> Kevin Grittner<kgrittn@wicourts.gov>  wrote:
>> Yeb Havinga<yebhavinga@gmail.com>  wrote:
>
>>> I personally tend to believe it doesn't even need to be an error.
>>> There is no technical reason not to allow it. All the user needs
>>> to do is make sure that the combination of named parameters and
>>> the positional ones together are complete and not overlapping.
>
>> If there are no objections, I suggest that Yeb implement the mixed
>> notation for cursor parameters.
>
> Hearing no objections -- Yeb, are you OK with doing this, and do you
> feel this is doable for this CF?

Attach is v6 of the patch, allowing mixed mode and with new test cases 
in the regression tests. One difference with calling functions remains: 
it is allowed to place positional arguments after named parameters. 
Including that would add code, but nothing would be gained.

regards,
Yeb Havinga