FreeTDS Development Group

["James K. Lowden" <jklowden AT freetds.org>]

Suren A. Chilingaryan wrote:
> 
> I have reworked a little bit my patch and have implemented unit tests. 
> Actually, in the old patch I have missed the comparison checking if 
> there is still data available. This comparison was again checking 
> processed number of bytes in output buffer against size of source. And 
> BINARY to CHAR conversion (which I said I haven't checked) was actually 
> wrong and even not needed.
> 
> Additionally, I have implemented the CHAR to BINARY conversion for 
> SQLPutData. The unit test is checking both SQLGetData and SQLPutData 
> functionality.

Hi Suren, 

Nice work.  Thanks for the patch!  I worked on it some myself to add the
ODBC error messages and some other tidying up.  Hope you like the result. 


I applied it to CVS HEAD.  I thought it's better if Frediano takes a look
at it before applying it to 0.82.  

Regards, 

--jkl