Message155440
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'. |
|
Date |
User |
Action |
Args |
2012-03-12 04:18:49 | orsenthil | set | recipients:
+ orsenthil, loewis, terry.reedy, roger.serwy, Ramchandra Apte |
2012-03-12 04:18:49 | orsenthil | set | messageid: <1331525929.48.0.812486953086.issue13532@psf.upfronthosting.co.za> |
2012-03-12 04:18:48 | orsenthil | link | issue13532 messages |
2012-03-12 04:18:48 | orsenthil | create | |
|