[freetds] [ freetds-Patches-1083162 ] Allow NULL values for dbrpcparam return parameters
SourceForge.net
noreply at sourceforge.net
Sat Dec 11 03:32:38 EST 2004
Patches item #1083162, was opened at 2004-12-10 21:29
Message generated for change (Comment added) made by freddy77
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=407808&aid=1083162&group_id=33106
Category: db-lib
Group: None
>Status: Closed
>Resolution: Accepted
Priority: 5
Submitted By: Adrian Daley (agd)
Assigned to: Nobody/Anonymous (nobody)
Summary: Allow NULL values for dbrpcparam return parameters
Initial Comment:
Hi,
Thanks for the great work that goes into FreeTDS.
It has been nearly seamless to start using
FreeTDS with an existing application and with some
small changes it is working well.
We're in the processing of updating an existing
application that uses the Sybase libsybdb library
to also use FreeTDS. This application has a
wrapper layer around the dblib calls. For any
stored procedure return parameters the wrapper
layer calls dbrpcparam with value=NULL, maxlen=-1
and datalen=0. The minor changes in the attached
patch update dbrpcparam to better match the Sybase
behavior and documentation.
http://sybooks.sybase.com:80/onlinebooks/group-cn/cng1251e/dblib/@Generic__BookTextView/31893;hf=0#X
With the patch applied the updated rpc.c test case
and all other dblib tests pass.
I'm new to using FreeTDS so please let me know if
these changes overlook anything.
Thank you,
Adrian Daley
Silicon Graphics
----------------------------------------------------------------------
>Comment By: Frediano Ziglio (freddy77)
Date: 2004-12-11 09:32
Message:
Logged In: YES
user_id=75766
Applied test case change. Work with CVS.
----------------------------------------------------------------------
Comment By: Adrian Daley (agd)
Date: 2004-12-10 23:06
Message:
Logged In: YES
user_id=474060
I just noticed that this patch overlaps with a recent
change that already took care of the issue. Maybe the
test case update would be useful. Otherwise please
ignore this patch.
Thanks,
Adrian
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=407808&aid=1083162&group_id=33106
More information about the FreeTDS
mailing list