New patch.
On 27-Apr-06, at 10:15 PM, Kris Jurka wrote:
>
>
> On Thu, 27 Apr 2006, Dave Cramer wrote:
>
>> Patch attached for review and test
>>
>
> This patch does not handle the situation where the number of
> registered parameters does not match the number of returned
> parameters as the attached tests show.
>
> Also the regression tests you sent me separately are no good. They
> do not pass because they mix creating and destroying the test
> function in the test method and the tearDown method. This should
> go into setUp and tearDown. Also you can't call executeQuery on a
> CallableStatement because it will complain with "No results were
> returned by the query."
>
> The whole Jdbc3CallableStatementTest sucks. It should have all of
> its function/table creation destruction moved into setUp/tearDown.
> Also the coding:
>
> try {
> // something
> } catch(Exception e) {
> fail(e.getMessage();
> }
>
> Is a waste of code. Not only does it not do anything, but it also
> obscures the source of the exception.
>
> Kris Jurka
> <OutFunc.java>
>
> ---------------------------(end of
> broadcast)---------------------------
> TIP 4: Have you searched our list archives?
>
> http://archives.postgresql.org