Senthil Kumaran <[email protected]> added the comment:
Just a minor comment on the patch - It should be %r instead of %s.
- raise TypeError('must be str, not %s' % type(s))
+ raise TypeError('must be str, not %r' % type(s))
If there is any test for this, it could be added. I find this approach 'OK'.
----------
nosy: +orsenthil
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue13532>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com