Jim Jewett <jimjjew...@gmail.com> added the comment:

The currently applied patch ( http://hg.python.org/cpython/rev/f7e05d205a52 ) 
left some dead code in unicodeobject.c

function fixup ( 
http://hg.python.org/cpython/file/f7e05d205a52/Objects/unicodeobject.c#l9386 ) 
has a shortcut for when the fixer doesn't make any actual changes.  The removed 
fixers (like fixupper ) returned 0 rather than maxchar to indicate that.  The 
only remaining fixer, fix_decimal_and_space_to_ascii (line 8839), does not.  (I 
think fix_decimal_and_space_to_ascii *should* add a touched flag, but until it 
does, the shortcut dedup code is dead.)

Also, around line 10502, there is an #if 0 section with code that relied on one 
of the removed fixers; is it time to remove that section?

----------
nosy: +Jim.Jewett

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

Reply via email to