This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

作者 bkabrda
收信人 benjamin.peterson, bkabrda, pitrou, serhiy.storchaka, stutzbach
日期 2014-06-16.09:40:28
SpamBayes Score -1.0
Marked as misclassified
Message-id <1402911629.56.0.531309355557.issue21679@psf.upfronthosting.co.za>
In-reply-to
内容
So, as pointed out by haypo, blksize_t is actually signed long on Linux. This means that my patch (as well as the current code) is not right.
Both with and without my patch, io_open function uses "int" to store blksize_t and it also passes it to one of PyBuffered{Random,Reader,Writer}_Type. These three however use Py_ssize_t, which is inconsistent with io_open, and it's not correct, too.
I'm unsure how to proceed here - should I fix buffer size types throughout the _io module to long and submit one big patch? It doesn't feel right to put two not-very-related changes into one patch (I'm afraid that changing buffer sizes to long everywhere will result in a rather large patch). Or should I first submit a patch that fixes the long usage and then rebase the patch attached to this issue on top of it?

Thanks a lot.
历史
日期 用户 动作 参数
2014-06-16 09:40:29bkabrda修改recipients: + bkabrda, pitrou, benjamin.peterson, stutzbach, serhiy.storchaka
2014-06-16 09:40:29bkabrda修改messageid: <1402911629.56.0.531309355557.issue21679@psf.upfronthosting.co.za>
2014-06-16 09:40:29bkabrda链接issue21679 messages
2014-06-16 09:40:28bkabrda创建