Re: [PATCH] Automatic fetching of refcursors

Hi Adrian,

As for the patch 0001, is DriverConfiguration necessary?
For example, how about the following patch?

diff --git a/winbuild/regress.ps1 b/winbuild/regress.ps1
index b5e1f85..f83a411 100644
--- a/winbuild/regress.ps1
+++ b/winbuild/regress.ps1
@@ -27,7 +27,7 @@
automatically unless this option is specified. Currently "4.0",
"12.0" or "14.0" is available.
.PARAMETER Configuration
- Specify "Release"(default) or "Debug".
+ Specify the configuration used for the regression tests.
"Release"(default) or "Debug".
.PARAMETER BuildConfigPath
Specify the configuration xml file name if you want to use
the configuration file other than standard one.
@@ -263,7 +263,7 @@ function SpecialDsn($testdsn, $testdriver)
$uid = $in
}
$in = read-host -assecurestring "Password [$passwd]"
- if ("$in" -ne "") {
+ if ($in.Length -ne 0) {
$ptr =
[System.Runtime.InteropServices.Marshal]::SecureStringToBSTR($in)
$passwd =
[System.Runtime.InteropServices.Marshal]::PtrToStringBSTR($ptr)
}
@@ -394,12 +394,12 @@ foreach ($pl in $pary) {
"Win32" {
$targetdir="test_x86"
$bit="32-bit"
- $dlldir="$objbase\x86_${ansi_dir_part}_Release"
+
$dlldir="$objbase\x86_${ansi_dir_part}_$Configuration"
}
default {
$targetdir="test_x64"
$bit="64-bit"
- $dlldir="$objbase\x64_${ansi_dir_part}_Release"
+
$dlldir="$objbase\x64_${ansi_dir_part}_$Configuration"
}
}
pushd $pushdir\$targetdir

regards,
Hiroshi Inoue

2021年5月17日(月) 15:35 Inoue,Hiroshi <hinoue205(at)gmail(dot)com>:

> Hi Adrian,
>
> I would take care of your patches.
>
> Thanks.
> Hiroshi Inoue
>
> 2021年5月17日(月) 11:21 Adrian Grucza <adrian(dot)grucza(at)iress(dot)com>:
>
>> Hi all,
>>
>> I've created a patch that enables automatic fetching of refcursors from a
>> function/procedure. Functions/procedures can return one or more refcursors,
>> and the client application can then consume them the way multiple result
>> sets are normally consumed via ODBC.
>>
>> Below is a summary of the attached patches:
>>
>> 0001 (`regress.ps1` changes unrelated to refcursors)
>> * Allow installation and testing of a debug build of the driver (useful
>> for debugging)
>> * Fixed a bug in `regress.ps1` where accepting the default password of
>> `postgres` did not work
>>
>> 0002
>> * Added a FetchRefcursors setting
>> * When FetchRefcursors is enabled, executes some refcursor code in
>> `statement.c` that already existed but was not compiled in due to
>> REFCUR_SUPPORT not being #defined
>>
>> 0003
>> * Enhanced refcursor code to support multiple refcursors
>> * Fixed an issue where output parameter bindings were not processed
>> when using refcursors
>> * Added a `fetch-refcursors` regression test
>>
>> Notes:
>> * Client application must execute the statement in a transaction to
>> avoid cursors being closed prematurely (i.e. turn auto-commit off)
>> * Works with both ODBC CALL and PostgreSQL CALL syntaxes
>> * A search through the mailing list archive shows that refcursor
>> support is a feature that multiple people have wanted
>>
>> Regards,
>> [image: iress.com] <https://www.iress.com/>
>> Adrian Grucza​
>> Technical Lead
>> Tel: +61390185800
>> *adrian(dot)grucza(at)iress(dot)com* <adrian(dot)grucza(at)iress(dot)com>
>> *www.iress.com* <https://www.iress.com/>
>> Level 16, 385 Bourke Street, Melbourne, Victoria, 3000
>> The contents of this email originated from Iress. For this purpose Iress
>> includes Iress Limited and/or any of its subsidiaries, holding companies
>> and trading entities. ​If you have received this email in error please
>> notify the sender immediately and delete this email.
>> nosig
>>
>