消息 [220084]
Thanks a lot for the code review! I'm attaching a revised version of the patch. Fixes I made:
- added check whether PyLong_AsLong returned an error
- removed "ADD_INTERNED(_blksize)" and "PyObject *_PyIO_str__blksize;" - I noticed that these are only necessary when exported by _iomodule.h, which isn't needed for _blksize ATM
- moved blksize to a place of fileio structure where it won't create unnecessary padding
I hope attaching the version 2 of the patch here is ok, if I should have attached it in the code review tool somehow, please let me know. |
|
| 日期 |
用户 |
动作 |
参数 |
| 2014-06-09 08:12:08 | bkabrda | 修改 | recipients:
+ bkabrda, pitrou, benjamin.peterson, stutzbach, serhiy.storchaka |
| 2014-06-09 08:12:08 | bkabrda | 修改 | messageid: <1402301528.34.0.543361606772.issue21679@psf.upfronthosting.co.za> |
| 2014-06-09 08:12:08 | bkabrda | 链接 | issue21679 messages |
| 2014-06-09 08:12:07 | bkabrda | 创建 | |
|