Author Jim.Jewett
Recipients Arfrever, Jean-Michel.Fauth, Jim.Jewett, belopolsky, benjamin.peterson, ezio.melotti, gvanrossum, mrabarnett, pitrou, python-dev, tchrist
Date 2012-01-12.17:17:24
SpamBayes Score 1.48307e-07
Marked as misclassified No
Message-id <>
The currently applied patch ( ) left some dead code in unicodeobject.c

function fixup ( ) 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?
Date User Action Args
2012-01-12 17:17:25Jim.Jewettsetrecipients: + Jim.Jewett, gvanrossum, belopolsky, pitrou, benjamin.peterson, ezio.melotti, mrabarnett, Arfrever, Jean-Michel.Fauth, python-dev, tchrist
2012-01-12 17:17:25Jim.Jewettsetmessageid: <>
2012-01-12 17:17:24Jim.Jewettlinkissue12736 messages
2012-01-12 17:17:24Jim.Jewettcreate