Author eric.smith
Recipients eric.smith, ezio.melotti, mark.dickinson
Date 2009-07-29.10:35:40
SpamBayes Score 4.35482e-06
Marked as misclassified No
Message-id <>
Since you're calling int() on the result, can't this code:
self._int = str(int((intpart+fracpart).lstrip('0') or '0'))
just be:
self._int = str(int(intpart+fracpart))

And here, you already know diag is not None, so do you need the "or '0'"
self._int = str(int(diag or '0')).lstrip('0')

And, in both calls to .lstrip('0'), what happens if you have a
non-European leading '0', like '\uff10'?

Otherwise, the patch looks good to me.
Date User Action Args
2009-07-29 10:35:43eric.smithsetrecipients: + eric.smith, mark.dickinson, ezio.melotti
2009-07-29 10:35:42eric.smithsetmessageid: <>
2009-07-29 10:35:40eric.smithlinkissue6595 messages
2009-07-29 10:35:40eric.smithcreate