Armin Rigo added the comment:

I reviewed your patch again.  It does look good after all: I find only one 
issue---it seems I implied there were several ones but I can't find more.  The 
issue is that PyString_AsString(result) will succeed if 'result' is a unicode.  
Then you call PyString_GET_SIZE(result), which gives nonsense for unicode 
objects.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue11145>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to