Walter Dörwald <wal...@livinglogic.de> added the comment:

The problem with your patch is that it calls PyUnicode_DecodeUTF8()
twice. It would be better if step 1 in the code would include the %s
format specifiers and step 3 would then call PyUnicode_DecodeUTF8() and
put the result into the callresults buffer.

BTW, I can't make your examples segfault.

Attached is a new version of the patch that implements this. Could you
test this new version to see if there's still a Valgrind report?

----------
nosy: +doerwalter
Added file: http://bugs.python.org/file13668/diff.txt

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

Reply via email to