消息 [132819]
Thanks for the review. I've made most of the changes you suggested, but there's
one thing I wanted to check about:
> - instead of "int(size)", use "size = size.__index__()" so as to forbid floats
The tests for readline() and readlines() expect a TypeError if size is None.
Calling size.__index__() in this case raises an AttributeError instead. Should I
change the tests to expect an AttributeError? Alternatively, something like this
would more closely match the behaviour of the old code:
try:
size = size.__index__()
except AttributeError:
raise TypeError("Integer argument expected") |
|
| 日期 |
用户 |
动作 |
参数 |
| 2011-04-02 23:34:22 | nadeem.vawda | 修改 | recipients:
+ nadeem.vawda, rhettinger, niemeyer, mdehoon, pitrou, wrobell, eric.araujo, asvetlov, MizardX, antlong, xuanji |
| 2011-04-02 23:34:22 | nadeem.vawda | 修改 | messageid: <1301787262.79.0.2065758755.issue5863@psf.upfronthosting.co.za> |
| 2011-04-02 23:34:21 | nadeem.vawda | 链接 | issue5863 messages |
| 2011-04-02 23:34:21 | nadeem.vawda | 创建 | |
|