Python API's: sql.Binary() fix

Sjoerd Mullender sjoerd at monetdb.org
Mon Feb 4 17:20:49 CET 2013


On 2013-02-04 17:08, Gijs Molenaar wrote:
> 
> i'll have a look at this.

Thanks.  But please update correct branch: Oct2012.

> 
> 
> On 02/03/2013 03:39 PM, Mate Birkas wrote:
>> Hi,
>>
>> I may have found a bug in sql.Binary(). Here is the patch:
>>
>> --- pythonize.py 2013-02-03 15:22:09.217186598 +0100
>> +++ pythonize.py.fix 2013-02-03 15:23:34.613187703 +0100
>> @@ -119,7 +119,7 @@
>>
>>  def Binary(data):
>>      """returns binary encoding of data"""
>> -    return ''.join([hex(ord(i))[2:] for i in data]).upper()
>> +    return ''.join(["%02X" % ord(i) for i in data])
>>
>>  def DateFromTicks(ticks):
>>      """Convert ticks to python Date"""
>>
>> The hex() built-in returns three character string for integers lower than
>> 15. This combined with the original code's [2:] results in corrupt string
>> representations of binary objects having bytes with these values.
>>
>> cheers
>> Mate
>>
>>
>>
>> _______________________________________________
>> developers-list mailing list
>> developers-list at monetdb.org
>> http://mail.monetdb.org/mailman/listinfo/developers-list
>>
> 
> 
> 
> 
> _______________________________________________
> developers-list mailing list
> developers-list at monetdb.org
> http://mail.monetdb.org/mailman/listinfo/developers-list
> 


-- 
Sjoerd Mullender

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 371 bytes
Desc: OpenPGP digital signature
URL: <http://www.monetdb.org/pipermail/developers-list/attachments/20130204/e07790d4/attachment.sig>


More information about the developers-list mailing list