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.

Author bkabrda
Recipients benjamin.peterson, bkabrda, pitrou, serhiy.storchaka, stutzbach
Date 2014-06-16.09:40:28
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1402911629.56.0.531309355557.issue21679@psf.upfronthosting.co.za>
In-reply-to
Content
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.
History
Date User Action Args
2014-06-16 09:40:29bkabrdasetrecipients: + bkabrda, pitrou, benjamin.peterson, stutzbach, serhiy.storchaka
2014-06-16 09:40:29bkabrdasetmessageid: <1402911629.56.0.531309355557.issue21679@psf.upfronthosting.co.za>
2014-06-16 09:40:29bkabrdalinkissue21679 messages
2014-06-16 09:40:28bkabrdacreate